| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-04 | |||
| 17:10:27 | mriedem | because it has to clear it on sighup | |
| 17:10:41 | gibi | mriedem: ooh | |
| 17:10:41 | mriedem | dansmith: i don't know the answer to that offhand | |
| 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 | mriedem | well, give a justifiable reason in the commit message to remove it | |
| 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: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 | |
| 18:39:39 | sean-k-mooney | im going to grab dinner. be back in an hour or so | |
| 18:49:49 | cdent | ah. aspiers, kind of figured it out. Those tests which are failing are using the placement database fixture directly, not through the Placement fixture, thus the the config opt is not registered, leading to plenty of confusion | |
| 18:53:34 | cdent | aspiers: https://bugs.launchpad.net/nova/+bug/1818560 | |
| 18:53:35 | openstack | Launchpad bug 1818560 in OpenStack Compute (nova) "Nova's use of the placement database fixture from test_report_client doesn't register opts" [Critical,Confirmed] | |
| 18:53:50 | cdent | in case I wasn't clear there, the fix is on the placement side | |
| 19:15:06 | cdent | aspiers: there are two levels of problem here | |
| 19:16:43 | cdent | requires fixes in both placement and nova, mabye | |
| 19:16:51 | cdent | still looking to try to figure out the cleanest way | |
| 19:19:24 | aspiers | cdent: awesome, thanks! | |
| 19:19:45 | mriedem | dansmith: am i missing what you're asking for here? https://review.openstack.org/#/c/636412/1/nova/scheduler/client/report.py@1758 | |
| 19:20:56 | dansmith | mriedem: ah, sorry I missed that the original change was already including one direction | |