| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-13 | |||
| 16:29:34 | mriedem | that looks like cellsv1 | |
| 16:30:16 | raghav | mriedem: i have devstack setup | |
| 16:30:22 | mriedem | raghav: note that you don't want to be using cellsv1 | |
| 16:30:29 | raghav | mriedem: with ocata release | |
| 16:30:35 | mriedem | raghav: what's your ENABLED_SERVICES variable? | |
| 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 | sdague | raghav: so... step one, please get back to a base clean environment | |
| 16:47:08 | raghav | melwitt: yes n-cell-region and n-cell-child | |
| 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 | sdague | that should, but a ./clean.sh is even safer | |
| 16:53:44 | mriedem | i would completely start over | |
| 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: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 17:30:04 | openstackgerrit | melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | |
| 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: Teach HostAPI about cells https://review.openstack.org/442162 | |
| 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: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:06 | openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | |
| 18:01:07 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | |
| 18:01:07 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 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:08 | openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | |
| 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:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861 | |
| 18:01:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Add SameCellFilter https://review.openstack.org/445035 | |
| 18:01:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |
| 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 | |