Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-13
16:30:45 mriedem you shouldn't have n-cell-region or n-cell-child
16:31:32 raghav mriedem: n-api,n-cpu,n-cond,n-sch,n-novnc,n-cauth
16:31:37 mriedem sorry just n-cell
16:36:40 mriedem raghav: in general, we don't want int id's exposed out of the API, which is why we have https://blueprints.launchpad.net/nova/+spec/service-hyper-pci-uuid-in-api
16:36:55 mriedem raghav: but that's going to be done in a microversion
16:37:57 sdague raghav: honestly, I think there is a lot of confusion about how you got to that kind of data, and the only way any of us know that to be possible is if you have a cells v1 setup
16:38:08 sdague raghav: so, perhaps a reproduce scenario would be useful
16:38:15 mriedem raghav: for now os-hypervisors will return an int for the id https://developer.openstack.org/api-ref/compute/?expanded=show-hypervisor-statistics-detail,show-hypervisor-details-detail
16:38:19 mriedem as shown in the docs
16:38:27 mriedem my blueprint is to change that to return the uuid rather than the int id
16:38:48 melwitt sdague: in cells v1, we don't officially support flavors in cell dbs. users like CERN have their own automation that syncs flavors from the api cell db to the child cell dbs. fwiw
16:39:08 mriedem raghav: as for the hypervisor hostname being "region!child@1" that looks like cellsv1
16:39:18 mriedem so it seems you have n-cells running somewhere in your devstack setup
16:39:26 sdague melwitt: right, it was definitely a goofy thing
16:40:25 raghav mriedem: yes its running
16:40:41 raghav mriedem: i could see in screen
16:40:49 mriedem raghav: yeah, so you don't want that
16:41:00 mriedem raghav: that's going to cause you all kinds of confusion
16:41:06 mriedem so i'd start over w/o n-cell enabled
16:41:13 raghav mriedem: so what should i do now
16:41:38 mriedem raghav: restack w/o n-cell for one
16:41:41 raghav mriedem: in local.conf i have not enabled n-cell
16:42:19 mriedem raghav: well, it's not enabled by default, so something is running it in your setup
16:42:40 mriedem raghav: also, be aware that cells v2, which is required in ocata, is not the same as the nova-cells service,
16:42:44 mriedem or n-cell service in devstack
16:42:47 mriedem i know it's confusing
16:43:58 raghav mriedem: i have n-cells-region and n-cells-child running
16:45:06 raghav mriedem: if i re-stack again those will get enabled
16:45:58 sdague raghav: why do you have those running?
16:46:35 raghav mriedem: i dont know how they are running, i just stacked without enabling them
16:46:36 melwitt raghav: what's in your stack-screenrc? are there any n-cell* in there?
16:47:01 raghav melwitt: yes n-cell-region and n-cell-chile
16:47:08 raghav melwitt: yes n-cell-region and n-cell-child
16:47:08 sdague raghav: so... step one, please get back to a base clean environment
16:48:07 melwitt raghav: okay. so it looks like you definitely have remnants of an earlier install where n-cell was part of ENABLED_SERVICES
16:48:16 mriedem dansmith: did you still want a specific unit test for https://review.openstack.org/#/c/443403/11/nova/context.py ?
16:49:09 openstackgerrit Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936
16:49:15 dansmith mriedem: yes, I'm not sure why we wouldn't
16:49:20 raghav melwitt: how can i remove it
16:49:32 dansmith mriedem: are you doing both of those in a follow-up?
16:50:49 mriedem dansmith: i can do them in this change
16:51:16 mriedem raghav: maybe posting your local.conf in a gist would help us
16:51:29 mriedem raghav: personally i'd re-clone devstack from stable/ocata to make sure it's completely clean
16:51:32 mriedem in a new vm
16:53:13 openstackgerrit John Garbutt proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809
16:53:25 melwitt yeah, I'm not 100% an unstack.sh/stack.sh after killing processes would result in a fresh read of ENABLED_SERVICES
16:53:44 mriedem i would completely start over
16:53:44 sdague that should, but a ./clean.sh is even safer
16:53:54 sdague yeh, completely fresh not a bad idea either
16:53:56 openstackgerrit John Garbutt proposed openstack/nova-specs master: WIP: Replace live migration steps and delays with memory iteration logic https://review.openstack.org/443339
16:54:04 sdague there are clearly bits left over from some past config
16:56:35 raghav ok
16:56:37 raghav thanks
17:15:08 openstackgerrit Juan Antonio Osorio Robles proposed openstack/nova master: Make versioned notifications topics configurable https://review.openstack.org/444947
17:30:04 openstackgerrit melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521
17:30:04 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
17:30:05 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
17:35:04 mriedem dansmith: i'm really confused about this test https://github.com/openstack/nova/blob/master/nova/tests/unit/test_context.py#L307-L310
17:35:21 mriedem shouldn't the assertion be such that the context db/mq connections within the target_cell context match what's in the mapping?
17:36:11 dansmith mriedem: it does
17:36:52 dansmith mriedem: maybe it should assert that get_context is called with the database url, and same for rpc, but otherwise it's checking that they get switched and restored
17:39:01 mriedem oh nvm
17:39:09 mriedem i was expecting to see it have the fake urls
17:46:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Decrement quota usage when deleting an instance in cell0 https://review.openstack.org/443395
17:46:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Temporarily untarget context when deleting from cell0 https://review.openstack.org/443403
18:01:05 openstackgerrit Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754
18:01:05 openstackgerrit Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
18:01:06 openstackgerrit Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163
18:01:06 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
18:01:07 openstackgerrit Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174
18:01:07 openstackgerrit Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292
18:01:08 openstackgerrit Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487
18:01:08 openstackgerrit Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293
18:01:09 openstackgerrit Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861
18:01:09 openstackgerrit Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025
18:01:10 openstackgerrit Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
18:01:10 openstackgerrit Dan Smith proposed openstack/nova master: Add SameCellFilter https://review.openstack.org/445035
18:05:09 dansmith johnthetubaguy: have you gone away for the day yet?
18:05:51 melwitt hah, my counting quota patch failed mriedem's recent quota regression functional test (the expected fail part) because I forgot it also fixes the problem
18:06:31 johnthetubaguy dansmith: about to go
18:06:42 johnthetubaguy dansmith: whats up?
18:06:48 dansmith johnthetubaguy: okay well, just heads up on https://review.openstack.org/#/c/445035/
18:08:03 johnthetubaguy dansmith: ah, cool, I will try go through that chain again tomorrow
18:08:44 johnthetubaguy dansmith: I forgot we don't get the cell from the host object already, which kinda sucks, but that cache seems like a good stop gap
18:09:57 dansmith johnthetubaguy: it should fill up pretty fast, not require too much memory, and only change when we add a host, so not too bad I think
18:16:36 siva_krishnan johnthetubaguy: can you unabandon thhese patches https://review.openstack.org/#/c/308198/ https://review.openstack.org/#/c/328280/ ?
18:52:02 mriedem rfolco: do you know who juergen leopold at ibm is? running ci on nova. i'm guessing a z person.
18:52:46 rfolco mriedem, hmm let me check, I knew Arne from zKVM CI from Germany these days...
18:54:39 rfolco mriedem, I guess his on the same team. They were having problems with their CI last week.
18:54:56 mriedem ok, i just emailed him and copied markus
18:55:05 mriedem asking to start using a system id rather than his email
18:55:12 mriedem and just asking what he's running
19:01:43 openstackgerrit Pushkar Umaranikar proposed openstack/nova master: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259
19:13:06 openstackgerrit Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228
19:13:51 openstackgerrit Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298
19:14:01 openstackgerrit Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185
19:14:11 openstackgerrit Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146
19:24:24 cfriesen someone pointed out to me that InstanceActionEvent.pack_action_event_finish() uses a "message" field, but the InstanceActionEvent model in the DB uses "details". Is this a bug?
19:26:08 cfriesen and given that it's been in there since 2014, is anyone actually looking at it?
19:28:50 openstackgerrit Pushkar Umaranikar proposed openstack/nova master: Remove config_drive attribute from build_requests table https://review.openstack.org/432511

Earlier   Later