| *** blarnath is now known as d34dh0r53 | 03:37 | |
| *** blarnath is now known as d34dh0r53 | 03:52 | |
| *** akekane is now known as abhishekk | 05:09 | |
| opendevreview | Ghanshyam proposed openstack/nova master: Enable new defaults and scope checks by default https://review.opendev.org/c/openstack/nova/+/866218 | 05:37 |
|---|---|---|
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce asym NUMA mixed CPU policy bug https://review.opendev.org/c/openstack/nova/+/862686 | 08:42 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Handle zero pinned CPU in a cell with mixed policy https://review.opendev.org/c/openstack/nova/+/862687 | 08:42 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce asym NUMA mixed CPU policy bug https://review.opendev.org/c/openstack/nova/+/862686 | 08:43 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Handle zero pinned CPU in a cell with mixed policy https://review.opendev.org/c/openstack/nova/+/862687 | 08:43 |
| opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | 09:58 |
| *** han-guangyu_ is now known as han-guangyu | 10:39 | |
| *** sfinucan is now known as stephenfin | 12:04 | |
| opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | 13:49 |
| *** dasm|off is now known as dasm | 14:00 | |
| *** akekane is now known as abhishekk | 14:10 | |
| opendevreview | Konrad Gube proposed openstack/nova-specs master: Use extend volume completion action https://review.opendev.org/c/openstack/nova-specs/+/855490 | 14:56 |
| bauzas | reminder : nova meeting in 5 mins | 15:55 |
| * bauzas rushes to amend the agenda | 15:55 | |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Dec 13 16:00:34 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 | hey folks | 16:00 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:00 |
| gmann | o/ | 16:01 |
| gibi | o/ | 16:01 |
| elodilles | o/ | 16:01 |
| Uggla | o/ | 16:01 |
| bauzas | okidoki let's start | 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 19 new untriaged bugs (+1 since the last meeting) | 16:02 |
| bauzas | sorry I was planning to do the triage yesterday with amit but given he wasn't here, I forgot | 16:02 |
| bauzas | so, I'll take the baton again | 16:03 |
| 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 | #info bug baton is being passed to bauzas | 16:04 |
| bauzas | any bug you want to discuss ? | 16:04 |
| bauzas | looks not | 16:04 |
| 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:05 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:05 |
| bauzas | #link https://review.opendev.org/c/openstack/tempest/+/866049 proposal to add more delay for centos9s-fips job | 16:05 |
| bauzas | as you can see, ^ was +Wd | 16:05 |
| bauzas | so, hopefully, it could be merged in a few days | 16:05 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:05 |
| bauzas | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | 16:05 |
| bauzas | voila for gate bugs | 16:05 |
| bauzas | gibi: you told me some change was fixing the issue for nova-ceph-multistore and nova-grenade-multinode | 16:06 |
| gibi | yes | 16:06 |
| gibi | it was pinning tox to < 4.0 | 16:06 |
| gibi | it is merged last week | 16:07 |
| gibi | since then we merged things to master so I think master gate is OK | 16:07 |
| elodilles | note that that workaround will be removed @ Dec 21 | 16:07 |
| gibi | elodilles: then we have a broken gate @ Dec 21 :) | 16:07 |
| clarkb | in the ensure-tox role which won't affect devstack but will affect unittests and other things | 16:07 |
| bauzas | gibi: cool thanks | 16:07 |
| bauzas | #link https://bugs.launchpad.net/devstack/+bug/1999183 | 16:08 |
| gibi | clarkb: ahh that sounds better | 16:08 |
| clarkb | you can either pin ensure-tox to <4 in your .zuul.yaml by setting ensure_tox_version or whatever I called the variable or fix your tox.ini to work under tox v4 | 16:08 |
| elodilles | clarkb: thanks for the clarification | 16:09 |
| gibi | clarkb: ack, based on the timing I guess we will fix this next year :) | 16:09 |
| clarkb | there are two main issues that we've found with tox v4 you need to convert whitelist_externals to allowlist_externals and that list must be complee as it doesn't just warn now but creates an error. And #'s in commands entries are interpreted as ini comments and you may need to escape them with \# if you intend for them to make it into the command | 16:09 |
| bauzas | so, we don't need to modify our .zuul.yaml file ? | 16:10 |
| * bauzas needs to understand more | 16:10 | |
| clarkb | you either fix your tox.ini so that it is compatibile with tox v4 (and v3) or you update your .zuul.yaml to pin tox <4 | 16:11 |
| gibi | bauzas: probably we are better of fixing our tox.ini instead | 16:11 |
| clarkb | the ideal is the first option as that way developers don't get confused when tox doesn't work for them locally | 16:11 |
| clarkb | I think if that wasn't a concern then we wouldn't be in a hurry to unpin and half force projects to address this, but it does actually represent a real world concern that we should be addressing | 16:12 |
| bauzas | gibi: do we have the same issue than devtack by our own tox.ini file ? | 16:12 |
| bauzas | about usage of allowlist_externals | 16:12 |
| gibi | clarkb: said about that the Dec 21 change will not effect devstack jobs | 16:13 |
| * bauzas tries to understand whether he needs to provide a new change (or someone else) before dec 21 | 16:13 | |
| clarkb | devstack installs and manages tox itself so it controls when tox gets unpinned if ever | 16:13 |
| gibi | clarkb thanks | 16:14 |
| clarkb | skimming the nova tox.ini it actually looks like it might be ok. I'll push a change in a second that forces nova to use tox 4 in unittests to try and confirm | 16:14 |
| bauzas | clarkb: okay, let ping us if you want after this | 16:15 |
| bauzas | clarkb: and thanks for working on it | 16:15 |
| bauzas | shall we move on ? | 16:16 |
| opendevreview | Clark Boylan proposed openstack/nova master: DNM force latest tox https://review.opendev.org/c/openstack/nova/+/867533 | 16:16 |
| bauzas | ack ^ | 16:16 |
| bauzas | moving on then | 16:17 |
| bauzas | #topic Release Planning | 16:17 |
| bauzas | #link https://releases.openstack.org/antelope/schedule.html | 16:17 |
| bauzas | #info Antelope-2 is in 3 weeks | 16:17 |
| bauzas | as a reminder, we will have a Spec review day tomorrow | 16:17 |
| bauzas | mark your calendars | 16:17 |
| bauzas | #info Spec review day planned for Dec 14th | 16:17 |
| bauzas | that's all I have here | 16:18 |
| bauzas | any release question ? | 16:18 |
| gibi | I will be a bit busy tomorrow witha in internal demo | 16:19 |
| gibi | but I will try to get some time on specs too | 16:19 |
| bauzas | ++ | 16:19 |
| bauzas | gibi: no worries | 16:19 |
| bauzas | #topic Review priorities | 16:19 |
| bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | 16:19 |
| bauzas | #info As a reminder, cores eager to review changes can +1 to indicate their interest, +2 for committing to the review | 16:19 |
| bauzas | btw. for tomorrow's spec review day, it would be nice if cores could +2 the r-p label for each of the specs they want to look at during the day | 16:20 |
| bauzas | so, when another core looks at a spec, he will know whether some other cores want to do this as well | 16:20 |
| bauzas | just sayin' | 16:20 |
| bauzas | movin' on | 16:22 |
| bauzas | #topic Stable Branches | 16:23 |
| bauzas | elodilles: your time | 16:23 |
| elodilles | in short: nothing to report :) | 16:23 |
| bauzas | \o/ | 16:23 |
| elodilles | in more length: | 16:23 |
| elodilles | #info stable branches seem to be unblocked / OK | 16:23 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:23 |
| bauzas | cool thanks | 16:23 |
| elodilles | and that is all | 16:23 |
| bauzas | quick and loving | 16:23 |
| elodilles | indeed | 16:23 |
| bauzas | moving on | 16:23 |
| bauzas | last topic | 16:24 |
| bauzas | #topic Open discussion | 16:24 |
| bauzas | first, I have a question | 16:24 |
| bauzas | (bauzas) We're entering the Christmas period. Would we want to skip some meetings ? | 16:24 |
| gibi | I will be off from next Monday so I vote to skip it | 16:24 |
| bauzas | at least for me, I'll be working on next Tuesday | 16:24 |
| * elodilles won't be availiable for the next 2-3 weeks | 16:24 | |
| bauzas | but if we don't have a quorum, then I'd prefer to punt this meeting | 16:24 |
| bauzas | I know sean-k-mooney won't be there too | 16:25 |
| bauzas | I don't know for others, but if 3 main folks aren't around... | 16:25 |
| Uggla | I guess we should not a lot of people in the next 2 weeks. | 16:25 |
| bauzas | #info Skip the two next Nova meetings | 16:25 |
| bauzas | #undo | 16:25 |
| opendevmeet | Removing item from minutes: #info Skip the two next Nova meetings | 16:25 |
| gmann | +1 on skipping | 16:25 |
| bauzas | #info Skip the two next Nova meetings on Dec 20 and Dec27 | 16:26 |
| bauzas | #undo | 16:26 |
| opendevmeet | Removing item from minutes: #info Skip the two next Nova meetings on Dec 20 and Dec27 | 16:26 |
| bauzas | #info Skip the two next Nova meetings on Dec 20 and Dec 27. See you next year ! | 16:26 |
| bauzas | ok was quick | 16:26 |
| bauzas | I'll send an email | 16:26 |
| bauzas | second question | 16:26 |
| bauzas | (bauzas) Document the volume detach issue | 16:26 |
| bauzas | as a reminder, this volume detach issue we have is related to a specific libvirt release | 16:27 |
| bauzas | so, I don't know how we could document it | 16:27 |
| bauzas | but I'd like to have a way to explain our users that they need to make sure to ask to detach once they can ssh | 16:27 |
| bauzas | and ideally attach as well | 16:28 |
| bauzas | thoughts on that ? | 16:28 |
| bauzas | hah | 16:29 |
| bauzas | you know what ? | 16:30 |
| bauzas | let's discuss this once more people are around like next year :) | 16:30 |
| gibi | I guess you are not happy with a simple reno | 16:30 |
| bauzas | gibi: yes, because this is not related to our repository, right? | 16:31 |
| gibi | right | 16:31 |
| bauzas | I mean, this isn't a nova bug | 16:31 |
| bauzas | by a specific release | 16:31 |
| gibi | it is only a nova bug if we assume nova should prevent the attach / detach | 16:31 |
| bauzas | we can track this in Launchpad and I'd be okay | 16:31 |
| gibi | but we never promised to prevent such attach / detach | 16:31 |
| bauzas | that's one way of documenting an open issue | 16:31 |
| bauzas | gibi: correct | 16:32 |
| bauzas | you know what ? | 16:33 |
| bauzas | lemme repropose this topic next year | 16:33 |
| gibi | OK | 16:33 |
| bauzas | (gmann) updates on switching the RBAC(scope and new defaults) defaults in nova | 16:33 |
| clarkb | as an end user I look at your api docs when tryingto understand expectations | 16:33 |
| clarkb | if anything is done I would suggest adding a note to the api docs for attach and detach | 16:33 |
| bauzas | clarkb: that's a solutioon | 16:33 |
| bauzas | which lgtm btw. | 16:34 |
| bauzas | I can propose this | 16:34 |
| bauzas | and people can vote on it | 16:34 |
| bauzas | gmann: you had a thought | 16:34 |
| gmann | for RBAC, | 16:34 |
| bauzas | on the RBAC defaults, yeah | 16:35 |
| gmann | I have added a new tempest job enabling the new defaults for Nova, neutron, cinder, glance and it is working fine #link https://zuul.openstack.org/builds?job_name=tempest-full-enforce-scope-new-defaults&skip=0 | 16:35 |
| gmann | and this patch which need more work on functional tests flipping the defaults in nova #link https://review.opendev.org/c/openstack/nova/+/866218 | 16:35 |
| gmann | just a info, nova patch is not ready to review as I need to fix functional tests there | 16:35 |
| gmann | especially on admin to be changed from context.is_admin to roles['admin'] | 16:36 |
| gmann | I should be ready by this week | 16:36 |
| gmann | that is all | 16:36 |
| gmann | there were few neutron bugs we found during tempest testing and all are fixed | 16:36 |
| gmann | that's it from me unless any question on rbac defaults | 16:38 |
| bauzas | gmann: ++ | 16:38 |
| bauzas | thanks for working on it, that's good to know it works :) | 16:39 |
| bauzas | gmann: you can ping me once you're done with the functests | 16:39 |
| gmann | sure, will do | 16:39 |
| bauzas | ok, then we're done | 16:41 |
| bauzas | any other item anyone wants to discuss ? | 16:41 |
| gibi | - | 16:42 |
| bauzas | thanks all | 16:42 |
| bauzas | #endmeeting | 16:42 |
| opendevmeet | Meeting ended Tue Dec 13 16:42:49 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:42 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-12-13-16.00.html | 16:42 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-12-13-16.00.txt | 16:42 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-12-13-16.00.log.html | 16:42 |
| gmann | thanks | 16:42 |
| Uggla | thx | 16:42 |
| gibi | thanks bauzas | 16:43 |
| elodilles | thanks bauzas o/ | 16:47 |
| clarkb | bauzas: to clarify a bit more on the tox stuff I'm trying to keep on top of it from a high level and in shared library tooling like the ensure-tox role. But I'm not planning to edit all the various tox.ini files to make them work :) | 16:48 |
| opendevreview | Merged openstack/nova stable/xena: Retry attachment delete API call for 504 Gateway Timeout https://review.opendev.org/c/openstack/nova/+/866083 | 16:48 |
| bauzas | ok | 16:48 |
| clarkb | but nova's actually looks really clean and if that test change passes then you are probably fine | 16:48 |
| gmann | elodilles: gibi: clarkb: bauzas: on tox. we have pinned it in devstack side for tox based job also https://github.com/openstack/devstack/blob/master/playbooks/tox/pre.yaml#L14 | 17:04 |
| gmann | so ensure-tox role unpinning on dev 21 should not impact devstack based tox job. but yes we need to move master to tox 4.0.0 soon | 17:04 |
| clarkb | right I think the main concern at this point is developers aren't going to want to maintain different versions of tox for different projects. Generally keeping up with latest avoids this problem. We pinned to stop the bleeding and now we can take a controlled approach to adding compatibility | 17:05 |
| gmann | yeah | 17:06 |
| opendevreview | Merged openstack/nova master: Create RequestGroups from InstancePCIRequests https://review.opendev.org/c/openstack/nova/+/852771 | 17:33 |
| opendevreview | Merged openstack/nova master: Support resource_class and traits in PCI alias https://review.opendev.org/c/openstack/nova/+/853316 | 17:33 |
| opendevreview | Merged openstack/nova master: Split PCI pools per PF https://review.opendev.org/c/openstack/nova/+/854440 | 17:33 |
| *** dasm is now known as dasm|off | 19:26 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!