)]}'
{"neutron_tempest_plugin/api/admin/test_ports.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d5ad764bb1a4a6b512e5c541870a4540510ff513","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        if (\u0027vxlan\u0027 not in"},{"line_number":86,"context_line":"                config.CONF.neutron_plugin_options.available_type_drivers):"},{"line_number":87,"context_line":"            msg \u003d \"Skipped as VXLAN driver is not available in config\""},{"line_number":88,"context_line":"            raise cls.skipException(msg)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    @classmethod"},{"line_number":91,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_912a127c","line":88,"updated":"2020-05-20 16:14:01.000000000","message":"Question - is there a list of type drivers that is valid for these tests?  If so, can we just use one of them if present in this list?  Seems better than skipping the tests completely.","commit_id":"9c6675c64a9408b6057fc1aae4b7adcb08f44f56"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"bd0b7e6b16a9bcc5bd9bd8dafaf17398020fd423","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        if (\u0027vxlan\u0027 not in"},{"line_number":86,"context_line":"                config.CONF.neutron_plugin_options.available_type_drivers):"},{"line_number":87,"context_line":"            msg \u003d \"Skipped as VXLAN driver is not available in config\""},{"line_number":88,"context_line":"            raise cls.skipException(msg)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    @classmethod"},{"line_number":91,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff570b3c_1dc7f047","line":88,"in_reply_to":"ff570b3c_912a127c","updated":"2020-05-25 14:36:18.000000000","message":"I executed the tests on a couple of environments with VLAN and GENEVE network types supported and test test_port_resource_request_no_provider_net_conflict passes.\nThat is the only test that uses this \u0027cls.network\u0027.","commit_id":"9c6675c64a9408b6057fc1aae4b7adcb08f44f56"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"216313a61425ba7e322e7d68ffc4b7fd103ea86e","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        cls.vnic_type \u003d \u0027normal\u0027"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        # Note(lajoskatona): to avoid creating provider network use vxlan"},{"line_number":93,"context_line":"        # as provider network type:"},{"line_number":94,"context_line":"        cls.network \u003d cls.create_network(provider_network_type\u003d\u0027vxlan\u0027)"},{"line_number":95,"context_line":"        cls.physnet_name \u003d CONF.neutron_plugin_options.provider_vlans[0]"},{"line_number":96,"context_line":"        base_segm \u003d CONF.neutron_plugin_options.provider_net_base_segm_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff570b3c_228447b1","side":"PARENT","line":93,"updated":"2020-05-26 07:27:51.000000000","message":"Looking at code in https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/base.py#L287 it seems for me that this method will create \"tenant\" network by default. So it will use whatever type which is defined in the neutron\u0027s config. IMO that should be ok here. But lets ask Lajos as author of this comment.","commit_id":"130eb1a340762610af87dc1466ac56c651e85cd6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"61177761b4f6b41926aeb4eec13177a886d38cad","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        cls.vnic_type \u003d \u0027normal\u0027"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        # Note(lajoskatona): to avoid creating provider network use vxlan"},{"line_number":93,"context_line":"        # as provider network type:"},{"line_number":94,"context_line":"        cls.network \u003d cls.create_network(provider_network_type\u003d\u0027vxlan\u0027)"},{"line_number":95,"context_line":"        cls.physnet_name \u003d CONF.neutron_plugin_options.provider_vlans[0]"},{"line_number":96,"context_line":"        base_segm \u003d CONF.neutron_plugin_options.provider_net_base_segm_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff570b3c_043b109c","side":"PARENT","line":93,"in_reply_to":"ff570b3c_228447b1","updated":"2020-05-26 08:06:26.000000000","message":"I can\u0027t recall exactly why this restriction I added, perhaps it was something in my environment and I kept is for zuul as well as there was no problem with it.","commit_id":"130eb1a340762610af87dc1466ac56c651e85cd6"}]}