)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"aa769369e4bf0fc1c9f8b5a4f863b7940cf2ba28","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     katarimanoj \u003cmkatari@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-09-14 19:22:59 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch c-bak default to A/A"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, cinder-backup deployment defaults to A/P."},{"line_number":10,"context_line":"This patch will update the environment file cinder-backup.yaml"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b6fab518_b6ef6082","line":7,"updated":"2022-09-16 18:57:20.000000000","message":"Use \"cinder-backup\" (c-bak is a devstack term)","commit_id":"8b0aa7b1c3ff9450e0e124928d27fb721844f6fe"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"b8e1454619568a72f8f170e1e2bb22aa78edd8ab","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     katarimanoj \u003cmkatari@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-09-14 19:22:59 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch c-bak default to A/A"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, cinder-backup deployment defaults to A/P."},{"line_number":10,"context_line":"This patch will update the environment file cinder-backup.yaml"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c49aac7e_963d0a3b","line":7,"in_reply_to":"b6fab518_b6ef6082","updated":"2022-09-19 05:56:58.000000000","message":"Done","commit_id":"8b0aa7b1c3ff9450e0e124928d27fb721844f6fe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"aa769369e4bf0fc1c9f8b5a4f863b7940cf2ba28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6ee2b77a_7ddb5c2d","updated":"2022-09-16 18:57:20.000000000","message":"This patch definitely needs a release note. The feature section should mention that when the cinder-backup service is deployed, it now defaults to A/A and not A/P. The upgrade section is important. It should state the service will be switched from A/P to A/A if the service was originally deployed using the cinder-backup.yaml env file. Cloud administrators who wish to keep it A/P should use the new cinder-backup-active-passive.yaml env file.","commit_id":"8b0aa7b1c3ff9450e0e124928d27fb721844f6fe"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"b8e1454619568a72f8f170e1e2bb22aa78edd8ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3bfb48ca_3d263315","in_reply_to":"6ee2b77a_7ddb5c2d","updated":"2022-09-19 05:56:58.000000000","message":"Done","commit_id":"8b0aa7b1c3ff9450e0e124928d27fb721844f6fe"}],"environments/cinder-backup-active-passive.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"aa769369e4bf0fc1c9f8b5a4f863b7940cf2ba28","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3e287c92_6d67d829","side":"PARENT","updated":"2022-09-16 18:57:20.000000000","message":"You need to add the new file, and retain the existing one alone in order to not break deployments that are using it.","commit_id":"ab4004e4aaaceb2854905f6e885a5e61ff894b49"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"b8e1454619568a72f8f170e1e2bb22aa78edd8ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3ade97a7_49a76028","side":"PARENT","in_reply_to":"3e287c92_6d67d829","updated":"2022-09-19 05:56:58.000000000","message":"Done","commit_id":"ab4004e4aaaceb2854905f6e885a5e61ff894b49"}],"releasenotes/notes/cinder_backup_default_switch_to_A-A-4adb439183be78f0.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"2592bf51d51fc521d0587d7adef95d122072d958","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    When the cinder-backup service is deployed, it now defaults to A/A"},{"line_number":5,"context_line":"    and not A/P."},{"line_number":6,"context_line":"upgrade:"},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Cinder backup service will be switched from A/P to A/A if the service"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"91e91bbe_200447fd","line":5,"range":{"start_line":4,"start_character":67,"end_line":5,"end_character":16},"updated":"2022-09-22 16:59:19.000000000","message":"Everything looks great, and my remaining doubt is that the release note should spell out \"active-active\" and \"active-passive\" instead of using the A/A and A/P terminology. I don\u0027t think we should rely on cloud administrators being familiar with our engineering shorthand.","commit_id":"26a47d10bca7025ff61da07526ba89b20e22ca9a"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"6bc5fa3f40835444d9e948f71a78f005cac6327e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    When the cinder-backup service is deployed, it now defaults to A/A"},{"line_number":5,"context_line":"    and not A/P."},{"line_number":6,"context_line":"upgrade:"},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Cinder backup service will be switched from A/P to A/A if the service"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8e09cf38_be618110","line":5,"range":{"start_line":4,"start_character":67,"end_line":5,"end_character":16},"in_reply_to":"91e91bbe_200447fd","updated":"2022-09-23 15:08:47.000000000","message":"Thank you for your insight.","commit_id":"26a47d10bca7025ff61da07526ba89b20e22ca9a"}]}
