| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-09 | |||
| 10:17:06 | bauzas | we could have specific libvirt objects like you said | |
| 10:17:24 | bauzas | but those objects would only have values | |
| 10:18:16 | gibi | and that is the point I disagree with out object model ^^ | |
| 10:18:47 | gibi | but as I said I rest my case. I don't want to change how nova looks at this point in its lifecycle | |
| 10:22:14 | ralonsoh | hi folks, do you know when we can set 'device_owner': 'compute:FAIL' for a port? Where is this 'compute:FAIL' coming from? | |
| 10:24:43 | gibi | the FAIL part is interesting | |
| 10:25:07 | ralonsoh | yeah, I've never seen that | |
| 10:25:23 | gibi | as it should be 'compute:%s' % instance.availability_zone | |
| 10:25:47 | ralonsoh | ahhh, that could make sense | |
| 10:25:49 | gibi | so I'm wondering if a) it is manually set by the user b) the availability_zone name is set to fAIL | |
| 10:27:08 | ralonsoh | I'll check that, thanks! | |
| 11:14:19 | opendevreview | Elod Illes proposed openstack/nova stable/victoria: DNM: CI test https://review.opendev.org/c/openstack/nova/+/873242 | |
| 11:15:18 | sean-k-mooney | ralonsoh: its the name of an AZ | |
| 11:15:35 | sean-k-mooney | ah gibi answered | |
| 11:22:18 | sean-k-mooney | gmann: do you think we can get the policy change for nova merged this cycle. specfically the service role support | |
| 11:22:43 | sean-k-mooney | the new defaults were just waiting on placmenemtn correct | |
| 11:29:44 | ralonsoh | sean-k-mooney, yeah, I found the code thanks! | |
| 12:11:58 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833437 | |
| 12:13:20 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833438 | |
| 12:44:02 | elodilles | bauzas gibi : os-traits release patch has merged | |
| 12:44:10 | gibi | elodilles: thanks | |
| 12:44:23 | bauzas | all cool | |
| 13:20:23 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 13:20:23 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 13:20:24 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 13:20:24 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 13:20:25 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 13:20:25 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 13:20:26 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 13:20:27 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 13:20:27 | opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | |
| 13:20:29 | opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | |
| 13:20:29 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 13:35:36 | opendevreview | Sylvain Bauza proposed openstack/nova master: cpu: interfaces for managing state and governor https://review.opendev.org/c/openstack/nova/+/868236 | |
| 13:35:36 | opendevreview | Sylvain Bauza proposed openstack/nova master: libvirt: let CPUs be power managed https://review.opendev.org/c/openstack/nova/+/821228 | |
| 13:35:38 | opendevreview | Sylvain Bauza proposed openstack/nova master: Enable cpus when an instance is spawning https://review.opendev.org/c/openstack/nova/+/868237 | |
| 13:36:02 | bauzas | gibi: sean-k-mooney: updated the series ^ | |
| 13:36:07 | bauzas | will provide a fup for docs | |
| 14:36:08 | opendevreview | Merged openstack/nova stable/yoga: Accept both 1 and Y as AMD SEV KVM kernel param value https://review.opendev.org/c/openstack/nova/+/843819 | |
| 15:30:24 | opendevreview | Elod Illes proposed openstack/nova stable/victoria: [stable-only] Remove broken sdk job from wallaby https://review.opendev.org/c/openstack/nova/+/873295 | |
| 15:57:59 | kashyap | bauzas: Maybe it's just best to quickly rename the function here - https://review.opendev.org/c/openstack/nova/+/821228 | |
| 15:58:36 | kashyap | bauzas: (Instead of a follow-up). Something like get_cpu_status() or get_cpu_power_status() -- as either of these names include both offline and online | |
| 16:23:00 | bauzas | what is the fuck this new CI issue ? TypeError: retry_bad_request() missing 1 required positional argument: 'fn' | |
| 16:23:01 | bauzas | https://6b96a36d22f450141095-9e4eb9f05513870f0e5456f29474f6b2.ssl.cf5.rackcdn.com/868236/5/check/nova-ceph-multistore/6011df8/testr_results.html | |
| 16:23:18 | bauzas | dansmith: ^ I'm pretty done with all those glance image issues | |
| 16:23:54 | dansmith | oh crap | |
| 16:24:02 | dansmith | that might be broken in the patch I just did | |
| 16:24:18 | dansmith | which might mean it did't actually get run in the tempest gate | |
| 16:24:20 | dansmith | eesh | |
| 16:24:56 | dansmith | crap yeah | |
| 16:25:06 | dansmith | so that's breaking anyone that runs that job 100% right now :( | |
| 16:25:40 | bauzas | then no worries, everyone can make mistakes | |
| 16:26:11 | dansmith | I was sure I checked, | |
| 16:26:20 | bauzas | including me, having a long list of bugs I created | |
| 16:26:20 | dansmith | but I have about a thousand open unit test report tabs right now :9 | |
| 16:26:23 | dansmith | :( | |
| 16:27:11 | dansmith | need a DNM nova patch against the fix I guess | |
| 16:28:33 | jungleboyj | dansmith: You around? | |
| 16:29:31 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Test tempest fix https://review.opendev.org/c/openstack/nova/+/873302 | |
| 16:29:37 | dansmith | bauzas: ^ | |
| 16:29:48 | dansmith | jungleboyj: yes, are you here to hassle me about breaking the ceph job? | |
| 16:29:57 | jungleboyj | No, not at all. | |
| 16:30:06 | jungleboyj | Have a question for a customer. | |
| 16:30:29 | jungleboyj | Do you, or do you know of anyone who has experience setting up diskless compute nodes? | |
| 16:31:12 | jungleboyj | Is it even possible with Nova? | |
| 16:31:29 | dansmith | it's been done, but no direct experience. it's really not what nova was intended to do, | |
| 16:31:58 | dansmith | so I imagine you need some writable ramdisk area and obviously ceph-backed would be the way to go | |
| 16:32:07 | dansmith | but no, nothing really solid to recommend | |
| 16:32:29 | dansmith | (ceph because you can avoid downloading the image locally) | |
| 16:32:37 | jungleboyj | Right. | |
| 16:33:06 | jungleboyj | Is there data that needs to persist from boot to boot for Nova on the compute? | |
| 16:33:39 | dansmith | well, libvirt guest configs technically, but I imagine we'll recreate them if they're gone | |
| 16:34:11 | dansmith | jungleboyj: honestly you're going to have to try and iterate on the issues, I definitely can't point out all the icebergs without trying | |
| 16:34:49 | kashyap | a thousand open unit test report tabs | |
| 16:35:04 | kashyap | Oh god; please close some of 'em | |
| 16:35:08 | dansmith | jungleboyj: oh and in master now, we persist a compute node id file, which you'd need to restore on boot yourself somehow | |
| 16:35:10 | jungleboyj | Ok. It helps to know that, technically, it is possible. Trying to figure out how to help a customer that has a hardware config that limits what disk is available locally. | |
| 16:35:42 | dansmith | jungleboyj: but like i say, it's really fighting against nova's design, so it's not going to be fun :) | |
| 16:36:09 | jungleboyj | Understood. Thanks for the input. That was what I needed. :-) | |
| 16:36:33 | kashyap | That's a very handy phrase - "fighting against Nova's design" | |
| 16:51:34 | bauzas | I'm pretty harassed for the day folks (13 hours of work having started at 6am :) ) | |
| 16:51:39 | bauzas | so, \o | |
| 16:51:49 | bauzas | dansmith: when I'm back, I'll look at your DNM | |
| 16:52:38 | dansmith | bauzas: it's just to actually run the tempest fix to validate, don't look. I tested locally it's all good, so hopefully we can get the tempest people to hit it quick | |
| 16:53:31 | sean-k-mooney | dansmith: +2 on the stable uuid doc, thanks for updating it https://review.opendev.org/c/openstack/nova/+/872977 | |
| 16:53:40 | dansmith | sean-k-mooney: thanks | |
| 16:53:54 | sean-k-mooney | gibi: if perhaps you could be the second +2 its pretty short | |
| 16:54:22 | sean-k-mooney | or melwitt i would as bauzas but they are done for today | |
| 16:57:00 | melwitt | I will check it later and review if someone's not gotten to it yet | |
| 16:58:56 | bauzas | I still have a bit of juice | |
| 16:59:07 | bauzas | (but a terrible internet connection) | |
| 17:02:11 | bauzas | okay, +Wd | |
| 17:02:27 | bauzas | this ends my long day | |
| 17:04:21 | bauzas | dansmith: oh, actually before ending, any bug report you wanna want to have for the glance issue ? | |
| 17:04:53 | dansmith | bauzas: it's a bug in a bugfix, I think it's okay to just sweep it in | |
| 17:17:08 | gibi | sean-k-mooney: done | |
| 17:29:14 | opendevreview | Sylvain Bauza proposed openstack/nova master: Enable cpus when an instance is spawning https://review.opendev.org/c/openstack/nova/+/868237 | |
| 17:30:33 | bauzas | This time done for the day ;-) | |
| 17:30:46 | bauzas | o/ | |
| 17:36:37 | opendevreview | Merged openstack/nova master: Add docs for stable-compute-uuid behaviors https://review.opendev.org/c/openstack/nova/+/872977 | |