opendevreview | Amit Uniyal proposed openstack/nova master: DNM: test evacuation https://review.opendev.org/c/openstack/nova/+/915923 | 05:49 |
---|---|---|
*** elodilles_pto is now known as elodilles | 07:17 | |
opendevreview | Kien Nguyen proposed openstack/python-novaclient master: support get and list server's metadata https://review.opendev.org/c/openstack/python-novaclient/+/915977 | 09:23 |
opendevreview | Amit Uniyal proposed openstack/nova master: DNM: test evacuation https://review.opendev.org/c/openstack/nova/+/915923 | 11:27 |
sean-k-mooney | auniyal: ^ fyi when you create DNM patches like that its better to disable the jobs that you dont need to save ci resouces | 11:33 |
dansmith | gibi: I cc'd you but just in case it's under a mountain of other notification emails, I'm looking for pre-approval on this method of determining the trailing threshold: https://review.opendev.org/c/openstack/nova/+/915911/1 | 13:50 |
dansmith | from our version support discussion | 13:50 |
dansmith | just the algorithm for picking the version, and then I'll flesh out everything else | 13:51 |
auniyal | sean-k-mooney ack, but I am not sure how to disable jobs, | 14:01 |
auniyal | by doing -1 to workflow ? | 14:02 |
auniyal | its a depends-on patch, so I do not really need to run any job in nova-ci for this | 14:03 |
dansmith | auniyal: by commenting out or removing jobs from the .zuul | 14:05 |
auniyal | dansmith, ack, thanks | 14:06 |
auniyal | yeah, I knew workflow is stop stop merging, but thought may it also work for jobs | 14:06 |
samcat116 | Hi all, I seem to be hitting the following issue pretty consistently on a 2023.2 deployment: https://bugs.launchpad.net/nova/+bug/1931707. Specifically when trying to delete a large number of instances with many ports attached to each at once. For instance on the last run we had computes with over 100 vms on a given compute node with 10 ports on each vm. Out of our deletion of about 100 vms, over 150 went into error due to this | 14:06 |
samcat116 | reason. I've also seen the error on vm creation, but much less common. Not sure if there's anything I can do to debug further. | 14:06 |
gibi | dansmith: thanks for the ping, otherwise I would have missed the patch | 14:06 |
dansmith | gibi: no problem and no big rush, just wanted to at least get some agreement while things are still fresh | 14:08 |
auniyal | dansmith, its okay to comment whole file right ? | 14:09 |
dansmith | auniyal: probably not, and obviously not if you want to run *any* tests | 14:14 |
dansmith | auniyal: sean-k-mooney was likely suggesting you comment out all but the tests you want to run on your WIP | 14:14 |
auniyal | ack | 14:20 |
sean-k-mooney | dansmith: auniyal yes | 14:20 |
erlon | sean-k-mooney: hey, can you give a look on the RBAC change (https://review.opendev.org/c/openstack/nova/+/811521) again when you have a chance? | 14:21 |
erlon | I was trying to add some core reviewers to the list as well, but I don't know how to find them anymore. It use to be in the project groups, core-reviewers, but now there's a lot of groups. | 14:23 |
sean-k-mooney | sure i see you created a blue print for it | 14:28 |
sean-k-mooney | so we still need to approve that in the nova team meeting | 14:28 |
sean-k-mooney | i dont really have any objection to enabing this as a specless blueprint | 14:28 |
erlon | The other thing I was wondering, is the likelihood of this being backported. The fact that it has a blueprint, make it sounds like there's a feature which this is not, but might raise some eyebrows | 14:33 |
erlon | The next meeting will be only next week right? Do I need to add the top in the agenda? Or there's already a slot for review requests | 14:42 |
erlon | sean-k-mooney: ^ | 14:42 |
dansmith | erlon: sounds like a feature to me, FWIW | 14:46 |
kashyap | gibi: When you get a min, can you look at this (needs to reload some context, though): https://review.opendev.org/c/openstack/nova/+/899185 | 14:50 |
sean-k-mooney | dansmith: erlon yes this is a feature and not something that is upstrema backporable | 14:51 |
*** efried1 is now known as efried | 15:19 | |
*** haleyb_ is now known as haleyb | 15:27 | |
gibi | kashyap: I remember we discussed doing that change in the past but I don't know where it is stopped, what is the next step there?j | 15:31 |
kashyap | gibi: The next step is discussed in the bug you filed: https://bugs.launchpad.net/nova/+bug/2039803 | 15:41 |
bauzas | reminder : nova meeting here in ~20 mins | 15:41 |
kashyap | gibi: Our aim is ge this fixed with as less churn as possible | 15:42 |
gibi | kashyap: OK your comment https://bugs.launchpad.net/nova/+bug/2039803/comments/2 make sense | 15:49 |
kashyap | s/ge/to get/ | 15:50 |
kashyap | gibi: Okay, please also give a quick look at the draft patch. Tomm; not tonight, it requires to load up on the context | 15:51 |
sean-k-mooney | so why is this listed as a bug | 15:52 |
sean-k-mooney | to me this is a feature | 15:52 |
sean-k-mooney | its technial debth | 15:52 |
sean-k-mooney | but i dotn thinkw e woudl backport this right | 15:52 |
sean-k-mooney | i mean we shoudl not eb breakign live migration by calling the worng api | 15:53 |
sean-k-mooney | but im nnot sure how i feel about changign all those libvirt api methods in somethign we would backport | 15:54 |
sean-k-mooney | i would epect the test fallout to be non trivial | 15:54 |
sean-k-mooney | although i guess we coudl see what it looks like and make the deicsion about if this could be backported later | 15:54 |
kashyap | Yeah, we don't want to update all those methods. If you see the WIP patch, it's targetted as a bug fix | 15:55 |
sean-k-mooney | i saw all the red ci jobs | 15:56 |
kashyap | That comment is just written to show what we've already considered and eliminated. | 15:56 |
sean-k-mooney | i briefly skimmed it | 15:56 |
sean-k-mooney | the ci jobs logs all rotated so this need a rebase at a minium | 15:56 |
kashyap | (Just on the bug comment: we shouldn't update all those methods. And not all those methods listed there make sense in this context.) | 15:57 |
kashyap | Yeah, needs a rebase for sure. | 15:57 |
kashyap | Thank you for looking (even if it's skimming) | 15:58 |
sean-k-mooney | the change is basically s/host_cpu = self._host.get_capabilities().host.cpu/host_cpu = self._host.get_domain_capabilities().host.cpu/ | 15:59 |
sean-k-mooney | the side effect being that domain caps the new api also has awareness of what the qemu instance on the host can emulate ectra | 16:00 |
sean-k-mooney | so we will be useing the capablity of the cpu that woudl be presented to the guest | 16:00 |
sean-k-mooney | rather the hte host caps | 16:00 |
bauzas | (meeting now)à | 16:00 |
bauzas | #startmeeting nova | 16:00 |
opendevmeet | Meeting started Tue Apr 16 16:00:53 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 | howdy folks | 16:01 |
dansmith | o/ | 16:01 |
bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
elodilles | o/ | 16:01 |
auniyal | o/ | 16:01 |
sean-k-mooney | o/ | 16:01 |
bauzas | let's try to have a short meeting, I could have someone going to my home around 4.30pm | 16:02 |
bauzas | #topic Bugs (stuck/critical) | 16:02 |
bauzas | #info No Critical bug | 16:02 |
bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 74 new untriaged bugs (+7 since the last meeting) | 16:02 |
bauzas | I definitely need to do what we agreed on the PTG :) | 16:02 |
bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:03 |
bauzas | so, | 16:03 |
bauzas | #info bug baton is ? | 16:03 |
elodilles | o/ | 16:03 |
bauzas | anyone wanting to take the baton ? | 16:03 |
elodilles | i can volunteer | 16:03 |
bauzas | cool thanks | 16:03 |
bauzas | #undo | 16:03 |
opendevmeet | Removing item from minutes: #info bug baton is ? | 16:03 |
bauzas | #info bug baton is for elodilles | 16:03 |
bauzas | elodilles: <3 | 16:03 |
bauzas | moving on | 16:04 |
bauzas | #topic Gate status | 16:04 |
bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:04 |
bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:04 |
bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:04 |
bauzas | all greens | 16:04 |
bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:04 |
bauzas | #info Please try to provide meaningful comment when you recheck | 16:04 |
bauzas | voila | 16:04 |
bauzas | any gate failure you had found ? | 16:04 |
bauzas | okayokay | 16:05 |
bauzas | #topic Release Planning | 16:05 |
bauzas | #link https://releases.openstack.org/dalmatian/schedule.html | 16:05 |
bauzas | #info Dalmatian-1 in 4 weeks | 16:05 |
bauzas | #action bauzas to add the nova deadlines in the Dalmatian schedule page | 16:05 |
bauzas | #topic vPTG Summary | 16:06 |
bauzas | #info The vPTG was held on Apr 8-12 | 16:06 |
bauzas | #link https://etherpad.opendev.org/p/nova-dalmatian-ptg Topics that were discussed | 16:06 |
bauzas | #action bauzas to provide a summary this week | 16:06 |
bauzas | anything people want to discuss about the vPTG ? | 16:07 |
bauzas | looks not | 16:07 |
bauzas | #topic Stable Branches | 16:07 |
bauzas | elodilles: heya | 16:07 |
elodilles | nothing news actually | 16:08 |
elodilles | zed branch is still broken due to nova-ceph-multistore job :/ | 16:08 |
elodilles | other maintained branches' gates are in OK state | 16:09 |
bauzas | kk | 16:09 |
bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | 16:09 |
bauzas | fwiesel: are you here? | 16:09 |
fwiesel | Yes, sorry for being late. | 16:10 |
fwiesel | #info No updates | 16:10 |
bauzas | cool no worries | 16:10 |
bauzas | #topic Open discussion | 16:10 |
bauzas | anyone ? | 16:10 |
bauzas | I don't see any item in our page | 16:10 |
sean-k-mooney | i had one thing | 16:11 |
bauzas | shoot | 16:11 |
sean-k-mooney | fwiesel: you have and had some patches up for vmware for a while | 16:11 |
sean-k-mooney | we might want to try an cordiante soem review of those | 16:11 |
sean-k-mooney | as otherwise i feel like they are just going to bitrot | 16:12 |
fwiesel | sean-k-mooney: That would be great. | 16:12 |
sean-k-mooney | im not really sure how to go about that othe rthen perhapsp tryign to use the review etherpad but im not sure if we have a new one for 2024.2 | 16:12 |
sean-k-mooney | bauzas: have you created one? | 16:12 |
bauzas | sean-k-mooney: yes, see above ^ | 16:13 |
bauzas | oh f... I forgot a topic :) | 16:13 |
bauzas | #link https://etherpad.opendev.org/p/nova-dalmatian-status | 16:13 |
sean-k-mooney | do we want to say chosse one to review before week or perhaps have a prioritesed list | 16:13 |
opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Avoid getCapabilities() to calculate host CPU definition https://review.opendev.org/c/openstack/nova/+/899185 | 16:13 |
Uggla_ | o/ | 16:14 |
fwiesel | The most important one would be the image import... as otherwise you cannot boot an instance :) | 16:14 |
fwiesel | Shall I add it then to the etherpad? | 16:14 |
sean-k-mooney | ack can you add that on eto the ether pad | 16:14 |
fwiesel | Okay, I will. | 16:14 |
bauzas | sean-k-mooney: we said in the PTG that we'll ask folks to ask to have feature liaisons | 16:14 |
sean-k-mooney | please do and perhaps we should add a vmware dirver section to list the in order of priority | 16:14 |
bauzas | sean-k-mooney: sure, we could organize a specific section | 16:15 |
bauzas | are we done with that ? | 16:15 |
sean-k-mooney | so the thing is currently non of the core team really know how that driver works in detail an di feel like if fwiesel does not remind us often that they are trying ot fix it we wont be encliend ot go looking to review vmware code | 16:16 |
sean-k-mooney | ya that all i wanted to bring ups | 16:16 |
bauzas | well, we could create a separate etherpad for that if you want | 16:16 |
sean-k-mooney | im fine with having it in the main one | 16:16 |
bauzas | or we could have some gmeet | 16:16 |
kashyap | gibi: Thanks for the rebase :) | 16:16 |
bauzas | anyway, it's related to what we discussed on the PTG | 16:17 |
sean-k-mooney | i just want to make sure if i look at some of theose patches someone else will | 16:17 |
sean-k-mooney | bauzas: ya i was not in those ptg dicussions | 16:17 |
sean-k-mooney | so im not aware of that discussion or if i agree with it | 16:17 |
bauzas | this cycle, we'll try to have some subteams for each of the blueprints | 16:17 |
sean-k-mooney | but i guess ill read back the notes | 16:17 |
bauzas | so each of the subteam people could organiaz some gmeets if they want | 16:18 |
sean-k-mooney | ok... im not sure how that is goign to work but maybe it could | 16:18 |
bauzas | here, we could have some kind of '"vmware subteam" | 16:18 |
bauzas | I mean, people reviewing vmware driver changes | 16:19 |
sean-k-mooney | well i would either like to fix the dirver or delete it. both work for me but im sure fwiesel has a prefernce given the work they have done | 16:19 |
fwiesel | Shall I get more of our people involved? | 16:19 |
bauzas | fwiesel: nope, we'll softly start | 16:19 |
sean-k-mooney | fwiesel: i dont think thats need | 16:19 |
bauzas | and see how we organize ourselves | 16:19 |
sean-k-mooney | we just need some cores to say yes ill review your patchs | 16:19 |
bauzas | zactly | 16:19 |
bauzas | anyway I think we're done here | 16:20 |
bauzas | thanks all | 16:20 |
sean-k-mooney | sure let end the meetign and we can chat after | 16:20 |
fwiesel | Thanks, have a good one. | 16:20 |
bauzas | #endmeeting | 16:20 |
opendevmeet | Meeting ended Tue Apr 16 16:20:38 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:20 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-04-16-16.00.html | 16:20 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-04-16-16.00.txt | 16:20 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-04-16-16.00.log.html | 16:20 |
sean-k-mooney | so i was previously, tentiitively +2 on https://review.opendev.org/c/openstack/nova/+/909474 | 16:20 |
fwiesel | So, how does the subteams work? | 16:20 |
sean-k-mooney | well normally the subteam has a meeting and then updtate the broader team like your already doing | 16:21 |
sean-k-mooney | we have not used subtemas out side of the stable team in a long time | 16:21 |
sean-k-mooney | it could somethign as simple as choose 15-30mins during the week to meet and disucss a given patch | 16:22 |
gibi | kashyap: sean-k-mooney: it is bug to me as it breaks live migration. If we decide not to backport the fix that is fine by mej | 16:24 |
gibi | I will try to check the CI results on the rebase tomorrow | 16:25 |
fwiesel | sean-k-mooney: How does it get organised? Next Nova meeting? | 16:28 |
sean-k-mooney | that really up to the memeber of the subteam i guess. i was not part of the ptg disucssion so im not sure what bauzas had in mind | 16:30 |
sean-k-mooney | bauzas: do you intend to review https://review.opendev.org/c/openstack/nova/+/909474 | 16:30 |
bauzas | sean-k-mooney: I'm not a super expert on the glance side, but I can try to help | 16:32 |
bauzas | that said, dansmith could also have some opinions | 16:33 |
dansmith | I might, but tied up right now | 16:34 |
kashyap | gibi: I agree it's a bug. I need to head out now; we can talk more tomm. | 16:37 |
fwiesel | I've to drop of as well. Have a good one. | 16:38 |
*** dmitriis is now known as Guest1333 | 23:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!