| *** seebaer is now known as seba | 00:14 | |
| *** dasm is now known as dasm|off | 01:06 | |
| opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-novaclient/+/835368 | 02:26 |
|---|---|---|
| opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add openssl in bindep.txt https://review.opendev.org/c/openstack/python-novaclient/+/835370 | 02:26 |
| opendevreview | Takashi Natsume proposed openstack/osc-placement master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/osc-placement/+/791972 | 02:36 |
| *** abhishekk is now known as akekane|home | 04:54 | |
| *** akekane|home is now known as abhishekk | 04:54 | |
| opendevreview | Merged openstack/nova master: Move FIPS jobs to experimental and periodic queue https://review.opendev.org/c/openstack/nova/+/833431 | 06:23 |
| opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'NovaObjectDictCompat' from 'Service' https://review.opendev.org/c/openstack/nova/+/835595 | 08:55 |
| opendevreview | Stephen Finucane proposed openstack/nova master: objects: Don't use generic 'Field' container https://review.opendev.org/c/openstack/nova/+/738239 | 08:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove unnecessary type aliases, exceptions https://review.opendev.org/c/openstack/nova/+/738240 | 08:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: objects: Use imports instead of type aliases https://review.opendev.org/c/openstack/nova/+/738018 | 08:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove wrappers around ovo mixins https://review.opendev.org/c/openstack/nova/+/738019 | 08:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'NovaObjectDictCompat' from 'Service' https://review.opendev.org/c/openstack/nova/+/835595 | 08:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: WIP: add ovo-mypy-plugin to type hinting o.vos https://review.opendev.org/c/openstack/nova/+/758851 | 08:59 |
| *** dasm|off is now known as dasm | 13:11 | |
| bauzas | folks, I have a problem with the nova meeting today | 13:37 |
| bauzas | since Europe is now on DST, our meeting will be at 4pm UTC but 6pm for me | 13:37 |
| bauzas | but I need to go to someway by 7pm | 13:37 |
| sean-k-mooney | ack we coudl just cancel it since the ptg is next week | 13:38 |
| bauzas | so either we have a small meeting until 4.15pm UTC or someone else could continue to discuss it | 13:38 |
| dansmith | ++ | 13:38 |
| sean-k-mooney | unless there is imporant topics that cant wait | 13:38 |
| dansmith | TC also canceled this week | 13:38 |
| bauzas | I have a small agenda at the time | 13:39 |
| bauzas | at least for asking folks to provide their PTG topics | 13:39 |
| bauzas | https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 13:39 |
| sean-k-mooney | ok do you want to just skip the normal standing itmes then and have a short meetign to cover that topic and wrap early | 13:39 |
| bauzas | sean-k-mooney: we could do this | 13:40 |
| bauzas | sean-k-mooney: just opening the meeting for the PTG | 13:40 |
| bauzas | and then I'll provide a email thread | 13:40 |
| noonedeadpunk | hey there! I was wondering - any reason we do this https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L986 ? | 13:45 |
| noonedeadpunk | Why I'm asking is bassically after talk in #virt regarding MPX flag, that makes IceLake CPUs being identified as Broadwell..... | 13:47 |
| noonedeadpunk | I mean - there's masking of flags being already implemented | 13:47 |
| noonedeadpunk | so technically comparing provided model to virsh capabilities is not needed | 13:48 |
| sean-k-mooney | if you have configured a cpu model that is not compatibale with the host the agent shoudl refuse to start | 13:50 |
| sean-k-mooney | so we have to check cpu compatiabity on agent start up | 13:50 |
| sean-k-mooney | noonedeadpunk: we have some patches in flight to use the newer libvirt api to do this which takes into account which flags are masked or emulated | 13:50 |
| noonedeadpunk | So I have intel xeon gold 6338 and trying to provide Icelake, but only Broadwell is available for it | 13:51 |
| noonedeadpunk | oh | 13:51 |
| sean-k-mooney | presumable icelake reference cpu feature flags that are not present on the host correct | 13:51 |
| sean-k-mooney | perhaps tsk | 13:52 |
| sean-k-mooney | *tsz | 13:52 |
| sean-k-mooney | ... tsx | 13:52 |
| noonedeadpunk | well, the only "streight" way is to add dozen of extra flags to cover difference... | 13:52 |
| noonedeadpunk | sean-k-mooney: so the falg that is missing is MPX which is sunset since 2019... But #virt said they can't jsut drop it for $reasons | 13:53 |
| noonedeadpunk | and they just suggested to mask it as they don't have other option | 13:53 |
| sean-k-mooney | ya https://paste.opendev.org/show/bmfNxhnzVYp54W9LQK0J/ so mpx is listed in the cpu model | 13:55 |
| noonedeadpunk | and I mean - https://lore.kernel.org/lkml/tip-eb012ef3b4e331ae479dd7cd9378041d9b7f851c@git.kernel.org/ and Intel also said that since 2019 no CPU will have it | 13:55 |
| noonedeadpunk | so I'm kind of... either use cpu model from 2014 or have dozens of flags specified in extras... | 13:56 |
| noonedeadpunk | or manually edit cpu_map which is too bad | 13:57 |
| sean-k-mooney | well that is a abi break on intels part which libvirt could accoutn for by adding a no_MPX verison of the cpu model | 13:57 |
| sean-k-mooney | but looking at https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L982-L1008 | 13:57 |
| sean-k-mooney | we coudl rework this os that we apply the extra cpu flags to the model before we do the compat test | 13:57 |
| sean-k-mooney | that woudl allow you to add cpu_model_extra_flags=-mpx | 13:58 |
| sean-k-mooney | and provide that combination was valid it would proceed | 13:59 |
| noonedeadpunk | well, self._get_cpu_info() is jsut `virsh capabilities`? | 13:59 |
| sean-k-mooney | right now it checks that the each model is valid on its own and then the default mode + the extra flags is valid | 13:59 |
| noonedeadpunk | and `virsh capabilities` falls back to Broadwell | 13:59 |
| noonedeadpunk | because all else have that damn MPX | 14:00 |
| sean-k-mooney | yes again its an abi break by intel effectivly | 14:00 |
| sean-k-mooney | this might be address by the new cpu compare api that we are moving too | 14:00 |
| sean-k-mooney | but if not we can rework that validation | 14:01 |
| sean-k-mooney | the curertn two pahse apprch is not really correct | 14:01 |
| sean-k-mooney | it predates the ablity to remove flags | 14:01 |
| sean-k-mooney | when we added the ability to remove flags this shoudl have been factored into one loop | 14:01 |
| sean-k-mooney | for now yes your only option is to ude broadwall and add the missing flags | 14:02 |
| noonedeadpunk | sean-k-mooney: ok, thanks a lot for help) | 14:04 |
| noonedeadpunk | hopefully it will be better soon ) | 14:04 |
| sean-k-mooney | well the currnt patch has been draging on for 3 cycle it might be betere to just rework this check as a simple backportable bugfix | 14:04 |
| noonedeadpunk | tbh that would be perfect but I can't say I see easy way how to re-work it | 14:05 |
| sean-k-mooney | something like this https://paste.opendev.org/show/b6PI9EzL6ucJUN7FKIBL/ | 14:07 |
| sean-k-mooney | that would allow you to set cpu_model=Icelake cpu_model_extra_flags=-mpx | 14:08 |
| sean-k-mooney | at least i think that would work | 14:08 |
| sean-k-mooney | the important thing is we apply the flag modification to the model before we ask libvirt to see if its compatible | 14:09 |
| sean-k-mooney | that quick refactor woudl loose some granualrity in the error message but i think it would allow better flexiblity | 14:11 |
| * noonedeadpunk reading it through | 14:11 | |
| sean-k-mooney | we could just print the requested feature and the host features and let it to the reader to do the intersection to get back most of the the previously useful info form the error | 14:12 |
| noonedeadpunk | But I think it won't affect _get_cpu_info result? | 14:13 |
| noonedeadpunk | as libvirt still would provide Broadwell as an aswer to that | 14:14 |
| noonedeadpunk | and we're comparing flags requested vs cpu model provided but virsh capabilities iirc | 14:14 |
| sean-k-mooney | _get_cpu_info i think shoudl provide the host cpu info | 14:15 |
| noonedeadpunk | not really I guess | 14:18 |
| noonedeadpunk | ant least not on Xena | 14:18 |
| noonedeadpunk | sean-k-mooney: I added some logging https://paste.opendev.org/show/b0AFUQdxdydGrXHQHn6o/ | 14:21 |
| noonedeadpunk | so _get_cpu_info for me looks exactly as `virsh capabilities` output | 14:21 |
| sean-k-mooney | yes we use libvirt as our interface to collect host infomation | 14:22 |
| noonedeadpunk | or welll | 14:22 |
| noonedeadpunk | flags are not from broadwell | 14:22 |
| sean-k-mooney | the flags should be the same or simlar to lscpu | 14:23 |
| noonedeadpunk | ok, yes, then your suggestion can make sense | 14:23 |
| sean-k-mooney | the model that is listed is the closet model that matches the cpu if i undersand correctly | 14:24 |
| noonedeadpunk | yeah, I think you're right here | 14:25 |
| ade_lee | dansmith, so what changed to make https://review.opendev.org/c/openstack/nova/+/831844 pass? | 14:53 |
| dansmith | ade_lee: the wait-for-sshable I think | 14:53 |
| ade_lee | ah ok | 14:54 |
| dansmith | there were a number of changes to various tests to make that work, but that seemed to be the problem with the quick boot-attach-detach workflow which was in a number of places | 14:54 |
| ade_lee | dansmith, excellent - we're going to run the cinder and glance fips tests again now to see if they pass | 14:56 |
| dansmith | I was seeing those qemu crashes in cs8, so I'm interested to see if those still pop up in cs9 now that this always-fail stuff is resolved | 15:01 |
| gmann | ade_lee: dansmith device tag test also is now ssh-able so may be that making it pass. | 15:13 |
| gmann | recently ssh-able | 15:13 |
| bauzas | reminder (special for EU folks) : nova meeting in 25 mins here | 15:36 |
| bauzas | we'll have a very short meeting | 15:36 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Mar 29 16:00:06 2022 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 | hola everybody | 16:00 |
| gmann | o/ | 16:00 |
| bauzas | let's do a very quick meeting (up to 15 mins) as I need to move on | 16:00 |
| elodilles | o/ | 16:00 |
| gmann | sure | 16:00 |
| bauzas | DST change here, lovely | 16:00 |
| gibi | o/ | 16:00 |
| bauzas | if someone wants to continue discussing, I can pass the chair baton | 16:00 |
| bauzas | but given next week will be PTG, I don't think we have a lot of things to discuss | 16:01 |
| bauzas | let's start then | 16:01 |
| gmann | ah its already 29. not DST is in sync in EU and USA :) | 16:01 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
| gmann | *now | 16:01 |
| bauzas | #topic Bugs (stuck/critical) | 16:01 |
| bauzas | #info No Critical bug | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 31 new untriaged bugs (+3 since the last meeting) | 16:02 |
| bauzas | #help Nova bug triage help is appreciated https://wiki.openstack.org/wiki/Nova/BugTriage | 16:02 |
| bauzas | #link https://storyboard.openstack.org/#!/project/openstack/placement 26 open stories (0 since the last meeting) in Storyboard for Placement | 16:02 |
| bauzas | nothing to say | 16:02 |
| * gibi is distracted by | 16:02 | |
| bauzas | moving on | 16:02 |
| bauzas | #topic Gate status | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:02 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly Placement periodic job status | 16:02 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:02 |
| bauzas | moving on too | 16:02 |
| bauzas | #topic Release Planning | 16:02 |
| bauzas | #link https://releases.openstack.org/yoga/schedule.html#y-final Yoga GA is planned on Wed | 16:02 |
| bauzas | #info no RC2 was needed | 16:02 |
| bauzas | there, we go to the main meat | 16:03 |
| bauzas | #topic PTG preparation | 16:03 |
| bauzas | #link https://etherpad.opendev.org/p/nova-zed-ptg Nova Zed PTG etherpad | 16:03 |
| bauzas | #action all please add your topics before Thursday EOB | 16:03 |
| bauzas | as you can see, we don't have a lof of topics | 16:03 |
| bauzas | #action bauzas to provide a mailing thread for asking for topics | 16:03 |
| bauzas | so, please add the topics before end of this week so I can plan the agenda | 16:04 |
| bauzas | and in case we need other teams for cross-project discussions, I could ping the other PTLs | 16:04 |
| bauzas | thanks to the folks who already provided some topics | 16:05 |
| *** EugenMayer4 is now known as EugenMayer | 16:05 | |
| bauzas | remind, hopefully this PTG would be the last virtual \o/ | 16:05 |
| bauzas | any questions about the PTG ? | 16:05 |
| bauzas | as you can see in the etherpad, we'll have the same room, the newton one | 16:06 |
| bauzas | for the whole PTG | 16:06 |
| bauzas | that's basically it for this | 16:06 |
| bauzas | can we move ? | 16:07 |
| bauzas | (punting review priorities for this meeting) | 16:07 |
| bauzas | #topic Stable Branches | 16:07 |
| bauzas | elodilles: ? | 16:07 |
| elodilles | #info xena branch is blocked (by nova-tox-functional-centos8-py36 job): agreed to set the job non-voting, patch: https://review.opendev.org | 16:07 |
| gmann | updates on centos8-py36 on stable/xena: now suds-jurko version conflict is happening #link https://zuul.opendev.org/t/openstack/build/4cb6ee2b453149998f8a489cfac9b0da | 16:07 |
| elodilles | #info pike branch is blocked - fix: https://review.opendev.org/c/openstack/nova/+/833666 | 16:07 |
| elodilles | reminder: Victoria Extended Maintenance transition is due ~ in a month (2022-04-27) | 16:08 |
| gmann | we discussed it in openinfra channel and we need these two backports #link https://review.opendev.org/c/openstack/oslo.vmware/+/835672 #link https://review.opendev.org/c/openstack/requirements/+/835671 | 16:08 |
| elodilles | gmann: but that's in the same job, right? | 16:08 |
| gmann | elodilles: same job , failure can be seen in https://review.opendev.org/c/openstack/nova/+/834765 | 16:08 |
| gmann | suds-jurko version conflict is consistent there from oslo/vmware which was fixed in yoga | 16:09 |
| elodilles | gmann: yes, thanks for that DNM patch | 16:09 |
| gmann | I was trying those mirror issue and traced to this issue which actually need to be fixed | 16:10 |
| elodilles | just for the record, we could unblock the gate via this patch: https://review.opendev.org/c/openstack/nova/+/834854 | 16:10 |
| elodilles | (the correct link) | 16:10 |
| gmann | elodilles: bauzas if we think req and oslo.vmware backport and then oslo.vmware release for stable/xena and then u-c update can take time then we can go with n-v for now and then revert | 16:10 |
| bauzas | ok | 16:11 |
| elodilles | gmann: ack | 16:11 |
| bauzas | how can I help ? | 16:11 |
| elodilles | thanks for the info | 16:11 |
| elodilles | bauzas: by reviewing the n-v patch (linked above ^^^) o:) | 16:12 |
| bauzas | elodilles: cool, thanks | 16:12 |
| bauzas | oh done then | 16:13 |
| gmann | yeah, I will add these info there and will propsoe revert once all backport and u-c are in place | 16:13 |
| elodilles | thanks gmann ! | 16:13 |
| elodilles | i'll also try to look at the vmware and req patches after Yoga release :) | 16:13 |
| bauzas | ok, moving on ? | 16:14 |
| gmann | elodilles: ack, thanks | 16:14 |
| bauzas | #topic Open discussion | 16:14 |
| bauzas | any item to discuss ? | 16:15 |
| bauzas | or can I end the meeting ? | 16:15 |
| gmann | nothing from me. | 16:15 |
| bauzas | thanks all then for this quick meeting and seeing hopefully all of you next week :) | 16:15 |
| bauzas | #endmeeting | 16:15 |
| opendevmeet | Meeting ended Tue Mar 29 16:15:40 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:15 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-03-29-16.00.html | 16:15 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-03-29-16.00.txt | 16:15 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-03-29-16.00.log.html | 16:15 |
| * bauzas rushes off the keyboard | 16:15 | |
| elodilles | o/ | 16:16 |
| gibi | o/ | 16:16 |
| *** sfinucan is now known as stephenfin | 16:31 | |
| opendevreview | kiran pawar proposed openstack/nova master: VMware: Early fail spwan if memory is not multiple of 4. https://review.opendev.org/c/openstack/nova/+/835739 | 17:31 |
| *** dasm is now known as dasm|afk | 17:33 | |
| opendevreview | melanie witt proposed openstack/nova-specs master: Move implemented specs for the Yoga release https://review.opendev.org/c/openstack/nova-specs/+/835272 | 20:08 |
| opendevreview | melanie witt proposed openstack/nova-specs master: Remove setup.py and setup.cfg https://review.opendev.org/c/openstack/nova-specs/+/835759 | 20:08 |
| melwitt | bauzas: rebased your change on top of a change meant to fix the docs and pep8 jobs in nova-specs ^ | 20:25 |
| melwitt | gmann: these two reviews have been sitting for awhile, they're about the perfload jobs that run on the placement repo, if you might be interested in reviewing https://review.opendev.org/q/topic:perfload-error | 21:14 |
| opendevreview | melanie witt proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | 21:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!