| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-09 | |||
| 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 | |
| 18:06:52 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 18:12:04 | 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 | |
| 19:27:46 | 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:29:09 | sean-k-mooney | it loks like some of the code has landed https://review.opendev.org/q/topic:DBLayer_Shard_Key | |
| 19:29:50 | sean-k-mooney | so next cycle would just be the ironic driver changes in nova | |
| 19:34:15 | gmann | sean-k-mooney: placement one is merged. nova service role is pending. I will try to update the patch soon. | |
| 19:34:47 | gmann | it has been lot of other things and gate failures and I did not get chance to update that | |
| 19:34:59 | sean-k-mooney | ack | |
| 19:35:08 | sean-k-mooney | i was more wondering if we still planned to land it this cycle | |
| 19:35:25 | sean-k-mooney | i was hopeing we could but if it slips we can do it early next cycle | |
| 19:35:35 | sean-k-mooney | after RC1 is shiped in ~4 weeks | |
| 19:43:49 | gmann | yeah, let see if I can do it before FF | |
| 19:44:05 | gmann | elodilles: you need to remove it from check pipeline also along with template https://review.opendev.org/c/openstack/nova/+/873242 | |
| 19:45:42 | 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:12 | 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:31 | gmann | elodilles: basically backporting this ti victoria ^^ | |
| 19:46:38 | gmann | I have updated the depend-on | |
| 21:34:22 | opendevreview | Merged openstack/nova stable/xena: nova-live-migration tests not needed for Ironic https://review.opendev.org/c/openstack/nova/+/855469 | |
| #openstack-nova - 2023-02-10 | |||
| 03:49:36 | opendevreview | Yusuke Okada proposed openstack/nova master: Fix failed count for anti-affinity check https://review.opendev.org/c/openstack/nova/+/873216 | |
| 04:10:02 | opendevreview | Yusuke Okada proposed openstack/nova master: Fix failed count for anti-affinity check https://review.opendev.org/c/openstack/nova/+/873216 | |
| 08:28:28 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Look for cpu controller on cgroups v2 https://review.opendev.org/c/openstack/nova/+/873127 | |
| 08:50:03 | gibi | bauzas: I start to think that the functional test failure is somehow changing behavior just becuase we added logging. | |
| 08:50:24 | bauzas | really ? | |
| 08:50:50 | bauzas | gibi: btw. saw your highlight yesterday, thanks, didn't had yet time to look at my series but I can surely rush for the nits | |
| 08:50:55 | gibi | we rechecked it through multiple days | |
| 08:51:09 | gibi | without hit | |
| 08:51:26 | bauzas | gibi: then we should merge it and see whether it magically solves our problem | |
| 08:51:32 | bauzas | gibi: we could prepare a revert | |
| 08:51:33 | gibi | lol :D | |
| 08:52:11 | bauzas | we're not really at risk for Feature Freeze and we have time to revert before RC1 | |
| 08:52:22 | bauzas | butn, | |
| 08:52:25 | gibi | OK, let me push a new PS to clean things a bit up | |
| 08:52:32 | gibi | then I'm OK to merge it with a pending revert | |
| 08:52:40 | bauzas | if that solves the problem, then honestly, I don't know what to say | |
| 08:53:04 | gibi | yeah, I feel the same | |
| 08:53:07 | bauzas | we could merge some log saying "meh, don't be afraid, we love you" | |
| 09:00:54 | bauzas | gibi: btw. you know that our master branch is broken broken ? (c) elodilles | |
| 09:01:12 | gibi | bauzas: that is a news to me | |
| 09:01:13 | bauzas | I gonna recheck https://review.opendev.org/c/openstack/tempest/+/873300 just looking up the root cause of the CI failure | |
| 09:03:09 | bauzas | gibi: tl;dr when dan fixed the image caching issue in Tempest, unfortunately we were not having coverage so we regressed | |
| 09:03:51 | bauzas | I was in the train with limited connection yesterday so I haven't send a status email, but basically our ceph-multistore job is unhappy | |
| 09:04:31 | bauzas | the tempest fix went accepted soon so now there is no need to send a signal to the community but I just hope to not face problems with merging it | |
| 09:09:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add logging to find test cases leaking libvirt threads https://review.opendev.org/c/openstack/nova/+/872975 | |
| 09:09:49 | gibi | bauzas: pimped out ^^ | |
| 09:10:14 | bauzas | lol | |
| 09:10:18 | gibi | bauzas: ack, thanks for the summary | |
| 09:10:22 | bauzas | it brightens | |
| 09:11:40 | bauzas | gibi: sean-k-mooney is on PTO today until wed, we need a second core | |
| 09:12:00 | gibi | hm mh | |
| 09:12:13 | sean-k-mooney[m] | i am but im reviewing your pm series | |
| 09:12:18 | sean-k-mooney[m] | right now | |
| 09:12:34 | sean-k-mooney[m] | so if you want me to look at something quickly i can | |
| 09:12:58 | bauzas | sean-k-mooney: oh, as you can | |
| 09:13:00 | gibi | sean-k-mooney[m]: just blindly merge https://review.opendev.org/c/openstack/nova/+/872975 please :D this is still trying to catch the functional tc | |
| 09:13:19 | gibi | that cause the libvirt import error in a later test | |
| 09:13:36 | bauzas | sean-k-mooney: for the PM series, I'll update the 3rd patch and work on a FUP given the method rename discussing in patch #2 | |
| 09:13:48 | gibi | after couple of days of constant rechecks we did not get a hit | |
| 09:14:00 | gibi | so we think if we merge the extra log it will fix the gate (kidding) | |
| 09:15:48 | gibi | bauzas: one thing hit me during the night and now I remember it. If we do the rename of the get_online_cpus and return the offlined ones as well then I think cpu_shared_set will allow listing offline CPUs without nova rejecting it. While in the past such config was rejected | |