)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"363d768191429f544a8327a37fa6fa9b92363a47","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix restarting cinder HA services on config change"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Specify the tripleo_ha_wrapper ansible role use a separate marker"},{"line_number":10,"context_line":"file to detect when it should restart the cinder-backup and"},{"line_number":11,"context_line":"cinder-volume services. This ensures both services get restarted"},{"line_number":12,"context_line":"when the config changes. Unless a separate marker file is used"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d24a43e9_5f9b3257","line":9,"range":{"start_line":9,"start_character":31,"end_line":9,"end_character":50},"updated":"2022-07-15 03:52:54.000000000","message":"nit: ansible role to use a","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"363d768191429f544a8327a37fa6fa9b92363a47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"93bf95a2_9b46c985","updated":"2022-07-15 03:52:54.000000000","message":"LGTM","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b703fdc7ecb6e3b0cf3c186f98cc5adaac9249e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7ed93164_044b070f","updated":"2022-07-13 14:44:52.000000000","message":"This would be specific to cinder currently because that is the only service which has two pacemaker resources sharing the same config directory.\n\nHowever wondering whether updating tripleo-ansible to add service name to state file path can be more global option and helps us prevent the same problem in case we introduce any additional resources.\n\nhttps://github.com/openstack/tripleo-ansible/blob/0cff3f9556fd6aa7b808c5d71941162fdfbfe0d3/tripleo_ansible/roles/tripleo_ha_wrapper/tasks/main.yml#L79","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f12586d894cf64fe47e66f96a41c3ab5ee36001d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"22aae356_c074b3a0","updated":"2022-07-13 17:52:45.000000000","message":"recheck\n\nMany RETRY_LIMIT and POST_FAILURE","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"b4906971216c484451abd7debe2b61d2c61d4ed5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6a4d6889_af9ca426","updated":"2022-07-14 00:53:01.000000000","message":"recheck\n\nanother POST_FAILURE","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"7609f5e92624a059210dd0d12b58544a9b60828a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a251403f_04926b2b","in_reply_to":"7ed93164_044b070f","updated":"2022-07-13 14:50:59.000000000","message":"Well, I\u0027m really trying to keep this focused, especially given the looming downstream deadline for RHOSP. Plus, the trend is to reduce the number of pcmk services, not expand them. I think the likelihood of this affecting some future TBD HA service is vanishingly small.","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"cc1fa9e8a039a619a378c50b6ba5b8ef2fcc413d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"25ff0402_84ea6a59","in_reply_to":"a251403f_04926b2b","updated":"2022-07-14 16:53:24.000000000","message":"As this requires backport to wallaby and train, I\u0027m ok with the current version to keep the change limited, which would help us avoid regressions.","commit_id":"7d76af34f54aa159b0e52d132705659be69b23a1"}]}