)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"175471cf059fbd2223c8d54faa8a32f697d6f403","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fa15151d_0a1a4824","updated":"2021-11-19 16:28:41.000000000","message":"I\u0027m still reviewing this path. However, I think it needs a release note: https://docs.openstack.org/cinder/latest/contributor/releasenotes.html","commit_id":"a68eefdb67ec41ce1f8ac38619f2bf1afa9a13ea"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"7a0b1da09639aecc87c2b824bc0287fe9c357478","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"21e8320c_fc17ad7a","in_reply_to":"fa15151d_0a1a4824","updated":"2023-03-23 15:10:39.000000000","message":"Done","commit_id":"a68eefdb67ec41ce1f8ac38619f2bf1afa9a13ea"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"16b823fec48c8ae9f306109607788741cba8468a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9348fb5a_80389d20","updated":"2023-05-19 14:47:43.000000000","message":"Hi, Pavlo, I found only a typo on release notes, but the patch looks is good to me.","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"1cd99492854e4b115c792c1c715748766c9414b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c4a422c3_1eccc390","updated":"2023-05-19 14:23:41.000000000","message":"Looks good to me - thank you!","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"1bdf7f8dda239e4436085a0d960e7cfa67d273bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e686c9e3_0b980474","updated":"2023-05-04 15:13:01.000000000","message":"recheck\n\nrefresh failure logs","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"7a0b1da09639aecc87c2b824bc0287fe9c357478","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"95c994a2_53ab261c","updated":"2023-03-23 15:10:39.000000000","message":"release note added\n\nI also took liberty and rebased this patch (as the previous version was quite old already, may not pass CI).","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"2bc8c8fd783115a5fa2d8539c89f57f67b8e6eb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f6c692ba_f6599874","updated":"2023-07-21 14:20:01.000000000","message":"typo in release notes need resolving","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f95359644f9efffafa06d47f301f5de2dbebe496","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e7df1981_0a4fb06d","updated":"2023-07-21 16:35:51.000000000","message":"Code change looks good; just need you to fix the formatting of the release note.  Thanks for the detailed bug report.","commit_id":"9c9dace97298f3e7c928e56f148cebe720f4aee2"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"4a0d203beb213a054326f6a91cd738661d28767b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0ad07605_b1b57fcd","updated":"2023-12-15 16:07:18.000000000","message":"Have had two cores look at this and say it looks reasonable.  Should be good.","commit_id":"9feee2e87b7a81da776f35d2b2454c08b0634280"},{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"2285d6df2017daa73467d9444c85ece2805dd291","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3cbb06b6_4931c3ae","updated":"2024-01-19 15:23:06.000000000","message":"Looks good to me - thank you!","commit_id":"9feee2e87b7a81da776f35d2b2454c08b0634280"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"69f034fa7e4ee3a59493d4a7a15666acbc55c755","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3f24fe68_a8bd973e","updated":"2023-07-27 18:02:47.000000000","message":"lgtm","commit_id":"9feee2e87b7a81da776f35d2b2454c08b0634280"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b244812a178038050bcdef9ecf2454e7db2235d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"35984cf1_40dc64fa","updated":"2023-11-17 15:03:31.000000000","message":"looks reasonable to do and fixing an important bug.","commit_id":"9feee2e87b7a81da776f35d2b2454c08b0634280"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"35234e30dbed58e928654bd62a2d71ad3bdf70bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9e2a38c4_26429986","updated":"2024-09-11 11:14:26.000000000","message":"recheck","commit_id":"9feee2e87b7a81da776f35d2b2454c08b0634280"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"7df3fe8d0a16143721d6773725521b867d30277d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e2f6861a_ab52fb88","updated":"2024-08-19 19:32:34.000000000","message":"run Pure Storage CI","commit_id":"9feee2e87b7a81da776f35d2b2454c08b0634280"}],"cinder/volume/volume_utils.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9c55b92c1a6b4937cd7a7abb2ddd781c2c921a11","unresolved":true,"context_lines":[{"line_number":1043,"context_line":"                 \"cinder\u0027s service context.\", encryption_key_id)"},{"line_number":1044,"context_line":"        conf \u003d CONF"},{"line_number":1045,"context_line":"        ks_loading.register_auth_conf_options(conf, \u0027keystone_authtoken\u0027)"},{"line_number":1046,"context_line":"        # NOTE(pas-ha) this will actually register conf options for the"},{"line_number":1047,"context_line":"        # specified auth_type"},{"line_number":1048,"context_line":"        ks_loading.load_auth_from_conf_options(conf, \u0027keystone_authtoken\u0027)"},{"line_number":1049,"context_line":"        service_context \u003d keystone_password.KeystonePassword("},{"line_number":1050,"context_line":"            password\u003dconf.keystone_authtoken.password,"}],"source_content_type":"text/x-python","patch_set":1,"id":"243fffb6_d4ec84f1","line":1047,"range":{"start_line":1046,"start_character":8,"end_line":1047,"end_character":29},"updated":"2021-10-15 14:57:27.000000000","message":"I\u0027m not sure this comment adds much value here -- may be better off just dropping it.","commit_id":"06c6fd636252c883846fcaf818e4b6fc7060a5c8"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"59016d45fe7c697116105f0d33367cf7f973b27f","unresolved":false,"context_lines":[{"line_number":1043,"context_line":"                 \"cinder\u0027s service context.\", encryption_key_id)"},{"line_number":1044,"context_line":"        conf \u003d CONF"},{"line_number":1045,"context_line":"        ks_loading.register_auth_conf_options(conf, \u0027keystone_authtoken\u0027)"},{"line_number":1046,"context_line":"        # NOTE(pas-ha) this will actually register conf options for the"},{"line_number":1047,"context_line":"        # specified auth_type"},{"line_number":1048,"context_line":"        ks_loading.load_auth_from_conf_options(conf, \u0027keystone_authtoken\u0027)"},{"line_number":1049,"context_line":"        service_context \u003d keystone_password.KeystonePassword("},{"line_number":1050,"context_line":"            password\u003dconf.keystone_authtoken.password,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1126d4fc_6c26e5c4","line":1047,"range":{"start_line":1046,"start_character":8,"end_line":1047,"end_character":29},"in_reply_to":"243fffb6_d4ec84f1","updated":"2021-10-18 13:36:32.000000000","message":"Done","commit_id":"06c6fd636252c883846fcaf818e4b6fc7060a5c8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f95359644f9efffafa06d47f301f5de2dbebe496","unresolved":true,"context_lines":[{"line_number":996,"context_line":"                 \"cinder\u0027s service context.\", encryption_key_id)"},{"line_number":997,"context_line":"        conf \u003d CONF"},{"line_number":998,"context_line":"        ks_loading.register_auth_conf_options(conf, \u0027keystone_authtoken\u0027)"},{"line_number":999,"context_line":"        ks_loading.load_auth_from_conf_options(conf, \u0027keystone_authtoken\u0027)"},{"line_number":1000,"context_line":"        service_context \u003d keystone_password.KeystonePassword("},{"line_number":1001,"context_line":"            password\u003dconf.keystone_authtoken.password,"},{"line_number":1002,"context_line":"            auth_url\u003dconf.keystone_authtoken.auth_url,"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf0f4d97_b682e827","line":999,"updated":"2023-07-21 16:35:51.000000000","message":"We do a similar thing in cinder/compute/nova.py when we\u0027re going to use credentials from the [nova] section of the cinder.conf file, so this looks correct to me.","commit_id":"9c9dace97298f3e7c928e56f148cebe720f4aee2"}],"releasenotes/notes/bug-1946483-cryptic-error-in-backup-fe03939577867e0a.yaml":[{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"16b823fec48c8ae9f306109607788741cba8468a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix an issue where admin user fails to delete backup of an ecrypted"},{"line_number":5,"context_line":"    volume with an ``oslo_config.cfg.NoSuchOptError`` error in logs"},{"line_number":6,"context_line":"    of cinder-backup service."},{"line_number":7,"context_line":"    With this fix cloud admin is able to delete backups of encrypted volumes"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"4c9ffcd2_09ba8c4d","line":4,"range":{"start_line":4,"start_character":63,"end_line":4,"end_character":71},"updated":"2023-05-19 14:47:43.000000000","message":"typo: encrypted","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"4179257d21ef6782fb85b2a666fe67f3494b2058","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix an issue where admin user fails to delete backup of an ecrypted"},{"line_number":5,"context_line":"    volume with an ``oslo_config.cfg.NoSuchOptError`` error in logs"},{"line_number":6,"context_line":"    of cinder-backup service."},{"line_number":7,"context_line":"    With this fix cloud admin is able to delete backups of encrypted volumes"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"60fe018e_640a1d7d","line":4,"range":{"start_line":4,"start_character":63,"end_line":4,"end_character":71},"in_reply_to":"4c9ffcd2_09ba8c4d","updated":"2023-07-23 18:19:12.000000000","message":"Done","commit_id":"35c89a121a0eacb5218ebdd289a9dd9ca0e97e58"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f95359644f9efffafa06d47f301f5de2dbebe496","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    of cinder-backup service."},{"line_number":7,"context_line":"    With this fix cloud admin is able to delete backups of encrypted volumes"},{"line_number":8,"context_line":"    created by other users if Barbican API policies allow it."},{"line_number":9,"context_line":"    For more details see https://bugs.launchpad.net/cinder/+bug/1946483."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f5d50fc1_2013dcb1","line":9,"updated":"2023-07-21 16:35:51.000000000","message":"Could you please put this in the required format as described here:\nhttps://docs.openstack.org/cinder/latest/contributor/releasenotes.html#bugs\n\nThanks!","commit_id":"9c9dace97298f3e7c928e56f148cebe720f4aee2"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"a26794c3e0774a2b85f51b2db0ec52904091857f","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    of cinder-backup service."},{"line_number":7,"context_line":"    With this fix cloud admin is able to delete backups of encrypted volumes"},{"line_number":8,"context_line":"    created by other users if Barbican API policies allow it."},{"line_number":9,"context_line":"    For more details see https://bugs.launchpad.net/cinder/+bug/1946483."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f430a64f_007a99eb","line":9,"in_reply_to":"f5d50fc1_2013dcb1","updated":"2023-07-23 18:18:32.000000000","message":"Done","commit_id":"9c9dace97298f3e7c928e56f148cebe720f4aee2"}]}
