)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"57142d33d25f1bf26c5da0cfac977ebefea80962","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9037cc86_72734584","updated":"2023-01-12 11:28:47.000000000","message":"Hi Alan,\n\nAbout your question, the post_upgrade_tasks don\u0027t have default become so you need it for nearly everything. You can see an example into deployment/ceph-ansible/ceph-osd.yaml for ceph.","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"7807a75e2d6f686ff4a3d0f2ca521d39ded2f6f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d5acedf1_965c0e46","updated":"2023-01-11 12:35:37.000000000","message":"Looks good to merge.","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"780005abc3a5f8fac28632d9d72620e259cdc381","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3b590812_5ef033cf","updated":"2023-01-12 14:24:46.000000000","message":"Thanks a lot Alan for the merge.","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"23d24449135122476addbb5c5c07ceb3dc6847e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"04b16428_6943f4fa","updated":"2022-11-07 11:11:08.000000000","message":"This THT uses the pacemaker_resource in several other places, and none of them need to become root. Why does the one in the post_upgrade_tasks require it?\n\nWhat about cinder-backup and manila-share? They have pcmk variants, and their THT don\u0027t even have post_upgrade_tasks. If cinder-volume needs them they I would think the others would as well.","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"c27e5b430cb88fbc1037d17c6a75612421638e1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"05a0d363_36416f07","updated":"2023-01-18 10:14:33.000000000","message":"recheck","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"ea8b08811bd18f1782b5a6d376d5d6a52b83db69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c3588c87_3605a83e","updated":"2023-01-17 07:57:31.000000000","message":"recheck","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"bfa9c4c23e91c69764d897c50e2b75669856cb28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"59f29a40_72e197e2","updated":"2023-01-17 17:19:13.000000000","message":"recheck dep fail","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"eca7a906cede947288ba79e72ac6417bb6653dd8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bf855f84_333711d1","in_reply_to":"9037cc86_72734584","updated":"2023-01-12 13:54:34.000000000","message":"Ack, thanks for the explanation.\n\nI still don\u0027t understand why cinder-volume needs post_upgrade_tasks to starts the service, but the cinder-backup and manila-share services that also run under pacemaker don\u0027t. I traced the cinder-volume code back to [1], but couldn\u0027t find any earlier references.\n\n[1] https://review.opendev.org/c/openstack/tripleo-heat-templates/+/626937/16/deployment/cinder/cinder-volume-pacemaker-puppet.yaml\n\nRegardless of that mystery, I\u0027ll move this patch along.","commit_id":"04519477eb255b6ecd9b3a54caa64a2fd1228d7b"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"5597b4c1762dac4333d09271fc92549adc2111aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d2479aeb_828c4dc1","updated":"2023-01-19 02:49:56.000000000","message":"The job tripleo-ci-centos-9-scenario001-standalone failed at:\n\nhttps://ddbcb83631d115022341-6698722f229587210f5dee93671360ee.ssl.cf2.rackcdn.com/863821/2/check/tripleo-ci-centos-9-scenario001-standalone/4dca09b/logs/undercloud/var/log/tempest/failing_tests.log\n\ntempest.scenario.test_snapshot_pattern.TestSnapshotPattern.test_snapshot_pattern[compute,id-608e604b-1d63-4a82-8e3e-91bc665c90b4,image,network,slow]\ntearDownClass (tempest.scenario.test_snapshot_pattern.TestSnapshotPattern)\ntempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern.test_volume_boot_pattern[compute,id-557cd2c2-4eb8-4dce-98be-f86765ff311b,image,slow,volume]\ntearDownClass (tempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern)","commit_id":"e7621f2b63459042d9dc4b7ed8711870e0551415"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"9e5a3da0546cc33a8f0c7d77d71443907c56d5e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c4918e5b_cb1841cd","updated":"2023-01-19 02:50:10.000000000","message":"recheck","commit_id":"e7621f2b63459042d9dc4b7ed8711870e0551415"}]}