)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32349,"name":"Thiago Paiva Brito","email":"thiago.brito@windriver.com","username":"outbrito"},"change_message_id":"ea4ab3d683f4e3209618377cad0493233eddc40e","unresolved":true,"context_lines":[{"line_number":10,"context_line":"with helmv3 since for a couple of jobs the helm-hooks and weights are"},{"line_number":11,"context_line":"not implemented correctly."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is already implemented on openstack-helm main branch, so this patch"},{"line_number":14,"context_line":"is simply bringing those updates to openstack-armada-app."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Test Plan:"},{"line_number":17,"context_line":"PASS - build-helm-charts.sh builds a basic FluxCD app tarball"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"bf08973b_4d4ea797","line":14,"range":{"start_line":13,"start_character":59,"end_line":14,"end_character":57},"updated":"2022-09-26 21:11:10.000000000","message":"You are bringing this through a patch temporarily, right? Please create a story to make sure this will get on the next OSH upversion and mention it here.","commit_id":"02378565ccfcd22a145fd92b3c6a3d1eba70a6ec"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"f1730691794fcf66d2c9af38797610b99a97cfe6","unresolved":false,"context_lines":[{"line_number":10,"context_line":"with helmv3 since for a couple of jobs the helm-hooks and weights are"},{"line_number":11,"context_line":"not implemented correctly."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is already implemented on openstack-helm main branch, so this patch"},{"line_number":14,"context_line":"is simply bringing those updates to openstack-armada-app."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Test Plan:"},{"line_number":17,"context_line":"PASS - build-helm-charts.sh builds a basic FluxCD app tarball"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"7f527476_e11da576","line":14,"range":{"start_line":13,"start_character":59,"end_line":14,"end_character":57},"in_reply_to":"7c445f88_a73aa791","updated":"2022-09-26 22:49:05.000000000","message":"Story-board created: story/2010321\nTask: 46425","commit_id":"02378565ccfcd22a145fd92b3c6a3d1eba70a6ec"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"511d1c55f8a66747d8a2d572f6d1ef60f2479a68","unresolved":false,"context_lines":[{"line_number":10,"context_line":"with helmv3 since for a couple of jobs the helm-hooks and weights are"},{"line_number":11,"context_line":"not implemented correctly."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is already implemented on openstack-helm main branch, so this patch"},{"line_number":14,"context_line":"is simply bringing those updates to openstack-armada-app."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Test Plan:"},{"line_number":17,"context_line":"PASS - build-helm-charts.sh builds a basic FluxCD app tarball"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"7c445f88_a73aa791","line":14,"range":{"start_line":13,"start_character":59,"end_line":14,"end_character":57},"in_reply_to":"bf08973b_4d4ea797","updated":"2022-09-26 21:19:27.000000000","message":"Sure thing, this (like the majority of our patches) is temporarily added until the next OSH upversion.\nThere is no current OSH upversion planned for stx/8.0 release (due to time and capacity constraints), but this would definitely be brought from osh master when our next upversion is done.","commit_id":"02378565ccfcd22a145fd92b3c6a3d1eba70a6ec"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34350,"name":"Rafael Vieira Falcão","display_name":"Rafael Falcão","email":"rafael.vieirafalcao@windriver.com","username":"rafaelvfalc"},"change_message_id":"5e5b62441847f9555b08a1a471168aaf644064b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"9acb6e22_4c006967","updated":"2022-09-26 21:04:31.000000000","message":"LGTM. Just a questions: Is this modification already in the master of the openstack-helm repo?","commit_id":"287aa6b4e3aad638f7b4a7364da4c1a3fdec6d15"},{"author":{"_account_id":34350,"name":"Rafael Vieira Falcão","display_name":"Rafael Falcão","email":"rafael.vieirafalcao@windriver.com","username":"rafaelvfalc"},"change_message_id":"a3af082e95442b7175f4680e49455a46ecfc720a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e336dfb0_e73e1dd2","in_reply_to":"9acb6e22_4c006967","updated":"2022-09-26 21:05:28.000000000","message":"*question","commit_id":"287aa6b4e3aad638f7b4a7364da4c1a3fdec6d15"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"d26208674c822f92384995d2b8c172348d702178","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"20baf957_8fd44d40","in_reply_to":"e336dfb0_e73e1dd2","updated":"2022-09-26 21:06:56.000000000","message":"It is. I mentioned on this commit message:\n\"This is already implemented on openstack-helm main branch, so this patch is simply bringing those updates to openstack-armada-app.\"","commit_id":"287aa6b4e3aad638f7b4a7364da4c1a3fdec6d15"},{"author":{"_account_id":34350,"name":"Rafael Vieira Falcão","display_name":"Rafael Falcão","email":"rafael.vieirafalcao@windriver.com","username":"rafaelvfalc"},"change_message_id":"f89d721eedce6c61802bd2a689d4275488beb30b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8018f6c5_509b39f7","in_reply_to":"e336dfb0_e73e1dd2","updated":"2022-09-26 21:07:10.000000000","message":"Oops, just see that you mentioned this in the commit message. Great!","commit_id":"287aa6b4e3aad638f7b4a7364da4c1a3fdec6d15"},{"author":{"_account_id":33987,"name":"Pedro Monteiro Azevedo de Moura Almeida","display_name":"Pedro Almeida","email":"pedro.monteiroazevedodemouraalmeida@windriver.com","username":"pedroalmd"},"change_message_id":"028e61468b9cd760d814175de8aad93c8fbd3643","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"0ea1100a_b15a97c9","updated":"2022-09-27 11:35:04.000000000","message":"LGTM","commit_id":"02378565ccfcd22a145fd92b3c6a3d1eba70a6ec"}]}