)]}'
{"tempest/scenario/test_minbw_allocation_placement.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"93dc6b8344e87912f3d89c2c5e7c87f5edb26de2","unresolved":false,"context_lines":[{"line_number":222,"context_line":"        \"\"\""},{"line_number":223,"context_line":"        server, valid_port \u003d self._boot_vm_with_min_bw("},{"line_number":224,"context_line":"            qos_policy_id\u003dself.qos_policy_valid[\u0027id\u0027])"},{"line_number":225,"context_line":"        allocations \u003d self.placement_client.list_allocations(server[\u0027id\u0027])"},{"line_number":226,"context_line":"        self._assert_allocation_is_as_expected(allocations, valid_port[\u0027id\u0027])"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        self.servers_client.migrate_server(server_id\u003dserver[\u0027id\u0027])"},{"line_number":229,"context_line":"        waiters.wait_for_server_status("}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_0e2c76aa","line":226,"range":{"start_line":225,"start_character":0,"end_line":226,"end_character":77},"updated":"2019-11-15 14:06:49.000000000","message":"these two calls seems to always called together so you can move the first into the second.","commit_id":"0374a9069d050f98941665b8df78c22a667a5aa2"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"93dc6b8344e87912f3d89c2c5e7c87f5edb26de2","unresolved":false,"context_lines":[{"line_number":231,"context_line":"            status\u003d\u0027VERIFY_RESIZE\u0027, ready_wait\u003dFalse, raise_on_error\u003dFalse)"},{"line_number":232,"context_line":"        allocations \u003d self.placement_client.list_allocations(server[\u0027id\u0027])"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"        # TODO(lajoskatona): Check that the allocations are ok for the"},{"line_number":235,"context_line":"        #  migration?"},{"line_number":236,"context_line":"        self._assert_allocation_is_as_expected(allocations, valid_port[\u0027id\u0027])"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"        self.servers_client.confirm_resize_server(server_id\u003dserver[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_8e3866ee","line":235,"range":{"start_line":234,"start_character":2,"end_line":235,"end_character":21},"updated":"2019-11-15 14:06:49.000000000","message":"If you mean looking that the allocation in placement on the source host held by the migration uuid, then I think you don\u0027t have to do that. It is pretty much implementation detail and already covered in nova functional test.","commit_id":"0374a9069d050f98941665b8df78c22a667a5aa2"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b3e957ca0a958159e0bf5cd7650bb8ca55eb7cc3","unresolved":false,"context_lines":[{"line_number":231,"context_line":"            status\u003d\u0027VERIFY_RESIZE\u0027, ready_wait\u003dFalse, raise_on_error\u003dFalse)"},{"line_number":232,"context_line":"        allocations \u003d self.placement_client.list_allocations(server[\u0027id\u0027])"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"        # TODO(lajoskatona): Check that the allocations are ok for the"},{"line_number":235,"context_line":"        #  migration?"},{"line_number":236,"context_line":"        self._assert_allocation_is_as_expected(allocations, valid_port[\u0027id\u0027])"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"        self.servers_client.confirm_resize_server(server_id\u003dserver[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_7e4af881","line":235,"range":{"start_line":234,"start_character":0,"end_line":235,"end_character":21},"updated":"2020-02-17 09:33:56.000000000","message":"That is pretty implementation specific that there is an extra migration allocation until you confirm (or revert). I think from tempest perspective what needs to be asserted that confirm and revert is possible without loosing the allocation. A working revert indirectly proves that source host allocation is not lost during the migration.\n\nSo I would add a revert test instead of looking for the migration allocation from tempest.","commit_id":"d1d89324f3370c9eb19db7befb86d9c8c4f9c0ba"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c0fae3839d8ebf559aafd4db38be34344296af11","unresolved":true,"context_lines":[{"line_number":210,"context_line":"        port \u003d self.os_admin.ports_client.show_port(not_valid_port[\u0027id\u0027])"},{"line_number":211,"context_line":"        self.assertEqual(0, len(port[\u0027port\u0027][\u0027binding:profile\u0027]))"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":214,"context_line":"    @decorators.idempotent_id(\u00278a98150c-a506-49a5-96c6-73a5e7b04ada\u0027)"},{"line_number":215,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":216,"context_line":"    def test_migrate_with_qos_min_bw_allocation(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"707e22eb_23f03888","line":213,"range":{"start_line":213,"start_character":0,"end_line":213,"end_character":33},"updated":"2020-12-02 15:46:18.000000000","message":"This test just take 36 sec, let\u0027s not mark it slow. iF we notice higher execution\u0027s time then we mark.  I think we should do the same for existing tests too but as separate patch.\n\n- https://zuul.opendev.org/t/openstack/build/01fbdf27f6b44ce0bc87688a5b25cefe/log/job-output.txt#59728","commit_id":"0bda5401f9ec6fa178efa949b6e46680c4cc62dc"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"89af6a9a62467984ad5a23532bf5db830f7853fd","unresolved":false,"context_lines":[{"line_number":210,"context_line":"        port \u003d self.os_admin.ports_client.show_port(not_valid_port[\u0027id\u0027])"},{"line_number":211,"context_line":"        self.assertEqual(0, len(port[\u0027port\u0027][\u0027binding:profile\u0027]))"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":214,"context_line":"    @decorators.idempotent_id(\u00278a98150c-a506-49a5-96c6-73a5e7b04ada\u0027)"},{"line_number":215,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":216,"context_line":"    def test_migrate_with_qos_min_bw_allocation(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"934a826a_ed517f25","line":213,"range":{"start_line":213,"start_character":0,"end_line":213,"end_character":33},"in_reply_to":"707e22eb_23f03888","updated":"2020-12-02 16:28:12.000000000","message":"Done","commit_id":"0bda5401f9ec6fa178efa949b6e46680c4cc62dc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c0fae3839d8ebf559aafd4db38be34344296af11","unresolved":true,"context_lines":[{"line_number":245,"context_line":"        allocations \u003d self.placement_client.list_allocations(server[\u0027id\u0027])"},{"line_number":246,"context_line":"        self._assert_allocation_is_as_expected(allocations, valid_port[\u0027id\u0027])"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":249,"context_line":"    @decorators.idempotent_id(\u0027c29e7fd3-035d-4993-880f-70819847683f\u0027)"},{"line_number":250,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":251,"context_line":"    def test_resize_with_qos_min_bw_allocation(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"1b5403b1_707c6448","line":248,"range":{"start_line":248,"start_character":0,"end_line":248,"end_character":33},"updated":"2020-12-02 15:46:18.000000000","message":"ditto","commit_id":"0bda5401f9ec6fa178efa949b6e46680c4cc62dc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93c22c9d73fcc95a9e34dd407e75937e9afcb209","unresolved":true,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(0, len(port[\u0027port\u0027][\u0027binding:profile\u0027]))"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    @decorators.idempotent_id(\u00278a98150c-a506-49a5-96c6-73a5e7b04ada\u0027)"},{"line_number":214,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":215,"context_line":"    def test_migrate_with_qos_min_bw_allocation(self):"},{"line_number":216,"context_line":"        \"\"\"Scenario to migrate VM with QoS min bw allocation in placement"},{"line_number":217,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"2f05dbec_74069782","line":214,"range":{"start_line":214,"start_character":40,"end_line":214,"end_character":41},"updated":"2020-12-03 17:56:41.000000000","message":"as this need multinode and cold migration available let\u0027s add skip checks for that\n\n    @testtools.skipUnless(CONF.compute_feature_enabled.cold_migration,\n                          \u0027Cold migration is not available.\u0027)\n    @testtools.skipUnless(CONF.compute.min_compute_nodes \u003e 1,\n                          \u0027Less than 2 compute nodes, skipping multinode \u0027\n                          \u0027tests.\u0027)","commit_id":"0515449c2c5b35ef42da30b6326a0abe14c313f3"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e2a31a05a034b92ee642062e7a89710597af9ebc","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(0, len(port[\u0027port\u0027][\u0027binding:profile\u0027]))"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    @decorators.idempotent_id(\u00278a98150c-a506-49a5-96c6-73a5e7b04ada\u0027)"},{"line_number":214,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":215,"context_line":"    def test_migrate_with_qos_min_bw_allocation(self):"},{"line_number":216,"context_line":"        \"\"\"Scenario to migrate VM with QoS min bw allocation in placement"},{"line_number":217,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"aad340a4_8d37f77f","line":214,"range":{"start_line":214,"start_character":40,"end_line":214,"end_character":41},"in_reply_to":"2f05dbec_74069782","updated":"2020-12-04 10:11:08.000000000","message":"Done","commit_id":"0515449c2c5b35ef42da30b6326a0abe14c313f3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93c22c9d73fcc95a9e34dd407e75937e9afcb209","unresolved":true,"context_lines":[{"line_number":245,"context_line":"        self._assert_allocation_is_as_expected(allocations, valid_port[\u0027id\u0027])"},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"    @decorators.idempotent_id(\u0027c29e7fd3-035d-4993-880f-70819847683f\u0027)"},{"line_number":248,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":249,"context_line":"    def test_resize_with_qos_min_bw_allocation(self):"},{"line_number":250,"context_line":"        \"\"\"Scenario to resize VM with QoS min bw allocation in placement."},{"line_number":251,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"adfd0594_b22f1262","line":248,"range":{"start_line":248,"start_character":40,"end_line":248,"end_character":41},"updated":"2020-12-03 17:56:41.000000000","message":"also we need to skip it if resixe is not available in env\n\n    @testtools.skipUnless(CONF.compute_feature_enabled.resize,\n                          \u0027Resize not available.\u0027)","commit_id":"0515449c2c5b35ef42da30b6326a0abe14c313f3"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e2a31a05a034b92ee642062e7a89710597af9ebc","unresolved":false,"context_lines":[{"line_number":245,"context_line":"        self._assert_allocation_is_as_expected(allocations, valid_port[\u0027id\u0027])"},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"    @decorators.idempotent_id(\u0027c29e7fd3-035d-4993-880f-70819847683f\u0027)"},{"line_number":248,"context_line":"    @utils.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":249,"context_line":"    def test_resize_with_qos_min_bw_allocation(self):"},{"line_number":250,"context_line":"        \"\"\"Scenario to resize VM with QoS min bw allocation in placement."},{"line_number":251,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"cf033159_d0d9f9b0","line":248,"range":{"start_line":248,"start_character":40,"end_line":248,"end_character":41},"in_reply_to":"adfd0594_b22f1262","updated":"2020-12-04 10:11:08.000000000","message":"Done","commit_id":"0515449c2c5b35ef42da30b6326a0abe14c313f3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a1b17d851d0564bd7695a2be391109bccc8bd436","unresolved":true,"context_lines":[{"line_number":10,"context_line":"#    distributed under the License is distributed on an \"AS IS\" BASIS, WITHOUT"},{"line_number":11,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":12,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from tempest.common import utils"},{"line_number":16,"context_line":"from tempest.common import waiters"}],"source_content_type":"text/x-python","patch_set":10,"id":"88d82fab_0e00ce5e","line":13,"range":{"start_line":13,"start_character":22,"end_line":13,"end_character":23},"updated":"2020-12-04 23:09:01.000000000","message":"need to import testtools","commit_id":"c4307461b1789f421a3c4b5007551ca02d0a22e2"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"42dd9ade27bbb262211ad0a2cf7e0431f3a3eb3e","unresolved":true,"context_lines":[{"line_number":10,"context_line":"#    distributed under the License is distributed on an \"AS IS\" BASIS, WITHOUT"},{"line_number":11,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":12,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from tempest.common import utils"},{"line_number":16,"context_line":"from tempest.common import waiters"}],"source_content_type":"text/x-python","patch_set":10,"id":"b94d384c_2174ddc9","line":13,"range":{"start_line":13,"start_character":22,"end_line":13,"end_character":23},"in_reply_to":"88d82fab_0e00ce5e","updated":"2020-12-07 14:43:14.000000000","message":"My bad.....","commit_id":"c4307461b1789f421a3c4b5007551ca02d0a22e2"}],"zuul.d/integrated-gate.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93c22c9d73fcc95a9e34dd407e75937e9afcb209","unresolved":true,"context_lines":[{"line_number":194,"context_line":"    vars:"},{"line_number":195,"context_line":"      devstack_localrc:"},{"line_number":196,"context_line":"        USE_PYTHON3: False"},{"line_number":197,"context_line":"      devstack_local_conf:"},{"line_number":198,"context_line":"        post-config:"},{"line_number":199,"context_line":"          \"/$NEUTRON_CORE_PLUGIN_CONF\":"},{"line_number":200,"context_line":"            ovs:"},{"line_number":201,"context_line":"              bridge_mappings: public:br-ex"},{"line_number":202,"context_line":"              resource_provider_bandwidths: br-ex:1000000:1000000"},{"line_number":203,"context_line":"        test-config:"},{"line_number":204,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":205,"context_line":"            network-feature-enabled:"},{"line_number":206,"context_line":"              qos_placement_physnet: public"},{"line_number":207,"context_line":"    group-vars:"},{"line_number":208,"context_line":"      subnode:"},{"line_number":209,"context_line":"        devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"392e97c4_d7109563","line":206,"range":{"start_line":197,"start_character":0,"end_line":206,"end_character":43},"updated":"2020-12-03 17:56:41.000000000","message":"please add these setting to tempest-full-py3 also so that non multinode test can run in that jobs too.","commit_id":"0515449c2c5b35ef42da30b6326a0abe14c313f3"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e2a31a05a034b92ee642062e7a89710597af9ebc","unresolved":false,"context_lines":[{"line_number":194,"context_line":"    vars:"},{"line_number":195,"context_line":"      devstack_localrc:"},{"line_number":196,"context_line":"        USE_PYTHON3: False"},{"line_number":197,"context_line":"      devstack_local_conf:"},{"line_number":198,"context_line":"        post-config:"},{"line_number":199,"context_line":"          \"/$NEUTRON_CORE_PLUGIN_CONF\":"},{"line_number":200,"context_line":"            ovs:"},{"line_number":201,"context_line":"              bridge_mappings: public:br-ex"},{"line_number":202,"context_line":"              resource_provider_bandwidths: br-ex:1000000:1000000"},{"line_number":203,"context_line":"        test-config:"},{"line_number":204,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":205,"context_line":"            network-feature-enabled:"},{"line_number":206,"context_line":"              qos_placement_physnet: public"},{"line_number":207,"context_line":"    group-vars:"},{"line_number":208,"context_line":"      subnode:"},{"line_number":209,"context_line":"        devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"66a05c67_dc7ed977","line":206,"range":{"start_line":197,"start_character":0,"end_line":206,"end_character":43},"in_reply_to":"392e97c4_d7109563","updated":"2020-12-04 10:11:08.000000000","message":"Done","commit_id":"0515449c2c5b35ef42da30b6326a0abe14c313f3"}]}