)]}'
{"nova/tests/functional/wsgi/test_services.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a948f598419abfaec0960598035198ef601804e2","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        resp \u003d self.placement_api.get(\u0027/resource_providers/%s\u0027 % rp_uuid)"},{"line_number":116,"context_line":"        self.assertEqual(404, resp.status)"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    def test_evacuate_then_delete_compute_service(self):"},{"line_number":119,"context_line":"        \"\"\"Tests a scenario where a server is created on a host, the host"},{"line_number":120,"context_line":"        goes down, the server is evacuated to another host, and then the"},{"line_number":121,"context_line":"        source host compute service is deleted. After that the deleted"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_c56ad2b1","line":118,"range":{"start_line":118,"start_character":8,"end_line":118,"end_character":49},"updated":"2019-06-06 19:24:19.000000000","message":"ci has not run on this yet but testing it locally it passes\nand the test makes sense to +1","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"a366f28428432716296c6ef2f17ac9eab4a04bad","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        rp_uuid \u003d self._get_provider_uuid_by_host(service[\u0027host\u0027])"},{"line_number":135,"context_line":"        # Make sure there is a resource provider for that compute node based"},{"line_number":136,"context_line":"        # on the uuid."},{"line_number":137,"context_line":"        resp \u003d self.placement_api.get(\u0027/resource_providers/%s\u0027 % rp_uuid)"},{"line_number":138,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":139,"context_line":"        # Down the compute service for host1 so we can evacuate from it."},{"line_number":140,"context_line":"        self.admin_api.put_service(service[\u0027id\u0027], {\u0027forced_down\u0027: True})"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_5a7d5de5","line":137,"range":{"start_line":137,"start_character":15,"end_line":137,"end_character":73},"updated":"2019-06-06 19:00:11.000000000","message":"or\n\n self._get_resource_provider_by_uuid(rp_uuid)","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"139a95f2fe0f7d02096851bee03b1a266aa4cf12","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        rp_uuid \u003d self._get_provider_uuid_by_host(service[\u0027host\u0027])"},{"line_number":135,"context_line":"        # Make sure there is a resource provider for that compute node based"},{"line_number":136,"context_line":"        # on the uuid."},{"line_number":137,"context_line":"        resp \u003d self.placement_api.get(\u0027/resource_providers/%s\u0027 % rp_uuid)"},{"line_number":138,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":139,"context_line":"        # Down the compute service for host1 so we can evacuate from it."},{"line_number":140,"context_line":"        self.admin_api.put_service(service[\u0027id\u0027], {\u0027forced_down\u0027: True})"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_65f4465c","line":137,"range":{"start_line":137,"start_character":15,"end_line":137,"end_character":73},"in_reply_to":"9fb8cfa7_1a18457f","updated":"2019-06-06 19:59:06.000000000","message":"_get_resource_provider_by_uuid won\u0027t work because I\u0027m checking the response status and _get_resource_provider_by_uuid only returns the body.","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a948f598419abfaec0960598035198ef601804e2","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        rp_uuid \u003d self._get_provider_uuid_by_host(service[\u0027host\u0027])"},{"line_number":135,"context_line":"        # Make sure there is a resource provider for that compute node based"},{"line_number":136,"context_line":"        # on the uuid."},{"line_number":137,"context_line":"        resp \u003d self.placement_api.get(\u0027/resource_providers/%s\u0027 % rp_uuid)"},{"line_number":138,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":139,"context_line":"        # Down the compute service for host1 so we can evacuate from it."},{"line_number":140,"context_line":"        self.admin_api.put_service(service[\u0027id\u0027], {\u0027forced_down\u0027: True})"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_1a18457f","line":137,"range":{"start_line":137,"start_character":15,"end_line":137,"end_character":73},"in_reply_to":"9fb8cfa7_5a7d5de5","updated":"2019-06-06 19:24:19.000000000","message":"i think matt was just being consitent with line 115 above but ya that would work too.","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"a366f28428432716296c6ef2f17ac9eab4a04bad","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        # FIXME(mriedem): This is bug 1829479 where the compute service is"},{"line_number":158,"context_line":"        # deleted but the resource provider is not because there are still"},{"line_number":159,"context_line":"        # allocations against the provider from the evacuated server."},{"line_number":160,"context_line":"        resp \u003d self.placement_api.get(\u0027/resource_providers/%s\u0027 % rp_uuid)"},{"line_number":161,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":162,"context_line":"        self.assertFlavorMatchesUsage(rp_uuid, flavor)"},{"line_number":163,"context_line":"        # Try to restart the host1 compute service to create a new resource"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_9a72f510","line":160,"range":{"start_line":160,"start_character":15,"end_line":160,"end_character":73},"updated":"2019-06-06 19:00:11.000000000","message":"ditto","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"139a95f2fe0f7d02096851bee03b1a266aa4cf12","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        # FIXME(mriedem): This is bug 1829479 where the compute service is"},{"line_number":158,"context_line":"        # deleted but the resource provider is not because there are still"},{"line_number":159,"context_line":"        # allocations against the provider from the evacuated server."},{"line_number":160,"context_line":"        resp \u003d self.placement_api.get(\u0027/resource_providers/%s\u0027 % rp_uuid)"},{"line_number":161,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":162,"context_line":"        self.assertFlavorMatchesUsage(rp_uuid, flavor)"},{"line_number":163,"context_line":"        # Try to restart the host1 compute service to create a new resource"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_c504124d","line":160,"range":{"start_line":160,"start_character":15,"end_line":160,"end_character":73},"in_reply_to":"9fb8cfa7_9a72f510","updated":"2019-06-06 19:59:06.000000000","message":"same","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a948f598419abfaec0960598035198ef601804e2","unresolved":false,"context_lines":[{"line_number":168,"context_line":"        # new uuid but the same name which results in a conflict. The service"},{"line_number":169,"context_line":"        # does not die, however, because _update_available_resource_for_node"},{"line_number":170,"context_line":"        # catches and logs but does not re-raise the error."},{"line_number":171,"context_line":"        log_output \u003d self.stdlog.logger.output"},{"line_number":172,"context_line":"        self.assertIn(\u0027Error updating resources for node host1.\u0027, log_output)"},{"line_number":173,"context_line":"        self.assertIn(\u0027Failed to create resource provider host1\u0027, log_output)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_45002202","line":171,"range":{"start_line":171,"start_character":6,"end_line":171,"end_character":46},"updated":"2019-06-06 19:24:19.000000000","message":"huh i have never dont that before.\nis the logger shared between all thecompute services in this test?","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"139a95f2fe0f7d02096851bee03b1a266aa4cf12","unresolved":false,"context_lines":[{"line_number":168,"context_line":"        # new uuid but the same name which results in a conflict. The service"},{"line_number":169,"context_line":"        # does not die, however, because _update_available_resource_for_node"},{"line_number":170,"context_line":"        # catches and logs but does not re-raise the error."},{"line_number":171,"context_line":"        log_output \u003d self.stdlog.logger.output"},{"line_number":172,"context_line":"        self.assertIn(\u0027Error updating resources for node host1.\u0027, log_output)"},{"line_number":173,"context_line":"        self.assertIn(\u0027Failed to create resource provider host1\u0027, log_output)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_85fe1a35","line":171,"range":{"start_line":171,"start_character":6,"end_line":171,"end_character":46},"in_reply_to":"9fb8cfa7_45002202","updated":"2019-06-06 19:59:06.000000000","message":"I believe so yeah. I learned this from one of gibi\u0027s tests.","commit_id":"2629d65fbc15d8698f98117e0d6072810f70da03"}]}