)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a76d57349887792a5c622bdab105df314f3c1a53","unresolved":true,"context_lines":[{"line_number":21,"context_line":"for everyone else."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Closes-Bug: 1988441"},{"line_number":24,"context_line":"Closes-Bug: 1988453"},{"line_number":25,"context_line":"Change-Id: If59a3d4cf77a6995d60fc2db7f21739cac5f72aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7480d639_1c64156e","line":24,"range":{"start_line":24,"start_character":0,"end_line":24,"end_character":19},"updated":"2022-09-04 14:20:26.000000000","message":"The error in bug 1988453 indicates the issue is more related to podman rather than the chown command itself. We can merge this now because this change is a valid improvement but we need to ensure whether this is the real cause of that error. I suspect the container failed to start because of the deleted file during relabeling process.","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"1dfd9f86aa08c17cdfe8cfe3a66f7221f447250e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2ff5fe6c_23ef9f8d","updated":"2023-02-18 13:49:19.000000000","message":"Couldn\u0027t we just make it a boolean and if it\u0027s being upgrade, we do it and if it\u0027s not an upgrade, we skip it ?   Or maybe a bit of both ?  Or why not running this in background with \"\u0026\" or async job ?","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"8127282d467fbdbde5432fe92d69c769c573e388","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2f792c09_696d0231","updated":"2022-09-02 07:35:34.000000000","message":"Looks like this hit https://bugs.launchpad.net/tripleo/+bug/1988500","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"3311c727eaea610a2ccc4081af28ca5fce09162e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c2821da1_a19b172a","updated":"2022-09-01 15:04:37.000000000","message":"it makes more sense now, thanks","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"e7742d221fd847d5be0be6445a2f3277fa733807","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6beeaef2_09da8caf","updated":"2022-09-02 06:45:04.000000000","message":"recheck","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"141e99371b0652a1188725f4e18a6b6188f7b378","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6e9281ec_eeb3c835","updated":"2022-09-03 18:22:33.000000000","message":"recheck","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"b68dd9ce9839f7fadddf53c8e646753d06dda07d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d836b8a5_ad6e047f","updated":"2022-09-04 11:25:16.000000000","message":"recheck","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"212e5e0ac579526367e9dbe0cab0240e80f25c70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c72c1ada_0654d26b","updated":"2022-09-01 18:17:33.000000000","message":"recheck\n\nErrors look unrelated to this change.","commit_id":"52c95b1d15631aff22d9d9efd943144676b44f8b"}],"deployment/swift/swift-storage-container-puppet.yaml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"608c36b44618a28c96b50df80f1e5a2deebecbe2","unresolved":true,"context_lines":[{"line_number":384,"context_line":"          # The puppet config sets this up but we don\u0027t have a way to mount the named"},{"line_number":385,"context_line":"          # volume during the configuration stage.  We just need to create this"},{"line_number":386,"context_line":"          # directory and make sure it\u0027s owned by swift."},{"line_number":387,"context_line":"          swift_setup_srv:"},{"line_number":388,"context_line":"            image: \u0026swift_account_image {get_attr: [RoleParametersValue, value, ContainerSwiftAccountImage]}"},{"line_number":389,"context_line":"            net: none"},{"line_number":390,"context_line":"            user: root"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e07e5d3c_d1b6e2fd","line":387,"range":{"start_line":387,"start_character":10,"end_line":387,"end_character":25},"updated":"2022-09-01 14:48:37.000000000","message":"or maybe this init container could be omitted in favor of adding \u0027owner: swift\u0027 here? https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/swift/swift-proxy-container-puppet.yaml#L468","commit_id":"fa8fcc03adcf7255d705fb10366e12a2c0bdd16c"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"3311c727eaea610a2ccc4081af28ca5fce09162e","unresolved":false,"context_lines":[{"line_number":384,"context_line":"          # The puppet config sets this up but we don\u0027t have a way to mount the named"},{"line_number":385,"context_line":"          # volume during the configuration stage.  We just need to create this"},{"line_number":386,"context_line":"          # directory and make sure it\u0027s owned by swift."},{"line_number":387,"context_line":"          swift_setup_srv:"},{"line_number":388,"context_line":"            image: \u0026swift_account_image {get_attr: [RoleParametersValue, value, ContainerSwiftAccountImage]}"},{"line_number":389,"context_line":"            net: none"},{"line_number":390,"context_line":"            user: root"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"48805eb7_c214563f","line":387,"range":{"start_line":387,"start_character":10,"end_line":387,"end_character":25},"in_reply_to":"e07e5d3c_d1b6e2fd","updated":"2022-09-01 15:04:37.000000000","message":"Done","commit_id":"fa8fcc03adcf7255d705fb10366e12a2c0bdd16c"}]}