Tuesday, 2025-07-01

ralonsohok folks, nova-ovs-hybrid-plug is green again: https://zuul.opendev.org/t/openstack/builds?job_name=nova-ovs-hybrid-plug&skip=008:18
ralonsohsorry for the inconveniences 08:18
gibiralonsoh: thanks for the quick fix108:32
opendevreviewBalazs Gibizer proposed openstack/nova master: Add spawn_on  https://review.opendev.org/c/openstack/nova/+/94807909:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on  https://review.opendev.org/c/openstack/nova/+/94818609:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on  https://review.opendev.org/c/openstack/nova/+/94818709:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private  https://review.opendev.org/c/openstack/nova/+/94818809:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR  https://review.opendev.org/c/openstack/nova/+/94808609:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Make the default executor configurable  https://review.opendev.org/c/openstack/nova/+/94808709:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Print ThreadPool statistics  https://review.opendev.org/c/openstack/nova/+/94834009:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Document threading mode and tuneables  https://review.opendev.org/c/openstack/nova/+/94936409:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Allow services to start with threading  https://review.opendev.org/c/openstack/nova/+/94831109:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode  https://review.opendev.org/c/openstack/nova/+/94845009:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Do not yield in threading mode  https://review.opendev.org/c/openstack/nova/+/95099409:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode  https://review.opendev.org/c/openstack/nova/+/95195709:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet  https://review.opendev.org/c/openstack/nova/+/95343609:13
opendevreviewBalazs Gibizer proposed openstack/nova master: Run unit test with threading mode  https://review.opendev.org/c/openstack/nova/+/95347509:13
opendevreviewBalazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively  https://review.opendev.org/c/openstack/nova/+/95381509:13
* gibi is sorry for spamming the CI 09:14
opendevreviewBalazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor  https://review.opendev.org/c/openstack/nova/+/95266609:17
mikalHerm, now the ceph CI test appears broken?10:04
mikalSome packages could not be installed. This may mean that you have10:04
mikalrequested an impossible situation or if you are using the unstable10:04
mikaldistribution that some required packages have not yet been created10:04
mikalor been moved out of Incoming.10:04
mikalThe following information may help to resolve the situation:10:04
mikalThe following packages have unmet dependencies:10:04
mikal qemu-block-extra : Depends: librbd1 (>= 19.2.1-0ubuntu0.24.04.1) but 19.2.0-0ubuntu0.24.04.2 is to be installed10:04
mikalE: Unable to correct problems, you have held broken packages.10:04
mikal+ functions-common:apt_get:1               :   exit_trap10:04
mikal+ ./stack.sh:exit_trap:543                 :   local r=10010:04
mikal++ ./stack.sh:exit_trap:544                 :   jobs -p10:04
mikal+ ./stack.sh:exit_trap:544                 :   jobs=10:04
mikal+ ./stack.sh:exit_trap:547                 :   [[ -n '' ]]10:04
mikal+ ./stack.sh:exit_trap:553                 :   '[' -f '' ']'10:04
mikal+ ./stack.sh:exit_trap:558                 :   kill_spinner10:04
mikal+ ./stack.sh:kill_spinner:453              :   '[' '!' -z '' ']'10:04
mikal+ ./stack.sh:exit_trap:560                 :   [[ 100 -ne 0 ]]10:04
mikal+ ./stack.sh:exit_trap:561                 :   echo 'Error on exit'10:04
mikalError on exit10:04
mikal+ ./stack.sh:exit_trap:563                 :   type -p generate-subunit10:04
mikal+ ./stack.sh:exit_trap:566                 :   [[ -z /opt/stack/logs ]]10:04
mikal+ ./stack.sh:exit_trap:569                 :   /usr/bin/python3.12 /opt/stack/devstack/tools/worlddump.py -d /opt/stack/logs10:04
mikal+ ./stack.sh:exit_trap:578                 :   exit 10010:04
mikal*** FINISHED ***10:04
sean-k-mooneythat sound like a mirror issue10:05
sean-k-mooneyi suspect that will resolve quickly10:06
mikalYeah, I just feel bad mashing retry over and over, but damn these releasenotes are hard to get through CI this week.10:07
sean-k-mooneyso lookging at https://packages.ubuntu.com/noble-updates/qemu-block-extra10:09
sean-k-mooneyit has librbd1 (>= 19.2.0-0ubuntu0.24.04.2) [not armhf]10:10
sean-k-mooneyso im wondering what verion of qemu you have10:10
sean-k-mooneymikal: do you have a link to the job10:11
mikalhttps://zuul.opendev.org/t/openstack/build/8d3bacc67bf5473993c8123ea03e969a10:11
sean-k-mooneyfor some reasons its askign for a newer dep then is enabled in the noble and noble-updates10:12
mikalAnd we're confident that job uses Ubuntu Noble?10:13
sean-k-mooneythey all do on nova except the nova-hybrid-plug job10:13
sean-k-mooneybut ya htis sjob is also noble10:13
mikalYeah, I agree that the Ubuntu packages shouldn't need 19.2.1 as best as I can tell.10:14
mikalPlucky and Questing has 19.2.1...10:15
sean-k-mooneyso the question is has a ppa or repo been enable dhat is messing that up10:16
sean-k-mooneyso we dont have the apt config and i cant see why it failed so i would say recheck it once and see if it happens again10:18
mikalI'd already done that, the previous failure was https://zuul.opendev.org/t/openstack/build/92953a7b8ed64aad84423567eab627b6 from 4.5 hours ago.10:21
mikalAnd https://zuul.opendev.org/t/openstack/build/02ab3d4b300d40b68effc5b19fdfcb21 from 7 hours ago10:22
mikalSo this started happening sometime in the last 13 hours, but more than 8 or so hours ago.10:22
fricklerI saw the same issue in the osa channel. looks like ubuntu messed up some packaging and reverted/fixed some hours ago, need to wait for our mirrors to catch up I guess11:42
jrosserfrickler: I was waiting until later in my day so ask if there was anything could be done to preempt the mirror update11:55
jrosser*to ask11:55
cardoeDid I mess up my calendar reminder for nova meeting?15:00
cardoeI'm really just looking to understand what else we need to do to get https://review.opendev.org/c/openstack/nova-specs/+/471815 to land?15:01
cardoeLanding that is the blocker for coordinating multiple projects to land the tests and the implementations.15:02
UgglaNova meeting in ~50mn15:11
Ugglacardoe, I think you can join to discuss it in the open discussion topic.15:12
UgglaFrom what I see the patch seems in a good shape.15:13
cardoeokay thanks15:28
UgglaNova meeting in around 10mn15:49
zigoUnder Python 3.13, nova-scheduler.log spits many of these: https://paste.opendev.org/show/bLJRfWD6kJr07tgShen2/15:54
zigoDoes anyone have a clue what's going on ?15:54
zigoFYI, this is with eventlet 0.40.0+2025.06.18.e470c1f493 (ie: master branch up to commit e470c1f493).15:56
zigoIt's doing this in loop.15:57
zigonova-conductor too.15:57
zigoShould I open a new bug?15:58
gibizigo: I can only add that I haven't seen similar in py31215:58
Uggla#startmeeting nova16:00
opendevmeetMeeting started Tue Jul  1 16:00:24 2025 UTC and is due to finish in 60 minutes.  The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'nova'16:00
UgglaHello everyone16:00
gibio/16:01
bauzaso/16:01
bauzas(I'll need to leave a bit early, like 15 mins before the end)16:01
masahitoo/16:02
Uggla#topic Bugs (stuck/critical) 16:02
Uggla#info No Critical bug 16:02
Uggla#topic Gate status 16:03
Uggla#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:03
Uggla#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:03
Uggla#link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status16:03
Uggla#info Please look at the gate failures and file a bug report with the gate-failure tag.16:03
Uggla#info Please try to provide a meaningful comment when you recheck16:03
Uggla#topic tempest-with-latest-microversion job status 16:04
Uggla#link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=016:04
Ugglagmaan, something you'd like to tell us ?16:04
fwieselo/16:04
Ugglanot sut gmaan is available today, so I'm gonna move on.16:05
Uggla#topic Release Planning 16:06
Uggla#link https://releases.openstack.org/flamingo/schedule.html16:06
Uggla#info Nova deadlines are set in the above schedule 16:06
Uggla#info Nova spec freeze is Thursday.16:06
Uggla⚠️ ^16:06
UgglaFYI, I have discussed with masahito about https://review.opendev.org/c/openstack/nova-specs/+/951636 and if he will be able to submit an update. So it will give us a chance to approve this one.16:08
opendevreviewStephen Finucane proposed openstack/nova master: api: Correct expected errors  https://review.opendev.org/c/openstack/nova/+/95164016:08
Uggla#topic Review priorities 16:08
masahitoyup. let me focus on updating my specs tomorrow.16:08
Ugglamasahito 👍16:09
Uggla#link https://etherpad.opendev.org/p/nova-2025.2-status16:09
UgglaI have updated the doc and updated launchpad accordingly. So blueprints, SL blueprints should be all in the correct status.16:10
UgglaIf you see something wrong in the doc, please let me know.16:10
Uggla#topic OpenAPI16:11
Uggla#link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned16:11
Uggla#info 19 decrease -6. \o/16:11
Ugglaelodilles is not available so I'll skip the stable branch topic.16:12
Uggla#topic vmwareapi 3rd-party CI efforts Highlights16:12
Ugglafwiesel do you have something to share ?16:12
Ugglafwiesel ?16:14
fwieselUggla: No, nothing from my side16:14
Ugglafwiesel ok thx16:14
Uggla#topic Gibi's news about eventlet removal. 16:14
Uggla#link Blog: https://gibizer.github.io/categories/eventlet/16:15
Uggla#link nova-scheduler series is ready for core review, starting at https://review.opendev.org/c/openstack/nova/+/94796616:15
gibio/16:15
Ugglagibi the mic is yours16:15
gibiso the scatter gather refactor patch has been landed 16:15
gibithanks for the reviews16:15
Uggla\o/16:15
gibithe next two patches are also approved but the gate was / is pretty flaky in the recent days so they are not landed yet16:16
gibi(hybrid-plug and ceph issuees)16:16
gibiI fixed the comments form bauzas on the next patch in the series about a possible race condition. Please take a look16:16
gibiOn the top of the series I started an effor to run unit tests without eventlet 16:17
gibithe effort starts here https://review.opendev.org/c/openstack/nova/+/953436/516:17
gibicouple of patches later in that series we have a job that runs most of our unit tests without monkey patching and with threading oslo.service and oslo.messaging backend16:18
gibithere is a list of excluded test I have to work through16:18
gibiI'm tracking to type of problems. Sqlaclhemy nonsense errors and simple test case hangs 16:18
gibis/to/two/16:19
gibiyou can see them noted in the test exclude list16:19
gibiI think we can make the new zuul job voting pretty soon 16:19
gibi(while keeping some excludes)16:19
gibithat is it from me16:20
sean-k-mooneyo/ sorry had this behind anohter window16:20
Ugglagibi do you have some kind of "urgent" review anymore ?16:21
bauzasI'll continue to review the series for sure16:22
gibiUggla: the bottom of the series is as urgent as the eventlet removal :)16:22
gibibauzas: hanks16:22
gibithanks 16:22
gibiI think we can move on :)16:24
UgglaBTW we have Nova Eventlet removal sync tomorrow too.16:24
Uggla14:30 UTC16:25
gibiyepp we have the call 16:25
gibias planned16:25
Ugglathanks gibi, next topic16:25
Uggla#topic Open discussion 16:25
Uggla#topic (fwiesel) Evacuate Action & InstanceInvalidState16:25
Ugglafwiesel please go ahead.16:26
fwieselSo, we are currently looking into doing HA by using the evacuate action on instances, and were running into various states, where it was not possible.16:26
fwieselFrom our side, evacuation usually happens when the host is down so can happen at any time.16:27
fwieselAnd I was wondering, if it wouldn't make sense to adjust the code so it can evacuate an instance in as many sitations as possible.16:27
fwieselSo, raising an InstanceInvalidState is merely a reflection of  what has been implemented. What do you say?16:28
gibido you have specific examples of states you want to evacuate from?16:29
cardoeSo I'll say internally we've got extra stuff which we don't see nova ever being able to handle.16:29
cardoeLike even human involved state16:29
fwieselLet me think. For starters: Verify-Resize.16:30
fwieselBut let's say, we have a live-migration, and that host falls down. I still would like to do a failover16:31
gibiverify-resize: so while you could evacuate from that state you also need to implement a proper cleanup in the nova-compute coming up in the source node of the original resize16:31
sean-k-mooneyso evacuate requires you the admin16:32
sean-k-mooneyto assert that the vm is not running on any host16:32
sean-k-mooneyso allowing it in verify resize and othe rstates is questionable16:32
sean-k-mooneyespically since in that state your in the middle of an ongoing migration16:32
cardoeYeah there's a lot of squirrely cases there which need to be monitored for.16:32
sean-k-mooneyand the flavor that is in use is kind of ill defiend16:33
sean-k-mooneyits technialy the new one in a real reaise but it has nto beeen confirm so you woudl want ot evacuate to the old one, but hte numa toplogy and ohter things are all in a weird hybrid state16:33
sean-k-mooneywhat you likely should be able to do is revert the resize in verify-resize16:34
gibiso in general it would be nice to support these but in practic it seems like a bunch of cans of worms 16:34
sean-k-mooneyeven if the dest host is down16:34
sean-k-mooneythen evacuate it16:34
sean-k-mooneyif needed16:34
cardoeWe've got a whole thing out of band that's specific to the hypervisor in use for these items.16:35
gibiyeah allowing revert-resize in souce host down is one way, even though a weird one. Who wants to revert a resize that will lead to the VM runing nowhere :)16:35
fwieselBut technically, an evacuate is practically a rebuild. So, I can start from "scratch" (nova wise)16:35
sean-k-mooneygibi: well it would revert reisze when dest down and conrim resize when source down16:35
fwieselI just have to build the VM up somewhere fitting the spec16:35
gibiyeah starting is easy, cleaning up the remnants is messy16:36
sean-k-mooneyfwiesel: yes and no16:36
cardoeyeah but you need to make sure that gets fully cleaned up.16:36
fwieselOf course16:36
sean-k-mooneywe have some api contract about preserving storage in some cases16:36
sean-k-mooneyplease there are the neutorn/cidner state clean up that we need to do16:36
fwieselExactly16:36
gibisean-k-mooney: revert-resize: ahh right16:36
fwieselRight now, I have the choice to implement all that outside of nova. Or within nova16:37
sean-k-mooneyto do this in nova woudl need a very detailed spec16:37
fwieselWell, I would not tackle all cases in one go.16:37
sean-k-mooneyand you woudl have to consier things liek cross cell resize16:37
fwieselAnd maybe then not resize :)16:38
cardoeits a worthy goal16:38
sean-k-mooneyright but even if you take an incremntal approch you need to have each incremantal chagne be valid16:38
sean-k-mooneyi think for rezie it woubld be siiler to allwo the migration to compelte or revert based on which end is alive16:38
cardoeI'll say when I did this with xen it was much easier because xapi had an object version field so I used that as a monotonic number which also then was stored in the nova DB16:38
sean-k-mooneyand if they are both dead well your kind of out of luck16:39
fwieselOf course. I was more fearing that it will be difficult to test that in the CI, and adds complexity. Which might be areason you might not want it at all.16:39
cardoeSo each time the instance got updated the monotonic number increased and that got pushed down into xapi16:39
gibifwiesel: start with a simpler state first, write up a spec how would you make sure we end up in the consistent state after the a successful evac and a recovered compute (or an failed evac and a recovered compute, of a failed evac and a retried evac and recovered compute, ...)16:39
cardoeBut libvirt doesn't have that16:39
cardoeSo when we'd go to clean up, anything that wasn't at the current version could get nuked.16:39
sean-k-mooneyfwiesel: so testing that in tempest would be hard, but it would be more doable in the functional suite16:39
cardoeBut there's all kinds of dangling references to storage and what not that really are a pain to fix16:40
gibiyepp use functional test for sure16:40
fwieselOkay, I'll take that with me. Thanks.16:40
gibiwe have a limited evact test somewhere after tempest16:40
sean-k-mooneycardoe: so we dont need montic object version in general becasue we assocate the instance with the comptue service that is currently manging it16:40
gibiin a palybook16:40
sean-k-mooneyin the post test hook in nova-next16:41
cardoeFor us we implemented this more as monitoring that an unexpected object was on a host.16:41
sean-k-mooneyand i think the cepch job?16:41
sean-k-mooneycardoe: ya so that kind of approch is not very comaptible with the nova desing although we do have some per host perodic that are intened ot clean up these types of issues16:42
cardoesean-k-mooney: when you perform a live migrate and the dest goes down at the very end during the resume16:42
sean-k-mooneyesplcially on compute agent start up16:42
sean-k-mooneycardoe: if the vm dies before resume libvirt will abort the migraiot16:42
sean-k-mooneyif we got to post live migrate 16:43
sean-k-mooneyits too late to abort but we will update the instance hsot to point to the dest16:43
gibithis is the evac testing in devstack https://github.com/openstack/nova/blob/master/roles/run-evacuate-hook/tasks/main.yaml16:43
Ugglaguys is it ok to move to the second point ?16:43
gibiOK for me16:44
cardoeI'm just saying it wasn't a simple problem when I was working in the VM world because hardware dies at the worst possible times and nova can be in many different code calls when that happens.16:44
sean-k-mooneyyep lets move on for now.16:44
Uggla#topic (jonnyb) Blueprint for new compute created weigher (https://blueprints.launchpad.net/nova/+spec/node-uptime-weigher)16:44
Ugglajonnyb wants to discuss about this weigher16:44
Ugglajonnyb please go ahead.16:45
jonnybso I propose a scheduler weigher based on creation date of the compute service16:45
jonnybthe initial idea was to use uptime but this is too dependend on the hypervisor and its driver16:45
jonnybfor us the creation is good enough and we use it since we migrate and deploy a lot of servers16:46
gibido you have the service creation time as part of the HostState the weigher running on?16:46
sean-k-mooneyi belive we do16:46
dansmiththat surprises me16:46
sean-k-mooneywe have  it form the compute node object i belive16:46
dansmithsure, but in hoststate?16:47
sean-k-mooneyso this is the poc https://review.opendev.org/c/openstack/nova/+/947503/9/nova/scheduler/weights/compute_created.py16:47
dansmithwe have an object, but I didn't think it was queried from the db and thus not complete, but maybe that's not right16:48
sean-k-mooneyso yes https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L17616:49
sean-k-mooneyyou could be correct about it being a partial object16:49
gibibased on the poc we have HostState.service.created_at  as created_at does not expect to change it is fair to assume it is up to date16:49
gibiisn't expected to change16:49
sean-k-mooneyso it looks like this is only populated via update its not populated iva __init__16:50
sean-k-mooneyso im not sure if its there if you disatble the track_isntace_state notifications stuff16:51
dansmithokay we are doing a scatter to gather nodes and hosts from cells16:51
sean-k-mooneywe woudl have to confirm that its always present in both confiugrations16:51
dansmithI was thinking we did an optimized construction from a list there16:51
dansmithsean-k-mooney: yeah, good point16:51
sean-k-mooneyi think that config only affect the instnace object and how those are populated16:52
dansmithbecause this could also be triggering massive lazy loads, which would be bad16:52
sean-k-mooneybut assuming we have that alwasy16:52
sean-k-mooneyi think it would be ok to use it in the weigher provide ya we are not lazy loading this16:52
dansmithwe need to make sure this is run in CI at least once so we can examine logs for lazy loads16:53
sean-k-mooneyya we can turn this on in one fo the jobs16:56
sean-k-mooneyit wont really have much of an effect with only 1-2 nodes16:57
dansmithwe don't have to permanently, but just a DNM on top of this to get a run and make sure we don't see "lazy loading service.created_at" :)16:57
sean-k-mooneywe are wrappign the sercie in a readonly dict class that i have not really seen before. i would assume that shoudl also block lazyloading?16:58
sean-k-mooneygoign back on topic16:58
sean-k-mooneyare we ok with appvoing this as a specless blueprint16:58
sean-k-mooneyand continuing to review16:58
dansmithidk16:59
Uggla:+1 for me it looks an interesting weigher.16:59
Ugglagibi, ok for you ?17:00
gibiI'm fine17:00
sean-k-mooneyi dont nessiarly have an objection just note that if there are lazy loadign issues they will need to be resolved ebfore we can merge it17:00
dansmithsame17:00
sean-k-mooneyso it might not be ready before FF 17:00
sean-k-mooneybut im ok to move to the review and take a look a ci logs ectra17:01
Ugglajonnyb is that ok for you ?17:01
jonnybthats fine with me, i wasnt aware that lazy loading could be an issue. but good point17:01
sean-k-mooneyyou defaulted to 1.0 so in theroy its enabled by default17:02
sean-k-mooneyso we should b eable to look at any of the current ci logs17:02
Ugglaok last topic:17:02
Uggla#topic (gibi) Workaround for cpython3.13 GC bug needs review https://review.opendev.org/c/openstack/nova/+/95296617:02
dansmithoh, definitely seems like it should be disabled by default17:02
dansmiththat's merged?17:02
sean-k-mooneywell we can defintly change that. in the review and ya the gc thing is merged17:03
gibiyepp it is merged17:03
Uggladansmith yes it seems17:03
gibiI proposed the backport to Epoxy as I know debian want it there17:03
gibi(the cpython bug fixing effort is progressing in the background)17:04
gibiso other than the backport there is nothing else left here17:05
gibiwe can move on17:05
sean-k-mooneyjonnyb: so without diging deeply https://zuul.opendev.org/t/openstack/build/8a9b0a0404034a79a95621b18e16534b/log/controller/logs/screen-n-sch.txt#1017 you schduler was enabeld and i dont see lazy loadign but we need to look at that properly after the meeting17:05
Ugglaok cardoe, wants to discuss about a patch17:05
Ugglacardoe please go ahead, and then we will close.17:06
cardoehttps://review.opendev.org/c/openstack/nova-specs/+/47181517:06
cardoeIt is a proposed spec change to add trunk ports with their vlan tags to the network_data.json17:06
sean-k-mooneyya im +2 on that gibi is +1 because they wont have review bandwith i belvie17:06
gibiI'm +2-1=+1 signalling review bandwidth yes17:07
cardoeIt's something written originally by someone not involved in the effort currently.17:07
cardoeBut neutron supports defining ports like that.17:07
cardoeThere have been a few patches floating around from different operators that are using that functionality as a downstream patch.17:07
dansmithsean-k-mooney: jonnyb: agree, I see the weigher being weighed and no lazy loads in that log, so seems fine17:07
gibihaving +1 from rubasov (Bence) is a good sign, he was involved in the original work of trunk17:08
cardoeI've been working to consolidate the downstream operators around a single implementation and wanting to get it upstreamed.17:08
sean-k-mooneyyep17:08
cardoeWe've got temptest tests written and an implementation.17:08
sean-k-mooneythere is code for noav and tempest test too17:08
sean-k-mooneyya https://review.opendev.org/q/topic:%22bp/expose-vlan-trunking%2217:09
sean-k-mooneythis is useful for both vms adn for ironic17:09
cardoeThis is just part of my TC work to try and get downstream operators to push back to the community.17:09
sean-k-mooneyso the requwst is bascily for another core to sponsor/review this before spec freeze on thruday 17:10
sean-k-mooneyand then to review the actul pactchs before m317:10
sean-k-mooneythe one thing i will say is i think we only have turnks in one job17:11
sean-k-mooneyso we will need to confirm that this si properly tested in our ci and that that job passes17:12
sean-k-mooneythe logs have rotated and the one nova patch that is requried https://review.opendev.org/c/openstack/nova/+/941227 is in merge conflict if i ty an rebase17:13
cardoeI'll work to get folks to update things based on feedback.17:13
sean-k-mooneyoh its just missign isnged off by17:13
Ugglacardoe something else you want to ask ?17:14
cardoeNo my only question is how can I get this on the roadmap17:15
UgglaWe are overtime, so we should close.17:18
UgglaThanks all17:18
Uggla#endmeeting17:18
opendevmeetMeeting ended Tue Jul  1 17:18:25 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:18
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-01-16.00.html17:18
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-01-16.00.txt17:18
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-01-16.00.log.html17:18
sean-k-mooneycardoe: if we cant proceed with it this cycle i woudl say its still wroth rebaseign for next cycle17:21
sean-k-mooneyover all i think geting this in to upstream is a good thing17:22
sean-k-mooneyand we shoud encurrage upstream operator change like this when they are in scope of nova ot address17:22
Ugglasean-k-mooney, if you have a min, what do you think of this bug: https://bugs.launchpad.net/nova/+bug/2115116  to my mind it is out of our scope but I'd like your feelings.17:23
sean-k-mooneyso i have been kind fo frustrated with some of the install guid work that has been sumited lately17:24
cardoeIt's this constant catch-22. The core teams have their own workloads and aren't there to respond to the whims of the operators. So the operators need to contribute more from a review standpoint. But then the operators are all counting spend and wanting to equate their return on investment.17:24
sean-k-mooneyUggla: i see where they are coming form but personlly i dont think the mysql info should be in the install guide either17:25
gibithe core team are basically employed by a company which is also counting profit, so operators could do the same, i.e. paying contributors to become cores17:25
sean-k-mooneybeyound a very high level over view of how to use teh config option ectra17:25
Ugglasean-k-mooney, btw there is so many ways to install it.17:26
sean-k-mooneyUggla: i personaly woudl mark it opipion17:26
sean-k-mooneyits not invalid i just think we would be better limiting the scope of the doc17:27
sean-k-mooneywe do not incldue it in the ubuntu ones either for what its worht17:28
sean-k-mooneyso it not a disto specific gap17:28
sean-k-mooneywe just doing docuemtn how to install rabbit in the nvoa guide17:28
sean-k-mooneywe tell you to configure the transport url transport_url = rabbit://openstack:RABBIT_PASS@controller:5672/17:29
sean-k-mooneybut that is reaonable17:29
sean-k-mooneywe coudl add a note to the prequistit section to say that rabbit must be installed adn aviable17:29
sean-k-mooneybut i would not say how to do that17:29
Ugglaok I will try to answer in this way tomorrow morning. 17:31
sean-k-mooneyso i mentioned im furstrated, there has been a number of new contiubotrs filing lots of small bugs for the install guide and submiting pathce for them17:32
sean-k-mooneywhil that is not a bad thign persay there has been no dicussion or cordiantion about this effort in genral17:32
sean-k-mooneyso it kind of feels lke some owne said to a bunch of interns ehre is a good way to do a first patch17:33
sean-k-mooneybut didnt actully talk to use about it first17:33
sean-k-mooneyim exagreating a bit17:33
sean-k-mooneybut that sort of how it feels, a buch of lower pririty bugs were created and folks are fixign them but no dicsussion of do we have time to review it17:34
Ugglaok but in this case there is no patch attached too. Or I miss something. 17:37
sean-k-mooneyno i just went out of my way to do some reviews in this area a while ago and sort of got burt out on it17:38
Ugglaok17:39
sean-k-mooneyim fine with fixing the docs in general and if they want to add a patch to add it as a preqeusit for all distos then ok but i dont wnat to need to keep updating how to install rabbit if we can avoid it17:39
sean-k-mooneyhttps://review.opendev.org/c/openstack/nova/+/93870217:40
sean-k-mooneyso there is a patch for it a diffent bug https://bugs.launchpad.net/nova/+bug/209158517:41
sean-k-mooneyso we proably shoudl clsoe the new one as a duplicate of that one17:41
sean-k-mooneyill just do that now if there is no objetion we do not need two bugs for that17:42
sean-k-mooneythere is apprenly a commen guide that covers the install we can just link too https://docs.openstack.org/install-guide/environment-messaging-rdo.html17:44
sean-k-mooneythere is a ubuntu version too https://docs.openstack.org/install-guide/environment-messaging-ubuntu.html17:44
mikalAt least the ceph CI error has changed this morning?21:55
mikalThe following packages have unmet dependencies:21:55
mikal qemu-block-extra : Depends: qemu-system-any (= 1:8.2.2+ds-0ubuntu1.7) or21:55
mikal                             qemu-utils (= 1:8.2.2+ds-0ubuntu1.7) but 1:8.2.2+ds-0ubuntu1.8 is to be installed21:55
mikalE: Unable to correct problems, you have held broken packages.21:55
opendevreviewMerged openstack/nova master: doc: Adding link for RabbitMQ installation during nova deployment on controller node.  https://review.opendev.org/c/openstack/nova/+/93870222:33

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!