| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-01 | |||
| 15:05:52 | lyarwood | mriedem: https://review.openstack.org/#/q/topic:bug/1784353 - nothing urgent, but if you have time this week would you mind taking a look at this BFV reschedule bug? | |
| 15:06:34 | bauzas | mriedem: well, I'm not really torn for your change | |
| 15:07:06 | mriedem | bauzas: you don't see the need for that cache? | |
| 15:07:12 | bauzas | mriedem: I mean, are we like calling a RPC API when we look at the bdms ? | |
| 15:07:18 | mriedem | yes | |
| 15:07:18 | mriedem | dude | |
| 15:07:35 | mriedem | we're getting all bdms for each instance to determine if it's volume-backed | |
| 15:07:44 | mriedem | that's a db query roundtrip through conductor | |
| 15:08:05 | bauzas | objects.BlockDeviceMappingList.get_by_instance_uuid() ? | |
| 15:08:06 | mriedem | and the bfv-ness of an instance isn't going to change | |
| 15:08:24 | bauzas | ah okay | |
| 15:08:27 | mriedem | yes https://github.com/openstack/nova/blob/master/nova/compute/utils.py#L215 | |
| 15:10:24 | bauzas | okay +2 then | |
| 15:11:33 | mriedem | thanks | |
| 15:11:49 | mriedem | lyarwood: sure | |
| 15:13:19 | lyarwood | mriedem: cheers, the actual bugfix is WIP as I'm not sure where the fix should land, any advice would be appreciated. | |
| 15:27:31 | gnuoy | Hi, I'm trying to understand this sentence from the cells layout doc: "The Neutron metadata API proxy should be global across all cells, and thus be configured as an API-level service with access to the [api_database]/connection information." | |
| 15:28:04 | openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova master: Hide hypervisor id on windows guests https://review.openstack.org/579897 | |
| 15:28:10 | gnuoy | Is it honestly saying that the neutron metadata proxy needs access to the nova api db ? | |
| 15:28:39 | gnuoy | https://docs.openstack.org/nova/latest/user/cellsv2-layout.html#neutron-metadata-api-proxy | |
| 15:29:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Improve NeutronFixture and remove unncessary stubbing https://review.openstack.org/587412 | |
| 15:31:39 | kosamara | sean-k-mooney: updated https://review.openstack.org/#/c/579897 | |
| 15:42:40 | openstackgerrit | Chen proposed openstack/nova master: Make nova-manage db archive_delete_rows take --all-cells https://review.openstack.org/587858 | |
| 15:57:33 | sean-k-mooney | kosamara: +1 from me. its annoying that that is needed but ya the only other thing i can think of is to remove the os_type=windows from the glance image | |
| 15:58:23 | sean-k-mooney | dansmith: see the message form openstackgerrit above | |
| 15:58:41 | dansmith | sean-k-mooney: yeah I see :) | |
| 15:58:56 | sean-k-mooney | dansmith: i guess infra turned it on across the board do we want to keep it on? | |
| 15:59:07 | dansmith | not sure we have a choice | |
| 15:59:09 | kosamara | sean-k-mooney: thanks | |
| 15:59:18 | dansmith | I was just discussing with them | |
| 15:59:27 | dansmith | apparently the sniper doesn't work as well in all channels, for whatever reason, | |
| 15:59:29 | dansmith | but still, | |
| 15:59:56 | dansmith | like email, managing spam is just one of those things.. closing the door on an open community is a bad solution, IMHO, but whatever | |
| 16:00:00 | sean-k-mooney | ok well hopefully this is temporay | |
| 16:00:38 | dansmith | until they just automate the registration process? yeah:) | |
| 16:01:44 | sean-k-mooney | ya... then we will learn how recapha's work over irc... | |
| 16:05:52 | openstackgerrit | Merged openstack/nova master: Add shelve/unshelve wrinkle to volume-backed disk func test https://review.openstack.org/584931 | |
| 16:13:43 | jangutter | sean-k-mooney: you guys are OK, but I continually fail my Turing test. | |
| 16:14:57 | stephenfin | I always knew there was something fishy about jungleboyj | |
| 16:15:27 | stephenfin | (RE: "jungleboyj has quit (Killed (Sigyn (Spam is off topic on freenode.)))") | |
| 16:15:56 | tssurya | dansmith: are we having a cells meeting today? | |
| 16:16:19 | dansmith | tssurya: I don't have anything to discuss and am also focused on other things, you? | |
| 16:16:24 | dansmith | mriedem_afk: melwitt ^ | |
| 16:16:25 | tssurya | I don't have anything either | |
| 16:16:56 | melwitt | +1 to skip | |
| 16:16:59 | tbachman | /WHOIS Turing^H^H^H^H | |
| 16:29:34 | dansmith | gawd, the testing of resource tracker... ugh | |
| 16:32:38 | melwitt | the ol RT | |
| 16:38:11 | mriedem_afk | dansmith: i had some things posted on the agenda | |
| 16:38:31 | dansmith | mriedem: who looks at that? | |
| 16:38:36 | mriedem | https://wiki.openstack.org/wiki/Meetings/NovaCellsv2#Agenda | |
| 16:38:49 | mriedem | i'll just dump them here | |
| 16:38:51 | mriedem | couple of new bugs | |
| 16:38:53 | openstack | Launchpad bug 1784074 in OpenStack Compute (nova) "Instances end up with no cell assigned in instance_mappings" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 16:38:53 | mriedem | https://bugs.launchpad.net/nova/+bug/1784074 | |
| 16:39:10 | melwitt | gnuoy: yes, it does need access to the api database in order to 'get_metadata_by_instance_id' it looks at the api database to determine what cell the instance is in | |
| 16:39:17 | mriedem | i started https://review.openstack.org/#/c/586713/ for that but need to do more probably, like collapse those 2 for loops as we said a year ago | |
| 16:39:37 | dansmith | mriedem: yeah, I'm not really sure where we are on that.. there are several patches up | |
| 16:39:41 | mriedem | also, mnaser has a script in that bug for healing some of these types of faliures | |
| 16:39:46 | openstack | Launchpad bug 1784093 in OpenStack Compute (nova) "Build requests can be orphaned without instance mappings" [Medium,In progress] - Assigned to Mohammed Naser (mnaser) | |
| 16:39:46 | mriedem | https://bugs.launchpad.net/nova/+bug/1784093 | |
| 16:39:59 | mriedem | ^ is about creating build requests, instance mappings and request specs in a single transaction | |
| 16:40:26 | mriedem | and then i was looking for a 2nd +2 on https://review.openstack.org/#/c/581910/ which is just documenting another up-call caveat | |
| 16:40:45 | dansmith | yeah, I think we also probably want a single transaction for converting a BR to a mapped mapping | |
| 16:41:04 | mriedem | i.e. create the instance in the cell + update the mapping? | |
| 16:41:14 | mriedem | how would that work across 2 dbs? | |
| 16:41:24 | melwitt | and destroying BR | |
| 16:41:27 | mnaser | my single transaction change, im not sure if it actually does it in a single transaction.. nor how to test it | |
| 16:41:45 | melwitt | mriedem: should be all api db, no? build request and instance mapping | |
| 16:41:46 | mnaser | i brought the requestspec, buildrequest and mapping to be a single transaction so that was a single db | |
| 16:42:25 | dansmith | mriedem: I mean the destroying the BR and setting cell_id on the instance mapping in one transaction | |
| 16:42:39 | dansmith | melwitt: right | |
| 16:43:26 | mnaser | i assume mriedem is concerned about updating instance mapping and creating instance in a single transaction | |
| 16:43:43 | mnaser | cause you can create but fail updating instance mapping | |
| 16:43:51 | mnaser | and end up with a listable but not retrievable instance | |
| 16:43:59 | mriedem | that's what https://review.openstack.org/#/c/586713/ is for | |
| 16:44:16 | mriedem | we update the mapping immediately after creating the instance, | |
| 16:44:25 | mriedem | but i'm not sure what should happen if we fail to update the mapping | |
| 16:44:32 | dansmith | destroy the instance | |
| 16:44:37 | dansmith | leave the BR in place | |
| 16:44:44 | mriedem | that's what it's doing right now | |
| 16:44:45 | mriedem | my patch | |
| 16:44:51 | dansmith | that's what we should do | |
| 16:44:57 | melwitt | mnaser: yeah, would have to take a look at what we can do with finding that out. jaypipes do you know how we can print/log whether something is occurring in a single db transaction when we're doing sqlalchemy queries while we're testing something? | |
| 16:45:00 | dansmith | if we double fail on all DBs then we can't do anything anyway | |
| 16:45:00 | mriedem | but, | |
| 16:45:12 | mriedem | if we have an empty mapping and no instance and a build request, | |
| 16:45:23 | dansmith | melwitt: one writer should mean one transaction AFAIK | |
| 16:45:24 | mriedem | the build request will always show as status BUILDING or whatever | |
| 16:45:53 | mnaser | thing is stale records in db that don't impact api aren't too much of a problem | |
| 16:46:19 | mnaser | having an instance mapping point to a cell where the instance doesnt exist => no confusing instance when listing instances | |
| 16:46:20 | dansmith | mriedem: the other thing we can do, which I'd rather not, but is look up mappings during a list and either filter those instances or fix the mappings on the fly | |
| 16:46:24 | mriedem | i don't think you can delete a scheduling instance though, unless you force delete it? | |
| 16:46:46 | dansmith | mriedem: I mean NFO the record you just created | |
| 16:46:46 | mriedem | dansmith: i thought about fixing on the fly, | |
| 16:46:47 | mriedem | however, | |
| 16:47:09 | mriedem | if we create the instance but fail to update the mapping, we won't have created the bdms and tags in the cell db either | |
| 16:47:11 | melwitt | dansmith: it should, but IIRC there are ways in which queries might not be a single transaction (from a db standpoint) even if you put them under the same decorator. it would be nice to be able to verify if we could | |
| 16:47:16 | mriedem | so the instance could be mapped, but it would be incomplete | |
| 16:47:42 | dansmith | melwitt: only if you're doing nested transactions or force commit, AFAIK | |