)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"f526bf70e4822219bd9efb3c93b112cf9f7dcf4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1e2e6ece_87db5ff1","updated":"2024-02-08 12:05:04.000000000","message":"recheck neutron-tempest-plugin-api-yoga","commit_id":"8e4c3790bc4072372b8f826ca1b646bdf9ce290d"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"09a438ac4b10e9ad3fa47234dbce19edfcc9f1aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"211e7f66_6e9f9ed9","updated":"2024-02-13 09:49:34.000000000","message":"Ok, now I think","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":29088,"name":"Candido Campos Rivas","email":"ccamposr@redhat.com","username":"ccamposr"},"change_message_id":"f532cb8c80740dbc366994667c23439b2ba84e60","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"48c9a65a_94710a53","updated":"2024-02-09 08:40:59.000000000","message":"Please we need more info. I think the ha/non-ha affects a ovn deployments too. And it is changed we should skip the complete class not only some tests and create other tests to cover the gap. The option should be removed this option from the api","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"fa8c95a365c0c7b0bd948347386a716f82bb124f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8ffa0894_f86912c4","updated":"2024-02-08 23:17:44.000000000","message":"recheck","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"2ae3129d5986bee451d67b750ae5a2c981c941b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a09f7e1b_ca97d621","updated":"2024-02-08 21:13:35.000000000","message":"recheck","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"4cf5f094f26e93d048ce37a2dd1d01f1f2fdeb38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"39906742_55b47332","updated":"2024-02-09 06:55:31.000000000","message":"recheck neutron-tempest-plugin-linuxbridge-2023-1 fails","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"85d96fb9341ef0fe2b9f5652bbb56288d8bea8e9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2a050980_bbef0203","in_reply_to":"1cd33559_3a78b18a","updated":"2024-02-12 09:09:41.000000000","message":"In tempest there is option \"available_features\" (https://github.com/openstack/tempest/blob/master/tempest/config.py#L802) which can be used here. You can specify to run those tests only if router flavors are or are not available.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"35746533bc4d50610a06805de39f287324e2a510","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"52e069f9_34440348","in_reply_to":"2a050980_bbef0203","updated":"2024-02-12 11:24:46.000000000","message":"Thank you for the feedback.\n\nThe setups I got have \"api_extensions\" option used instead, option has \"l3-ha\" listed due to HA added to OVN router flavors as in here:\nhttps://review.opendev.org/c/openstack/neutron/+/901513/10\n\nWrote more thoughts in other comment, let me know what you think.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"63a4e030351c991d0caf8402c08f14a0b69c67ee","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1cd33559_3a78b18a","in_reply_to":"48c9a65a_94710a53","updated":"2024-02-09 16:29:40.000000000","message":"+1\n\nseems this is specific to ovn router flavors, so those tests likely need to be skipped when that extension is enabled.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"fbbc27428317da18c58881b20eda08d479d983d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"83ac8463_2811cf11","in_reply_to":"52e069f9_34440348","updated":"2024-02-18 19:34:18.000000000","message":"I added changes according to discussion, commit has more info to explain the need of this change.\nLet me know if looks better, thanks.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"4e391b9ac986752bbc1785537501db6974697d0a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6c7d227c_41b4b370","updated":"2024-02-19 11:11:17.000000000","message":"Job \"neutron-tempest-plugin-openvswitch-enforce-scope-old-defaults\" had unrelated error in a test case:\nhttps://zuul.opendev.org/t/openstack/build/caf5602da0c14a74a4151553590474a9","commit_id":"17d21474bfddf61522a427fca8248ee4909fac1d"}],"neutron_tempest_plugin/api/admin/test_routers_ha.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"85d96fb9341ef0fe2b9f5652bbb56288d8bea8e9","unresolved":true,"context_lines":[{"line_number":65,"context_line":"        as opposed to a \"High Availability Router\""},{"line_number":66,"context_line":"        \"\"\""},{"line_number":67,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":68,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":69,"context_line":"        name \u003d data_utils.rand_name(\u0027router\u0027)"},{"line_number":70,"context_line":"        router \u003d self._create_admin_router(name, ha\u003dFalse)"},{"line_number":71,"context_line":"        self.assertFalse(router[\u0027ha\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"c6f62f4f_f17d171c","line":68,"updated":"2024-02-12 09:09:41.000000000","message":"I don\u0027t really understand need for those new skips. If You look at the line 22 this whole class requires \u0027l3-ha\u0027 extension to be available. And OVN backend don\u0027t supports that extension: https://github.com/openstack/neutron/blob/7d2bb8c86f7537f56d7ac13fac800031d56f75e0/neutron/common/ovn/extensions.py#L103 so all those tests should be skipped in such case.\nAnd if You check results of the \u0027neutron-tempest-plugin-ovn\u0027 job, like e.g. https://a33a625d46947ad5c6a4-b506db2b6a14acf4c94c5e6756f47161.ssl.cf5.rackcdn.com/908408/5/check/neutron-tempest-plugin-ovn/56504ea/job-output.txt You will see that those tests are indeed skipped because \"SKIPPED: l3-ha extension not enabled.\"","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"09a438ac4b10e9ad3fa47234dbce19edfcc9f1aa","unresolved":true,"context_lines":[{"line_number":65,"context_line":"        as opposed to a \"High Availability Router\""},{"line_number":66,"context_line":"        \"\"\""},{"line_number":67,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":68,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":69,"context_line":"        name \u003d data_utils.rand_name(\u0027router\u0027)"},{"line_number":70,"context_line":"        router \u003d self._create_admin_router(name, ha\u003dFalse)"},{"line_number":71,"context_line":"        self.assertFalse(router[\u0027ha\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"d776b938_9680a33b","line":68,"in_reply_to":"2fc18081_7f162f61","updated":"2024-02-13 09:49:34.000000000","message":"ok, now I think I understand why this check would be useful even if this is really some kind of hack only.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"911ea9a4cc9d2a3711b10a9cecdc6cc43d05c992","unresolved":true,"context_lines":[{"line_number":65,"context_line":"        as opposed to a \"High Availability Router\""},{"line_number":66,"context_line":"        \"\"\""},{"line_number":67,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":68,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":69,"context_line":"        name \u003d data_utils.rand_name(\u0027router\u0027)"},{"line_number":70,"context_line":"        router \u003d self._create_admin_router(name, ha\u003dFalse)"},{"line_number":71,"context_line":"        self.assertFalse(router[\u0027ha\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"d450d3ce_4680bda6","line":68,"in_reply_to":"75b271c5_5574b2bf","updated":"2024-02-12 11:27:25.000000000","message":"I will also add more info to the commit message.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"35746533bc4d50610a06805de39f287324e2a510","unresolved":true,"context_lines":[{"line_number":65,"context_line":"        as opposed to a \"High Availability Router\""},{"line_number":66,"context_line":"        \"\"\""},{"line_number":67,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":68,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":69,"context_line":"        name \u003d data_utils.rand_name(\u0027router\u0027)"},{"line_number":70,"context_line":"        router \u003d self._create_admin_router(name, ha\u003dFalse)"},{"line_number":71,"context_line":"        self.assertFalse(router[\u0027ha\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"75b271c5_5574b2bf","line":68,"in_reply_to":"c6f62f4f_f17d171c","updated":"2024-02-12 11:24:46.000000000","message":"Sorry for the lack of related information.\n\nI needed to skip some non relevant test methods since they ran when tested HA capabilities added for router flavors support with OVN as in this patch [1].\n\nI checked 2 setups deployed by same CI job with/without such patch used [1], both has identical ML2_SUPPORTED_API_EXTENSIONS list on controllers, without \"l3-ha\" extension listed.\n\nOnly the env with extra HA router flavors change [1] got \"l3-ha\" listed in under \"api_extensions\" in tempest.conf, and also \"l3-ha\" in response from \"openstack extension list\".\n\nIf a subset of these tests can be useful against HA with OVN router flavors, isn\u0027t it better to run them in that case?\n\nFirst type of skip iiuc affects OVN specifically, when OVN doesn\u0027t accept ha\u003dFalse attribute in request since all routers support HA.\n\nSecond type of skip done according to implementation assumption that verifies certain networks details and isn\u0027t relevant for OVN, I think only relevant for OVS backend with HA\n(I can adjust to skip unless OVS).\n\nWDYT?\n\n[1] https://review.opendev.org/c/openstack/neutron/+/901513/10","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"7c16df68b63ce0d19eec1a571b7f93bd3ddc0886","unresolved":true,"context_lines":[{"line_number":65,"context_line":"        as opposed to a \"High Availability Router\""},{"line_number":66,"context_line":"        \"\"\""},{"line_number":67,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":68,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":69,"context_line":"        name \u003d data_utils.rand_name(\u0027router\u0027)"},{"line_number":70,"context_line":"        router \u003d self._create_admin_router(name, ha\u003dFalse)"},{"line_number":71,"context_line":"        self.assertFalse(router[\u0027ha\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"2fc18081_7f162f61","line":68,"in_reply_to":"d450d3ce_4680bda6","updated":"2024-02-13 08:48:59.000000000","message":"We also noticed this bug that I recently opened due to these tests running with OVN too [1], so this is why I tend towards running the subsset that is still relevant also to OVN with/without router flavors and other future changes, noticing possible future regressions.\n\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d2263225","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"fbbc27428317da18c58881b20eda08d479d983d6","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        as opposed to a \"High Availability Router\""},{"line_number":66,"context_line":"        \"\"\""},{"line_number":67,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":68,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":69,"context_line":"        name \u003d data_utils.rand_name(\u0027router\u0027)"},{"line_number":70,"context_line":"        router \u003d self._create_admin_router(name, ha\u003dFalse)"},{"line_number":71,"context_line":"        self.assertFalse(router[\u0027ha\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"7c0af0f4_75f64acf","line":68,"in_reply_to":"d776b938_9680a33b","updated":"2024-02-18 19:34:18.000000000","message":"I added changes according to discussion, commit has more info to explain the need of this change.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a27ce30c6501ec59a88f508d422760d0ccf96082","unresolved":true,"context_lines":[{"line_number":108,"context_line":"        regression of https://bugs.launchpad.net/neutron/+bug/1732543."},{"line_number":109,"context_line":"        \"\"\""},{"line_number":110,"context_line":"        if not self.is_driver_ovs:"},{"line_number":111,"context_line":"            raise self.skipException(\"Test meant only for openvswitch driver\")"},{"line_number":112,"context_line":"        for i in range(2):"},{"line_number":113,"context_line":"            router \u003d self._create_admin_router("},{"line_number":114,"context_line":"                data_utils.rand_name(\u0027router%d\u0027 % i),"}],"source_content_type":"text/x-python","patch_set":6,"id":"30ffa31c_9451edd8","line":111,"range":{"start_line":111,"start_character":7,"end_line":111,"end_character":78},"updated":"2024-02-19 07:57:23.000000000","message":"I see this tests runs even for linuxbridge so reason looks wrong here or need to add more context:-\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a2a/908557/2/gate/neutron-tempest-plugin-openvswitch/a2a5d75/testr_results.html\n\nAlso with this skip it\u0027s not running even in ovs job https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_e5d/908408/6/check/neutron-tempest-plugin-openvswitch/e5d91bd/job-output.txt","commit_id":"f532056ddafc6ec4d6e2202edd4bc3d5053e8369"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"c2987d206e2a210eb89ea6bfd42d81385414a83d","unresolved":true,"context_lines":[{"line_number":108,"context_line":"        regression of https://bugs.launchpad.net/neutron/+bug/1732543."},{"line_number":109,"context_line":"        \"\"\""},{"line_number":110,"context_line":"        if not self.is_driver_ovs:"},{"line_number":111,"context_line":"            raise self.skipException(\"Test meant only for openvswitch driver\")"},{"line_number":112,"context_line":"        for i in range(2):"},{"line_number":113,"context_line":"            router \u003d self._create_admin_router("},{"line_number":114,"context_line":"                data_utils.rand_name(\u0027router%d\u0027 % i),"}],"source_content_type":"text/x-python","patch_set":6,"id":"896c6bcc_73e2988e","line":111,"range":{"start_line":111,"start_character":7,"end_line":111,"end_character":78},"in_reply_to":"30ffa31c_9451edd8","updated":"2024-02-19 08:32:17.000000000","message":"Thanks, I will notice CI for for these next times. \nMaybe OVS similar HA implementation is preserved with linux bridge used, I will skip only when driver is OVN, when HA is certainly implemented differently.\nI will remove the OVS check since it is not accurate, negation of OVN check seems sufficient for now.","commit_id":"f532056ddafc6ec4d6e2202edd4bc3d5053e8369"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"5f73a6acf6b50a2ad8b197a23b8d57860b0fa9ac","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        regression of https://bugs.launchpad.net/neutron/+bug/1732543."},{"line_number":109,"context_line":"        \"\"\""},{"line_number":110,"context_line":"        if not self.is_driver_ovs:"},{"line_number":111,"context_line":"            raise self.skipException(\"Test meant only for openvswitch driver\")"},{"line_number":112,"context_line":"        for i in range(2):"},{"line_number":113,"context_line":"            router \u003d self._create_admin_router("},{"line_number":114,"context_line":"                data_utils.rand_name(\u0027router%d\u0027 % i),"}],"source_content_type":"text/x-python","patch_set":6,"id":"2f04cbf8_15a8a60a","line":111,"range":{"start_line":111,"start_character":7,"end_line":111,"end_character":78},"in_reply_to":"896c6bcc_73e2988e","updated":"2024-02-19 09:02:30.000000000","message":"Done","commit_id":"f532056ddafc6ec4d6e2202edd4bc3d5053e8369"}],"neutron_tempest_plugin/api/base.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"09a438ac4b10e9ad3fa47234dbce19edfcc9f1aa","unresolved":true,"context_lines":[{"line_number":1214,"context_line":"    def resource_setup(cls):"},{"line_number":1215,"context_line":"        super(BaseAdminNetworkTest, cls).resource_setup()"},{"line_number":1216,"context_line":"        if hasattr(cls, \u0027os_admin\u0027):"},{"line_number":1217,"context_line":"            cls.IS_DRIVER_OVN \u003d cls.is_driver_ovn()"},{"line_number":1218,"context_line":""},{"line_number":1219,"context_line":"    @classmethod"},{"line_number":1220,"context_line":"    def create_metering_label(cls, name, description):"}],"source_content_type":"text/x-python","patch_set":5,"id":"8873845e_3950be7a","line":1217,"updated":"2024-02-13 09:49:34.000000000","message":"You can probably do it in \"lazy\" way instead. Something like:\n\n\n    self._is_ovn_driver \u003d None\n\n    @attribute\n    def is_ovn_driver(self):\n        if self._is_driver_ovn is None:\n            if hasattr(self, \u0027os_admin\u0027):\n                self._is_ovn_driver \u003d self.is_driver_ovn()\n        return self._is_ovn_driver","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"fbbc27428317da18c58881b20eda08d479d983d6","unresolved":false,"context_lines":[{"line_number":1214,"context_line":"    def resource_setup(cls):"},{"line_number":1215,"context_line":"        super(BaseAdminNetworkTest, cls).resource_setup()"},{"line_number":1216,"context_line":"        if hasattr(cls, \u0027os_admin\u0027):"},{"line_number":1217,"context_line":"            cls.IS_DRIVER_OVN \u003d cls.is_driver_ovn()"},{"line_number":1218,"context_line":""},{"line_number":1219,"context_line":"    @classmethod"},{"line_number":1220,"context_line":"    def create_metering_label(cls, name, description):"}],"source_content_type":"text/x-python","patch_set":5,"id":"1e095feb_782d0cca","line":1217,"in_reply_to":"8873845e_3950be7a","updated":"2024-02-18 19:34:18.000000000","message":"I like that way, I assume the intention was for \"property\" decorator, added, thank you.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"}],"neutron_tempest_plugin/api/test_revisions.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"85d96fb9341ef0fe2b9f5652bbb56288d8bea8e9","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        # updates from CVR to CVR-HA are supported on every release,"},{"line_number":346,"context_line":"        # but only the admin can forcibly create a non-HA router"},{"line_number":347,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":348,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":349,"context_line":"        router_args \u003d {\u0027tenant_id\u0027: self.client.project_id,"},{"line_number":350,"context_line":"                       \u0027ha\u0027: False}"},{"line_number":351,"context_line":"        router \u003d self.admin_client.create_router(\u0027r1\u0027, True,"}],"source_content_type":"text/x-python","patch_set":5,"id":"e8f4a019_85fe088e","line":348,"updated":"2024-02-12 09:09:41.000000000","message":"here is the same thing as I described in one of the previous files - this test requires l3-ha extension to be enabled to be run.","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"fbbc27428317da18c58881b20eda08d479d983d6","unresolved":false,"context_lines":[{"line_number":345,"context_line":"        # updates from CVR to CVR-HA are supported on every release,"},{"line_number":346,"context_line":"        # but only the admin can forcibly create a non-HA router"},{"line_number":347,"context_line":"        if self.IS_DRIVER_OVN:"},{"line_number":348,"context_line":"            raise self.skipException(\"Test not meant for OVN driver\")"},{"line_number":349,"context_line":"        router_args \u003d {\u0027tenant_id\u0027: self.client.project_id,"},{"line_number":350,"context_line":"                       \u0027ha\u0027: False}"},{"line_number":351,"context_line":"        router \u003d self.admin_client.create_router(\u0027r1\u0027, True,"}],"source_content_type":"text/x-python","patch_set":5,"id":"f1f5a7ba_94bdb092","line":348,"in_reply_to":"e8f4a019_85fe088e","updated":"2024-02-18 19:34:18.000000000","message":"Acknowledged","commit_id":"9695660a18d6b64aedce46c7fc72792e6ed95ec2"}]}