| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-04 | |||
| 17:09:18 | gibi | bauzas: does compute/manager have access to the ProvideTree? | |
| 17:09:19 | bauzas | gibi: but you understand my concern, right? | |
| 17:09:32 | dansmith | mriedem: the evac case seems ripe for just moving where we wait, but I haven't gone digging | |
| 17:09:39 | sean-k-mooney | the flag would only fix the rollback case. artom change would fix the cisco aci/old odl case | |
| 17:09:39 | bauzas | gibi: that's the problem, I don't think so | |
| 17:09:41 | gibi | bauzas: I understand that we call placement a lot | |
| 17:09:47 | sean-k-mooney | in addtion to rollbacks | |
| 17:10:12 | dansmith | artom: thinking of this: vifs_already_plugged=False, | |
| 17:10:18 | bauzas | gibi: and say you have two pci requests for the same interface... | |
| 17:10:22 | mriedem | gibi: bauzas: pretty sure the compute manager has a handle to the provider tree, | |
| 17:10:26 | bauzas | it's a bit sad to query it twice | |
| 17:10:27 | mriedem | because it has to clear it on sighup | |
| 17:10:41 | mriedem | dansmith: i don't know the answer to that offhand | |
| 17:10:41 | gibi | mriedem: ooh | |
| 17:10:47 | artom | dansmith, ah, OK. It's still only a bandaid tho... | |
| 17:10:59 | mriedem | gibi: bauzas: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L551 | |
| 17:11:02 | artom | dansmith, thanks, that'll allow me to eat :P | |
| 17:11:10 | gibi | bauzas: sure your concern is valid either I would like to use a ProviderTree or add a local cache | |
| 17:12:00 | gibi | mriedem: looking | |
| 17:12:13 | mdbooth | mriedem: The purpose of the timestamp stuff IIUC is to assert that the contents of the volume are preserved across the migration. I personally think it's an important test, but if it's problematic in practise we'd need to be pragmatic, e.g. removing or guarding it. NP if you don't have a strong opinion, lyarwood and I can fling fedoras at each other internally :) | |
| 17:13:26 | gibi | bauzas, mriedem: so here in the compute/manager we only need the name of the RP and we know the UUID of it. So I guess I should be able to use the ProvideTree cache for that | |
| 17:13:26 | mriedem | well, give a justifiable reason in the commit message to remove it | |
| 17:14:27 | mriedem | need another core on the 2.71 novaclient change https://review.openstack.org/#/c/640657/ so we can get that into the final release | |
| 17:14:43 | bauzas | gibi: the cache is a non-public information from the reportclient | |
| 17:14:51 | bauzas | gibi: at the moment, I mean | |
| 17:15:10 | bauzas | mriedem: on it | |
| 17:15:27 | gibi | bauzas: but I call reportclient anyhow so I can call something else on the reportclient that uses the internal cache | |
| 17:16:20 | gibi | bauzas: it I fail then I will add a local cache to my code to query each RP once | |
| 17:16:27 | bauzas | gibi: sure, I'm just saying that if you wanna use its cache, you have to make its attribute accessible | |
| 17:16:44 | bauzas | either thru a getter or whatever | |
| 17:17:08 | gibi | bauzas: I will go with a new method (if not exists already) in reportclient that reads an RP from the cache | |
| 17:17:09 | bauzas | I dislike at most accessing internal attributes in python | |
| 17:17:17 | bauzas | even if that's allowed | |
| 17:17:37 | bauzas | gibi: fine with me | |
| 17:18:01 | bauzas | my point being "please don't just access self.reportclient.provider_tree" or whatever it's named | |
| 17:20:17 | gibi | bauzas: understood | |
| 17:20:25 | bauzas | sorry but I have to bail out | |
| 17:20:28 | bauzas | ++ | |
| 17:24:54 | gibi | bauzas: thanks a lot | |
| 17:24:59 | gibi | bauzas: I will bail soon as well | |
| 17:25:09 | gibi | bauzas: I will finish updating the patch tomorrow morning | |
| 17:29:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: fakelibvirt: Add ability to generate fake PCI devices https://review.openstack.org/640409 | |
| 17:29:21 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Create PCI tests for NUMA'y tests https://review.openstack.org/640826 | |
| 17:31:41 | openstackgerrit | Chris Friesen proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706 | |
| 17:32:09 | cfriesen | mriedem: this addresses almost all your comments. will split out the hardware.py stuff as you asked in the next revision | |
| 17:56:59 | openstackgerrit | Chris Dent proposed openstack/nova master: Correct lower-constraints.txt and the related tox job https://review.openstack.org/622972 | |
| 17:57:19 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: qemu: Make disk image conversion dramatically faster https://review.openstack.org/640781 | |
| 17:58:52 | mriedem | cfriesen: wouldn't a deactivate image (even if it doesn't change) on resize fail the resize if the dest host can't download the image (and it's not cached)? | |
| 18:00:08 | cfriesen | mriedem: good question. I seem to remember some scenarios where it copies it from the cache on the source...but let me check. | |
| 18:04:52 | mriedem | maybe if you're using shared storage? | |
| 18:04:55 | mriedem | but not everyone does | |
| 18:05:13 | mriedem | i believe the libvirt driver code falls back to pull from the source if it's not in the dest image cache | |
| 18:05:16 | mriedem | or tries to | |
| 18:05:22 | mriedem | mdbooth probably knows better | |
| 18:18:04 | aspiers | gah, looks like functional tests for https://review.openstack.org/#/c/538498/ failed in the gate | |
| 18:18:23 | aspiers | http://logs.openstack.org/98/538498/22/gate/nova-tox-functional/d76478b/testr_results.html.gz has a bunch of errors like "oslo_config.cfg.NoSuchOptError: no such option sync_on_startup in group [placement_database]" | |
| 18:18:28 | aspiers | does that make sense to anyone | |
| 18:18:30 | aspiers | ? | |
| 18:24:47 | cdent | aspiers: that's interesting, it should default to false | |
| 18:25:07 | cdent | except that's not what the error says, it says the conf option doesn't exist | |
| 18:25:11 | aspiers | right | |
| 18:25:19 | aspiers | totally weird that this should appear in the gate but not in the check | |
| 18:25:41 | cdent | which means the gate doesn't have the right code... | |
| 18:25:56 | sean-k-mooney | aspiers ist it posble that it was usign the inrepo placement? | |
| 18:26:43 | sean-k-mooney | it does not exist here https://github.com/openstack/nova/blob/master/nova/conf/placement.py | |
| 18:27:11 | cdent | sean-k-mooney: the fix should be using an indepdent conf | |
| 18:27:34 | cdent | and according to the failures it's using the external placement | |
| 18:27:47 | cdent | however | |
| 18:27:57 | cdent | it might be that the fixture needs to be tweakd. | |
| 18:28:15 | cdent | aspiers: if you have the time can you try to run the functional tests locally on a clean tox? | |
| 18:28:32 | aspiers | cdent: sure | |
| 18:28:51 | sean-k-mooney | well looking at code search it only exists in placement http://codesearch.openstack.org/?q=sync_on_startup&i=nope&files=&repos= | |
| 18:28:58 | aspiers | right | |
| 18:29:24 | aspiers | cdent: but the zuul check already did that and passed ... | |
| 18:30:10 | cdent | aspiers: yes, but that change (to add that config) merged to placement several minutes ago so maybe it wasn't there in check | |
| 18:31:40 | sean-k-mooney | cdent: if it only merged a few minuts ago its not in pypi | |
| 18:31:56 | sean-k-mooney | the last relase was feb 13th https://pypi.org/project/osc-placement/#history | |
| 18:32:05 | sean-k-mooney | sorry wrong repo | |
| 18:32:25 | sean-k-mooney | january 7th https://pypi.org/project/openstack-placement/#history | |
| 18:32:32 | cdent | sean-k-mooney: it should be installed a tox-sibling, not from pypi | |
| 18:32:48 | cdent | otherwise it wouldn't work as a fixture-to-use-master | |
| 18:32:57 | cdent | aspiers: i've repeated here | |
| 18:33:04 | cdent | so I reckon it's a bug in the fixture | |
| 18:33:05 | sean-k-mooney | perhaps in which case then ya it could be jsut a a timing window | |
| 18:33:12 | aspiers | ah | |
| 18:33:15 | sean-k-mooney | aspiers: do you have a depends on on that placmenet change | |
| 18:33:25 | sean-k-mooney | aspiers: that would have prevented any race | |
| 18:33:32 | aspiers | no I didn't know about the change until just now | |
| 18:33:34 | cdent | sean-k-mooney: not in the case | |
| 18:33:37 | cdent | the fixture has a real bug | |
| 18:33:52 | sean-k-mooney | cdent: oh ok | |
| 18:34:08 | sean-k-mooney | aspiers: generally depens on helps bug not in all cases | |
| 18:34:38 | aspiers | is it possible to do rechecks on the gate? | |
| 18:34:43 | sean-k-mooney | yep | |
| 18:34:48 | sean-k-mooney | just a normal recheck | |
| 18:34:59 | sean-k-mooney | we used to have reverify years ago but not anymore | |
| 18:35:03 | sean-k-mooney | reverify just ran gate | |
| 18:35:20 | sean-k-mooney | recheck will run check and gate | |
| 18:38:12 | cdent | aspiers: I'll see about fixing this | |
| 18:38:20 | cdent | as in, now | |