)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"3e5a4e64e0dbb4ec834dae3dc6cf9c5ad2168ade","unresolved":true,"context_lines":[{"line_number":14,"context_line":"- subnets: auto_allocated_subnet_v4 \u0026 auto_allocated_subnet_v6"},{"line_number":15,"context_line":"- router: auto_allocated_router"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Story: 40406"},{"line_number":18,"context_line":"Change-Id: I8cc77f7d47918d8329298ee32733d569294f11b7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"585b50d7_bf93b86d","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":12},"updated":"2021-07-13 07:44:21.000000000","message":"Story: 2007941\nTask: 40406","commit_id":"d469e67de0bb5966e7a59ff8ef9595fbe8468e87"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"77bb7c077c93379ed21d5272d3a97e4b66d90e13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"740e7c0d_b53fcc26","updated":"2022-04-25 06:54:14.000000000","message":"lgtm, let\u0027s go ahead while gates like us ","commit_id":"4769f92f6b78994ba1cdf593f9646daa77217699"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"4a501e16cb06e3d3c6676307e67dc61718229a60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f32791b2_79516207","updated":"2022-03-10 13:21:13.000000000","message":"recheck","commit_id":"4769f92f6b78994ba1cdf593f9646daa77217699"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"75138f0c3c3afd01614df6e76d7859a1840bdc16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"41e9df98_ad7eca99","updated":"2022-04-22 14:45:03.000000000","message":"recheck gates seem to be fixed","commit_id":"4769f92f6b78994ba1cdf593f9646daa77217699"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9ebcabd1cf32659f8adca5bed1031ab24615adfd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3541b794_37e4a852","updated":"2021-11-03 17:50:43.000000000","message":"thanks, lgtm but gate need fixes for volume tests.","commit_id":"4769f92f6b78994ba1cdf593f9646daa77217699"}],"patrole_tempest_plugin/tests/api/network/test_auto_allocated_topology_rbac.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"558275d3392790a48c95c0cc09073b5190002756","unresolved":true,"context_lines":[{"line_number":38,"context_line":"        \"\"\"Test show auto_allocated_topology."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        RBAC test for the neutron \"get_auto_allocated_topology\" policy"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        with self.override_role():"},{"line_number":43,"context_line":"            self.ntp_client.get_auto_allocated_topology("},{"line_number":44,"context_line":"                tenant_id\u003dself.os_primary.credentials.tenant_id)"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        self.ntp_client.delete_auto_allocated_topology("},{"line_number":47,"context_line":"            tenant_id\u003dself.os_primary.credentials.tenant_id)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _ensure_network_not_in_use(cls, network_id):"},{"line_number":50,"context_line":"        ports \u003d cls.ntp_client.list_ports(network_id\u003dnetwork_id)[\"ports\"]"}],"source_content_type":"text/x-python","patch_set":3,"id":"4c4f552b_ebad3818","line":47,"range":{"start_line":41,"start_character":0,"end_line":47,"end_character":60},"updated":"2021-09-29 02:53:33.000000000","message":"I did not get where we are creating the resorce in this tests?","commit_id":"284ea4f48a0662403673b3b250d4646444b9d365"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0e130936c58976445e7d78ebe58bc9ee3f64fc71","unresolved":true,"context_lines":[{"line_number":38,"context_line":"        \"\"\"Test show auto_allocated_topology."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        RBAC test for the neutron \"get_auto_allocated_topology\" policy"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        with self.override_role():"},{"line_number":43,"context_line":"            self.ntp_client.get_auto_allocated_topology("},{"line_number":44,"context_line":"                tenant_id\u003dself.os_primary.credentials.tenant_id)"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        self.ntp_client.delete_auto_allocated_topology("},{"line_number":47,"context_line":"            tenant_id\u003dself.os_primary.credentials.tenant_id)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _ensure_network_not_in_use(cls, network_id):"},{"line_number":50,"context_line":"        ports \u003d cls.ntp_client.list_ports(network_id\u003dnetwork_id)[\"ports\"]"}],"source_content_type":"text/x-python","patch_set":3,"id":"ee07b7e0_7122eca5","line":47,"range":{"start_line":41,"start_character":0,"end_line":47,"end_character":60},"in_reply_to":"2d07fb24_ca9a5f17","updated":"2021-11-02 18:34:43.000000000","message":"Ah i see, It seems if auto allocated network is not there for that project then neutron provision new topology in GET call.\n- https://github.com/openstack/neutron/blob/3b3398b8c5c0d597e0995ab7fdccef6d64d456cf/neutron/services/auto_allocate/db.py#L143-L152\n\nMay be we can make sure auto allocated network is there otherwise please add a NOTE here about ^^ it so that we know why GET create topology and we need to cleanup that.","commit_id":"284ea4f48a0662403673b3b250d4646444b9d365"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"112a2209acdd1d950429f3da618faf6db92e20b4","unresolved":true,"context_lines":[{"line_number":38,"context_line":"        \"\"\"Test show auto_allocated_topology."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        RBAC test for the neutron \"get_auto_allocated_topology\" policy"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        with self.override_role():"},{"line_number":43,"context_line":"            self.ntp_client.get_auto_allocated_topology("},{"line_number":44,"context_line":"                tenant_id\u003dself.os_primary.credentials.tenant_id)"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        self.ntp_client.delete_auto_allocated_topology("},{"line_number":47,"context_line":"            tenant_id\u003dself.os_primary.credentials.tenant_id)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _ensure_network_not_in_use(cls, network_id):"},{"line_number":50,"context_line":"        ports \u003d cls.ntp_client.list_ports(network_id\u003dnetwork_id)[\"ports\"]"}],"source_content_type":"text/x-python","patch_set":3,"id":"2d07fb24_ca9a5f17","line":47,"range":{"start_line":41,"start_character":0,"end_line":47,"end_character":60},"in_reply_to":"4c4f552b_ebad3818","updated":"2021-10-05 11:22:59.000000000","message":"It looks like the resources get created when the API call to show the auto allocated topology is made [1]. \n\nI tried to go through the code with a debugger and the resources got created exactly when the API call was made [2]. I am not sure why is that. The API call is supposed to only show the auto allocated topology if I understand it correctly.\n\n[1] https://docs.openstack.org/api-ref/network/v2/?expanded\u003dshow-auto-allocated-topology-details-detail#show-auto-allocated-topology-details\n[2] https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/services/network/json/network_client.py#L764","commit_id":"284ea4f48a0662403673b3b250d4646444b9d365"}]}