| *** bauzas_ is now known as bauzas | 01:47 | |
| *** bauzas_ is now known as bauzas | 03:11 | |
| *** bauzas_ is now known as bauzas | 06:13 | |
| opendevreview | Alexander Hild proposed openstack/nova master: feat: Include "vm uuid" in baseBoard asset tag in libvirt domain xml https://review.opendev.org/c/openstack/nova/+/924697 | 07:29 |
|---|---|---|
| opendevreview | Alexander Hild proposed openstack/nova master: feat: Include "vm uuid" in baseBoard asset tag in libvirt domain xml https://review.opendev.org/c/openstack/nova/+/924697 | 07:31 |
| *** bauzas_ is now known as bauzas | 07:38 | |
| mikal | sean-k-mooney: are you around? I think https://review.opendev.org/c/openstack/nova/+/922544 is ready for review but I have questions about if there needs to be a tempest test for it. | 08:06 |
| sean-k-mooney | mikal: im not sure how you would tempet test that spice is secure. we recently tired to do that for vnc and vcrypt and we could not come up with a reasonable way ot do that just with a tcp connection becuase vnc uses tls to encypt the vnc packate payload not the entire tcp packet payload | 08:39 |
| sean-k-mooney | so you can just connect without implementing the nvc protocal at least enought to negoicate the vencrypt session | 08:40 |
| sean-k-mooney | unless there is a simple way to asssert that spice is using a secure connection without implemeting a portion of the spice protocoal i think unit and perhaps functional tests are enough | 08:41 |
| sean-k-mooney | mikal: it might be worth considerign extending the libvirt funcitonal tests to be able to test some of the spice direct support but im not sure where in the series it makes sense to do that | 08:43 |
| mikal | sean-k-mooney: so, it depends on if tempest can carry additional software. Kerbside does have a simple client which can parse enough of the protocol to know if its been redirected to a secure connection or not. | 09:20 |
| mikal | sean-k-mooney: but then we'd have to carry extra code in tempest for that. https://github.com/shakenfist/kerbside/blob/develop/kerbside/spiceprotocol/__init__.py#L102 is approximate the right code. | 09:21 |
| mikal | sean-k-mooney: also, should I move this spec from the "to be determined" bit of the dalmation etherpad to the "needs review" bit, or is that someone else's job? | 09:29 |
| sean-k-mooney | mikal: tempest is generally not allowed to have depencies on external tools unless you are writing your own plugin | 09:30 |
| sean-k-mooney | there are some things like ssh ectra but for the most part we try to keep core tempest as depency free as is reasonable | 09:31 |
| mikal | sean-k-mooney: yeah, that's going to make any meaningful testing pretty hard to do. | 09:31 |
| sean-k-mooney | mikal: again it would be totally fine to depend on kerbside in a kerbside tempest plugin | 09:31 |
| mikal | sean-k-mooney: I'm happy enough to take a swing at writing such a plugin, but I'd like to see some more traction on the patch series and implement more of the changes requested (extra specs, new API flow) before pivoting to that. | 09:32 |
| sean-k-mooney | and that can be in the kerbside repo or in a seperate one. if we want to do real end ot end testing i would suggest creating both a devstack adn tempest plugin which we can enable in a upstream job in the future | 09:33 |
| sean-k-mooney | ya i personally woudl love to see end ot end testing but im ok with it happenign after the feature is completed | 09:33 |
| mikal | I'd need someone to explain more to me what a tempest plugin in a separate repo gains as compared to just doing end to end CI with github actions in the kerbside project. I've never done much with tempest. | 09:33 |
| sean-k-mooney | i think unit and functional tests woudl be enouch for now | 09:33 |
| mikal | Well, I'm a bit worried about only having like three or four weeks to get all this landed at the moment. | 09:34 |
| sean-k-mooney | yep understandable | 09:34 |
| mikal | The patches have unit tests already (except for the new bits). | 09:34 |
| mikal | I'm not particularly asking for special treatment, but there is only so much time before the deadline and I don't see a lot of value in writing lots of tempest stuff before the actual code is complete. | 09:35 |
| mikal | sean-k-mooney: so should I move the heading in the etherpad? | 09:45 |
| sean-k-mooney | sorry which etherpad? | 10:00 |
| sean-k-mooney | mikal: was distracted downstream | 10:00 |
| mikal | sean-k-mooney: no worries. The dalmatian etherpad (https://etherpad.opendev.org/p/nova-dalmatian-status) lists the SPICE stuff under "to be determined" not "needs code review". Should I just change that, or is it done by a designated person? | 10:31 |
| sean-k-mooney | mikal: oh ya you can move it to needs core review since its approved and you have code not in WIP state | 10:40 |
| mikal | sean-k-mooney: thanks done. I'm hesitant to un-WIP the next few patches because I'll have to redo CI on them, but I guess I'll do that too. | 10:52 |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | 10:54 |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | 10:54 |
| opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | 10:54 |
| opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | 10:54 |
| opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | 10:54 |
| opendevreview | Alexander Hild proposed openstack/nova master: feat: Include "vm uuid" in baseBoard asset tag in libvirt domain xml https://review.opendev.org/c/openstack/nova/+/924717 | 10:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | 11:02 |
| opendevreview | Alexander Hild proposed openstack/nova master: feat: Include "vm uuid" in baseBoard asset tag in libvirt xml https://review.opendev.org/c/openstack/nova/+/924720 | 11:05 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Fix typo https://review.opendev.org/c/openstack/nova/+/924722 | 11:06 |
| opendevreview | Alexander Hild proposed openstack/nova master: feat: "vm uuid" in baseBoard asset tag https://review.opendev.org/c/openstack/nova/+/924723 | 11:08 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for port interface APIs https://review.opendev.org/c/openstack/nova/+/924589 | 11:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for availability zone APIs https://review.opendev.org/c/openstack/nova/+/924590 | 11:11 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for bare metal node APIs https://review.opendev.org/c/openstack/nova/+/924591 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for console auth token APIs https://review.opendev.org/c/openstack/nova/+/924592 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor access API https://review.opendev.org/c/openstack/nova/+/924593 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor extra specs APIs https://review.opendev.org/c/openstack/nova/+/924594 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavors APIs https://review.opendev.org/c/openstack/nova/+/924595 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP pool APIs https://review.opendev.org/c/openstack/nova/+/924596 | 11:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add contributor docs for response body validation https://review.opendev.org/c/openstack/nova/+/924597 | 11:12 |
| *** chuanm9 is now known as chuanm | 12:16 | |
| *** iurygregory__ is now known as iurygregory | 12:17 | |
| elodilles_ooo | bauzas_: fyi, i'm off this week, so i won't participate on today's meeting | 13:03 |
| bauzas_ | elodilles_ooo: ack thanks | 13:04 |
| bauzas_ | have a good rest | 13:04 |
| *** bauzas_ is now known as bauzas | 13:08 | |
| sean-k-mooney | elodilles_ooo: in that case please ignore irc and email until your pto is over :) you deserve a break | 13:13 |
| opendevreview | Julien Le Jeune proposed openstack/nova master: Fix test_vmdk_bad_descriptor_mem_limit and test_vmdk_bad_descriptor_mem_limit_stream_optimized https://review.opendev.org/c/openstack/nova/+/924148 | 13:14 |
| opendevreview | sean mooney proposed openstack/nova master: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924731 | 14:01 |
| jlejeune | sean-k-mooney: hello, what du you mean by your remark on my proposal in https://review.opendev.org/c/openstack/nova/+/924148 ? | 14:01 |
| opendevreview | sean mooney proposed openstack/nova stable/2024.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924732 | 14:01 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.2: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924733 | 14:02 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924734 | 14:03 |
| opendevreview | sean mooney proposed openstack/nova stable/2024.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924732 | 14:04 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.2: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924733 | 14:05 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924734 | 14:05 |
| sean-k-mooney | jlejeune: i woudl prefer not to try and do the convert and catch the exepction and instead check via the help text if qemu image is present and supprots vmdk | 14:06 |
| jlejeune | hm ok, so a check with --help option before running the qemu-img command, just to verify that qemu(img binary is present or not, right ? | 14:10 |
| sean-k-mooney | jlejeune: basically i dont want to do any of the extra setup in that function if its not installed and does not supprot the format | 14:10 |
| dansmith | personally I'd just go for consistency with the one above (which the current version pretty much does) and not worry about it as much since this is all moving to oslo | 14:11 |
| dansmith | I'd also say that I think a dependency on qemu-img being in path and available for tests is very reasonable | 14:12 |
| dansmith | ...for anything in nova for sure | 14:12 |
| dansmith | and definitely for oslo.utils, since it has a bunch of tests for qemu-img info already | 14:12 |
| jlejeune | dansmith: that's was my first proposal, do the same as other tests and waiting for a global move in oslo | 14:12 |
| jlejeune | :/ | 14:12 |
| dansmith | jlejeune: yep, that's fine with me fwiw.. I don't think we need to probe --help, but maybe sean-k-mooney can explain more | 14:13 |
| sean-k-mooney | our downsstream version dont have some format compiled in and as you are aware many disues have an unsafe qemu image so i didnt want to run it even if its just on our local system | 14:14 |
| sean-k-mooney | what proaposed is fine i just wanted to aovid calllign convert/create ectra if we coudl | 14:14 |
| dansmith | changing FileNotFound to Exception (as the previous one you pointed to does) will catch both cases right? | 14:15 |
| sean-k-mooney | yep and i dont think we want to differenciate | 14:15 |
| sean-k-mooney | ill admign because of the other things i just pushed i only quickly skimmed the patch | 14:16 |
| sean-k-mooney | so i might have missed somethign | 14:16 |
| dansmith | yeah, so I think just using bare exception like the case above makes the most sense, jlejeune bauzas probably just didn't realize (as mentioned by the comment above) that it covers format issues and missing file issues | 14:17 |
| jlejeune | dansmith: totally agree, that was my opinion at first | 14:18 |
| bauzas | yeah I missed the first catch | 14:19 |
| bauzas | what I explained is that I prefer to catch specific exceptions mostly | 14:20 |
| bauzas | but we have some exceptions in code where we bare-catch | 14:20 |
| bauzas | here, sean-k-mooney is just pointing out an already existing pattern, so I'm cool with it | 14:20 |
| bauzas | sorry if that created some misunderstanding | 14:20 |
| dansmith | yep, honestly we should raise on FileNotFound and call qemu-img a requirement, IMHO, and skip on anything else, but this code shouldn't be staying in nova so let's just do the consistent thing for now | 14:21 |
| jlejeune | dansmith: ok, so let's go back on my patchset1 :) | 14:21 |
| bauzas | agreed with dansmith here | 14:21 |
| bauzas | keep consistency for now, but I'd appreciate some further rework on that whole crazy module :) | 14:22 |
| jlejeune | :) | 14:22 |
| opendevreview | Julien Le Jeune proposed openstack/nova master: Fix test_vmdk_bad_descriptor_mem_limit and test_vmdk_bad_descriptor_mem_limit_stream_optimized https://review.opendev.org/c/openstack/nova/+/924148 | 14:27 |
| sean-k-mooney | jlejeune: we have some thing in ci that we dont want this to conflict with so we will likely loop back to your review tomorrow or later today | 14:29 |
| jlejeune | sean-k-mooney: no problem | 14:31 |
| dansmith | sean-k-mooney: ++ | 14:31 |
| bauzas | sean-k-mooney: about that, I missed some comment, but could you please rebase the stable patches by adding [stable-only] tag so the order wouldn't be important for merging ? | 14:31 |
| dansmith | that's not necessary, IMHO | 14:32 |
| dansmith | they're up, we can land them in normal order now | 14:32 |
| sean-k-mooney | bauzas: i spoke to gibi about that and came to the same conlcution | 14:32 |
| sean-k-mooney | we dont need to do that in this case | 14:32 |
| dansmith | they just need to be posted so we have URLs for the advisory, but no need to force fast merge them or anything | 14:32 |
| bauzas | well, sure | 14:32 |
| sean-k-mooney | i also want a full set of check results so dont want to promot to gate directly | 14:33 |
| bauzas | I'm saying that because our SLURP branches could merge before the Bobcat one | 14:33 |
| sean-k-mooney | bauzas: that techncially breaking policy | 14:33 |
| bauzas | but we can proceed by order, I'm fine | 14:33 |
| sean-k-mooney | we could but they are equally supported while instable | 14:33 |
| sean-k-mooney | we shoudl not priorities slurps over supported non slurps | 14:34 |
| bauzas | it was just a pragmatic question in terms of merging | 14:34 |
| bauzas | but I'm cool | 14:34 |
| sean-k-mooney | with that said i have promoated zuul to its own monitor which i will be watching until thsee land :) | 14:35 |
| bauzas | ++ | 14:35 |
| frickler | sean-k-mooney that might be a night shift ;-/ | 14:38 |
| frickler | depending on the state of the gate I would still promote them to the head of gate if needed. also ping me if re-gates are needed due to unrelated failures | 14:39 |
| dansmith | we should at least wait until we see a clean run in check, but yeah, thanks | 14:40 |
| frickler | yes, I won't submit them to gate before that | 14:40 |
| sean-k-mooney | frickler: ya we can consider that once the iniall run is complete. at least this time its one commit per branch | 14:48 |
| sean-k-mooney | awsome post failure in one of the jobs.... | 14:56 |
| sean-k-mooney | and its related. | 14:56 |
| sean-k-mooney | well i know what im doing for the next little while | 14:56 |
| sean-k-mooney | https://4b125a6587520ffd7605-3e126156a22dab1f4ba4bdb0e9add880.ssl.cf2.rackcdn.com/924731/1/check/nova-live-migration/a83a99e/testr_results.html | 14:56 |
| dansmith | oye | 14:57 |
| dansmith | sean-k-mooney: thanks for doing that, I gotta run in a sec, but I'll be back later | 14:57 |
| dansmith | sean-k-mooney: in glance we found some tempest jobs were uploading qcows as AMI because AMI is the first format in the disk_image_formats list, so that could be related | 14:58 |
| sean-k-mooney | maybe btu theses are migration test failrues | 14:58 |
| dansmith | sean-k-mooney: https://review.opendev.org/c/openstack/glance/+/923321 | 14:58 |
| sean-k-mooney | but ya ill check if that job has that fixed or not | 14:59 |
| dansmith | sean-k-mooney: yeah I know | 14:59 |
| dansmith | just saying, something to look for | 14:59 |
| sean-k-mooney | i think this is the job we use uec images in so it proably related | 14:59 |
| dansmith | ahh right | 14:59 |
| sean-k-mooney | if thats all it is then we can fix the ci without respining the patch | 14:59 |
| sean-k-mooney | whic would be good | 14:59 |
| sean-k-mooney | lets see if it passes the same test in the hybrid plug job | 15:00 |
| sean-k-mooney | since that is using qcow | 15:00 |
| sean-k-mooney | oh yep it did https://zuul.opendev.org/t/openstack/build/be52199567724f90a6017b7a33fd0e3b | 15:00 |
| sean-k-mooney | so ya this looks liek a job config issue not a patch issue | 15:00 |
| bauzas | guys, we'll have a nova meeting | 15:06 |
| bauzas | but I don't want to conflict with the effort here | 15:06 |
| bauzas | dansmith: sean-k-mooney: are you okay if we run the meeting for a short period of time, say 15 mins | 15:07 |
| sean-k-mooney | its fine | 15:07 |
| sean-k-mooney | we likely wont need to cordiate much | 15:07 |
| sean-k-mooney | so run it as normal | 15:07 |
| bauzas | okay, anyway I'll try to make it quick | 15:07 |
| bauzas | reminder : nova meeting in ~ 1 hour here | 15:08 |
| opendevreview | sean mooney proposed openstack/nova master: update nova-live-migration to use corrrect disk-formats https://review.opendev.org/c/openstack/nova/+/924746 | 15:14 |
| opendevreview | Merged openstack/nova master: conf: Add '[api] response_validation' option https://review.opendev.org/c/openstack/nova/+/915740 | 15:16 |
| *** bauzas_ is now known as bauzas | 15:41 | |
| sbauza | sorry folks, intermittent problems with my internet | 15:46 |
| *** sbauza is now known as bauzas__ | 15:46 | |
| bauzas__ | for the moment, I'll stick with bauzas__ | 15:47 |
| bauzas__ | can someone ping me please in order to doublecheck whether you can hear me ? | 15:47 |
| sean-k-mooney | bauzas__: nope sorry cant do that :P | 15:47 |
| bauzas__ | okay cool | 15:47 |
| bauzas__ | fucking French local fiber operator | 15:48 |
| * bauzas__ raises fist with rage to XPFibre | 15:48 | |
| gibi | sean-k-mooney: re live-migration post failure, I see that the the live-migraton job passed on 2023.2 and 2023.1 on the backports | 15:48 |
| sean-k-mooney | gibi: ya i think https://review.opendev.org/c/openstack/nova/+/924746 will fix it on master | 15:48 |
| sean-k-mooney | it also passed in the hybrid plug job | 15:48 |
| sean-k-mooney | which uses the same storage but qcow images | 15:49 |
| gibi | fingres crossed | 15:49 |
| sean-k-mooney | that just completed devstack so we will know one way or another soon | 15:49 |
| sean-k-mooney | if it fails ill update it with the other possibel option i need to set before the jobs completes | 15:49 |
| gibi | OK | 15:50 |
| sean-k-mooney | https://zuul.openstack.org/stream/9fb4f36837c644ae8ff100d21f8a0666?logfile=console.log so far so good | 15:50 |
| *** bauzas_ is now known as bauzas | 15:51 | |
| bauzas__ | don't trust this guy, that's my znc bouncer that continues to fail ^ | 15:51 |
| bauzas__ | :) | 15:51 |
| bauzas__ | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Jul 23 16:00:05 2024 UTC and is due to finish in 60 minutes. The chair is bauzas__. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'nova' | 16:00 |
| bauzas__ | hey folks | 16:00 |
| tkajinam | o/ | 16:00 |
| bauzas__ | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:00 |
| * gibi lurks a bit | 16:00 | |
| bauzas__ | sorry, have unstable internet connection at home | 16:00 |
| fwiesel | o/ | 16:00 |
| bauzas__ | let's try to make this meeting quick for a reason we'll discuss in a sec | 16:01 |
| sean-k-mooney | o/ | 16:01 |
| bauzas__ | #topic Bugs (stuck/critical) | 16:01 |
| bauzas__ | #info One Critical bug | 16:01 |
| bauzas__ | #link https://bugs.launchpad.net/nova/+bug/2071734 | 16:01 |
| bauzas__ | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:02 |
| bauzas__ | so, voila we have another CVE fix | 16:02 |
| bauzas__ | do people want to discuss about it ? | 16:02 |
| bauzas__ | if not, I have a question | 16:02 |
| bauzas__ | given the problem with the image module, should we working on it this cycle ? | 16:04 |
| bauzas__ | if so, it would be a priority | 16:04 |
| sean-k-mooney | there may be more work we do to harden it but likely we will need to take a step back and do tha tnext cycle | 16:04 |
| sean-k-mooney | this cycle we may do so prep work or adtopation with the moving of the code to oslo | 16:05 |
| sean-k-mooney | i know dan cant attend today and he has some tought on this topic | 16:05 |
| sean-k-mooney | so we can bring it up again next week | 16:05 |
| bauzas__ | then at least we should maybe stop to merge new changes for that module | 16:05 |
| bauzas__ | unless it's a priority | 16:06 |
| bauzas__ | the more we accept, the more it could be difficult for the oslo usage | 16:06 |
| bauzas__ | is dansmith around ? | 16:06 |
| tkajinam | are you talking about the image module in oslo or one in nova ? | 16:08 |
| sean-k-mooney | bauzas__: no they are away for the next while | 16:08 |
| sean-k-mooney | tkajinam: the one in nova | 16:08 |
| bauzas__ | tkajinam: we want to move the module to oslo | 16:09 |
| sean-k-mooney | yep so until that happens we likely shoudl stablise it | 16:09 |
| sean-k-mooney | we dont wnat to have them diverge more while we are porting | 16:09 |
| bauzas__ | if we don't have quorum for now, then let's disucss this next week | 16:09 |
| sean-k-mooney | +1 | 16:10 |
| bauzas__ | moving on then | 16:10 |
| tkajinam | ok then I'll try to follow the discussion before I do +2+A actual implementations | 16:10 |
| bauzas__ | #topic Gate status | 16:10 |
| bauzas__ | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:11 |
| bauzas__ | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:11 |
| bauzas__ | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:11 |
| bauzas__ | #link https://zuul.openstack.org/build/132eaea6e65c41a1a508163b9c1f82b2 tempest-integrated-placement FAILURE on unmaintained/zed branch | 16:11 |
| bauzas__ | I just looked at it ^ | 16:11 |
| bauzas__ | this is just a problem when creating dbcounter | 16:12 |
| bauzas__ | hopefully the next run won't have problems | 16:12 |
| bauzas__ | so unless someone finds something for this job run, let's continue | 16:12 |
| bauzas__ | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:13 |
| bauzas__ | #info Please try to provide meaningful comment when you recheck | 16:14 |
| bauzas__ | anything else about the gate ? | 16:14 |
| frickler | ah, is that the setuptools issue? I think nova was having something similar | 16:14 |
| bauzas__ | frickler: yeah looks to me | 16:14 |
| frickler | fix should be https://review.opendev.org/c/openstack/devstack/+/924740 then, but maybe also dansmith will have some other idea | 16:15 |
| bauzas__ | ack but this is for zed | 16:16 |
| bauzas__ | as said, this branch is "unmaintained" | 16:16 |
| bauzas__ | anyway, moving on | 16:16 |
| bauzas__ | #topic Release Planning | 16:17 |
| bauzas__ | #link https://releases.openstack.org/dalmatian/schedule.html | 16:17 |
| bauzas__ | #info Dalmatian-3 in 5 weeks | 16:17 |
| bauzas__ | #info Spec approval freeze happened | 16:17 |
| bauzas__ | #link https://blueprints.launchpad.net/nova/2024.2 16 approved blueprints | 16:17 |
| bauzas__ | you can see all of them here ^ | 16:18 |
| bauzas__ | anything else to discuss ? | 16:18 |
| bauzas__ | #topic Review priorities | 16:18 |
| bauzas__ | #link https://etherpad.opendev.org/p/nova-dalmatian-status | 16:18 |
| bauzas__ | all of the approved blueprint series are there ^ | 16:18 |
| bauzas__ | #topic Stable Branches | 16:19 |
| bauzas__ | elodilles is on PTO this week | 16:19 |
| bauzas__ | so let's discuss that topic next week | 16:19 |
| bauzas__ | #topic vmwareapi 3rd-party CI efforts Highlights | 16:19 |
| fwiesel | #info No updates. | 16:20 |
| bauzas__ | fwiesel: anything to report ? | 16:20 |
| bauzas__ | cool | 16:20 |
| bauzas__ | moving on then | 16:20 |
| bauzas__ | #topic Open discussion | 16:20 |
| bauzas__ | I have one item | 16:20 |
| bauzas__ | (bauzas) Aug 6, 13 and 20 nova meetings | 16:20 |
| bauzas__ | I'll be on PTO those weeks | 16:20 |
| bauzas__ | if someone wants to run them, cool | 16:20 |
| bauzas__ | if not, I gonna skip those | 16:20 |
| sean-k-mooney | we proably shoudl keep them but we dont need to decied that now | 16:22 |
| bauzas__ | okay, I'll tell about that again next week | 16:22 |
| bauzas__ | that's it for me | 16:22 |
| weanew | Hi! I've submitted a patch https://review.opendev.org/c/openstack/nova/+/923395 Could someone take a look at it please? | 16:22 |
| bauzas__ | anything else people want to discuss ? | 16:22 |
| bauzas__ | weanew: we try to avoid review requests during nova meetings | 16:22 |
| sean-k-mooney | that is a pretty old bug | 16:22 |
| sean-k-mooney | and im not sure its one we really want to fix | 16:23 |
| sean-k-mooney | i dont have time today to review but it will depend on the perfomace impact of the fix | 16:24 |
| sean-k-mooney | we proably dot want to downcall to all cells for example | 16:24 |
| sean-k-mooney | with that said ill try and review it properly in the next few days | 16:25 |
| sean-k-mooney | if there is nothign else i think we can close the meeting | 16:26 |
| weanew | Thank you, I don't have quations | 16:26 |
| bauzas__ | cool | 16:26 |
| bauzas__ | then we can close this meeting | 16:27 |
| bauzas__ | thanks all | 16:27 |
| bauzas__ | #endmeeting | 16:27 |
| opendevmeet | Meeting ended Tue Jul 23 16:27:09 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-23-16.00.html | 16:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-23-16.00.txt | 16:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-23-16.00.log.html | 16:27 |
| fwiesel | thanks a lot | 16:27 |
| opendevreview | sean mooney proposed openstack/nova master: [DNM] testing nova-live-migation fix https://review.opendev.org/c/openstack/nova/+/924762 | 16:32 |
| opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | 16:34 |
| tkajinam | bauzas: Could you check the follow up of SEV-ES spec when you have time ? https://review.opendev.org/c/openstack/nova-specs/+/924563 | 16:38 |
| tkajinam | I left the comment in it as well as https://review.opendev.org/c/openstack/nova-specs/+/907702 (original one) | 16:39 |
| clarkb | frickler: I think the correct fix is to bump https://opendev.org/openstack/requirements/src/branch/stable/2023.1/upper-constraints.txt#L339 to package 22.0 or newer | 16:39 |
| clarkb | and note that I expect 2023.1 is broken and not just unmaintained | 16:40 |
| clarkb | due to upper contraints specifying 21.3 there | 16:40 |
| sean-k-mooney | fun are there fixes in flight for that | 16:41 |
| clarkb | not that I am aware of. The only thing I've seen is the patch taht disable the dbcounter which isn't really a fix | 16:42 |
| clarkb | https://github.com/pypa/setuptools/issues/4483 is the upstream issue fwiw | 16:42 |
| sean-k-mooney | so we could fix this by pinning setuptools on stable until they move to unmaintainted | 16:45 |
| sean-k-mooney | welll or after i guess | 16:45 |
| sean-k-mooney | specificlay 2023.1 and older | 16:46 |
| clarkb | no you cannot pin setuptools | 16:46 |
| sean-k-mooney | no ? | 16:46 |
| sean-k-mooney | i know we cant do it in requrieemnts | 16:46 |
| clarkb | setuptools must be installed before your package installations occur. It is a chicken and egg problem unless you use pyproject.toml universally (which we do not) | 16:46 |
| sean-k-mooney | but cant we do it explcitly in the jobs to <70.0.0 | 16:46 |
| clarkb | no | 16:47 |
| sean-k-mooney | ok cause im pretty sure we pinned in in the past | 16:47 |
| clarkb | I mean maybe you could make the ci jobs do that but it is a bad approach because you can't do that in the outside world | 16:47 |
| sean-k-mooney | im aware of the chicken and egg situration | 16:47 |
| clarkb | its the wrong approach until you switch to pyproject.toml | 16:47 |
| clarkb | once you do that you can specify what packages you need to build and install your package | 16:47 |
| sean-k-mooney | well the other option is to raise the min of packaging on stable | 16:47 |
| clarkb | until then you need to assume someone is pulling latest setuptools and it will explode for them | 16:48 |
| sean-k-mooney | which we coudl do but its agaisnt stable policy to raise miniums | 16:48 |
| clarkb | yes this is a perfect example of where that policy should probably be bent | 16:48 |
| clarkb | the software is broken as is. The fix is to change the constraints. Policy shouldn't stand in the way of that | 16:48 |
| sean-k-mooney | is there a patch to do that? it would be good to get ci results and see what the fallout is | 16:48 |
| clarkb | I am not aware of any such patch | 16:49 |
| *** bauzas_ is now known as bauzas | 16:56 | |
| clarkb | I ended up pushing https://review.opendev.org/c/openstack/requirements/+/924764 | 17:07 |
| clarkb | We can see if testing works and take it from there | 17:08 |
| *** bauzas_ is now known as bauzas | 17:26 | |
| sean-k-mooney | clarkb: sorry was on an internal call but yes that is what i was thinking of doing too | 17:40 |
| opendevreview | sean mooney proposed openstack/nova master: update nova-live-migration to use corrrect disk-formats https://review.opendev.org/c/openstack/nova/+/924746 | 17:43 |
| opendevreview | sean mooney proposed openstack/nova master: [DNM] testing nova-live-migation fix https://review.opendev.org/c/openstack/nova/+/924762 | 17:43 |
| *** bauzas_ is now known as bauzas | 17:48 | |
| opendevreview | Merged openstack/os-vif master: Remove old excludes https://review.opendev.org/c/openstack/os-vif/+/917611 | 18:05 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Test additional AMI exclusion https://review.opendev.org/c/openstack/nova/+/924775 | 18:45 |
| dansmith | sean-k-mooney: we can squash this ^ if it works | 18:45 |
| dansmith | might as well start the check timer on it while we look further | 18:46 |
| sean-k-mooney | ack | 18:57 |
| *** bauzas_ is now known as bauzas | 19:09 | |
| _colby | Hey All. We have been runing vGPU on openstack Yoga on centos 8 just fine. We recently added a new hypervisor and went with centos9 since 8 has been moved to EOL. The node with CentOS9 seems to be acting very differently. Its creating all the resource providers (even though we specify the addresses to use in nove). Are there issues with the software versionf of libvirt in centos 9 that are causing issues? | 19:19 |
| *** bauzas_ is now known as bauzas | 19:34 | |
| opendevreview | sean mooney proposed openstack/nova master: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924731 | 19:38 |
| _colby | also when we attempt to spin up a vgpu instance it does not seem to be creating any of the mediated devices. Is there a way to get this working on centos9? We plan to upgrade openstack versions soo but would like to get this hypervisor running | 19:57 |
| opendevreview | sean mooney proposed openstack/nova stable/2024.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924732 | 20:00 |
| opendevreview | sean mooney proposed openstack/nova stable/2024.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924732 | 20:03 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.2: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924733 | 20:03 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924734 | 20:05 |
| sean-k-mooney | _colby: it is possibel to use vgpus on centos 9 however nvidia chagned the feature set for some of there existign gpus in later driver version and remvoed vgpu support for some older cards | 20:07 |
| sean-k-mooney | _colby: that may be whaat you are encoutering but im not sure | 20:07 |
| _colby | we are using A40 cardsd | 20:08 |
| _colby | I did upgrade the nvidia drivers from the other hypervisors | 20:09 |
| _colby | Ill try reverting nvidia software version to match other hypervisors and see what happens | 20:17 |
| sean-k-mooney | _colby: bauzas know more about this then i do and might be able to tell you what version things changed in | 20:22 |
| sean-k-mooney | _colby: but they are based in france and done for today so they may be able to respond in the morning | 20:22 |
| _colby | ok thanks @sean-k-mooney | 20:28 |
| *** bauzas_ is now known as bauzas | 21:20 | |
| opendevreview | Merged openstack/nova master: conf: Clarify '[api] response_validation help' text https://review.opendev.org/c/openstack/nova/+/924501 | 21:23 |
| *** bauzas_ is now known as bauzas | 23:01 | |
| opendevreview | Merged openstack/nova master: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924731 | 23:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!