)]}'
{"id":"openstack%2Fopenstack-helm~782510","triplet_id":"openstack%2Fopenstack-helm~master~Ibc99cb20482864f55daa12321e8d81414c1ef9f8","project":"openstack/openstack-helm","branch":"master","topic":"heat_annotations","hashtags":[],"change_id":"Ibc99cb20482864f55daa12321e8d81414c1ef9f8","subject":"Added post-install and post-upgrade for jobs in heat","status":"MERGED","created":"2021-03-23 12:45:18.000000000","updated":"2021-09-04 22:18:55.000000000","submitted":"2021-09-04 22:16:50.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"782510-heat_annotations","meta_rev_id":"36cadfdcf2d601ae866d65d0c662e665826c8421","_number":782510,"virtual_id_number":782510,"owner":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-09-04 22:16:49.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"value":0,"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-08-31 19:51:13.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":1,"date":"2021-08-23 16:32:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"value":2,"date":"2021-09-04 21:04:56.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"value":1,"date":"2021-09-04 21:04:56.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"}],"reviewers":{"REVIEWER":[{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-23 13:44:04.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":"2021-08-19 10:14:56.000000000","updated_by":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"reviewer":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"state":"REVIEWER"},{"updated":"2021-08-19 10:14:56.000000000","updated_by":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"reviewer":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"state":"REVIEWER"},{"updated":"2021-08-19 10:14:56.000000000","updated_by":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"reviewer":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"state":"REVIEWER"},{"updated":"2021-08-19 10:14:56.000000000","updated_by":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"reviewer":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"state":"REVIEWER"}],"messages":[{"id":"35086ce706d7850fccee3e6287a1f90fbf12a7b3","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"date":"2021-03-23 12:45:18.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"803ab5276b3937e63b83fb5cd426043f7613cde7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-23 13:44:04.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/97a221b5a16842d3bdf9c4d8cf526c99 : SUCCESS in 6m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/b590f0eb5a74425d92dc22c45def59a2 : SUCCESS in 4m 23s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/c49d50545b2b432daea48ef54b25adcf : SUCCESS in 5m 45s\n- openstack-helm-cinder-stein-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/1dc2d9f9b49f4cbab9577a1db5e87397 : SUCCESS in 43m 39s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/dd601ec299db4cd084a2386cc5701ffb : SUCCESS in 42m 38s\n- openstack-helm-cinder-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/9a1345741aaf4f048b8a7007adef7f7f : SUCCESS in 47m 40s\n- openstack-helm-compute-kit-stein-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/382a74f3ea6b44efbc11ec072b7dfd86 : SUCCESS in 45m 45s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/aad753ca22954ac195dd409089172ae5 : SUCCESS in 56m 17s\n- openstack-helm-compute-kit-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/18374b8f56b347bda2cbbc0cb00dbe64 : SUCCESS in 50m 40s","accounts_in_message":[],"_revision_number":1},{"id":"4553bd64dda3bf2be6ca443da4d7d959749a831e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"date":"2021-08-19 10:12:32.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"14a20dcce25cfd4254f5294115fe043531a083e7","author":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"date":"2021-08-19 10:14:56.000000000","message":"Patch Set 2:\n\nCan you please have a look at it when you got time?","accounts_in_message":[],"_revision_number":2},{"id":"d8285e8a0098a5808fca2ef6a6a1299b8897e150","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-19 11:11:06.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4ddefd25328a4941bc5974a68076cdcd : SUCCESS in 6m 12s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/e6a190ba98b14c5b83eff26bc4373d87 : SUCCESS in 4m 48s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/15b5fbf28d1d4ce3aaac8c2782645516 : SUCCESS in 7m 06s\n- openstack-helm-helm3-lint https://zuul.opendev.org/t/openstack/build/abfea671483a4b8f8d1f5907df54e03a : SUCCESS in 2m 56s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/d629f55fa550437baf769cfc6a4efa4d : SUCCESS in 44m 58s\n- openstack-helm-cinder-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/92ffc09e063b4889b02d4762712d609b : SUCCESS in 45m 39s\n- openstack-helm-cinder-victoria-ubuntu_focal https://zuul.opendev.org/t/openstack/build/96c0ad6800a446379eed5f26fb9de585 : SUCCESS in 46m 31s\n- openstack-helm-cinder-wallaby-ubuntu_focal https://zuul.opendev.org/t/openstack/build/0f785ecd87c14e96b2142be2adf77daa : SUCCESS in 49m 21s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/fda095c5b44c494883ebfb92f0d002ff : SUCCESS in 54m 09s\n- openstack-helm-compute-kit-ussuri-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/6c49a28c95164280a6e335af4eb663e0 : SUCCESS in 54m 24s\n- openstack-helm-compute-kit-victoria-ubuntu_focal https://zuul.opendev.org/t/openstack/build/a71a7f66f5d2447bb294f7e0f5e03d4d : SUCCESS in 51m 09s\n- openstack-helm-compute-kit-wallaby-ubuntu_focal https://zuul.opendev.org/t/openstack/build/e1324fc15ce3487a861f3a66aa24f115 : SUCCESS in 49m 52s","accounts_in_message":[],"_revision_number":2},{"id":"6f968a893ae622105c5793f6910ae1949e9aa90c","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-08-20 15:15:08.000000000","message":"Patch Set 2: Code-Review+1\n\nalso","accounts_in_message":[],"_revision_number":2},{"id":"c5170a28d3770466534c205844aa7f912906e0eb","author":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"date":"2021-08-23 16:32:01.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"ee1b70ebeac0a5a37a28cfb2213f93ee05dbc6bb","author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"date":"2021-08-31 19:51:13.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"25ddd6e551651cbd7c8dbb5ed77389a16b077b0a","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-09-04 21:04:56.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"0f9d9eef57848865b75ec1cf89af64ac8f2dd423","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-09-04 21:05:17.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"4e6baba68ca727068f813b3df8ade74c84f079df","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-09-04 22:16:49.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/39d93052041248a3abed6fce0669c31c : SUCCESS in 9m 48s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/926a89ab5dc64f7abcf2556fc2b531a8 : SUCCESS in 6m 14s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/b7e9b331c8de41b48f0f1ebf61ae2b62 : SUCCESS in 6m 07s\n- openstack-helm-tls https://zuul.opendev.org/t/openstack/build/da9c946a0a624c088300ca9036cd3f24 : SUCCESS in 1h 09m 11s\n- openstack-helm-cinder-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/12a38fa6b85248439f2d03235074013e : SUCCESS in 44m 32s\n- openstack-helm-compute-kit-train-ubuntu_bionic https://zuul.opendev.org/t/openstack/build/52b0fe810d964bea929f28d64fec6608 : SUCCESS in 52m 43s","accounts_in_message":[],"_revision_number":2},{"id":"39d8f722a2dc309b84a13474a808a6d98fe418e3","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-09-04 22:16:50.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"36cadfdcf2d601ae866d65d0c662e665826c8421","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-09-04 22:18:55.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/be89ca349a1c4a4297cfbf78e098ad41 : SUCCESS in 1m 43s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/cf7338a128b044a5938cc700bfe15f03 : SUCCESS in 56s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"841fb577d9edc518a112624b86edcbbe48f7f33f","revisions":{"444a5770a48549b340f9511cfa6513a9212e4341":{"kind":"REWORK","_number":1,"created":"2021-03-23 12:45:18.000000000","uploader":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"ref":"refs/changes/10/782510/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/10/782510/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/1"}}},"commit":{"parents":[{"commit":"48ab9f6961b87aec4ec4e8847cdb0e64f796ae82","subject":"Merge \"Enable tls for glance storage init\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/48ab9f6961b87aec4ec4e8847cdb0e64f796ae82"}]}],"author":{"name":"Susanta Gautam","email":"xuxant02@gmail.com","date":"2021-03-23 12:44:05.000000000","tz":345},"committer":{"name":"Susanta Gautam","email":"xuxant02@gmail.com","date":"2021-03-23 12:44:09.000000000","tz":345},"subject":"Added post-install and post-upgrade for jobs in heat","message":"Added post-install and post-upgrade for jobs in heat\n\nChart upgrading was failing due to some immutable fields are needed to upgrade before the jobs can be upgraded. For solving this issue, we\nhave added the helm.sh/hook annotations with post-install and post-upgrade values.\nAs for hook-weight annotations, we have added these to control the flow of the jobs with hook creation as the jobs are dependent. Like,\ndb-init jobs need to run before db-sync and so on.\n\nChange-Id: Ibc99cb20482864f55daa12321e8d81414c1ef9f8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/444a5770a48549b340f9511cfa6513a9212e4341"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/444a5770a48549b340f9511cfa6513a9212e4341"}]},"branch":"refs/heads/master"},"841fb577d9edc518a112624b86edcbbe48f7f33f":{"kind":"REWORK","_number":2,"created":"2021-08-19 10:12:32.000000000","uploader":{"_account_id":33130,"name":"Susanta gautam","email":"xuxant02@gmail.com","username":"sgautam"},"ref":"refs/changes/10/782510/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/10/782510/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/10/782510/2"}}},"commit":{"parents":[{"commit":"2f80bd3ff22e0d46f531fd4b7190f80e26018e90","subject":"Merge \"Add V and W jobs to zuul check\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/2f80bd3ff22e0d46f531fd4b7190f80e26018e90"}]}],"author":{"name":"Susanta Gautam","email":"xuxant02@gmail.com","date":"2021-03-23 12:44:05.000000000","tz":345},"committer":{"name":"xuxant02@gmail.com","email":"xuxant02@gmail.com","date":"2021-08-19 10:10:38.000000000","tz":345},"subject":"Added post-install and post-upgrade for jobs in heat","message":"Added post-install and post-upgrade for jobs in heat\n\nChart upgrading was failing due to some immutable fields are needed to upgrade before the jobs can be upgraded. For solving this issue, we have added the helm.sh/hook annotations with post-install and post-upgrade values. As for hook-weight annotations, we have added these to control the flow of the jobs with hook creation as the jobs are dependent. Like, db-init jobs need to run before db-sync and so on. Also values helm3_hook is introduced in values.yaml from which hooks can be disabled if needed.\n\nChange-Id: Ibc99cb20482864f55daa12321e8d81414c1ef9f8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/841fb577d9edc518a112624b86edcbbe48f7f33f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/841fb577d9edc518a112624b86edcbbe48f7f33f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}