| gibi | sean-k-mooney: thanks for the reviews on https://review.opendev.org/q/topic:%22bp/igb-vif-model%22 | 07:50 |
|---|---|---|
| gibi | bauzas, melwitt, dansmith : can some of you be the second core on ^^ I promise it is simple :) | 07:51 |
| bauzas | shit, "Your message to openstack-discuss@lists.openstack.org awaits moderator approval" | 08:25 |
| bauzas | because "The message is larger than the 40 KB maximum size" | 08:25 |
| * bauzas facepalms | 08:25 | |
| bauzas | gibi: sure, looking | 08:26 |
| bauzas | okay, I'll prioritize your series review this morning | 08:26 |
| frickler | bauzas: is this intentionally large message or html by accident. I can look into approving it if needed unless you want to wait for fungi to be back | 08:54 |
| bauzas | frickler: I attached a txt file, hence the message size | 08:55 |
| bauzas | do you want me to send again the message without the etherpas backup file ? | 08:55 |
| bauzas | etherpad* | 08:56 |
| frickler | bauzas: linking to the etherpad should be enough I guess? | 08:59 |
| bauzas | frickler: I always do that by linking to an etherpad readonly copy, but this time I also attached the file backup | 09:06 |
| frickler | bauzas: ok, so let me just try to approve your message, then, so you don't need to resend | 09:15 |
| frickler | oh, wait, you just did | 09:15 |
| bauzas | frickler: wait | 09:15 |
| bauzas | yeah I just sent it again without the attached file | 09:15 |
| frickler | ok, then I'll try to drop the other mail instead to avoid ending up on the list as duplicate | 09:16 |
| frickler | hmm, I didn't manage to do that, will need some advice from fungi who'll hopefully read this before picking up moderation chores | 09:27 |
| bauzas | ++ | 09:27 |
| *** ykarel_ is now known as ykarel | 09:49 | |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/933636 | 10:06 |
| gibi | sean-k-mooney, bauzas: a small doc bug fix ^^ | 10:06 |
| fungi | frickler: bauzas: sure, i can discard the larger duplicate, but the moderation hold notice you received should have also included instructions on how to cancel your own post instead | 10:47 |
| fungi | (if you saved it) | 10:48 |
| semantic | Hello, guys! Could someone with +2 power please look into https://review.opendev.org/c/openstack/nova/+/931737 when it's possible? | 10:53 |
| sean-k-mooney | gibi: i breifly looked at my email last night but said i would respond in the mornining | 11:06 |
| sean-k-mooney | gibi: so ya the socket policy should work via the alias and flaovr, it does not work via the neutron port because we never got around to adding it to the enum on the neutron side | 11:07 |
| gibi | sean-k-mooney: cool, thanks for confirming | 11:41 |
| gibi | bauzas: thanks for the review on igb. I will respin the doc patch at the top of the series in a minute. I see you did not added +W to the rest of the series either. Is it intentional? | 11:50 |
| sean-k-mooney | gibi: fixing the fact that you cant do the socket polocy vai neutron is a nice lowish hanging frut but customers have not asked for it recently so it just never was a priority | 11:51 |
| sean-k-mooney | im pretty happy with the igb state so if you need me to rereview ping me | 11:52 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | 11:52 |
| gibi | sean-k-mooney: just respined the doc patch to fix the q35 nit in it ^^ | 11:54 |
| sean-k-mooney | cool im +2 on the full stack so ill leave +w to bauzas when they have tiem to re review | 11:56 |
| gibi | OK cool | 11:56 |
| bauzas | sean-k-mooney: gibi: ack, will click on the submit magical button | 12:55 |
| bauzas | gibi: sorry was outside for my usual 'gym' | 12:55 |
| *** dc is now known as Guest7866 | 13:27 | |
| gibi | bauzas: ack, thanks | 13:29 |
| *** ykarel_ is now known as ykarel | 13:59 | |
| bauzas | reminder for EU people : the nova meeting will happen in 49 mins given the DST change for us | 15:11 |
| fungi | bauzas: i went ahead and discarded your larger ptg summary that was held due to the text attachment, just remember that you can cancel your own held posts too | 15:35 |
| bauzas | fungi: sorry, I got no way for knowing how to delete the post by the back email | 15:36 |
| fungi | ah, i think it was a link in the bottom of the message in the past, but maybe mailman v3 changed that. i'll test it myself | 15:37 |
| frickler | just guessing it might have been move into a header similar to the Archived-At? | 15:46 |
| zigo | bauzas: As I told you I was on holidays during the PTG. I didn't even open my laptop even once ! :) | 15:53 |
| frickler | that sounds like a true holiday :) | 15:54 |
| zigo | Yeah. | 15:56 |
| zigo | Went to Malaga (next to it...). | 15:56 |
| bauzas | zigo: haha cool | 16:00 |
| bauzas | anyway, let me start the meeting | 16:00 |
| bauzas | fungi: sorry I was on another meeting, I can provide you the exact email I got | 16:00 |
| bauzas | (after the meeting, of course) | 16:00 |
| bauzas | anyway, starting now | 16:00 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Oct 29 16:00:57 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 | what's up folks ! | 16:00 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
| bauzas | who's around ? | 16:01 |
| tkajinam | o/ | 16:01 |
| dansmith | o/ | 16:01 |
| bauzas | let's try to have a very short but productive meeting, since we had the PTG last week | 16:01 |
| bauzas | #topic Bugs (stuck/critical) | 16:02 |
| bauzas | #info No Critical bug | 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 | any important bug you'd like to raise ? | 16:03 |
| * gibi finishing a meeting | 16:03 | |
| bauzas | okay moving on so | 16:03 |
| 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 | #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 | I looked at some changes this week, but I haven't seen any problem so far | 16:04 |
| bauzas | anything people saw on our gate ? | 16:05 |
| Uggla | o/ | 16:05 |
| sean-k-mooney | o/ | 16:06 |
| tkajinam | nothing I'm aware of | 16:06 |
| fwiesel | o/ | 16:06 |
| bauzas | before moving on, I'm still awaiting the results of the periodic checks | 16:06 |
| bauzas | gosh, it takes a while | 16:06 |
| tkajinam | py38 was removed from master u-c so now py38 job may be completely broken in master, but we believe we already bumped the min version of all nova deliverables so no job may still run in py38 for master | 16:07 |
| bauzas | ah, got it, all periodics in green except nova-emulation on 2023.2 stable branch | 16:07 |
| tkajinam | https://review.opendev.org/c/openstack/requirements/+/925201 | 16:07 |
| tkajinam | just fyi | 16:07 |
| tkajinam | next big event may be switch to noble, I expect | 16:07 |
| bauzas | tkajinam: thanks for the note, I'll mention something about that in the next topic | 16:07 |
| bauzas | tkajinam: when is planned that big change ? | 16:08 |
| tkajinam | I saw gmann started submitting a few patches for it but I've seen a few projects have problems when run in noble so it may take some time | 16:08 |
| sean-k-mooney | it should happen in the next few weeks | 16:08 |
| sean-k-mooney | so ideally before m1 perhaps m2 at the very latest | 16:09 |
| tkajinam | yeah | 16:09 |
| sean-k-mooney | nova-next has been on it since last cycle | 16:09 |
| tkajinam | https://review.opendev.org/c/openstack/nova/+/932648 | 16:09 |
| sean-k-mooney | i breifly looked at the docs failure | 16:09 |
| sean-k-mooney | nova is in a pretty good state | 16:09 |
| tkajinam | probably https://review.opendev.org/q/topic:%22migrate-to-noble%22 is a better link | 16:10 |
| sean-k-mooney | i think we just need to add the right dep for pcr2 to bindep | 16:10 |
| bauzas | yeah I'm not afraid of the noble change in the gate, but I'd like somehow to get noticed when it happens | 16:10 |
| sean-k-mooney | well its for us to review | 16:10 |
| sean-k-mooney | there are two parts | 16:10 |
| sean-k-mooney | one is devstack will change its default nodeset | 16:10 |
| sean-k-mooney | the other is we need to update our in repo jobs https://review.opendev.org/c/openstack/nova/+/932648 | 16:10 |
| sean-k-mooney | does the latter | 16:11 |
| frickler | there's an ML thread on the noble upgrade https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/JOMDY26TCW7OX3NXRGOYQCIDXNNJ4E25/ | 16:11 |
| bauzas | ok, I'll cc myself on gmann's patch | 16:11 |
| frickler | I assume gmann will update once ready | 16:11 |
| sean-k-mooney | yep so at the momp[ent the only blocker on teh nova side that im aware of is including the correct package for prce.h | 16:12 |
| sean-k-mooney | in the docs jobs (api ref ectra) | 16:12 |
| frickler | ack, lots of other projects look much worse off | 16:12 |
| bauzas | hence the failure | 16:13 |
| sean-k-mooney | yep so in my devstack i have pcre 2 and 3 and maybe 1 installed | 16:13 |
| sean-k-mooney | we just need to see which one actully provides the correct dep | 16:13 |
| bauzas | ack | 16:13 |
| sean-k-mooney | the packaging of that is a bit weired and may have changed | 16:13 |
| bauzas | I guess we're cool then | 16:13 |
| sean-k-mooney | more or less | 16:13 |
| opendevreview | Merged openstack/nova master: Refactor obj_make_compatible to reduce complexity https://review.opendev.org/c/openstack/nova/+/928590 | 16:14 |
| sean-k-mooney | i think we can just review this as normal | 16:14 |
| sean-k-mooney | and check in each week until merged | 16:14 |
| bauzas | ok, moving on then | 16:15 |
| bauzas | #topic Release Planning | 16:15 |
| bauzas | #link https://releases.openstack.org/epoxy/schedule.html | 16:16 |
| bauzas | #action bauzas to add Epoxy nova deadlines in the schedule | 16:16 |
| bauzas | if you were not present at the PTG, you need to know that we'll have a spec soft freeze by mid-Dec | 16:16 |
| bauzas | plan in advance to write your spec accordingly | 16:16 |
| bauzas | the earlier the better as always | 16:17 |
| bauzas | #topic Review priorities | 16:17 |
| bauzas | #link https://etherpad.opendev.org/p/nova-2025.1-status | 16:17 |
| bauzas | I moved some stuff, but I need to do a scan check of all the items in there | 16:17 |
| bauzas | #topic PTG summary | 16:18 |
| bauzas | for those who were not able to attend : | 16:18 |
| bauzas | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/HZZPADKTWNMBGRSOEMSPLN432DXZYYTL/ | 16:18 |
| bauzas | feel free to raise questions or remarks in that thread | 16:18 |
| bauzas | #topic Stable Branches | 16:19 |
| bauzas | hah, elodilles_pto is on PTO as the nick says | 16:19 |
| bauzas | let's move on then to the next topic unless someone wants to raise something about stable branches | 16:19 |
| zigo | FYI (about this link): We'd love to have "vTPM live migration" implemented... :P | 16:19 |
| bauzas | zigo: make a wish | 16:20 |
| sean-k-mooney | many woudl and there is at least some hope that it will be doen in epoxy | 16:20 |
| sean-k-mooney | its non trivial however without wekening security | 16:20 |
| bauzas | there is already some code there so the wish wouldn't be too hard to do | 16:20 |
| sean-k-mooney | yes and no the current code is not accpabel as is as it wont work in the general case without custom policy | 16:21 |
| bauzas | zigo: if you really like the case, please review the spec then once it's uploaded | 16:21 |
| sean-k-mooney | but that is why we are goign to have a spec and try and delvier a better generic solution | 16:22 |
| bauzas | zigo: as sean-k-mooney is explaining, we discussed about user bond of trust with admins | 16:22 |
| bauzas | so I'd like your op voice to be shared on that spec | 16:22 |
| bauzas | anyway, let's not derail the meeting into a technical discussion about a specific feature | 16:23 |
| bauzas | moving on (and I think we can skip the stable branches topic for today) | 16:24 |
| bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | 16:24 |
| bauzas | fwiesel: want to share something ? | 16:24 |
| fwiesel | Yes, sorry. I found a regression in our networking driver and fixed it. But there is another one. | 16:27 |
| fwiesel | Some race-condition in the startup. I am optimistic that I can fix it this week. | 16:28 |
| fwiesel | That's from my side. Back to you bauzas. | 16:28 |
| bauzas | thanks | 16:29 |
| bauzas | and good luck with the investigation | 16:29 |
| bauzas | #topic Open discussion | 16:29 |
| bauzas | just refreshed the page, nothing in there | 16:29 |
| bauzas | so, anything anyone ? | 16:29 |
| tkajinam | I've proposed oslo.utils release so we hopefully get it soon and can use it for some work in nova https://review.opendev.org/c/openstack/releases/+/933627 | 16:30 |
| sean-k-mooney | i tought there was somethign we said to cover in the meeting instead of the ptg | 16:30 |
| sean-k-mooney | but i dont recall | 16:30 |
| tkajinam | dansmith, ^^^ just fyi | 16:30 |
| dansmith | tkajinam: thanks | 16:30 |
| bauzas | tkajinam: thanks | 16:30 |
| tkajinam | I'll work on https://bugs.launchpad.net/nova/+bug/2083518 once the release is created | 16:31 |
| sean-k-mooney | i might be thinkign of https://review.opendev.org/c/openstack/nova-specs/+/932653 but i highlighted that in the cidner cross project | 16:31 |
| tkajinam | sean-k-mooney, ah, yeah | 16:32 |
| bauzas | yeah, we had no topics that were left to be discussed | 16:32 |
| sean-k-mooney | tkajinam: you are porting that to oslo yes | 16:32 |
| sean-k-mooney | and we will just consume the predecate form there once its released | 16:32 |
| tkajinam | sean-k-mooney, yes. you are correct. | 16:32 |
| sean-k-mooney | ya so i dont see anythign contoversion with that so im happy to review that when its ready | 16:33 |
| tkajinam | :-) | 16:33 |
| tkajinam | sean-k-mooney, talking about the spec you raised... I think you spotted a nice point and we need a new mechanism in cinder (qos specs which are not used by cinder but by nova during attachment) | 16:34 |
| tkajinam | I overlooked the fact that the prpoosal is trying to use the existing qos specs but these are cinder internal one and can't be used by nova | 16:34 |
| sean-k-mooney | well frontend qos policies are use by the hypervior (libvirt in this case) | 16:34 |
| sean-k-mooney | to enforce it where as backend qos polcies are done on the stroage backend | 16:35 |
| tkajinam | but that qos is associated with flavor, right ? | 16:35 |
| sean-k-mooney | no | 16:35 |
| sean-k-mooney | its assocated witht eh volume | 16:35 |
| sean-k-mooney | we also have the ablity to do it via the flavor for nova created block devices | 16:35 |
| sean-k-mooney | using the quota:* extra specs | 16:35 |
| sean-k-mooney | i belive the current state is you can configure hard limits today | 16:36 |
| sean-k-mooney | but not burst | 16:36 |
| sean-k-mooney | and the brust limist are what they want to enabel | 16:36 |
| sean-k-mooney | but that was my question to the cinder folk | 16:36 |
| tkajinam | ok | 16:36 |
| sean-k-mooney | to have them confirm what is and is not expected to be requestable vai cinder today | 16:36 |
| tkajinam | I thought the qos specs are not propagated via connection info but it's set in storage backend | 16:36 |
| tkajinam | I'll recheck it and fix my comments in that spec if needed | 16:37 |
| sean-k-mooney | you set them on the voluem type and im not sure how they are propegated to nova | 16:37 |
| sean-k-mooney | so this is where we need input form the cider folk | 16:37 |
| sean-k-mooney | i.e are tehy on teh attachmetn? conection info? do we need to look them up ectra | 16:38 |
| bauzas | can we end the meeting ? | 16:38 |
| sean-k-mooney | yep i think so | 16:38 |
| tkajinam | yeah we can discuss details in the spec | 16:38 |
| sean-k-mooney | this can be an async dicussion | 16:38 |
| bauzas | cool | 16:38 |
| bauzas | then thanks all | 16:38 |
| bauzas | #endmeeting | 16:38 |
| opendevmeet | Meeting ended Tue Oct 29 16:38:49 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-10-29-16.00.html | 16:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-10-29-16.00.txt | 16:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-10-29-16.00.log.html | 16:38 |
| sean-k-mooney | tkajinam: for what its worht i have only ever used cinder qos with ceph using the backend mode so i have nerver personally tried to use the frontend mode | 16:42 |
| sean-k-mooney | i think you can use it with cinder-lvm | 16:42 |
| sean-k-mooney | but im fuzzy on the details | 16:42 |
| sean-k-mooney | thats a long way to say i think this coudl be configured and enabeld in tempest or whitebox tests and validated end to end in 1st party ci. | 16:43 |
| sean-k-mooney | so im not really conserned about the testablity ro the libvirt xml generaion just the api contract between nova/cinder | 16:44 |
| sean-k-mooney | i woudl hope the exisating cidner tempest plugins or core tempest could verify frontend qos and the proposer could jsut extend that (in adddtion to unit/fucntional tests) | 16:46 |
| tkajinam | https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L2698-L2756 | 16:46 |
| tkajinam | ahh, wait | 16:47 |
| tkajinam | https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1831-L1832 | 16:47 |
| tkajinam | ok I understand what you mean | 16:47 |
| sean-k-mooney | right cinder has an enum https://github.com/openstack/cinder/blob/master/cinder/volume/qos_specs.py#L33 for diffent type of qos | 16:48 |
| sean-k-mooney | and as you foudn the generic manager code shoudl include the qos if its front-end or both | 16:48 |
| sean-k-mooney | it looks like its part of the connection info | 16:48 |
| sean-k-mooney | in a optional qos_specs section | 16:49 |
| tkajinam | yeah | 16:49 |
| sean-k-mooney | looking at https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1836-L1839 | 16:49 |
| sean-k-mooney | im not sure they the burst values they are propsoing are supproted by cinder | 16:50 |
| sean-k-mooney | so i think there shoudl be a cinder spec first to extend the set of qos types | 16:50 |
| sean-k-mooney | and then we woudl just enable them | 16:50 |
| sean-k-mooney | the cinder team may be ok just using the nova spec for both but we would need them to say one way or another | 16:50 |
| tkajinam | sean-k-mooney, quickly looking at cinder code it seems cinder has no mechanism to validate keys in qos specs | 16:54 |
| tkajinam | the documentation even describes the "delay" which is not actually used IIUC https://docs.openstack.org/api-ref/block-storage/v3/index.html#id768 | 16:54 |
| sean-k-mooney | it filters them when puting them in the connection info | 16:54 |
| sean-k-mooney | so it may not filter for backend qos polciies btu it does for frontend ones | 16:55 |
| tkajinam | I mean there is not filtering mechanism in api layer, so what we may need to update is the filtering logic in manager.py which you pointed | 16:56 |
| sean-k-mooney | yes which cinder may or may not consider an api change | 16:56 |
| sean-k-mooney | so they may be happy with jsut the nova spec and say a bug on there side or they may ask for a spec on both sides if they consider it an api change | 16:57 |
| sean-k-mooney | i posted this on the spec more or less | 16:57 |
| sean-k-mooney | although not in those exact words | 16:57 |
| tkajinam | yeah I saw these | 16:58 |
| sean-k-mooney | if you think any addtional context woudl help please reply so the author can condier that. are you overall infaovr or agasitn this feature? | 16:59 |
| tkajinam | sean-k-mooney, I now think that's a good idea. My -1 came from the fact that I overlooked that frontend qos handling | 17:05 |
| * tkajinam is trying to put a comment to summarize the discussion | 17:05 | |
| opendevreview | Merged openstack/nova master: [ovo]Add igb value to hw_vif_model image property https://review.opendev.org/c/openstack/nova/+/928456 | 17:05 |
| opendevreview | Merged openstack/nova master: [libvirt]Support hw_vif_model = igb https://review.opendev.org/c/openstack/nova/+/928584 | 17:20 |
| opendevreview | Merged openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | 17:20 |
| sean-k-mooney | gibi: :) ^ | 17:20 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | 17:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | 17:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 17:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 17:23 |
| gibi | sean-k-mooney: \o/ | 17:26 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | 17:27 |
| gibi | a quick win for Epoxy | 17:27 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 17:27 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 17:27 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: pwr mgmt: power down free PCPUS when updating compute node https://review.opendev.org/c/openstack/nova/+/932926 | 20:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!