)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"a401fd87179901fbb13a01db48ef2000fea69bb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"45ab1783_6ac5d113","updated":"2022-04-19 18:42:56.000000000","message":"I\u0027m seeing an issue with a circular import somewhere in it, \n2022-04-19 15:47:50.410 3168483 INFO tempest [-] Using tempest config file /opt/stack/tempest/etc/tempest.conf                                                    \n   2 2022-04-19 15:47:50.411 3168483 ERROR tempest.test_discover.plugins [-] Could not load \u0027designate\u0027: partially initialized module \u0027designate_tempest_plugin.plugin\u0027      has no attribute \u0027DesignateTempestPlugin\u0027 (most likely due to a circular import)\n   3 202","commit_id":"8b3e7efdeb401f81c6a95412dee6d8e100eff7d3"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"75046e3baccfa4277fd1878967ac988c0fd5ec8b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"44cc5e66_08bd9b17","in_reply_to":"45ab1783_6ac5d113","updated":"2022-04-20 08:13:09.000000000","message":"Michael has found the problem, I\u0027m going to fix that.\nThanks!","commit_id":"8b3e7efdeb401f81c6a95412dee6d8e100eff7d3"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"05be2c995beab15a8e161baee4924fa869369bea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"abc423c4_f6581297","updated":"2022-05-10 22:04:57.000000000","message":"LGTM","commit_id":"6d6b27d53764e885dd030dc5b6296043808508e6"},{"author":{"_account_id":19298,"name":"Nicolas Bock","email":"nicolas.bock@canonical.com","username":"nicolasbock"},"change_message_id":"d944f588e9ea76572174691fa4480754a655b2d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b000e83e_1088eb6a","updated":"2022-05-11 14:27:15.000000000","message":"LGTM","commit_id":"6d6b27d53764e885dd030dc5b6296043808508e6"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"9e061fda8352544af2c77e2976c3072c7ddd1795","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"50022534_85c6a1d0","updated":"2022-04-25 21:19:38.000000000","message":"There are a couple unrelated issues being addressed in this patch. TLDs, \u0027_\u0027 and reorganizing the tld tests into their own files. It is better to put into a patch each item where they are left to their own verses stacking many changes into a single patch.","commit_id":"6d6b27d53764e885dd030dc5b6296043808508e6"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"67fac273069972d4b5a2d45c9be76a10a7a5fb1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"618be933_31b265e6","in_reply_to":"50022534_85c6a1d0","updated":"2022-04-26 08:27:35.000000000","message":"Hi Don!\n\nAll patch\u0027s changes do related to TLDs and as agreed in the past, we are fixing \"underscores\" issues in a particular patch\u0027s related files only.\n\nThanks!","commit_id":"6d6b27d53764e885dd030dc5b6296043808508e6"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"1c1719d5308b2145e6f323d5013d18696a03b66a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"dac236cf_ff76db5f","in_reply_to":"618be933_31b265e6","updated":"2022-04-27 21:28:44.000000000","message":"Understand, but it would be nice to address one issue in a patch.","commit_id":"6d6b27d53764e885dd030dc5b6296043808508e6"}],"designate_tempest_plugin/data_utils.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6a61fbe128b71abcedcefc9de605f8a5b82fcd25","unresolved":true,"context_lines":[{"line_number":268,"context_line":"    return \u0027\u0027.join(random.choice(ascii_lowercase) for _ in range(size))"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"def rand_domain_name(tld\u003dCONF.dns.tld_suffix):"},{"line_number":272,"context_line":"    \"\"\"Create random valid domain name"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    :param tld (optional) - TLD that will be used to random domain name"}],"source_content_type":"text/x-python","patch_set":5,"id":"0aea96aa_f4a46a98","line":271,"updated":"2022-04-19 18:39:20.000000000","message":"You can\u0027t access CONF here.\nChange this to tld\u003dNone, then set it from CONF if it\u0027s None down below.","commit_id":"8b3e7efdeb401f81c6a95412dee6d8e100eff7d3"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"75046e3baccfa4277fd1878967ac988c0fd5ec8b","unresolved":true,"context_lines":[{"line_number":268,"context_line":"    return \u0027\u0027.join(random.choice(ascii_lowercase) for _ in range(size))"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"def rand_domain_name(tld\u003dCONF.dns.tld_suffix):"},{"line_number":272,"context_line":"    \"\"\"Create random valid domain name"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    :param tld (optional) - TLD that will be used to random domain name"}],"source_content_type":"text/x-python","patch_set":5,"id":"a1c4603d_b3e66dac","line":271,"in_reply_to":"0aea96aa_f4a46a98","updated":"2022-04-20 08:13:09.000000000","message":"Yep, that\u0027s correct.\nWill be fixed in the next patch","commit_id":"8b3e7efdeb401f81c6a95412dee6d8e100eff7d3"}]}