)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"2c636030b01c52abb8df6813c899507f3b93df96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"40646d80_61f2ad21","updated":"2022-10-31 22:14:13.000000000","message":"run \"grep -rs node-role.kubernetes.io/master stx-openstack-helm-fluxcd\" to see what files need to be updated.","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":34350,"name":"Rafael Vieira Falcão","display_name":"Rafael Falcão","email":"rafael.vieirafalcao@windriver.com","username":"rafaelvfalc"},"change_message_id":"aaee96df925b432f92ce6ba56b87ad852409dffd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a4db8c43_92d0e640","updated":"2022-11-23 17:31:26.000000000","message":"I think you forgot to add the \u0027node-role.kubernetes.io/control-plane\u0027 label in the garbd static overrides. If it is not added there, the bug might still happen in a standard/storage environment.","commit_id":"6344cd013c87e8fd22ed4cd95dec8597594f22dc"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"90a3bd2351215f6f3a7c12920276ae1d27d8439a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"084436a8_af7399cd","updated":"2022-11-18 12:23:51.000000000","message":"Looking good so far. Will wait for the Test Plan to be updated before bumping to +2\n\nThanks for updating those overrides :)","commit_id":"6344cd013c87e8fd22ed4cd95dec8597594f22dc"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8e57677a_8cbbdbe9","updated":"2022-11-14 05:06:10.000000000","message":"Looking good, just need to coordinate the testing to make sure nothing has broken.","commit_id":"6344cd013c87e8fd22ed4cd95dec8597594f22dc"},{"author":{"_account_id":34350,"name":"Rafael Vieira Falcão","display_name":"Rafael Falcão","email":"rafael.vieirafalcao@windriver.com","username":"rafaelvfalc"},"change_message_id":"e45114e62ddeb3a36dfc063c6e5581d15814e7e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7744dd25_526e91f0","in_reply_to":"a4db8c43_92d0e640","updated":"2022-11-23 18:11:01.000000000","message":"Done","commit_id":"6344cd013c87e8fd22ed4cd95dec8597594f22dc"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2b18234bb36157b6e83206f10178559d6af79a99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9f7f73e0_854a7b1d","updated":"2022-11-23 18:05:31.000000000","message":"Thanks for jumping in Falcão. Will wait for the tests results to bump my +2","commit_id":"4dc32c8671b31ff16bcb853b2ed230eb2ff75cf2"}],"openstack-helm-infra/debian/deb_folder/patches/0026-Add-control-plane-toleration-for-Openstack-services.patch":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"f61965a145e80ebb337e056b6b247658c15dcaf1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f062a52f_096791ce","updated":"2022-10-31 22:21:30.000000000","message":"given that we\u0027re setting the tolerations in the manifests and static overrides, I\u0027m not sure if these changes are actually needed as a patch.","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c80175d4_aa9e1c9f","in_reply_to":"13af4b72_821b03ad","updated":"2022-11-14 05:06:10.000000000","message":"Done","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"317c0825c06f85f1a89cb2de5372992b69ccb944","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"13af4b72_821b03ad","in_reply_to":"f062a52f_096791ce","updated":"2022-11-10 23:42:25.000000000","message":"Again, I don\u0027t think this patch is needed since we\u0027re overriding the values in the static manifest files anyway.  I think you can probably get rid of this patch altogether.","commit_id":"68791931425108362151a6fa66aff30eb3a06780"}],"openstack-helm-infra/files/0026-Add-control-plane-toleration-for-Openstack-services.patch":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"2df8fb6c401ee42df2abd1d32d23315302900d0e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95384d71_25ad5c6f","updated":"2022-10-31 22:09:45.000000000","message":"This is for CentOS, so it only makes sense to modify this file if you are also going to modify centos/openstack-helm-infra.spec","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"55c27190_ccd41b08","in_reply_to":"95384d71_25ad5c6f","updated":"2022-11-14 05:06:10.000000000","message":"Done","commit_id":"68791931425108362151a6fa66aff30eb3a06780"}],"openstack-helm/debian/deb_folder/patches/0020-Add-control-plane-toleration-for-Openstack-service.patch":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"f61965a145e80ebb337e056b6b247658c15dcaf1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5420c926_bb121171","updated":"2022-10-31 22:21:30.000000000","message":"given that we\u0027re setting the tolerations in the manifests and static overrides, I\u0027m not sure if these changes are actually needed as a patch.","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"317c0825c06f85f1a89cb2de5372992b69ccb944","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ae5f15c8_96758bc2","in_reply_to":"5420c926_bb121171","updated":"2022-11-10 23:42:25.000000000","message":"I suspect we can simply drop this patch entirely since we\u0027re overriding the values via the static overrides files.","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d0159a50_735f0876","in_reply_to":"ae5f15c8_96758bc2","updated":"2022-11-14 05:06:10.000000000","message":"Done","commit_id":"68791931425108362151a6fa66aff30eb3a06780"}],"openstack-helm/files/0020-Add-control-plane-toleration-for-Openstack-service.patch":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"2df8fb6c401ee42df2abd1d32d23315302900d0e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5c86d9b8_553cc9a3","updated":"2022-10-31 22:09:45.000000000","message":"This is for CentOS builds, so it only makes sense to modify this if you are also going to modify centos/openstack-helm.spec","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5aec856f_144b7d3e","in_reply_to":"5c86d9b8_553cc9a3","updated":"2022-11-14 05:06:10.000000000","message":"Done","commit_id":"68791931425108362151a6fa66aff30eb3a06780"}],"stx-openstack-helm/stx-openstack-helm/helm-charts/nova-api-proxy/values.yaml":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"317c0825c06f85f1a89cb2de5372992b69ccb944","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3ac6fb49_5e3459df","updated":"2022-11-10 23:42:25.000000000","message":"Pretty sure we don\u0027t need this given the change in stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/nova-api-proxy/nova-api-proxy-static-overrides.yaml","commit_id":"f43ce5e0f4c92484d93190a9d443337875b89944"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ceb62e61_c9ea7d37","in_reply_to":"3ac6fb49_5e3459df","updated":"2022-11-14 05:06:10.000000000","message":"Done","commit_id":"f43ce5e0f4c92484d93190a9d443337875b89944"}],"stx-openstack-helm/stx-openstack-helm/manifests/manifest.yaml":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"0c922d3eb37906d17b58140d050b13cb728a676b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"724b72b1_07381fca","updated":"2022-10-31 22:13:25.000000000","message":"This file is not used by FluxCD, can be omitted.  (It\u0027s a legacy file used by Armada.)\n\nThere are a bunch of files under stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd that need to be modified though.","commit_id":"68791931425108362151a6fa66aff30eb3a06780"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"31978aab8d94ea0ae0e2734c6dc898a10644310e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1313dbf1_597d25b6","in_reply_to":"724b72b1_07381fca","updated":"2022-11-14 05:06:10.000000000","message":"Done","commit_id":"68791931425108362151a6fa66aff30eb3a06780"}]}