)]}'
{"id":"openstack%2Fwatcher~736494","triplet_id":"openstack%2Fwatcher~master~I8ce04ce8d08f504b8a70787d159e046334e40e0d","project":"openstack/watcher","branch":"master","topic":"optimization/pip-no-cache-dir","hashtags":[],"change_id":"I8ce04ce8d08f504b8a70787d159e046334e40e0d","subject":"optimize size and time using \"--no-cache-dir\"","status":"ABANDONED","created":"2020-06-18 08:07:30.000000000","updated":"2020-06-18 12:24:15.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"40eb2c5674ac2a56625880bb8ddf87299f9071a7","_number":736494,"virtual_id_number":736494,"owner":{"_account_id":31020,"name":"Pratik raj","email":"Rajpratik71@gmail.com","username":"rajpratik71"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},{"tag":"autogenerated:zuul:check","value":1,"date":"2020-06-18 09:23:29.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"all":[{"value":-1,"date":"2020-06-18 11:47:06.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":-1,"date":"2020-06-18 11:58:23.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"date":"2020-06-18 12:06:28.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Standard Change","+1":"Important Change","+2":"High Priority Change"},"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-06-18 09:23:29.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-06-18 11:47:06.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2020-06-18 11:58:23.000000000","updated_by":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"reviewer":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"state":"REVIEWER"},{"updated":"2020-06-18 12:06:28.000000000","updated_by":{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},"reviewer":{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},"state":"REVIEWER"}],"messages":[{"id":"86d45079cbc522c823894b8ced7593a9e29c4e05","author":{"_account_id":31020,"name":"Pratik raj","email":"Rajpratik71@gmail.com","username":"rajpratik71"},"date":"2020-06-18 08:07:30.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"36747104b9d339cbd622bc6ed6d3323da87c9b1c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-18 09:23:29.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-api-ref https://zuul.opendev.org/t/openstack/build/51663d69af1a488492898b955620801d : SUCCESS in 6m 33s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/aa71412dce6942d886fca1fb2c6ea1bc : SUCCESS in 8m 35s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/19823d4f3657426f904b974e5e29f581 : SUCCESS in 8m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e563b158415b45e5b3e9ea842ea0bc1c : SUCCESS in 7m 04s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d253d07d001a4a5b90f82493d8237253 : SUCCESS in 7m 07s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/898147b11b5a4ce8bbdc43bd9f376ac3 : SUCCESS in 8m 22s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2f3e1d0f410642dc8448cb2b83d354e0 : SUCCESS in 10m 23s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/19978d8c5486448dab42350efb872d69 : SUCCESS in 7m 23s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/e683bf7fad094949a6f969e03c570d49 : SUCCESS in 28m 51s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/56548348a20e41c395fff2bda92987ca : SUCCESS in 55m 32s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/f3271265a5d244cfb5257d70f30223fa : SUCCESS in 49m 37s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/e2595582f1894ce689905968d282aee0 : SUCCESS in 56m 14s\n- watcher-tls-test https://zuul.opendev.org/t/openstack/build/6013f823fa14431b8f6182e36d4e806b : SUCCESS in 45m 18s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/46b944152ab24448a4b1b182b7911c06 : SUCCESS in 37m 51s","accounts_in_message":[],"_revision_number":1},{"id":"468a0009e292b1c53bd1c6f41ff6ca0e65f7043d","author":{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},"date":"2020-06-18 10:24:11.000000000","message":"Patch Set 1: Code-Review+2\n\nI think this is a good addition not only since it will reduce the size for docker images but since there is no reason to want to use cached packages when setting up the tox environments.","accounts_in_message":[],"_revision_number":1},{"id":"555c43cd061a923376ba3cf1c4382e81b85a5335","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2020-06-18 11:47:06.000000000","message":"Patch Set 1: Code-Review-1\n\nPlease don\u0027t land this like this - the user docs should absolutely not be changed in this way.","accounts_in_message":[],"_revision_number":1},{"id":"45d893d38971ccb384ea0ae2b4fbaaf7873db6be","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2020-06-18 11:58:23.000000000","message":"Patch Set 1: Code-Review-1\n\nthis is wrong and not needed, please abandon","accounts_in_message":[],"_revision_number":1},{"id":"c925ad9fce2dcf07ebd3b0d7bc36339f16efcb32","author":{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},"date":"2020-06-18 12:01:07.000000000","message":"Removed Code-Review+2 by Dantali0n \u003cinfo@dantalion.nl\u003e\n","accounts_in_message":[],"_revision_number":1},{"id":"241b3ac5d0c39091e251273eba5676f28df9d8ee","author":{"_account_id":29911,"name":"Dantali0n","email":"info@dantalion.nl","username":"Dantali0n"},"date":"2020-06-18 12:06:28.000000000","message":"Patch Set 1:\n\n\u003e this is wrong and not needed, please abandon\n\nCare to elaborate? I can imagine this can manually be done by loci / openstack-images removing the need to have it in tox or userdocs.\n\nStill when (re)creating a tox environment when would you want to use the cache? Perhaps when creating multiple environments in succession it would prevent to have to download the packages multiple times I suppose.\n\nVery well I see why this is very undesirable.","accounts_in_message":[],"_revision_number":1},{"id":"1ac67d8de9f4c30845d2a90ae30c6f0c08ce46d0","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2020-06-18 12:08:07.000000000","message":"Patch Set 1:\n\n\u003e \u003e this is wrong and not needed, please abandon\n \u003e \n \u003e Care to elaborate? I can imagine this can manually be done by loci\n \u003e / openstack-images removing the need to have it in tox or userdocs.\n \nloci etc. are not using tox to install the package. And neither do they use the documentation with copy \u0026 paste.\n\n \u003e Still when (re)creating a tox environment when would you want to\n \u003e use the cache? Perhaps when creating multiple environments in\n \u003e succession it would prevent to have to download the packages\n \u003e multiple times I suppose.\n\nAs a developer, I want to use a cache when running things locally.\n \n \u003e Very well I see why this is very undesirable.","accounts_in_message":[],"_revision_number":1},{"id":"f1e8a3a5fac17ad8508b6c7b5ce0b6681d7bfa7e","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-06-18 12:24:15.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"6ea13e6ae6abaa0f21e36c577ef02cbd5faa29fa","revisions":{"6ea13e6ae6abaa0f21e36c577ef02cbd5faa29fa":{"kind":"REWORK","_number":1,"created":"2020-06-18 08:07:30.000000000","uploader":{"_account_id":31020,"name":"Pratik raj","email":"Rajpratik71@gmail.com","username":"rajpratik71"},"ref":"refs/changes/94/736494/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/736494/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/736494/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/736494/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/736494/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/736494/1"}}},"commit":{"parents":[{"commit":"097ac06f0b81377a4184c20433d1cb0845de5623","subject":"Use uwsgi binary from path and mark grenade non-voting","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/097ac06f0b81377a4184c20433d1cb0845de5623"}]}],"author":{"name":"Pratik Raj","email":"rajpratik71@gmail.com","date":"2020-06-18 04:56:35.000000000","tz":330},"committer":{"name":"Pratik Raj","email":"rajpratik71@gmail.com","date":"2020-06-18 08:07:23.000000000","tz":330},"subject":"optimize size and time using \"--no-cache-dir\"","message":"optimize size and time using \"--no-cache-dir\"\n\nUsing \"--no-cache-dir\" flag in pip install ,make sure dowloaded packages\nby pip don\u0027t cached on system . This is a best practise which make sure\nto fetch ftom repo instead of using local cached one . Further , in case\nof Docker Containers , by restricing caching , we can reduce image size.\nIn term of stats , it depends upon the number of python packages\nmultiplied by their respective size . e.g for heavy packages with a lot\nof dependencies it reduce a lot by don\u0027t caching pip packages.\n\nFurther , more detail information can be found at\n\nhttps://medium.com/sciforce/strategies-of-docker-images-optimization-2ca9cc5719b6\n\nChange-Id: I8ce04ce8d08f504b8a70787d159e046334e40e0d\nSigned-off-by: Pratik Raj \u003crajpratik71@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/6ea13e6ae6abaa0f21e36c577ef02cbd5faa29fa"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/6ea13e6ae6abaa0f21e36c577ef02cbd5faa29fa"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}