)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"3c625534e35f58128d4d302ff3082c30fc79b7e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bf270a74_60decc11","updated":"2023-08-08 10:43:09.000000000","message":"Looks good to me! Thank you!","commit_id":"c67d4f2faf175d579590b4f20a24461b4134b21d"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"9be2b8e0abb951429b7f6567913b6acfdc51193e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"13a5bc5f_ae55ddd4","updated":"2023-08-07 15:06:50.000000000","message":"This looks good to me! Waiting for the job results for +1.\n\nI think that skipping the test is the best what we can do here. If we skip the cleanup like suggested here [1], it might cause troubles when pre-prov creds are used (I think). \n\nAlso, a same user is used by two test cases which do the password update. This could be avoided by using an extra user but we will probably encounter the pre-prov creds problem.\n\n[1] https://review.opendev.org/c/openstack/tempest/+/838726","commit_id":"c67d4f2faf175d579590b4f20a24461b4134b21d"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"513f21032ab40df6479b122cab4f9b4c3aea6511","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8f5bdc75_fd383740","updated":"2023-08-08 17:06:04.000000000","message":"lgtm, can you please add the release notes for the new config option\n\nexample: https://review.opendev.org/c/openstack/tempest/+/860722/13/releasenotes/notes/image_multiple_locations-cda4453567953c1d.yaml","commit_id":"c67d4f2faf175d579590b4f20a24461b4134b21d"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"c1d15b89c70dd65d2a3d87bfecaba634ff1fc4bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5d95392b_ba3c3cc0","updated":"2023-08-08 07:50:09.000000000","message":"recheck\nThe failure in tempest was not caused by the patch","commit_id":"c67d4f2faf175d579590b4f20a24461b4134b21d"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"3d42c010d8399522317f5bf507d5c6fad2eb473d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c28e11a7_6c404a11","updated":"2023-08-09 09:18:35.000000000","message":"I added the release notes, hope it is correctly written.","commit_id":"13e2ae5a34b62a23ef866fbf1221a2104b370e27"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f837be4465b063e616bfdddbad86c565e098fda7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"38df3196_0380389f","updated":"2023-08-09 16:05:43.000000000","message":"one minor missing of config option name in releae notes otherwise lgtm","commit_id":"13e2ae5a34b62a23ef866fbf1221a2104b370e27"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"82423eb0a1866a04ccd7d572eb352a24c8d582cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b5f16deb_ec117b7b","updated":"2023-08-10 07:27:27.000000000","message":"Looks good to me!","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"2821c146db1482e228084e05649324c3139c28f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"23836b61_836b1525","updated":"2023-08-10 08:05:47.000000000","message":"lgtm, thanks!","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"a71db2a95c4cfb08c209d268d52dd180707b2e46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3af1dd17_784d4896","updated":"2023-08-14 07:27:03.000000000","message":"recheck post failure","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"2772e5a06c5930d3bbff5e0d58bc3b06c78e4d92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8582ea10_346ef377","updated":"2023-08-12 07:41:58.000000000","message":"recheck random issue","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ca0a9a8a4011747c937c046c6619a6c30aff1c04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d8e29e6f_1fa8b4b2","updated":"2023-08-13 20:17:04.000000000","message":"recheck random issue","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8042d92751414b0e969653b9ebcb65db0112cc95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6bbe69d4_8f9efc0f","updated":"2023-08-10 16:39:50.000000000","message":"recheck test_rebuild_server_with_volume_attached failing for detach_volume in cleanup. we are already improving this test in https://review.opendev.org/c/openstack/tempest/+/890821","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"0555ca0f5390af86d4e550926e422f20018fd0c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5bee2e16_9a118602","updated":"2023-08-14 12:03:07.000000000","message":"recheck timeout","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"15e57af5e0c78f6bdb6ce48fd25da5285810e96a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a2784b97_00e84bc6","updated":"2023-08-11 14:13:35.000000000","message":"recheck unrelated error","commit_id":"702f0befcea128ac2bc95bad787d592056f18ad4"}],"releasenotes/notes/add-keystone-config-opt-minimum-password-age-426e9d225f743137.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f837be4465b063e616bfdddbad86c565e098fda7","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Adding a new config option `[identity]/user_minimum_password_age`"},{"line_number":5,"context_line":"    which allows to specify the number of days that a password"},{"line_number":6,"context_line":"    must be used before the user can change it. For this option"},{"line_number":7,"context_line":"    to take effect, identity-feature-enabled must be set to True."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f42b417c_3f3472d6","line":7,"range":{"start_line":7,"start_character":20,"end_line":7,"end_character":44},"updated":"2023-08-09 16:05:43.000000000","message":"you mean identity-feature-enabled.security_compliance ?","commit_id":"13e2ae5a34b62a23ef866fbf1221a2104b370e27"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8042d92751414b0e969653b9ebcb65db0112cc95","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    Adding a new config option `[identity]/user_minimum_password_age`"},{"line_number":5,"context_line":"    which allows to specify the number of days that a password"},{"line_number":6,"context_line":"    must be used before the user can change it. For this option"},{"line_number":7,"context_line":"    to take effect, identity-feature-enabled must be set to True."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"fcdeb002_c2e4eac9","line":7,"range":{"start_line":7,"start_character":20,"end_line":7,"end_character":44},"in_reply_to":"530f4151_2be35d34","updated":"2023-08-10 16:39:50.000000000","message":"Done","commit_id":"13e2ae5a34b62a23ef866fbf1221a2104b370e27"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"42131a290b561910f3257cdb333e8062678f6070","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Adding a new config option `[identity]/user_minimum_password_age`"},{"line_number":5,"context_line":"    which allows to specify the number of days that a password"},{"line_number":6,"context_line":"    must be used before the user can change it. For this option"},{"line_number":7,"context_line":"    to take effect, identity-feature-enabled must be set to True."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"530f4151_2be35d34","line":7,"range":{"start_line":7,"start_character":20,"end_line":7,"end_character":44},"in_reply_to":"f42b417c_3f3472d6","updated":"2023-08-10 07:23:48.000000000","message":"You\u0027re right, thank you for the correction.","commit_id":"13e2ae5a34b62a23ef866fbf1221a2104b370e27"}]}