)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"4cc53f539b8e8b2841263a29038d856e51f880df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6e840db2_c1c515ed","updated":"2024-08-15 14:37:24.000000000","message":"run Pure Storage CI","commit_id":"16657fd3759bb5f1afbc3b116104d77ed04b0466"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"0e4c649aa8285802b8cecc7e9d71162c78fe46e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7afc9106_e5c8cd9e","updated":"2024-08-15 14:24:45.000000000","message":"run Pure Storage CI","commit_id":"16657fd3759bb5f1afbc3b116104d77ed04b0466"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"4d93689cbfffdc70eaea7d4833d8aaef135845be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"90e2ebd7_c92ed4b8","updated":"2024-08-15 21:46:56.000000000","message":"run Pure Storage CI","commit_id":"16657fd3759bb5f1afbc3b116104d77ed04b0466"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"a44b2ec09f187073eb523b873fa676a2d775b624","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e7345c96_0bd03f42","updated":"2024-08-14 16:22:46.000000000","message":"run Pure Storage CI","commit_id":"16657fd3759bb5f1afbc3b116104d77ed04b0466"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a099923475d4f17d08ea628fa4746e2c828383d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d2ad5bf5_09539aa8","updated":"2024-08-21 22:23:23.000000000","message":"Typo noted inline if you have to push a revision.\n\nCode and tests LGTM; logging looks appropriate.\n\nThe cinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes tests are passing, so it looks like this change does not cause a regression.","commit_id":"d436d39b6e3ac6cbc7c1c972446075b8b6c6f185"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7ead8f3eedeac216974f0837db6df8c5c1c0248a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ac00679d_9fa7ac9a","updated":"2024-08-21 14:47:15.000000000","message":"at today\u0027s cinder meeting: this patch was determined release-critical for Dalmatian","commit_id":"d436d39b6e3ac6cbc7c1c972446075b8b6c6f185"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"8f38f298a628237113a20754e9c517a49330d3b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fb9c0a32_6f52bac8","updated":"2024-08-22 19:14:09.000000000","message":"Code looks good to me, docstrings are helpful, test coverage looks good, CI indicates no regressions.","commit_id":"2b42257eff03615816dc66d3171b87c22ad67eb1"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b069caf96a0b75a0e85d4a6fd910a3a91ebbb548","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f612e559_d0a63e50","updated":"2024-08-22 12:28:56.000000000","message":"Revision LGTM.","commit_id":"2b42257eff03615816dc66d3171b87c22ad67eb1"},{"author":{"_account_id":27665,"name":"Markus Hentsch","email":"markus.hentsch@cloudandheat.com","username":"mhen"},"change_message_id":"83750cae727032a32c901c834ab18db0a4502f54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1d0783fc_c8261c12","updated":"2024-08-22 08:01:42.000000000","message":"Thanks you for the review Brian! I fixed the typo you noticed.","commit_id":"2b42257eff03615816dc66d3171b87c22ad67eb1"}],"os_brick/utils.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a099923475d4f17d08ea628fa4746e2c828383d4","unresolved":true,"context_lines":[{"line_number":474,"context_line":"    type is \u0027passphrase\u0027 explicitly)."},{"line_number":475,"context_line":""},{"line_number":476,"context_line":"    :param key: Key Manager Secret containing the encryption key"},{"line_number":477,"context_line":"    :type key: castellan.common.objects.maanged_object.ManagedObject"},{"line_number":478,"context_line":"    :return: passphrase"},{"line_number":479,"context_line":"    :rtype: str"},{"line_number":480,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"7f1d2957_85771717","line":477,"range":{"start_line":477,"start_character":40,"end_line":477,"end_character":47},"updated":"2024-08-21 22:23:23.000000000","message":"nit: managed","commit_id":"d436d39b6e3ac6cbc7c1c972446075b8b6c6f185"},{"author":{"_account_id":27665,"name":"Markus Hentsch","email":"markus.hentsch@cloudandheat.com","username":"mhen"},"change_message_id":"83750cae727032a32c901c834ab18db0a4502f54","unresolved":false,"context_lines":[{"line_number":474,"context_line":"    type is \u0027passphrase\u0027 explicitly)."},{"line_number":475,"context_line":""},{"line_number":476,"context_line":"    :param key: Key Manager Secret containing the encryption key"},{"line_number":477,"context_line":"    :type key: castellan.common.objects.maanged_object.ManagedObject"},{"line_number":478,"context_line":"    :return: passphrase"},{"line_number":479,"context_line":"    :rtype: str"},{"line_number":480,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"0d1a6d60_20e858e3","line":477,"range":{"start_line":477,"start_character":40,"end_line":477,"end_character":47},"in_reply_to":"7f1d2957_85771717","updated":"2024-08-22 08:01:42.000000000","message":"Fixed. Thank you for spotting this!","commit_id":"d436d39b6e3ac6cbc7c1c972446075b8b6c6f185"}]}
