Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2023-02-09
10:00:36 Uggla bauzas, not sure it is worth creating an object in the libvirt part, I'll see.
10:05:37 gibi bauzas: we have examples where we use ovo not just to persist data. We even have examples to driver specific data in ovo https://github.com/openstack/nova/blob/master/nova/objects/migrate_data.py
10:06:19 gibi or driver specific ovo
10:09:27 bauzas gibi: sure, but those objects are used for being passing between services
10:10:07 bauzas and those are only for fields
10:10:31 Uggla sounds like https://miro.medium.com/v2/resize:fit:720/format:webp/1*LzKppsHIMabQhnCCdpdQXg.jpeg :)
10:10:40 bauzas gibi: while the previous ShareMappingLibvirt objects were directly calling libvirt
10:14:54 gibi bauzas: I never really got why we are not coupling data and the methods using that data together into the ovos. That is one of the main reason classes exists to couple data and code acting on that data together :)
10:15:26 gibi but I rest my case. It is a theorethical discussion
10:15:51 bauzas gibi: well I remember when dansmith created those objects
10:16:13 bauzas gibi: and I'm pretty sure I can find you some links explaining how this works
10:16:26 bauzas I mean, not how, but why
10:16:46 bauzas (sorry, not explaining you're bad, just about the situation)
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: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755
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:24 opendevreview melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933
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: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:25 opendevreview melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934
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: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938
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:29 opendevreview melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273
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: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: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: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 dansmith but I have about a thousand open unit test report tabs right now :9
16:26:20 bauzas including me, having a long list of bugs I created
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

Earlier   Later