Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-26
14:56:19 jaypipes bauzas: I'm on firefox...
14:56:31 bauzas when I say FTW, it's sarcastic
14:56:44 bauzas so yeah, I'm hitted too
14:57:09 bauzas I suspected a French regulation cause, but looks like it's not :p
14:58:07 mriedem gibi: good point, there is no unit test coverage of the libvirt driver change or the images change to use the new flag
14:58:14 mriedem gibi: -1 that sonofa
14:59:21 gibi mriedem: I think the image part is covered by https://review.openstack.org/#/c/505673/5/nova/tests/unit/virt/libvirt/test_utils.py@181
15:00:31 bauzas mriedem: not sure I love assigning a new module attribute for the QEMU version, but I guess it's probably the only easy way
15:00:51 bauzas mriedem: I'm thinking of cases where you don't reload your module and you're doomed
15:00:57 mriedem oh i see
15:01:21 bauzas but I guess upgrading qemu would be part of a nova minor upgrade as well, so that would require restarting n-cpu as well
15:01:33 mriedem bauzas: jamespage went an alternate route where the version was passed down through the many plumbing layers, and it was quickly unmanageable
15:01:53 mriedem another alternative is handling the error, parsing the output and retrying with the flag, which also sucks
15:01:56 bauzas mriedem: yeah, I was thinking the image module wasn't written like that, but unfortunately it is
15:02:06 bauzas it's just an helper module
15:02:35 bauzas so no way to pass a flag but just adding a global var which sucks in my mind but which sucks less than the other possibilities you envisaged
15:10:44 mriedem weird, seeing this in the logs when creating 500 servers using the fake driver
15:10:46 mriedem Sep 26 15:09:38 devstack nova-compute[30351]: DEBUG nova.compute.resource_tracker [None req-f0c0e899-18e6-47f9-b13b-34829646d07e demo demo] Instance 62604071-77c2-46e1-9f57-d7192edc3f82 has been deleted (perhaps locally). Deleting allocations that remained for this instance against this compute host: {u'resources': {u'VCPU': 1, u'MEMORY_MB': 512, u'DISK_GB': 1}}. {{(pid=30351) _remove_deleted_instances_allocations /opt/stack/
15:10:46 mriedem /nova/compute/resource_tracker.py:1171}}
15:10:46 mriedem Sep 26 15:09:38 devstack nova-compute[30351]: INFO nova.scheduler.client.report [None req-f0c0e899-18e6-47f9-b13b-34829646d07e demo demo] Deleted allocation for instance 62604071-77c2-46e1-9f57-d7192edc3f82
15:11:33 mriedem doesn't really matter for what i'm testing, but it's odd
15:15:14 mriedem stephenfin: what's your actual -1 on this now? https://review.openstack.org/#/c/457532/
15:15:18 mriedem i'd like to see this get in so i can use it in devstack
15:15:33 mriedem dtroyer told us to make the command names "openstack resource provider <action>"
15:15:42 mriedem to be consistent with everything else in osc
15:15:52 mriedem being new to osc, i'm going to follow his recommendations
15:16:19 stephenfin mriedem: There isn't one. Removed now
15:16:24 mriedem thanks
15:16:35 mriedem and yeah, it's a bit large, but it lays the base crud ops for RPs
15:16:48 mriedem plus the common framework stuff
15:21:42 cdent jaypipes: one of the other things I did almost immediately after you asked was limiting aggregate checks: https://review.openstack.org/#/c/489633/
15:23:00 sdague gibi: it's tested in images.py
15:23:08 sdague mriedem / gibi: https://review.openstack.org/#/c/505673/5/nova/tests/unit/virt/libvirt/test_utils.py
15:23:24 jaypipes cdent: cool, will check shortly.
15:23:55 mriedem sdague: i think he meant there is no unit test for the driver code setting the variable in https://review.openstack.org/#/c/505673/5/nova/tests/unit/virt/libvirt/test_driver.py
15:23:58 gibi sdague: that test sets the images.QEMU_VERSION directly, but you have some code in the driver.py that sets images.QEMU_VERSION
15:24:46 mriedem like test_next_min_qemu_version_ok
15:24:55 openstackgerrit Chris Dent proposed openstack/nova master: [placement] gabbi tests for shared custom resource class https://review.openstack.org/485209
15:25:03 mriedem gibi: you can -1 and i can add the test later
15:25:10 sdague mriedem: he already did
15:25:16 mriedem ok
15:26:01 gibi :)
15:27:02 mriedem dansmith: you said os reboot and restack would cleanup devstack?
15:27:29 dansmith mriedem: reboot and stack
15:27:38 mriedem ok
15:32:23 openstackgerrit Evgeny Antyshev proposed openstack/nova master: Add ploop procedures to privsep.libvirt https://review.openstack.org/507569
15:40:38 openstackgerrit Sean Dague proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505673
15:40:47 sdague gibi / mriedem unit test added
15:41:55 gibi sdague: looking
15:45:20 gibi sdague: thanks, looks good, +2
15:45:47 sdague mriedem: ok, you next, reminding you of the "be faster and merge things" principle from the PTG :)
15:47:17 dansmith sdague: he's currently working on being fast and merge things on my instance list patch
15:47:28 dansmith sdague: but I'll look at yours so he can continue to be fast and merge on my patch
15:47:32 sdague :)
15:47:40 sdague it's pretty small and should be straight forward
15:50:36 mriedem sdague: i shouldn't approve my own change
15:51:48 sdague mriedem: it's barely your change any more
16:02:22 gibi I've just realized that I have to migrate the notification burndown chart from openshift 2 to 3 until end of September. This will be joyful
16:02:35 mdbooth stephenfin: Are the changes to https://review.openstack.org/#/c/507488/ evil?
16:03:07 mdbooth Thinking specifically of the 'if six.py2:' in libvirt/guest.py
16:03:26 mdbooth I still think it's saner to have unicode everywhere, and convert to something else at the point of use
16:04:12 dansmith cdent: are you working your way up that migration uuid series? if so, I'll hold off pushing that fix you just identified until you have a chance to nit out on anything else
16:04:27 stephenfin mdbooth: No, I'd probably do the same thing
16:04:40 dansmith meaning, you can identify more nit-ish things and I can fix them, vs. just pushing for nits later
16:04:59 cdent dansmith: yup (on phone at the moment though)
16:05:15 dansmith cdent: okay, np
16:06:55 stephenfin mdbooth: though I would like to see one of the tests slightly reworked, if that makes sense to you?
16:07:47 mdbooth stephenfin: Sure thing. I'll take a look in the morning.
16:07:51 openstackgerrit Merged openstack/nova master: Remove SCREEN_LOGDIR from devstack install setting https://review.openstack.org/507425
16:07:54 stephenfin (y)
16:20:46 bauzas edleafe: jaypipes: cdent: I thought we would be talking of how we would lead reschedules in https://review.openstack.org/#/c/498830/
16:21:34 bauzas edleafe: jaypipes: cdent: I see a couple of comments in that spec review, but have you settled down on discussing reschedules as being out of scope for that spec?
16:21:59 dansmith cdent: ah I see you have +1s on most of the rest of the set anyway, so I'll just push
16:22:14 cdent dansmith: i’m in the midst of the last one now
16:22:25 dansmith cdent: ah okay then I'll wait
16:22:37 cdent so give me a couple of minutes (phone call was shorter than expected)
16:22:48 dansmith yeah not trying to rush you
16:23:20 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404
16:23:21 dansmith I just want to push that review button and get that squirt of dopamine you know :P
16:24:14 edleafe bauzas: reschedules will be a different spec
16:24:55 bauzas edleafe: so the spec is litterally just for mentioning which object the scheduler will return to conductor ?
16:25:01 rybridges Hey guys. I am using the ocata release and am wondering if there is any way to print the user data associated with an instance that i own from the cli with the openstack client
16:25:02 bauzas edleafe: looks uber too much
16:25:21 bauzas I mean, super heavy
16:26:03 edleafe bauzas: since this will be sent over RPC, we needed agreement on it so that we don't find ourselves changing it later
16:26:04 bauzas I'd be up concentrating our minds on how we plan to pass that object
16:26:23 bauzas edleafe: we did a couple of RPC changes that didn't require a spec fortunately
16:26:34 bauzas but I leave the mic to mriedem
16:26:45 edleafe bauzas: the idea is to get it close to correct before we make the change
16:26:46 dansmith bauzas: specs are cheap
16:27:06 dansmith if edleafe wants separate specs, I don't think there's a problem
16:27:11 edleafe bauzas: and given the amount of discussion on the Selection object spec, I'd say it was a good thing to do
16:27:16 dansmith we should focus on getting the work done and not the process
16:27:16 bauzas dansmith: well, I'd rather then look at code, but okay :)
16:27:22 bauzas yeah that
16:27:37 edleafe bauzas: the work is being done in parallel
16:28:00 bauzas edleafe: well, okay
16:28:37 bauzas edleafe: the thing is, if you want a spec, fine with me, but then precise the scope
16:28:48 bauzas since it was a work item, I was expecting more
16:29:14 dansmith mriedem: so I was just looking at this for evac and live migration.. this method _moves_ allocations to the destination, not copies AFAICT: https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L222-L224

Earlier   Later