)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"547090eafc15368e7386aca8dd3d3442aff7528a","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As tap-as-a-service was moved under Neutron governance its tests should"},{"line_number":10,"context_line":"be in neutron-tempest-plugin."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3055d8512c099eea2e25376f3525d96a040e70fa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aceb717_9f707567","line":11,"updated":"2021-10-21 09:25:23.000000000","message":"Perhaps a few line that this is not a copy-paste, but reworked tests and classes to adopt them to current n-t-p and remove the duplications.","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As tap-as-a-service was moved under Neutron governance its tests should"},{"line_number":10,"context_line":"be in neutron-tempest-plugin."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3055d8512c099eea2e25376f3525d96a040e70fa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b3e4ee37_b7fc7fba","line":11,"in_reply_to":"3aceb717_9f707567","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7b7f518cc9fe632227f42454ac43a8012d384348","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"836e7f9b_6aa4b993","updated":"2021-11-22 11:03:02.000000000","message":"Why there are some skipped tests [1]? \n\n[1]https://384cfbf25b69835aeffb-93a25d8b6838f520f662813cb255d034.ssl.cf2.rackcdn.com/814876/13/check/neutron-tempest-plugin-tap-as-a-service/55e539f/testr_results.html","commit_id":"852807115511dee6a39f8f347d1956b77b8dcd0a"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"aa0f4abe53c7e55a62371bb60524ae451993180f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"01822991_63ccec36","updated":"2021-11-24 13:59:02.000000000","message":"As Rodolfo pointed 3 tests are skipped, for example: \"test_taas_forwarded_traffic_provider_net_negative ... SKIPPED: provider_physical_network not provided\" - though I see this parameter defined in job","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1ae6498ae1d49531d3f368f0317635a48c7ae3eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"a11d0bfb_03624c08","in_reply_to":"01822991_63ccec36","updated":"2021-11-26 14:16:41.000000000","message":"hmm, I think now the config will be ok, though I am interested what will happen as those tests are for sriov, but perhaps \"plain\" providernet will be ok for them here.","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"72811b26cd0cfc13da6e320626cd9beb49f0c4a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"01318c2b_c8844e13","updated":"2021-12-14 08:54:43.000000000","message":"recheck","commit_id":"6e7da1a8dec9a8de44cca5d69bc8f60600971779"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a308b42eff10829f06359abef2187996d877e861","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"0aae4848_cf84e27e","updated":"2021-12-08 13:01:35.000000000","message":"recheck","commit_id":"6e7da1a8dec9a8de44cca5d69bc8f60600971779"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"6d1182e01e54de36e2617b2d59cedc50a587e909","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"5838d6aa_f341743b","updated":"2021-12-13 09:36:13.000000000","message":"recheck\nSeems to be on victoria designate this hits: https://bugs.launchpad.net/neutron/+bug/1944201","commit_id":"6e7da1a8dec9a8de44cca5d69bc8f60600971779"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"de9df77dc26317e10dd5309c7ff95af6f930b4f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"47b11d34_f21a16d8","updated":"2021-12-13 15:50:24.000000000","message":"recheck\nglance-api cannot start","commit_id":"6e7da1a8dec9a8de44cca5d69bc8f60600971779"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"1b1b4911edba79827f5390b83006ac8b4082d1e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"7a3d70e3_73cc63ac","updated":"2022-03-16 23:18:55.000000000","message":"I\u0027ll let Rodolfo approve the patch since he indicated he wants to review the logs again","commit_id":"1f275e4b0d8c108ae36dc50aa5cbd981159b3bbc"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3f8c1f156bc4d9659f93cdbde38e8118270f224b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"57218da9_0b818bca","updated":"2022-03-16 16:13:13.000000000","message":"recheck\n\nI would like to check the logs again","commit_id":"1f275e4b0d8c108ae36dc50aa5cbd981159b3bbc"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"215b49ddbea915736e1312bfaa1ccfacf0d6ad32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"3a766f15_04ebc410","in_reply_to":"57218da9_0b818bca","updated":"2022-03-17 12:09:43.000000000","message":"+1 thanks (I should have done that)","commit_id":"1f275e4b0d8c108ae36dc50aa5cbd981159b3bbc"}],"neutron_tempest_plugin/config.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"547090eafc15368e7386aca8dd3d3442aff7528a","unresolved":true,"context_lines":[{"line_number":218,"context_line":"TaasGroup \u003d ["},{"line_number":219,"context_line":"    cfg.StrOpt(\u0027provider_physical_network\u0027,"},{"line_number":220,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":221,"context_line":"               help\u003d\u0027Physical network to be used for creating SRIOV network.\u0027),"},{"line_number":222,"context_line":"    cfg.StrOpt(\u0027provider_segmentation_id\u0027,"},{"line_number":223,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":224,"context_line":"               help\u003d\u0027Segmentation-id to be used for creating SRIOV network.\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"ae0cdcf8_77321e24","line":221,"range":{"start_line":221,"start_character":21,"end_line":221,"end_character":76},"updated":"2021-10-21 09:25:23.000000000","message":"This is not only for SRIOV, but for \"barefoot\" provider network (at least if I understand well what we test in these tests, and how taas works)","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":218,"context_line":"TaasGroup \u003d ["},{"line_number":219,"context_line":"    cfg.StrOpt(\u0027provider_physical_network\u0027,"},{"line_number":220,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":221,"context_line":"               help\u003d\u0027Physical network to be used for creating SRIOV network.\u0027),"},{"line_number":222,"context_line":"    cfg.StrOpt(\u0027provider_segmentation_id\u0027,"},{"line_number":223,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":224,"context_line":"               help\u003d\u0027Segmentation-id to be used for creating SRIOV network.\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"37aec44c_46c15eae","line":221,"range":{"start_line":221,"start_character":21,"end_line":221,"end_character":76},"in_reply_to":"ae0cdcf8_77321e24","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"547090eafc15368e7386aca8dd3d3442aff7528a","unresolved":true,"context_lines":[{"line_number":221,"context_line":"               help\u003d\u0027Physical network to be used for creating SRIOV network.\u0027),"},{"line_number":222,"context_line":"    cfg.StrOpt(\u0027provider_segmentation_id\u0027,"},{"line_number":223,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":224,"context_line":"               help\u003d\u0027Segmentation-id to be used for creating SRIOV network.\u0027),"},{"line_number":225,"context_line":"    cfg.StrOpt(\u0027vlan_filter\u0027,"},{"line_number":226,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":227,"context_line":"               help\u003d\u0027Comma separated list of VLANs to be mirrored \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"c6c09c8c_11b2eb86","line":224,"range":{"start_line":224,"start_character":21,"end_line":224,"end_character":75},"updated":"2021-10-21 09:25:23.000000000","message":"ditto","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":221,"context_line":"               help\u003d\u0027Physical network to be used for creating SRIOV network.\u0027),"},{"line_number":222,"context_line":"    cfg.StrOpt(\u0027provider_segmentation_id\u0027,"},{"line_number":223,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":224,"context_line":"               help\u003d\u0027Segmentation-id to be used for creating SRIOV network.\u0027),"},{"line_number":225,"context_line":"    cfg.StrOpt(\u0027vlan_filter\u0027,"},{"line_number":226,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":227,"context_line":"               help\u003d\u0027Comma separated list of VLANs to be mirrored \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"b152ad09_9e7c06ba","line":224,"range":{"start_line":224,"start_character":21,"end_line":224,"end_character":75},"in_reply_to":"c6c09c8c_11b2eb86","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"}],"neutron_tempest_plugin/scenario/base.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"aa0f4abe53c7e55a62371bb60524ae451993180f","unresolved":true,"context_lines":[{"line_number":611,"context_line":"        self.assertEqual(0, result.exit_status)"},{"line_number":612,"context_line":"        return result.stdout"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"    def _get_router(self, client\u003dNone, tenant_id\u003dNone):"},{"line_number":615,"context_line":"        \"\"\"Retrieve a router for the given tenant id."},{"line_number":616,"context_line":""},{"line_number":617,"context_line":"        If a public router has been configured, it will be returned."}],"source_content_type":"text/x-python","patch_set":14,"id":"b407c5c5_8b096da3","line":614,"range":{"start_line":614,"start_character":8,"end_line":614,"end_character":19},"updated":"2021-11-24 13:59:02.000000000","message":"nit: _ensure_public_router might better explain what this method does","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1ae6498ae1d49531d3f368f0317635a48c7ae3eb","unresolved":true,"context_lines":[{"line_number":611,"context_line":"        self.assertEqual(0, result.exit_status)"},{"line_number":612,"context_line":"        return result.stdout"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"    def _get_router(self, client\u003dNone, tenant_id\u003dNone):"},{"line_number":615,"context_line":"        \"\"\"Retrieve a router for the given tenant id."},{"line_number":616,"context_line":""},{"line_number":617,"context_line":"        If a public router has been configured, it will be returned."}],"source_content_type":"text/x-python","patch_set":14,"id":"baa07c0f_311052b1","line":614,"range":{"start_line":614,"start_character":8,"end_line":614,"end_character":19},"in_reply_to":"b407c5c5_8b096da3","updated":"2021-11-26 14:16:41.000000000","message":"True, I renamed it.\nThe issue is that these tempest-plugins were mostly copy-pasted and then slightly adopted, and than with stadium changes (move tests under neutron-tempest-plugin) all code was left as it is.\nI added a todo to my todo list to make all _get_router calls in different scenario tests to use this _ensure_public_router method","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":611,"context_line":"        self.assertEqual(0, result.exit_status)"},{"line_number":612,"context_line":"        return result.stdout"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"    def _get_router(self, client\u003dNone, tenant_id\u003dNone):"},{"line_number":615,"context_line":"        \"\"\"Retrieve a router for the given tenant id."},{"line_number":616,"context_line":""},{"line_number":617,"context_line":"        If a public router has been configured, it will be returned."}],"source_content_type":"text/x-python","patch_set":14,"id":"3425d5f3_544dc573","line":614,"range":{"start_line":614,"start_character":8,"end_line":614,"end_character":19},"in_reply_to":"baa07c0f_311052b1","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"7e7f290702667c697e0abad76b165fd5f4a7700a","unresolved":true,"context_lines":[{"line_number":638,"context_line":"            raise Exception(\"Neither of \u0027public_router_id\u0027 or \""},{"line_number":639,"context_line":"                            \"\u0027public_network_id\u0027 has been defined.\")"},{"line_number":640,"context_line":""},{"line_number":641,"context_line":"    def _create_router(self, client\u003dNone, tenant_id\u003dNone,"},{"line_number":642,"context_line":"                       namestart\u003d\u0027router-smoke\u0027):"},{"line_number":643,"context_line":"        if not client:"},{"line_number":644,"context_line":"            client \u003d self.client"}],"source_content_type":"text/x-python","patch_set":19,"id":"821d73ad_529120dd","line":641,"updated":"2021-12-06 13:56:13.000000000","message":"why we need new method to create router instead of using existing ones?","commit_id":"fd7bef827d7921c604dbc25d386f237dba033402"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"37adfa8902d2ca4a7a4e267dfebc89a9a16b8f26","unresolved":true,"context_lines":[{"line_number":638,"context_line":"            raise Exception(\"Neither of \u0027public_router_id\u0027 or \""},{"line_number":639,"context_line":"                            \"\u0027public_network_id\u0027 has been defined.\")"},{"line_number":640,"context_line":""},{"line_number":641,"context_line":"    def _create_router(self, client\u003dNone, tenant_id\u003dNone,"},{"line_number":642,"context_line":"                       namestart\u003d\u0027router-smoke\u0027):"},{"line_number":643,"context_line":"        if not client:"},{"line_number":644,"context_line":"            client \u003d self.client"}],"source_content_type":"text/x-python","patch_set":19,"id":"9b87a9c3_1345ff6a","line":641,"in_reply_to":"821d73ad_529120dd","updated":"2021-12-06 19:20:53.000000000","message":"true, moving these old tests really full of repeated code.","commit_id":"fd7bef827d7921c604dbc25d386f237dba033402"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":638,"context_line":"            raise Exception(\"Neither of \u0027public_router_id\u0027 or \""},{"line_number":639,"context_line":"                            \"\u0027public_network_id\u0027 has been defined.\")"},{"line_number":640,"context_line":""},{"line_number":641,"context_line":"    def _create_router(self, client\u003dNone, tenant_id\u003dNone,"},{"line_number":642,"context_line":"                       namestart\u003d\u0027router-smoke\u0027):"},{"line_number":643,"context_line":"        if not client:"},{"line_number":644,"context_line":"            client \u003d self.client"}],"source_content_type":"text/x-python","patch_set":19,"id":"8395221e_0b8ab299","line":641,"in_reply_to":"9b87a9c3_1345ff6a","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"fd7bef827d7921c604dbc25d386f237dba033402"}],"neutron_tempest_plugin/tap_as_a_service/scenario/manager.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7b7f518cc9fe632227f42454ac43a8012d384348","unresolved":true,"context_lines":[{"line_number":191,"context_line":"            self._log_net_info(e)"},{"line_number":192,"context_line":"            raise"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def _get_router(self, client\u003dNone, tenant_id\u003dNone):"},{"line_number":195,"context_line":"        \"\"\"Retrieve a router for the given tenant id."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        If a public router has been configured, it will be returned."}],"source_content_type":"text/x-python","patch_set":13,"id":"5be9e9c0_4865fdcd","line":194,"range":{"start_line":194,"start_character":8,"end_line":194,"end_character":19},"updated":"2021-11-22 11:03:02.000000000","message":"nit: we have several methods duplicated in n-t-p, like \"_get_router\", \"_create_router\" or \"create_networks\", that could be implemented in BaseTempestTestCase.\n\nBut this is just a nit.","commit_id":"852807115511dee6a39f8f347d1956b77b8dcd0a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":191,"context_line":"            self._log_net_info(e)"},{"line_number":192,"context_line":"            raise"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def _get_router(self, client\u003dNone, tenant_id\u003dNone):"},{"line_number":195,"context_line":"        \"\"\"Retrieve a router for the given tenant id."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        If a public router has been configured, it will be returned."}],"source_content_type":"text/x-python","patch_set":13,"id":"acb69588_cd8a4f3b","line":194,"range":{"start_line":194,"start_character":8,"end_line":194,"end_character":19},"in_reply_to":"50ce0b77_12ed5fc1","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"852807115511dee6a39f8f347d1956b77b8dcd0a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"567c91d9f9735f0a5a7728bb7efa6d5f701ec91a","unresolved":true,"context_lines":[{"line_number":191,"context_line":"            self._log_net_info(e)"},{"line_number":192,"context_line":"            raise"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def _get_router(self, client\u003dNone, tenant_id\u003dNone):"},{"line_number":195,"context_line":"        \"\"\"Retrieve a router for the given tenant id."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        If a public router has been configured, it will be returned."}],"source_content_type":"text/x-python","patch_set":13,"id":"50ce0b77_12ed5fc1","line":194,"range":{"start_line":194,"start_character":8,"end_line":194,"end_character":19},"in_reply_to":"5be9e9c0_4865fdcd","updated":"2021-11-22 16:14:41.000000000","message":"true, now I moved some from taas manager, and added a todo to myself to push other sfc, bgp... managers common code to scenario/base.py","commit_id":"852807115511dee6a39f8f347d1956b77b8dcd0a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7b7f518cc9fe632227f42454ac43a8012d384348","unresolved":true,"context_lines":[{"line_number":337,"context_line":"        }"},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"        segmentation_id \u003d CONF.taas.provider_segmentation_id"},{"line_number":340,"context_line":"        if segmentation_id and segmentation_id \u003d\u003d \"0\":"},{"line_number":341,"context_line":"            network_kwargs[\u0027provider:network_type\u0027] \u003d \u0027flat\u0027"},{"line_number":342,"context_line":"        elif segmentation_id:"},{"line_number":343,"context_line":"            network_kwargs[\u0027provider:segmentation_id\u0027] \u003d segmentation_id"},{"line_number":344,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"36a30c25_4a29ba35","line":341,"range":{"start_line":340,"start_character":8,"end_line":341,"end_character":60},"updated":"2021-11-22 11:03:02.000000000","message":"just one question: if we use \"flat\" on \"provider_physical_network\", we\u0027ll be able to create just one single provider network. Could this affect to other tests running in parallel trying to do the same?\n\nEDITED: I see \"tempest_concurrency\u003d1\". So the only concern here is to properly delete the used resources when the test is finished.","commit_id":"852807115511dee6a39f8f347d1956b77b8dcd0a"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"aa0f4abe53c7e55a62371bb60524ae451993180f","unresolved":true,"context_lines":[{"line_number":61,"context_line":"            **cls.os_primary.default_params)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _create_subnet(self, network, subnets_client\u003dNone,"},{"line_number":64,"context_line":"                       routers_client\u003dNone, namestart\u003d\u0027subnet-smoke\u0027,"},{"line_number":65,"context_line":"                       **kwargs):"},{"line_number":66,"context_line":"        \"\"\"Create a subnet for the given network"},{"line_number":67,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"300c07e5_0531d6be","line":64,"range":{"start_line":64,"start_character":23,"end_line":64,"end_character":37},"updated":"2021-11-24 13:59:02.000000000","message":"unused","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1ae6498ae1d49531d3f368f0317635a48c7ae3eb","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            **cls.os_primary.default_params)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _create_subnet(self, network, subnets_client\u003dNone,"},{"line_number":64,"context_line":"                       routers_client\u003dNone, namestart\u003d\u0027subnet-smoke\u0027,"},{"line_number":65,"context_line":"                       **kwargs):"},{"line_number":66,"context_line":"        \"\"\"Create a subnet for the given network"},{"line_number":67,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"01ac66ae_fa6daf53","line":64,"range":{"start_line":64,"start_character":23,"end_line":64,"end_character":37},"in_reply_to":"300c07e5_0531d6be","updated":"2021-11-26 14:16:41.000000000","message":"Done","commit_id":"d04026fdf3de6337abc877b7b83d7f07c9fc81bf"}],"neutron_tempest_plugin/tap_as_a_service/scenario/test_taas.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"547090eafc15368e7386aca8dd3d3442aff7528a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bd067006_4e0a79b5","line":250,"updated":"2021-10-21 09:25:23.000000000","message":"Here some traffic test is needed....","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3a9e3ea5_c258971c","line":250,"in_reply_to":"bd067006_4e0a79b5","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"}],"neutron_tempest_plugin/tap_as_a_service/scenario/test_traffic_impact.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"547090eafc15368e7386aca8dd3d3442aff7528a","unresolved":true,"context_lines":[{"line_number":181,"context_line":"                self.monitor_client,"},{"line_number":182,"context_line":"                self.left_port[\u0027fixed_ips\u0027][0][\u0027ip_address\u0027])"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"    @decorators.idempotent_id(\u0027ff414b7d-e81c-47f2-b6c8-53bc2f1e9b00\u0027)"},{"line_number":185,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":186,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":187,"context_line":"    def test_taas_provider_network_connectivity(self):"},{"line_number":188,"context_line":"        self._test_taas_connectivity(use_provider_net\u003dTrue)"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    @decorators.idempotent_id(\u0027e3c52e91-7abf-4dfd-8687-f7c071cdd333\u0027)"},{"line_number":191,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":192,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":193,"context_line":"    def test_taas_network_connectivity(self):"},{"line_number":194,"context_line":"        self._test_taas_connectivity(use_provider_net\u003dFalse)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    @decorators.idempotent_id(\u0027fcb15ca3-ef61-11e9-9792-f45c89c47e11\u0027)"},{"line_number":197,"context_line":"    @testtools.skipUnless(CONF.neutron_plugin_options.advanced_image_ref,"}],"source_content_type":"text/x-python","patch_set":1,"id":"8c2324bd_e1d22d7e","line":194,"range":{"start_line":184,"start_character":0,"end_line":194,"end_character":60},"updated":"2021-10-21 09:25:23.000000000","message":"drop these or add traffic check, but below tests cover the same scenario with traffic test....","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5f508410e581151d32339c0909255a7e664dc8b6","unresolved":false,"context_lines":[{"line_number":181,"context_line":"                self.monitor_client,"},{"line_number":182,"context_line":"                self.left_port[\u0027fixed_ips\u0027][0][\u0027ip_address\u0027])"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"    @decorators.idempotent_id(\u0027ff414b7d-e81c-47f2-b6c8-53bc2f1e9b00\u0027)"},{"line_number":185,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":186,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":187,"context_line":"    def test_taas_provider_network_connectivity(self):"},{"line_number":188,"context_line":"        self._test_taas_connectivity(use_provider_net\u003dTrue)"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    @decorators.idempotent_id(\u0027e3c52e91-7abf-4dfd-8687-f7c071cdd333\u0027)"},{"line_number":191,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":192,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":193,"context_line":"    def test_taas_network_connectivity(self):"},{"line_number":194,"context_line":"        self._test_taas_connectivity(use_provider_net\u003dFalse)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"    @decorators.idempotent_id(\u0027fcb15ca3-ef61-11e9-9792-f45c89c47e11\u0027)"},{"line_number":197,"context_line":"    @testtools.skipUnless(CONF.neutron_plugin_options.advanced_image_ref,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ec9d676c_4d8d67f2","line":194,"range":{"start_line":184,"start_character":0,"end_line":194,"end_character":60},"in_reply_to":"8c2324bd_e1d22d7e","updated":"2022-02-14 10:41:44.000000000","message":"Done","commit_id":"59197544ccab7201af804384c8fd3bf136e95ac6"}],"zuul.d/master_jobs.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"7e7f290702667c697e0abad76b165fd5f4a7700a","unresolved":true,"context_lines":[{"line_number":916,"context_line":"      devstack_localrc:"},{"line_number":917,"context_line":"        NETWORK_API_EXTENSIONS: \"{{ (network_api_extensions_common + network_api_extensions_tempest) | join(\u0027,\u0027) }}\""},{"line_number":918,"context_line":"        DOWNLOAD_DEFAULT_IMAGES: false"},{"line_number":919,"context_line":"        IMAGE_URLS: http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-i386-disk.img,https://cloud-images.ubuntu.com/releases/bionic/release/ubuntu-18.04-server-cloudimg-amd64.img"},{"line_number":920,"context_line":"        ADVANCED_IMAGE_NAME: ubuntu-18.04-server-cloudimg-amd64"},{"line_number":921,"context_line":"        ADVANCED_INSTANCE_TYPE: ntp_image_384M"},{"line_number":922,"context_line":"        ADVANCED_INSTANCE_USER: ubuntu"}],"source_content_type":"text/x-yaml","patch_set":19,"id":"98f38460_76edc5b1","line":919,"range":{"start_line":919,"start_character":53,"end_line":919,"end_character":85},"updated":"2021-12-06 13:56:13.000000000","message":"nit (no action needed in this patch): maybe we could also use newer Cirros in that job?","commit_id":"fd7bef827d7921c604dbc25d386f237dba033402"}]}