[WIP] - Experiment caching#172
Conversation
|
[test] |
|
@miteshvp Your image is available in the registry: |
|
[test] |
|
@miteshvp Your image is available in the registry: |
| # Get the response if already cached earlier | ||
| # Sort package_list elements first | ||
| for i in input_json: | ||
| i['package_list'] = sorted(i['package_list']) |
There was a problem hiding this comment.
Hey @miteshvp I don't think this is going to work the way you want it to. This does not alter the original list.
There was a problem hiding this comment.
@rootAvish - I did check on some local json and it works as expected. Also, the tests does prove the point that caching is in play. Still, what makes you think so?
There was a problem hiding this comment.
Oh right sorry, I was thinking along the lines of a normal list. Cool, thanks.
|
Are we still not required to merge this? |
|
@rootAvish - we already did https://github.com/fabric8-analytics/fabric8-analytics-stack-analysis/blob/master/analytics_platform/kronos/deployment/rest_api.py#L138. |
3 participants
No description provided.