| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-29 | |||
| 17:51:07 | melwitt | but I thought you were saying "what if compute manager uses a different context" and it can't | |
| 17:51:08 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates https://review.openstack.org/513526 | |
| 17:51:10 | melwitt | https://github.com/openstack/nova/blob/master/nova/service.py#L294-L295 | |
| 17:51:35 | dansmith | melwitt: I'm saying if we have another context we got from not the periodic one we were handed, not in the scope of the periodic itself | |
| 17:51:44 | dansmith | we have contexts everywhere, on every object, etc | |
| 17:51:56 | dansmith | I thought we actually stored on in the RT too but I don't think we do | |
| 17:52:07 | melwitt | okay, yeah I thought you were talking only about periodics | |
| 17:52:25 | dansmith | ah in the claim we do, that's probably what I'm thinking of | |
| 17:52:43 | dansmith | melwitt: I mean a periodic that accesses one created outside the scope of the periodic, like claim for example | |
| 17:52:54 | melwitt | oh, I see | |
| 17:52:57 | melwitt | yeah | |
| 17:57:52 | melwitt | mriedem: I think you might have to not do your own _setup_compute_service | |
| 17:58:38 | cdent | jaypipes, efried_rollin : is n-r-p stack clean and reviewable? Last I checked last night might rebase was in progress. | |
| 18:07:20 | openstackgerrit | melanie witt proposed openstack/nova master: qemu-img do not use cache=none if no O_DIRECT support https://review.openstack.org/523554 | |
| 18:12:15 | dansmith | stephenfin: https://review.openstack.org/#/c/523914 | |
| 18:13:45 | melwitt | mriedem: I pulled down your patch to see if I can find anything | |
| 18:15:53 | mriedem | dansmith: that root_bdm thing with using compute_utils.is_volume_backed opens up some funny doors | |
| 18:16:06 | dansmith | mriedem: yeah? | |
| 18:16:06 | mriedem | probably nothing we'd ever hit in real life | |
| 18:16:24 | mriedem | is_volume_backed_instance returns True if there is no root bdm but the image_ref is '' | |
| 18:16:36 | mriedem | so "volume_id = root_bdm.volume_id" could result in a NoneType | |
| 18:17:05 | mriedem | nova.tests.unit.compute.test_compute.ComputeAPITestCase.test_rebuild_no_image apparently tickles that path | |
| 18:17:58 | mriedem | in that case i guess i just log an error or something, not sure | |
| 18:18:02 | mriedem | shouldn't really happen | |
| 18:29:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008 | |
| 18:29:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243 | |
| 18:29:22 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Use QEMU's native LUKS support https://review.openstack.org/523958 | |
| 18:37:53 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/523765 | |
| 18:38:58 | INeedaUUID | Hi everyone. Can anyone tell me how best to modify a systems' uuid using openstack methods? | |
| 18:39:03 | INeedaUUID | Does openstack support this? (I know it's a stupid request, but an application depends on them being set) | |
| 18:39:11 | INeedaUUID | Ocata/KVM | |
| 18:39:47 | jaypipes | cdent: efried_rollin is currently rebasing./ | |
| 18:40:12 | jaypipes | cdent: ooh, hold up, he may have pushed a fresh series. | |
| 18:45:54 | mriedem | gdi, that moment when you rebase a series and forget git add / git rebase --continue on the last change, and did git commit -a --amend instead, thus squashing the last change and ruining everything | |
| 18:47:38 | mriedem | INeedaUUID: no you can't specify the uuid that gets used with the vm | |
| 18:47:49 | INeedaUUID | mriedem: Is there not a workaround at all? | |
| 18:47:52 | mriedem | no | |
| 18:48:03 | INeedaUUID | What about modifying from virsh? | |
| 18:48:18 | mriedem | let me guess, you're doing some kind of instance HA | |
| 18:48:44 | INeedaUUID | actually, no. It's to do with a 3rd party application and the VERY stupid way it licences. | |
| 18:48:49 | mriedem | anything you do to a vm via virsh out of band from nova will likely break nova's ability to do things with that vm | |
| 18:49:31 | INeedaUUID | I thought that. But I will need to try. :-( | |
| 18:49:35 | mriedem | pass metadata through to the guest and have the 3rd party app pull from that | |
| 18:50:06 | dansmith | presumably the app is unchangeable | |
| 18:50:11 | dansmith | getting the uuid from smbios, | |
| 18:50:16 | dansmith | so metadata won't help even if hacked | |
| 18:50:36 | INeedaUUID | dansmith: is correct. Unchangable. | |
| 18:51:18 | dansmith | your options are: | |
| 18:51:28 | INeedaUUID | It's crap, I know it is. But... It's essential. | |
| 18:51:30 | dansmith | 1. lots of scary nova db surgery. don't even tell us about it | |
| 18:51:40 | dansmith | 2. wrap your application in something that fools it | |
| 18:51:43 | dansmith | 3. Get a new application | |
| 18:52:05 | dansmith | I suggest 3, I would do 2, please don't do 1 | |
| 18:53:31 | INeedaUUID | The requirements suggest exactly the opposite of the numbers you propose. But I apprieciate the honesty. It is what it is. | |
| 18:53:55 | INeedaUUID | 3. not an option (don't ask why) | |
| 18:53:57 | dansmith | not an accident :) | |
| 18:54:04 | dansmith | I'm sure | |
| 18:54:11 | INeedaUUID | 2. Not sure if I can do this. Can I have the system uuid somehow I wonder? | |
| 18:54:20 | dansmith | windows or linux guest? | |
| 18:54:29 | INeedaUUID | 1. I'm too stupid, but given the lack of options, it's a few sleepless nights. | |
| 18:54:50 | INeedaUUID | linux | |
| 18:55:10 | dansmith | LD_PRELOAD | |
| 18:56:02 | INeedaUUID | If this is an option, I will make this work! :-D | |
| 18:56:46 | cdent | dansmith: you’re potentially a freakin’ genius | |
| 18:56:58 | dansmith | o.O | |
| 18:57:35 | jaypipes | INeedaUUID: is this a telco VNF vendor by any chance? | |
| 18:59:24 | dansmith | gawd jaypipes, there _are_ other closed-source stupid-licensed industries out there | |
| 18:59:36 | dansmith | why you always pickin' on those lovable telco people? | |
| 18:59:48 | jaypipes | dansmith: just a wild guess. ;) | |
| 18:59:48 | dansmith | (just kidding, I'm _sure_ that's what this is) | |
| 19:00:16 | mriedem | that reminds me, | |
| 19:00:33 | mriedem | jaypipes: did i tell you how 2 different groups of people approached me about their worries about deprecating the ability to do file injection? | |
| 19:00:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/521391 | |
| 19:00:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200 | |
| 19:00:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660 | |
| 19:00:39 | mriedem | at the summit | |
| 19:00:47 | mriedem | verizon and ericsson | |
| 19:00:50 | dansmith | only two? | |
| 19:00:56 | jaypipes | mriedem: yeah, only two? | |
| 19:01:02 | INeedaUUID | jaypipes: I could tell you. But then i'd have to murder you. | |
| 19:01:04 | dansmith | two seems like a mandate to me | |
| 19:01:06 | mriedem | well, most people only say things 3 years after the change is merged | |
| 19:01:44 | mriedem | mikal told the verizon guys that their vnf vendor was "shit" in his aussy words and that they should talk to him | |
| 19:01:48 | jaypipes | mriedem: also, it's probably all rooted in the same dependency on a specific particular vendor VNF or MANO system that requires file injection (and stuff like multi-attach boot-from-volume | |
| 19:02:36 | mriedem | multiattach bfv is the bees knees | |
| 19:02:49 | mriedem | i wonder how rebuild is going to handle that... | |
| 19:03:02 | dansmith | probably as well as it handles everything else | |
| 19:03:12 | dansmith | by opening security holes, leaking resource accounting, and destroying data | |
| 19:03:22 | jaypipes | luckily that shotgun I ordered through the mail should be here by the time multi-attach bfv rolls into Nova. | |
| 19:03:33 | dansmith | jaypipes: you're welcome, from america | |
| 19:03:39 | jaypipes | murka | |
| 19:03:50 | mriedem | dansmith: ok i did the deeds in https://review.openstack.org/#/c/521391/3/nova/compute/api.py | |
| 19:04:02 | mriedem | if i can't find a root bdm and image_ref is '', i just log an error and keep trucking | |
| 19:04:45 | mriedem | i don't know if that case is some scary pre-bdmv1 stuff or what, it predates me | |
| 19:05:45 | dansmith | I don't understand where that's ever going to happen | |
| 19:06:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660 | |
| 19:06:11 | mriedem | i don't either | |
| 19:06:18 | mriedem | which is why i just log an error and punt | |
| 19:06:34 | dansmith | but you don't punt, | |
| 19:06:50 | dansmith | you set it to None, which causes the need for that extra else and then kick the can to the next thing in line | |