)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"647b5e651a6cbe48d9ecc9b77e7bc77d967d6f00","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This script is being removed as it was only required for apps that"},{"line_number":10,"context_line":"needed to migrate from helm v2 to helm v3 and this is no longer"},{"line_number":11,"context_line":"required as of stx.5 release. Also remove"},{"line_number":12,"context_line":"63-helm-v2-to-v3-rel_migration.sh."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1997879"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"02cfeb87_7c60d18a","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":20},"updated":"2022-12-01 20:44:35.000000000","message":"This was added for stx.6.0 to stx.7.0 migration. If we want to support stx.6.0 to stx.8.0 direct upgrades, this will be needed.\n\nThis will also be need for stx-openstack upgrades with is still TBD when/how the optional upgrade will occur. This can\u0027t be removed yet","commit_id":"5626b09b24816121994c39e6328738cad036c906"},{"author":{"_account_id":28424,"name":"Frank Miller","email":"frank.miller@eng.windriver.com","username":"sensfan22"},"change_message_id":"a2adb4c7967eeba1c7e270cfafb796f1e495a24f","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This script is being removed as it was only required for apps that"},{"line_number":10,"context_line":"needed to migrate from helm v2 to helm v3 and this is no longer"},{"line_number":11,"context_line":"required as of stx.5 release. Also remove"},{"line_number":12,"context_line":"63-helm-v2-to-v3-rel_migration.sh."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1997879"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3c0901c5_31d169c7","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":20},"in_reply_to":"02cfeb87_7c60d18a","updated":"2022-12-01 22:16:40.000000000","message":"I didn\u0027t realize there was a dependency between stx-openstack and helm v2 (due to armada). Since stx-openstack has now migrated to FluxCD, it will no longer have a dependency on helm v2 as of stx8.0 release.  Therefore I suggest Fabricio puts a WFL -1 on this commit until the stx.9.0 release and this code gets merged at that time.","commit_id":"5626b09b24816121994c39e6328738cad036c906"},{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"787023b7d9153ea2d064e20f883798ffee4680f7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This script is being removed as it was only required for apps that"},{"line_number":10,"context_line":"needed to migrate from helm v2 to helm v3 and this is no longer"},{"line_number":11,"context_line":"required as of stx.5 release. Also remove"},{"line_number":12,"context_line":"63-helm-v2-to-v3-rel_migration.sh."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1997879"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"51441415_26bfb72a","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":20},"in_reply_to":"1d66d49b_5e119820","updated":"2023-03-27 17:06:08.000000000","message":"Raised this to track Bob\u0027s concerns: https://bugs.launchpad.net/starlingx/+bug/2012956 . Based on what Frank said the implementation is probably done, the work left is to double check the stx-openstack documentation for upgrades section (or upgrades documentation for stx-openstack section?).","commit_id":"5626b09b24816121994c39e6328738cad036c906"},{"author":{"_account_id":33631,"name":"Fabricio Henrique Ramos","email":"fabriciohenrique.ramos@windriver.com","username":"fhramos"},"change_message_id":"e16f8550343ba7f8645dc58eb9f896fc846f8119","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This script is being removed as it was only required for apps that"},{"line_number":10,"context_line":"needed to migrate from helm v2 to helm v3 and this is no longer"},{"line_number":11,"context_line":"required as of stx.5 release. Also remove"},{"line_number":12,"context_line":"63-helm-v2-to-v3-rel_migration.sh."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1997879"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1d66d49b_5e119820","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":20},"in_reply_to":"3c0901c5_31d169c7","updated":"2023-03-24 16:40:49.000000000","message":"Ack","commit_id":"5626b09b24816121994c39e6328738cad036c906"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"647b5e651a6cbe48d9ecc9b77e7bc77d967d6f00","unresolved":true,"context_lines":[{"line_number":11,"context_line":"required as of stx.5 release. Also remove"},{"line_number":12,"context_line":"63-helm-v2-to-v3-rel_migration.sh."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1997879"},{"line_number":15,"context_line":"Signed-off-by: Fabricio Henrique Ramos \u003cfabriciohenrique.ramos@windriver.com\u003e"},{"line_number":16,"context_line":"Change-Id: If15ecf3cca4538a6e377a9524e09e7bd65d3d5fa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"c440a38f_12065ff7","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":19},"updated":"2022-12-01 20:44:35.000000000","message":"This LP is filed to the wrong project?","commit_id":"5626b09b24816121994c39e6328738cad036c906"},{"author":{"_account_id":33631,"name":"Fabricio Henrique Ramos","email":"fabriciohenrique.ramos@windriver.com","username":"fhramos"},"change_message_id":"5b4dd52803a8c9fbd5cba4ee4eb5db90847792b8","unresolved":false,"context_lines":[{"line_number":11,"context_line":"required as of stx.5 release. Also remove"},{"line_number":12,"context_line":"63-helm-v2-to-v3-rel_migration.sh."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1997879"},{"line_number":15,"context_line":"Signed-off-by: Fabricio Henrique Ramos \u003cfabriciohenrique.ramos@windriver.com\u003e"},{"line_number":16,"context_line":"Change-Id: If15ecf3cca4538a6e377a9524e09e7bd65d3d5fa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9d179c68_dc7b65fe","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":19},"in_reply_to":"c440a38f_12065ff7","updated":"2023-03-24 16:40:22.000000000","message":"Done","commit_id":"5626b09b24816121994c39e6328738cad036c906"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"4b2e8b53a34568ccf0a28dfd99411558eb480bb0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1135ab12_a5da335c","updated":"2022-11-29 17:29:45.000000000","message":"We have an upgrade script referencing this: 63-helm-v2-to-v3-rel_migration.sh. Shouldn\u0027t we remove that script first?","commit_id":"469272a33602e473d6daa161afb1804bea9d1c4c"},{"author":{"_account_id":33631,"name":"Fabricio Henrique Ramos","email":"fabriciohenrique.ramos@windriver.com","username":"fhramos"},"change_message_id":"f6fe42759c4bc00adfd25b893c8f5eb9f747093d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fa07310d_e7b4b865","in_reply_to":"1135ab12_a5da335c","updated":"2022-12-01 15:40:25.000000000","message":"Done","commit_id":"469272a33602e473d6daa161afb1804bea9d1c4c"},{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"787023b7d9153ea2d064e20f883798ffee4680f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"862445f5_b5ba09ed","updated":"2023-03-27 17:06:08.000000000","message":"lgtm\n\nReasoning: migrate_helm_release.py needs to be removed since Armada and HelmV2 is removed in https://review.opendev.org/q/topic:remove-armada-helmv2 (stx.9.0). migrate_helm_release.py\n\nRaised https://bugs.launchpad.net/starlingx/+bug/2012956 which can be looked at by openstack team.\n","commit_id":"563cb2e9888925b4ef703dc3f6f88c190c24a03e"}]}