)]}'
{"nova/api/openstack/wsgi_app.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"36d63cbbba7c49efbe9fd91e14e592692c8def0c","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            service_obj.topic \u003d None"},{"line_number":58,"context_line":"            service_obj.report_count \u003d 0"},{"line_number":59,"context_line":"            service_obj.create()"},{"line_number":60,"context_line":"            LOG.info(\u0027Created service record for binary %s with version %s\u0027,"},{"line_number":61,"context_line":"                     binary, service_obj.version)"},{"line_number":62,"context_line":"        except (exception.ServiceTopicExists,"},{"line_number":63,"context_line":"                exception.ServiceBinaryExists):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_3a09d9da","line":60,"range":{"start_line":60,"start_character":12,"end_line":60,"end_character":15},"updated":"2018-12-11 21:02:38.000000000","message":"damn\n\nhttp://logs.openstack.org/38/624438/1/check/tempest-full/64de2ef/controller/logs/screen-n-api.txt.gz#_Dec_11_18_28_14_615480","commit_id":"3547dcddd578ea3131fc3d91c5a9de180c094123"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6dd99b6d5f19024eabf26d00f9cd0dfe3fe03bc9","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            service_obj.report_count \u003d 0"},{"line_number":59,"context_line":"            service_obj.create()"},{"line_number":60,"context_line":"            logging.getLogger(__name__).info("},{"line_number":61,"context_line":"                \u0027Created service record for binary %s with version %s\u0027,"},{"line_number":62,"context_line":"                binary, service_obj.version)"},{"line_number":63,"context_line":"        except (exception.ServiceTopicExists,"},{"line_number":64,"context_line":"                exception.ServiceBinaryExists):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_2e9afbc2","line":61,"updated":"2018-12-12 00:51:38.000000000","message":"Looks good:\n\nhttp://logs.openstack.org/38/624438/2/check/tempest-full/c51f2aa/controller/logs/screen-n-api.txt.gz#_Dec_11_22_56_33_854813\n\nDec 11 22:56:33.854813 ubuntu-xenial-inap-mtl01-0001136117 devstack@n-api.service[6565]: INFO nova.api.openstack.wsgi_app [None req-050c42c5-fb67-4c6c-8020-10ec403d1b5b None None] Created service record for binary nova-osapi_compute with version 36","commit_id":"2b13abb0c91012f436beb115f928ddd40933b51c"}],"nova/compute/api.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6dd99b6d5f19024eabf26d00f9cd0dfe3fe03bc9","unresolved":false,"context_lines":[{"line_number":2459,"context_line":"            # deployments have upgraded from a base cells v2 setup with"},{"line_number":2460,"context_line":"            # mappings."},{"line_number":2461,"context_line":"            if not CONF.cells.enable:"},{"line_number":2462,"context_line":"                LOG.warning(\u0027The nova-osapi_compute service version (%s) is \u0027"},{"line_number":2463,"context_line":"                            \u0027from before Ocata and may cause problems looking \u0027"},{"line_number":2464,"context_line":"                            \u0027up instances in a cells v2 setup. Check your \u0027"},{"line_number":2465,"context_line":"                            \u0027nova-api service configuration and cell \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_8e7eaf6a","line":2462,"updated":"2018-12-12 00:51:38.000000000","message":"Hit this and the version is 0:\n\nhttp://logs.openstack.org/38/624438/2/check/tempest-full/c51f2aa/controller/logs/screen-n-api.txt.gz#_Dec_11_23_04_57_191719\n\nDec 11 23:04:57.191719 ubuntu-xenial-inap-mtl01-0001136117 devstack@n-api.service[6565]: WARNING nova.compute.api [None req-958da0f6-4472-4e06-9515-dce59932d249 tempest-ServersAdminTestJSON-1780563434 tempest-ServersAdminTestJSON-1780563434] The nova-osapi_compute service version (0) is from before Ocata and may cause problems looking up instances in a cells v2 setup. Check your nova-api service configuration and cell mappings. You may need to remove stale nova-osapi_compute service records from the cell database.\nDec 11 23:04:57.200291 ubuntu-xenial-inap-mtl01-0001136117 devstack@n-api.service[6565]: INFO nova.compute.api [None req-958da0f6-4472-4e06-9515-dce59932d249 tempest-ServersAdminTestJSON-1780563434 tempest-ServersAdminTestJSON-1780563434] Found 0 nova-osapi_compute services in DB: mysql+pymysql://root:secretdatabase@127.0.0.1/nova_cell0?charset\u003dutf8\nDec 11 23:04:57.200664 ubuntu-xenial-inap-mtl01-0001136117 devstack@n-api.service[6565]: INFO nova.compute.api [None req-958da0f6-4472-4e06-9515-dce59932d249 tempest-ServersAdminTestJSON-1780563434 tempest-ServersAdminTestJSON-1780563434] Service version cache: {u\u0027nova-osapi_compute\u0027: 36}\n\nSo we\u0027re getting 0 back for the min service version, but there are 0 entries in the services table in cell0, even though we created a service record with version 36, and the cache has 36 in it, so why isn\u0027t objects.Service.get_minimum_version using the cache?\n\nI wonder if the context is targeted to cell1?\n\nI sort of remember bug 1757472 being somewhat related to this, but it\u0027s different. The context must be targeted at cell1...because earlier in the logs for the same request ID we lookup the instance and it doesn\u0027t warn about an ocata API service version:\n\nhttp://logs.openstack.org/38/624438/2/check/tempest-full/c51f2aa/controller/logs/screen-n-api.txt.gz#_Dec_11_23_04_56_794396\n\nSo where is _get_instance being re-entered with a targeted context?","commit_id":"2b13abb0c91012f436beb115f928ddd40933b51c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"997e18baa715872486f7001927836327ac46c7a4","unresolved":false,"context_lines":[{"line_number":2459,"context_line":"            # deployments have upgraded from a base cells v2 setup with"},{"line_number":2460,"context_line":"            # mappings."},{"line_number":2461,"context_line":"            if not CONF.cells.enable:"},{"line_number":2462,"context_line":"                LOG.warning(\u0027The nova-osapi_compute service version (%s) is \u0027"},{"line_number":2463,"context_line":"                            \u0027from before Ocata and may cause problems looking \u0027"},{"line_number":2464,"context_line":"                            \u0027up instances in a cells v2 setup. Check your \u0027"},{"line_number":2465,"context_line":"                            \u0027nova-api service configuration and cell \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_aef7ebd2","line":2462,"in_reply_to":"3f79a3b5_8e7eaf6a","updated":"2018-12-12 00:53:08.000000000","message":"Next step here is probably logging a traceback if the context is already targeted and we hit this warning.","commit_id":"2b13abb0c91012f436beb115f928ddd40933b51c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"693fa3dbbf93da40750001d3073b707d3339f69e","unresolved":false,"context_lines":[{"line_number":2470,"context_line":"                # Is the context already targeted? Because if so, we don\u0027t"},{"line_number":2471,"context_line":"                # need the warning."},{"line_number":2472,"context_line":"                if context.db_connection:"},{"line_number":2473,"context_line":"                    traceback.print_stack()"},{"line_number":2474,"context_line":"                # Get all of the API services in the DB to see if one is bad."},{"line_number":2475,"context_line":"                services \u003d objects.ServiceList.get_by_binary("},{"line_number":2476,"context_line":"                    context, \u0027nova-osapi_compute\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_9a57bb5d","line":2473,"updated":"2018-12-12 14:23:26.000000000","message":"http://logs.openstack.org/38/624438/3/check/tempest-full-py3/0dc3c7e/controller/logs/screen-n-api.txt.gz#_Dec_12_01_50_09_309255\n\nshows we\u0027re hitting this during rebuild:\n\nDec 12 01:50:09.309255 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/compute/servers.py\", line 991, in _action_rebuild\nDec 12 01:50:09.309476 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     instance \u003d self._get_server(context, req, id, is_detail\u003dTrue)\nDec 12 01:50:09.309785 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/compute/servers.py\", line 288, in _get_server\nDec 12 01:50:09.310113 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     cell_down_support\u003dFalse)\nDec 12 01:50:09.310366 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/common.py\", line 469, in get_instance\nDec 12 01:50:09.310607 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     cell_down_support\u003dcell_down_support)\nDec 12 01:50:09.310844 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/compute/api.py\", line 2536, in get\nDec 12 01:50:09.311059 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     expected_attrs, cell_down_support\u003dcell_down_support)\nDec 12 01:50:09.311284 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/compute/api.py\", line 2473, in _get_instance\nDec 12 01:50:09.311509 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     traceback.print_stack()\n\nand other extensions:\n\nDec 12 01:55:19.365824 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/compute/attach_interfaces.py\", line 156, in create\nDec 12 01:55:19.366026 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     return self.show(req, server_id, vif[\u0027id\u0027])\nDec 12 01:55:19.366227 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/wsgi.py\", line 655, in wrapped\nDec 12 01:55:19.366421 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     return f(*args, **kwargs)\nDec 12 01:55:19.366616 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/compute/attach_interfaces.py\", line 83, in show\nDec 12 01:55:19.366820 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     common.get_instance(self.compute_api, context, server_id)\nDec 12 01:55:19.367015 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/api/openstack/common.py\", line 469, in get_instance\nDec 12 01:55:19.367218 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     cell_down_support\u003dcell_down_support)\nDec 12 01:55:19.367421 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/compute/api.py\", line 2536, in get\nDec 12 01:55:19.367622 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     expected_attrs, cell_down_support\u003dcell_down_support)\nDec 12 01:55:19.367822 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:   File \"/opt/stack/nova/nova/compute/api.py\", line 2473, in _get_instance\nDec 12 01:55:19.368023 ubuntu-xenial-rax-iad-0001138939 devstack@n-api.service[13125]:     traceback.print_stack()\n\nIn both of those cases, we lookup the instance from the db using an untargeted context, so we get the instance mapping to figure out which cell the instance is in, and then we target the context. Then later the instance is retrieved again, this time with the targeted context, to build the response.\n\nSo the warning is unnecessary but nothing is harmed either way. I\u0027m not sure why this just changed in Stein though.","commit_id":"5dfea9f29336951c7209a0cc59b329513eaa1f46"}]}