| opendevreview | Yusuke Okada proposed openstack/nova master: Fix failed count for anti-affinity check https://review.opendev.org/c/openstack/nova/+/873216 | 03:26 |
|---|---|---|
| opendevreview | Yusuke Okada proposed openstack/nova master: Fix failed count for anti-affinity check https://review.opendev.org/c/openstack/nova/+/873216 | 03:30 |
| opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Add 'COMPUTE_ADDRESS_SPACE_*' traits support https://review.opendev.org/c/openstack/nova/+/873221 | 06:05 |
| Uggla | bauzas, gibi o/ | 09:14 |
| Uggla | bauzas, gibi looking at the https://review.opendev.org/c/openstack/nova/+/839401/22 are we all agree to remove the ShareMappingLibvirt* although that is not following the spec and might be more difficult if we want to create a dedicated module (os-share) later ? | 09:16 |
| Uggla | bauzas, gibi, of course doing that will simplify the code but I will have to review lot of stuff. | 09:17 |
| bauzas | Uggla: I agree with you | 09:39 |
| gibi | ...loading context | 09:40 |
| gibi | For me the content of ShareMappingLibvirt is OK, it is just placed to a too public place. | 09:44 |
| gibi | i.e. I'm OK to use inheritance to model that a ShareMapping my need to be mounted by a libvirt or by other virt driver | 09:45 |
| Uggla | gibi, hum I think what bauzas and john expect is the removal of convert and inheritance and move that in the driver itself. | 09:48 |
| gibi | sure tha is also an option | 09:48 |
| Uggla | gibi, if not in the object where do you put the act code ? | 09:49 |
| gibi | the libvirt specific classes can be placed to nova.virt.libvirt afaik. But as you said bauzas and johnthetubaguy might want not to have them as ovos even if they are moved under nova.virt.libvirt | 09:50 |
| gibi | so to expedite things I think it would be easier to simply remove the driver specific ovo | 09:51 |
| gibi | and create functions in the driver that takes a generic ShareMapping and do the driver specific bits with it | 09:51 |
| gibi | I guess that what bauzas and johnthetubaguy suggests | 09:52 |
| Uggla | gibi, yes sounds like it is ok for you to go in this way as well ? | 09:54 |
| gibi | yeah, OK with me | 09:54 |
| Uggla | ok so I'll change the code in this way. | 09:56 |
| bauzas | gibi: tbc, I w | 09:57 |
| bauzas | I don't want to have a ovo inheritance for this object | 09:58 |
| bauzas | Uggla: ^ | 09:58 |
| bauzas | since this object is not about providing fields or persisting them | 09:58 |
| bauzas | by RPC calls | 09:58 |
| bauzas | but yeah, we can have a specific python object for it in libvirt if Uggla wants | 09:59 |
| Uggla | bauzas, yep I will keep only the ShareMapping / ShareMappingList objects | 09:59 |
| Uggla | bauzas, not sure it is worth creating an object in the libvirt part, I'll see. | 10:00 |
| 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:05 |
| gibi | or driver specific ovo | 10:06 |
| bauzas | gibi: sure, but those objects are used for being passing between services | 10:09 |
| bauzas | and those are only for fields | 10:10 |
| Uggla | sounds like https://miro.medium.com/v2/resize:fit:720/format:webp/1*LzKppsHIMabQhnCCdpdQXg.jpeg :) | 10:10 |
| bauzas | gibi: while the previous ShareMappingLibvirt objects were directly calling libvirt | 10:10 |
| 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:14 |
| gibi | but I rest my case. It is a theorethical discussion | 10:15 |
| bauzas | gibi: well I remember when dansmith created those objects | 10:15 |
| bauzas | gibi: and I'm pretty sure I can find you some links explaining how this works | 10:16 |
| bauzas | I mean, not how, but why | 10:16 |
| bauzas | (sorry, not explaining you're bad, just about the situation) | 10:16 |
| bauzas | we could have specific libvirt objects like you said | 10:17 |
| bauzas | but those objects would only have values | 10:17 |
| gibi | and that is the point I disagree with out object model ^^ | 10:18 |
| 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:18 |
| 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:22 |
| gibi | the FAIL part is interesting | 10:24 |
| ralonsoh | yeah, I've never seen that | 10:25 |
| gibi | as it should be 'compute:%s' % instance.availability_zone | 10:25 |
| ralonsoh | ahhh, that could make sense | 10:25 |
| gibi | so I'm wondering if a) it is manually set by the user b) the availability_zone name is set to fAIL | 10:25 |
| ralonsoh | I'll check that, thanks! | 10:27 |
| opendevreview | Elod Illes proposed openstack/nova stable/victoria: DNM: CI test https://review.opendev.org/c/openstack/nova/+/873242 | 11:14 |
| sean-k-mooney | ralonsoh: its the name of an AZ | 11:15 |
| sean-k-mooney | ah gibi answered | 11:15 |
| 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 |
| sean-k-mooney | the new defaults were just waiting on placmenemtn correct | 11:22 |
| ralonsoh | sean-k-mooney, yeah, I found the code thanks! | 11:29 |
| opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833437 | 12:11 |
| opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833438 | 12:13 |
| elodilles | bauzas gibi : os-traits release patch has merged | 12:44 |
| gibi | elodilles: thanks | 12:44 |
| bauzas | all cool | 12:44 |
| opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | 13:20 |
| 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 |
| opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | 13:20 |
| opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | 13:20 |
| opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | 13:20 |
| 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 |
| opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | 13:20 |
| opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | 13:20 |
| 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 |
| 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 |
| opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | 13:20 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: cpu: interfaces for managing state and governor https://review.opendev.org/c/openstack/nova/+/868236 | 13:35 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: libvirt: let CPUs be power managed https://review.opendev.org/c/openstack/nova/+/821228 | 13:35 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Enable cpus when an instance is spawning https://review.opendev.org/c/openstack/nova/+/868237 | 13:35 |
| bauzas | gibi: sean-k-mooney: updated the series ^ | 13:36 |
| bauzas | will provide a fup for docs | 13:36 |
| *** dasm|offline is now known as dasm|rover | 13:47 | |
| 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 | 14:36 |
| *** dasm|rover is now known as dasm|bbiab | 15:25 | |
| 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:30 |
| kashyap | bauzas: Maybe it's just best to quickly rename the function here - https://review.opendev.org/c/openstack/nova/+/821228 | 15:57 |
| 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 | 15:58 |
| bauzas | what is the fuck this new CI issue ? TypeError: retry_bad_request() missing 1 required positional argument: 'fn' | 16:23 |
| bauzas | https://6b96a36d22f450141095-9e4eb9f05513870f0e5456f29474f6b2.ssl.cf5.rackcdn.com/868236/5/check/nova-ceph-multistore/6011df8/testr_results.html | 16:23 |
| bauzas | dansmith: ^ I'm pretty done with all those glance image issues | 16:23 |
| dansmith | oh crap | 16:23 |
| dansmith | that might be broken in the patch I just did | 16:24 |
| dansmith | which might mean it did't actually get run in the tempest gate | 16:24 |
| dansmith | eesh | 16:24 |
| dansmith | crap yeah | 16:24 |
| dansmith | so that's breaking anyone that runs that job 100% right now :( | 16:25 |
| bauzas | then no worries, everyone can make mistakes | 16:25 |
| dansmith | I was sure I checked, | 16:26 |
| bauzas | including me, having a long list of bugs I created | 16:26 |
| dansmith | but I have about a thousand open unit test report tabs right now :9 | 16:26 |
| dansmith | :( | 16:26 |
| dansmith | need a DNM nova patch against the fix I guess | 16:27 |
| jungleboyj | dansmith: You around? | 16:28 |
| *** dasm|bbiab is now known as dasm|rover | 16:28 | |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Test tempest fix https://review.opendev.org/c/openstack/nova/+/873302 | 16:29 |
| dansmith | bauzas: ^ | 16:29 |
| dansmith | jungleboyj: yes, are you here to hassle me about breaking the ceph job? | 16:29 |
| jungleboyj | No, not at all. | 16:29 |
| jungleboyj | Have a question for a customer. | 16:30 |
| jungleboyj | Do you, or do you know of anyone who has experience setting up diskless compute nodes? | 16:30 |
| jungleboyj | Is it even possible with Nova? | 16:31 |
| dansmith | it's been done, but no direct experience. it's really not what nova was intended to do, | 16:31 |
| dansmith | so I imagine you need some writable ramdisk area and obviously ceph-backed would be the way to go | 16:31 |
| dansmith | but no, nothing really solid to recommend | 16:32 |
| dansmith | (ceph because you can avoid downloading the image locally) | 16:32 |
| jungleboyj | Right. | 16:32 |
| jungleboyj | Is there data that needs to persist from boot to boot for Nova on the compute? | 16:33 |
| dansmith | well, libvirt guest configs technically, but I imagine we'll recreate them if they're gone | 16:33 |
| 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 |
| kashyap | a thousand open unit test report tabs | 16:34 |
| kashyap | Oh god; please close some of 'em | 16:35 |
| 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 |
| 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 |
| dansmith | jungleboyj: but like i say, it's really fighting against nova's design, so it's not going to be fun :) | 16:35 |
| jungleboyj | Understood. Thanks for the input. That was what I needed. :-) | 16:36 |
| kashyap | That's a very handy phrase - "fighting against Nova's design" | 16:36 |
| bauzas | I'm pretty harassed for the day folks (13 hours of work having started at 6am :) ) | 16:51 |
| bauzas | so, \o | 16:51 |
| bauzas | dansmith: when I'm back, I'll look at your DNM | 16:51 |
| 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:52 |
| sean-k-mooney | dansmith: +2 on the stable uuid doc, thanks for updating it https://review.opendev.org/c/openstack/nova/+/872977 | 16:53 |
| dansmith | sean-k-mooney: thanks | 16:53 |
| sean-k-mooney | gibi: if perhaps you could be the second +2 its pretty short | 16:53 |
| sean-k-mooney | or melwitt i would as bauzas but they are done for today | 16:54 |
| melwitt | I will check it later and review if someone's not gotten to it yet | 16:57 |
| bauzas | I still have a bit of juice | 16:58 |
| bauzas | (but a terrible internet connection) | 16:59 |
| bauzas | okay, +Wd | 17:02 |
| bauzas | this ends my long day | 17:02 |
| bauzas | dansmith: oh, actually before ending, any bug report you wanna want to have for the glance issue ? | 17:04 |
| dansmith | bauzas: it's a bug in a bugfix, I think it's okay to just sweep it in | 17:04 |
| gibi | sean-k-mooney: done | 17:17 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Enable cpus when an instance is spawning https://review.opendev.org/c/openstack/nova/+/868237 | 17:29 |
| bauzas | This time done for the day ;-) | 17:30 |
| bauzas | o/ | 17:30 |
| opendevreview | Merged openstack/nova master: Add docs for stable-compute-uuid behaviors https://review.opendev.org/c/openstack/nova/+/872977 | 17:36 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | 18:06 |
| gibi | bauzas sean-k-mooney: I'm more or less OK with the power management series. I'm happy with the sys fs fixture now. I left some nits but nothing serious | 18:12 |
| sean-k-mooney | johnthetubaguy: jayF am i right in saying no work has happend on https://specs.openstack.org/openstack/nova-specs/specs/2023.1/approved/ironic-shards.html and it will hopefully happne next cycle? | 19:27 |
| sean-k-mooney | it loks like some of the code has landed https://review.opendev.org/q/topic:DBLayer_Shard_Key | 19:29 |
| sean-k-mooney | so next cycle would just be the ironic driver changes in nova | 19:29 |
| gmann | sean-k-mooney: placement one is merged. nova service role is pending. I will try to update the patch soon. | 19:34 |
| gmann | it has been lot of other things and gate failures and I did not get chance to update that | 19:34 |
| sean-k-mooney | ack | 19:34 |
| sean-k-mooney | i was more wondering if we still planned to land it this cycle | 19:35 |
| sean-k-mooney | i was hopeing we could but if it slips we can do it early next cycle | 19:35 |
| sean-k-mooney | after RC1 is shiped in ~4 weeks | 19:35 |
| gmann | yeah, let see if I can do it before FF | 19:43 |
| gmann | elodilles: you need to remove it from check pipeline also along with template https://review.opendev.org/c/openstack/nova/+/873242 | 19:44 |
| opendevreview | Ghanshyam proposed openstack/nova stable/victoria: [stable-only] Remove broken sdk job from wallaby https://review.opendev.org/c/openstack/nova/+/873295 | 19:45 |
| opendevreview | Ghanshyam proposed openstack/nova stable/victoria: [stable-only] Remove broken sdk job from wallaby https://review.opendev.org/c/openstack/nova/+/873295 | 19:46 |
| gmann | elodilles: basically backporting this ti victoria ^^ | 19:46 |
| gmann | I have updated the depend-on | 19:46 |
| opendevreview | Merged openstack/nova stable/xena: nova-live-migration tests not needed for Ironic https://review.opendev.org/c/openstack/nova/+/855469 | 21:34 |
| *** dasm|rover is now known as dasm|off | 22:18 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
