Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-04-02
14:36:39 Uggla dansmith, you know better than me. :)
14:36:43 dansmith he needs to talk to cinder about the same topic as nova, but we don't all need to take the global lock, AFAIK
14:37:07 dansmith don't change it now, just saying
14:38:29 Uggla dansmith, yep let's keep it like this. As booking is more difficult than releasing people.
14:39:11 dansmith yup
14:49:19 elodilles gibi is right, but we couldn't help releasing / announcing a bit earlier: https://lists.openstack.org/archives/list/openstack-announce@lists.openstack.org/thread/ACVRVHIKT5K2TKZIPVT4P2OGAK3LZVVL/
14:49:33 elodilles thanks everyone :) \o/
14:49:49 elodilles 'OpenStack 2025.1 Epoxy is officially released!'
14:50:10 gibi \o/
14:50:30 Uggla elodilles, thanks \o/
14:51:27 dansmith Uggla: https://review.opendev.org/c/openstack/releases/+/946162
14:51:33 dansmith just so you don't have to do it ^
14:52:15 Uggla dansmith, thx I'll review ASAP.
14:54:06 dansmith aside from the mistake that was splitting out placement, having os-traits a library that requires a commit there, a release, a u-c bump, and requirements change to get a string constant into nova is about the most ridiculous process we could have designed :)
15:09:43 mnaser Has there been any changes in how network_info is generated/cached/etc? I have an 2024.1 environment and it seems that the Ironic driver is getting the following as network_info from Nova inside spawn: https://paste.openstack.org/show/b6wcQ0hWl9JvdpwHEOks/ -- this environment uses routed provider networks
15:10:05 mnaser As a result of this, it naturally generates a metadata of `{'links': [], 'networks': [], 'services': []}` only.
15:10:41 mnaser But after the fact if I check the instance_info_cache, the value is populated correctly, I got that value by adding a bunch of logs, so it tells me that i'm somehow getting an "unresolved" network_info
15:13:04 Uggla gibi, is "Nova needs to adjust its DB sche ma to Glance's regarding ImageProperties." something we could discuss with glance too ?
15:14:01 gibi as far as I understand from the past glance does not really want to change anything so we can bring it up with them if we have time but I think we can handled that as a nova only bug to fix
15:14:14 dansmith agree
15:46:59 Uggla melwitt, is 1h enough to covert your PTG topics (power mgmt & Tempest testing requirements for API changes
16:19:59 melwitt Uggla: I hope so :) it "should" be
16:20:29 Uggla melwitt, I try to put a convenient slot for you.
16:20:41 melwitt thanks!
16:20:45 Uggla not too early.
16:26:26 Uggla Hi all, please have a look at https://etherpad.opendev.org/p/nova-2025.2-ptg — if you spot anything incorrect, feel free to let me know.
17:21:51 opendevreview Dan Smith proposed openstack/nova master: Support "one-time-use" PCI devices https://review.opendev.org/c/openstack/nova/+/943816
17:21:51 opendevreview Dan Smith proposed openstack/nova master: Add one-time-use devices docs and reno https://review.opendev.org/c/openstack/nova/+/944262
17:21:52 opendevreview Dan Smith proposed openstack/nova master: Functional tests for one-time-use devices https://review.opendev.org/c/openstack/nova/+/946065
17:32:21 dansmith gmann: do you know why this fails like this? https://zuul.opendev.org/t/openstack/build/82b63cd467c342fa89cd0c29f599431d
17:32:33 dansmith for this requirements bump: https://review.opendev.org/946177
17:32:50 gmann checking
17:38:09 gmann dansmith: it should pass, i think it is just mirror issue?
17:38:51 dansmith oh is it? it seemed to be complaining about the conflict of versions and not that it couldn't actually find the version requested
17:39:33 gmann but conflict versions are not valid - 3.4.0 is >=3.3.0
17:40:06 dansmith you mean the complaint it has is not valid right?
17:40:30 gmann yeah, I have seen these in past due to mirror issue
17:40:40 dansmith okay, weird
17:40:59 gmann 3.4.0 is just released right?
17:41:13 dansmith related, why can't we set the u-c for things like os-traits to like <4.0.0? It's just a bunch of constants, there's no harm in running with a newer one of the same major version
17:41:27 dansmith having to bump u-c for this is kinda silly
17:41:33 dansmith gmann: yes, minutes before
17:41:58 dansmith gmann: I just didn't even consider it possibly a mirror issue since it's not failing to download, but I'm sure you're right
17:43:06 gmann dansmith: on constraints, I agree that there is no point to pin it in u-c especially we only maintain this lib and we can have (might have) master code testing before any new release and made sure new version will not break
17:43:18 dansmith yeah
17:43:32 gmann from testing tools, we consider the same approach and do not have constraint for tempest, hacking or so
17:43:59 gmann and if any project want to pin it can be done in requirements.txt for safer side like we do for hacking version
17:45:07 dansmith yeah, makes sense for nova to require the version with the constants it needs, but u-c seems like too much trouble for this
17:45:56 gmann dansmith: here we can propose to exclude it from u-c, there are many neutron lib in that list https://github.com/openstack/requirements/blob/be97f2507812c9047c5a39ee4a35b88ea7f677d7/denylist.txt#L23
17:46:52 dansmith gmann: so add it to the denylist and remove from u-c.txt?
17:47:24 dansmith I can propose that on top of this bump for discussion
17:47:35 gmann dansmith: once you add it in denylist at same time you can remove it from u-c
17:47:44 dansmith yeah that's what I meant
17:47:54 gmann yeah
17:49:17 Uggla dansmith, fyi added a slot for One-time-use devices (30mn). Is it ok for you ?
17:50:04 dansmith Uggla: yeah tbh I'm not sure there's much discussion needed at this point, but sure
17:50:24 dansmith Uggla: maybe schedule it near something else so we can go straight to the other topic if there are no issues to discuss?
17:51:21 Uggla ok
17:52:14 dansmith gmann: https://review.opendev.org/c/openstack/requirements/+/946183
17:53:21 gmann ++
17:55:12 gmann dansmith: this job failure gives clear reason for failure "ERROR: No matching distribution found for os-traits===3.4.0" https://zuul.opendev.org/t/openstack/build/69b690d9b5744edd8a96a53fa2899193
17:55:58 dansmith gmann: ah indeed
18:45:34 Uggla dansmith, https://review.opendev.org/c/openstack/releases/+/946162 merged and 3.4.0 available on pypi.
18:46:06 Uggla dansmith, fyi ^
18:46:11 dansmith Uggla: yep, thanks I already have the u-c patch up and the nova patches bumped (but I was a few minutes above the mirrors)
18:46:23 dansmith s/above/before/
18:49:35 Uggla cool, have a good day. Time for me to afk.
18:50:17 frickler dansmith: I'd rather go with the bot generated patch that has all the metadata if you don't mind https://review.opendev.org/c/openstack/requirements/+/946181 I'll also leave the decision about removing it from u-c to the "real" reqs ppl (cc tonyb)
18:52:08 dansmith frickler: okay, I've just been blocked for a while and was trying to get this moving
18:53:42 frickler dansmith: sure thing, don't hesitate to ping me if this gets stuck again (or other reqs+release stuff)
18:54:02 opendevreview Dan Smith proposed openstack/nova master: Support "one-time-use" PCI devices https://review.opendev.org/c/openstack/nova/+/943816
18:54:02 opendevreview Dan Smith proposed openstack/nova master: Add one-time-use devices docs and reno https://review.opendev.org/c/openstack/nova/+/944262
18:54:03 opendevreview Dan Smith proposed openstack/nova master: Functional tests for one-time-use devices https://review.opendev.org/c/openstack/nova/+/946065
18:54:12 dansmith rebased to depends-on the bot patch ^
18:54:26 dansmith frickler: once it passes tests can you approve it?
18:54:37 dansmith (the bot requirements patch I mean
18:54:59 frickler I'll approve now, might not be around long enough today
18:55:08 dansmith even better thanks :)
18:56:32 dansmith the bot patch looks like it happened at basically the same time mine did, I thought there was (used to be?) a substantial delay there
19:23:37 frickler there may be delays if the bot job needs to wait for a node, but that should seldom be long. other than that there's only a 10min delay to give the newly uploaded pkg time to appear in the pypi index cached by our mirrors
23:16:39 cardoe mnaser: that paste was an unbound interface which seemed weird.
23:17:23 mnaser cardoe: right, there's routed provider networks here in the equation which only assign a port/subnet until the port is plugged since it doesn't know what subnet it can pick an ip in
23:18:22 mnaser but good point, it seems like it wasn't bound yet in neutron with the baremetal stuff
23:37:21 cardoe Yeah I feel like there’s been some patches around that recently. Maybe in NGS. If that’s what you’re using.
#openstack-nova - 2025-04-03
00:43:05 opendevreview Mohammed Naser proposed openstack/nova master: Add reproducer for bug#2106073 https://review.opendev.org/c/openstack/nova/+/946221
00:44:28 mnaser cardoe: well that's a reproducer alongside https://bugs.launchpad.net/nova/+bug/2106073 .. now to figure out how to nicely fix this :)
00:45:17 cardoe mnaser: btw did you see my note about the release GH action not working on keystoneauth-websso?
00:45:29 mnaser ahhhh sorry cardoe got distracted
00:45:36 mnaser i'll add it to my tomorrow list!
00:46:25 cardoe hmm. That's gonna be a tough one to solve cause config_drive is generated before the update_port_postcommit() which will resolve that deferred allocation no?
02:36:00 opendevreview Mohammed Naser proposed openstack/nova master: Add reproducer for bug#2106073 https://review.opendev.org/c/openstack/nova/+/946221
02:37:33 mnaser https://bugs.launchpad.net/nova/+bug/2106073 - revised the bug, seems to be affecting nova at wide since it feeds in ports without ip allocation (and doesnt wait for it ip_allocation=deferred) and pushed another repro..
03:14:35 opendevreview Yaguang Tang proposed openstack/nova stable/2024.1: Fix deepcopy usage for BlockDeviceMapping in get_root_info https://review.opendev.org/c/openstack/nova/+/926696
03:35:03 opendevreview melanie witt proposed openstack/nova master: Call volume detach rollback API if detach fails https://review.opendev.org/c/openstack/nova/+/880399
03:35:03 opendevreview melanie witt proposed openstack/nova master: Reproducer for bug 2016173 https://review.opendev.org/c/openstack/nova/+/946222
03:52:56 opendevreview Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to lock API https://review.opendev.org/c/openstack/nova/+/946223
04:14:22 opendevreview Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to lock API https://review.opendev.org/c/openstack/nova/+/946223
08:46:07 opendevreview Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to lock API https://review.opendev.org/c/openstack/nova/+/946223
09:15:54 opendevreview Masahito Muroi proposed openstack/nova master: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/939658
09:18:35 masahito gmann: sean-k-mooney: ack. thanks for the review. updated the commit. https://review.opendev.org/c/openstack/nova/+/939658

Earlier   Later