)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34913,"name":"Mert Kırpıcı","email":"mert.kirpici@canonical.com","username":"mertkirpici"},"change_message_id":"f0afde7c9878d250178efee35235329c18640cfe","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"eda52e83_3743a0de","updated":"2023-08-01 12:05:08.000000000","message":"Hi Martin,\nI am observing a \n\"ValueError: Nameservers list cannot be empty and it should contain DNS backend IPs to \"dig\" for\"\n\nduring some of the designate tests during the devstack-admin-plugins CI runs. Do you think this is related to the change?","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"9468332f1d96383278f92094828c504f56c08dae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"df3f669e_1097080b","updated":"2023-07-27 21:15:56.000000000","message":"check experimental","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":34913,"name":"Mert Kırpıcı","email":"mert.kirpici@canonical.com","username":"mertkirpici"},"change_message_id":"0243f74540e5dbc94dd191a2e312b84287a21c49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3b8469e6_cebf5b6a","updated":"2023-08-04 09:33:27.000000000","message":"recheck","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":34913,"name":"Mert Kırpıcı","email":"mert.kirpici@canonical.com","username":"mertkirpici"},"change_message_id":"369d35611b8197a38562a40f837c216dacdd58d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5fc73a33_ad9f374c","updated":"2023-08-01 07:30:26.000000000","message":"recheck timeout","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"002cd87266a51fd92cfc7ab3f626b9111ed66d13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bf38448f_cbbc2acc","updated":"2023-07-27 21:16:05.000000000","message":"recheck timeout","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":34913,"name":"Mert Kırpıcı","email":"mert.kirpici@canonical.com","username":"mertkirpici"},"change_message_id":"ff5cea5e41474c5da4de492dbd43d2b63311b0c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"45497209_28b47bfa","in_reply_to":"3e843ff7_4830e16b","updated":"2023-08-08 15:46:59.000000000","message":"OK, thanks for the update","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"402b6934dcbb184c45e1c97c35e7dae1f9d06a7e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3e843ff7_4830e16b","in_reply_to":"eda52e83_3743a0de","updated":"2023-08-07 09:19:35.000000000","message":"the error isn\u0027t related to this change - i\u0027m trying to resolve it here: \nhttps://review.opendev.org/c/openinfra/python-tempestconf/+/889988","commit_id":"767e287fa18cfd375abcd809afdad051aee11de5"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"43652f55fa44a5e1abcfc4bcf3b25bed5e20a4fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"75aef559_6f96448d","updated":"2023-08-14 13:52:04.000000000","message":"Looks good to me!","commit_id":"c5d0c69dae221ca8a2a33cc0b95bd4c141c21a76"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"25cbf5b9db140fb57f75755db7d4cb45ad102381","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"05c20f37_cfc9b42c","updated":"2023-08-21 11:28:56.000000000","message":"Soooo... rebase?","commit_id":"c5d0c69dae221ca8a2a33cc0b95bd4c141c21a76"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b448b3beec644570b301749148f73abb951b6271","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"923a797a_86e10950","updated":"2023-08-21 09:55:07.000000000","message":"hm, that previous comment somehow triggered the jobs o.O","commit_id":"c5d0c69dae221ca8a2a33cc0b95bd4c141c21a76"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5c5f6e531b416812420e7b21dbacb4cb0df81430","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2973c0b7_c2f93cc9","updated":"2023-08-21 09:50:47.000000000","message":"hmm, submittable but not merged? weird .. maybe a rebase would help","commit_id":"c5d0c69dae221ca8a2a33cc0b95bd4c141c21a76"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"4789bf12231259322c19aa95b41c765c9dfa02ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"53e60a76_21a89860","updated":"2023-08-11 14:08:46.000000000","message":"looks good to me and the jobs pass (experimental ones too)","commit_id":"c5d0c69dae221ca8a2a33cc0b95bd4c141c21a76"},{"author":{"_account_id":34913,"name":"Mert Kırpıcı","email":"mert.kirpici@canonical.com","username":"mertkirpici"},"change_message_id":"c360779da9877b5bcc29cf52251d85d507102490","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dc5b595c_3c291f9d","updated":"2023-08-21 15:19:01.000000000","message":"recheck","commit_id":"58e32bbe5ad6b225c6db6b14dd768fa085663fd5"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"90cc77102e1e860124f48e469da6f94a57ad792c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fd6a5907_ba70e8ec","updated":"2023-08-21 13:18:02.000000000","message":"recheck\n\nunrelated POST_FAILURE in python-tempestconf-tempest-devstack-admin-zed","commit_id":"58e32bbe5ad6b225c6db6b14dd768fa085663fd5"}],"config_tempest/main.py":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"43652f55fa44a5e1abcfc4bcf3b25bed5e20a4fd","unresolved":true,"context_lines":[{"line_number":461,"context_line":"                     \u0027admin_project_name\u0027,"},{"line_number":462,"context_line":"                     cloud_creds[\u0027project_name\u0027])"},{"line_number":463,"context_line":"            conf.set(\u0027auth\u0027, \u0027admin_password\u0027, cloud_creds[\u0027password\u0027])"},{"line_number":464,"context_line":"            for cred in ["},{"line_number":465,"context_line":"                \u0027domain_name\u0027, \u0027project_domain_name\u0027, \u0027user_domain_name\u0027"},{"line_number":466,"context_line":"            ]:"},{"line_number":467,"context_line":"                if cred in cloud_creds:"},{"line_number":468,"context_line":"                    conf.set(\u0027auth\u0027, f\u0027admin_{cred}\u0027, cloud_creds[cred])"},{"line_number":469,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9694e273_d282b5eb","line":466,"range":{"start_line":464,"start_character":24,"end_line":466,"end_character":14},"updated":"2023-08-14 13:52:04.000000000","message":"nit: I would rewrite this section like this:\n```\ncreds \u003d [\u0027domain_name\u0027, \u0027project_domain_name\u0027, \u0027user_domain_name\u0027]\nfor cred in creds:\n   ...\n```\n\nBut this is probably not that important.","commit_id":"c5d0c69dae221ca8a2a33cc0b95bd4c141c21a76"}]}
