)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"36b79940b0817d79d87afa3458c19ec503d70f04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e34255fd_aee38a59","updated":"2021-12-14 11:43:48.000000000","message":"Do we need to spend 50 seconds [1] for a test that overrides \"test_create_port_with_dns_name\"? What is the value of having both? You can (1) create the port with an \"invalid\" dns-name, (2) validate the DNS port details, (3) update the port with the correct value, (4) validate again the DNS port details (now updated) and (5) spawn the VM.\n\nIs there any value on running this new test and \"test_create_port_with_dns_name\"?\n\n[1]\n2021-12-12 11:04:55.577097 | controller | {1} neutron_tempest_plugin.scenario.test_internal_dns.InternalDNSTest.test_update_port_with_dns_name [50.455896s] ... ok","commit_id":"578ff05af58edfcee13186e4fc2fe088127ea13a"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"7210580d912b831517e77147c0a1dbaec87b7adb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5a365ada_d10c97de","in_reply_to":"e34255fd_aee38a59","updated":"2021-12-14 11:59:51.000000000","message":"I like the idea of combining both tests. Thx Rodolfo.","commit_id":"578ff05af58edfcee13186e4fc2fe088127ea13a"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"b0e2cfe6921164d39e34b7bedc45a311718d7237","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"29102aa7_c9e00e79","updated":"2021-12-14 13:50:38.000000000","message":"Thanks, done","commit_id":"ce029bb71a91276e5ba9f49a2158dc4a49205192"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"39b4bc847524a468d84e80f94d4a6307fabeb3d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"058577a2_faebc16a","updated":"2021-12-14 15:26:14.000000000","message":"recheck","commit_id":"7e0bf6183b28033033450c6d85505e6af7ff90ed"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"18c900ebb2777225fdedf3cb3d0fc80170962b64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"2b6ac306_4c732917","updated":"2021-12-15 09:40:15.000000000","message":"Waiting for a fix in those git mirrors. Patch looks ok.","commit_id":"c18067fa3489dd488d9fc80127ee28fd05182f18"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"d7af40cbbadc814d897636350b5327ee0421cb7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"00f4da04_697feb77","updated":"2021-12-15 16:52:43.000000000","message":"recheck","commit_id":"c18067fa3489dd488d9fc80127ee28fd05182f18"}],"neutron_tempest_plugin/scenario/test_internal_dns.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"d30f324272790f72d5c4af582c4693961e0be989","unresolved":true,"context_lines":[{"line_number":199,"context_line":"        self.create_router_interface(self.router[\u0027id\u0027], subnet[\u0027id\u0027])"},{"line_number":200,"context_line":"        # create port with wrong dns-name"},{"line_number":201,"context_line":"        dns_port \u003d self.create_port(network,"},{"line_number":202,"context_line":"                                    dns_name\u003d\u0027bazinga\u0027,"},{"line_number":203,"context_line":"                                    security_groups\u003d[self.secgroup[\u0027id\u0027]],"},{"line_number":204,"context_line":"                                    name\u003dself._rand_name(\u0027port\u0027))"},{"line_number":205,"context_line":"        # update port with correct dns-name (as VM name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ad3d086_1c8fabf5","line":202,"range":{"start_line":202,"start_character":46,"end_line":202,"end_character":53},"updated":"2021-12-08 21:04:43.000000000","message":"I like that name. Big bang theory rules! :D","commit_id":"97c08d97fbc67c92adfb20c3ec876cea7952426e"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"95c79cd30f1ea886f1a0ac01d7796cb2013715a5","unresolved":true,"context_lines":[{"line_number":199,"context_line":"        self.create_router_interface(self.router[\u0027id\u0027], subnet[\u0027id\u0027])"},{"line_number":200,"context_line":"        # create port with wrong dns-name"},{"line_number":201,"context_line":"        dns_port \u003d self.create_port(network,"},{"line_number":202,"context_line":"                                    dns_name\u003d\u0027bazinga\u0027,"},{"line_number":203,"context_line":"                                    security_groups\u003d[self.secgroup[\u0027id\u0027]],"},{"line_number":204,"context_line":"                                    name\u003dself._rand_name(\u0027port\u0027))"},{"line_number":205,"context_line":"        # update port with correct dns-name (as VM name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"33f42b83_2f4852ed","line":202,"range":{"start_line":202,"start_character":46,"end_line":202,"end_character":53},"in_reply_to":"1ad3d086_1c8fabf5","updated":"2021-12-09 08:08:50.000000000","message":"Yes! I\u0027m glad it was noticed :D","commit_id":"97c08d97fbc67c92adfb20c3ec876cea7952426e"}]}