opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | 03:10 |
---|---|---|
bauzas | sean-k-mooney: can you review https://review.opendev.org/c/openstack/nova/+/893462 ? | 08:38 |
*** ralonsoh_ is now known as ralonsoh | 08:41 | |
sean-k-mooney | i need to leave fo a doctors approintment soon but ill take a look at it today | 08:49 |
opendevreview | Merged openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | 08:57 |
* bauzas disappears early this morning due to an appointment, but I'll be back in 2 hours roundly | 09:05 | |
*** dmitriis is now known as Guest1871 | 09:12 | |
opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 09:58 |
plibeau | hello guys, I need your help on this change https://review.opendev.org/c/openstack/nova/+/861172 | 10:10 |
dvo-plv | sean-k-mooney, dansmith Uggla sahid Hello, Mayby you will have a chance to review patchset https://review.opendev.org/c/openstack/nova/+/876075 | 12:21 |
sean-k-mooney | dvo-plv: unfortunetlly we are past feature freeze fo this cycle | 12:27 |
sean-k-mooney | i could take a look but it cant merge until next cycle after RC1 is cut so thats 2-3 weeks form now | 12:28 |
sean-k-mooney | bauzas: unless we want to consider a FFE for ^ | 12:29 |
sean-k-mooney | i was +2 on it a month ago | 12:29 |
sean-k-mooney | it would have been nice to land that but we are now out of time | 12:29 |
bauzas | I'm not super happy to create a FFE for this due to https://review.opendev.org/c/openstack/nova/+/876075/23/nova/objects/image_meta.py | 12:30 |
dvo-plv | I see,maybe it will be good to take a look, if you have new comments, we will have a time to fix until next cycle | 12:30 |
bauzas | I'd prefer to look at it in two weeks, after RC1 while the 2023.2 branch is created and then master is for 2024.1 | 12:31 |
opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 12:39 |
opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 13:05 |
opendevreview | Sylvain Bauza proposed openstack/nova master: doc: mark the maximum microversion for 2023.2 Bobcat https://review.opendev.org/c/openstack/nova/+/893742 | 14:20 |
opendevreview | Sylvain Bauza proposed openstack/nova master: Update compute rpc alias for bobcat https://review.opendev.org/c/openstack/nova/+/893744 | 14:26 |
ygk_12345 | some of my nodes in an aggregate dont have the aggregate mapping in the placement db for those nodes | 14:26 |
ygk_12345 | is this a bug in nova ? | 14:26 |
ygk_12345 | i am trying to use the command "openstack resource provider aggregate list <resource-id> | 14:27 |
ygk_12345 | that is returning empty result for those nodes in that aggregate | 14:27 |
ygk_12345 | i tried placement syc aggregates also | 14:28 |
ygk_12345 | but no result | 14:28 |
ygk_12345 | *sync | 14:28 |
opendevreview | Merged openstack/nova master: Follow up for unified limits: PCPU and documentation https://review.opendev.org/c/openstack/nova/+/893462 | 14:36 |
opendevreview | Sylvain Bauza proposed openstack/nova master: Add service version for Bobcat https://review.opendev.org/c/openstack/nova/+/893749 | 14:41 |
ygk_12345 | any idea anyone ? | 14:49 |
dansmith | ygk_12345: not really enough information to say (like which release are you running?) but in general the placement aggregates should match the nova ones, yes | 14:50 |
ygk_12345 | dansmith: it is yoga | 14:51 |
ygk_12345 | the aggregate matching ismissing for one set of nodes in the az | 14:52 |
dansmith | ygk_12345: and that set of nodes is different how? | 14:53 |
ygk_12345 | dansmith: they are just added to the az along with others | 14:54 |
ygk_12345 | do I need to manually add all these nodes from that aggregate to match that aggregate using the command openstack resource provider aggregate set ? | 14:56 |
ygk_12345 | it succeeded for one node now | 14:56 |
dansmith | ygk_12345: when you add a node to an aggregate in nova, it should mirror that to placement for you | 14:56 |
ygk_12345 | but why did not it add those nodes automatically ? | 14:56 |
ygk_12345 | dansmith: but it did not do so for an aggregate | 14:57 |
*** dasm is now known as Guest1929 | 14:57 | |
dansmith | ygk_12345: not sure what to say without logs and things... try removing it from the aggregate in nova, re-add it and look at the nova-api logs to see if there's a failure talking to placement or something | 14:58 |
ygk_12345 | dansmith: ok | 14:59 |
bauzas | reminder: nova meeting in 1 hour | 15:00 |
bauzas | ygk_12345: we have some method that calls placement for creating the same aggregates, sec | 15:00 |
ygk_12345 | bauzas: ok | 15:01 |
ygk_12345 | bauzas: please let me know | 15:01 |
*** Guest1929 is now known as dasm | 15:02 | |
bauzas | eventually found the change https://review.opendev.org/c/openstack/nova/+/553597 | 15:04 |
ygk_12345 | bauzas: I have removed and re added and now it is working. Thank you | 15:07 |
bauzas | ok good to know | 15:07 |
dansmith | ygk_12345: that likely means nova-api wasn't able to talk to placement at the time you added those nodes originally, for whatever reason | 15:09 |
ygk_12345 | dansmith: seems so. Thanks for your time and patience | 15:09 |
bauzas | #startmeeting nova | 16:00 |
opendevmeet | Meeting started Tue Sep 5 16:00:02 2023 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 | welcome folks | 16:00 |
gibi | o/ | 16:00 |
sean-k-mooney | o/ | 16:00 |
Uggla | o/ | 16:00 |
elodilles | o/ | 16:00 |
auniyal | o/ | 16:00 |
bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:00 |
dansmith | o/ | 16:00 |
bauzas | cool, people arriving fast, lovin' | 16:00 |
bauzas | let's start | 16:00 |
bauzas | #topic Bugs (stuck/critical) | 16:00 |
bauzas | #info No Critical bug | 16:01 |
bauzas | whoops | 16:01 |
bauzas | #info No Critical bug | 16:01 |
bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 49 new untriaged bugs (+0 since the last meeting) | 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:01 |
bauzas | #info bug baton is bauzas | 16:01 |
auniyal | bauzas, I have few bugs to discuss | 16:01 |
bauzas | auniyal: any questions you have ? | 16:01 |
bauzas | cool, shoot | 16:01 |
auniyal | actully this : https://etherpad.opendev.org/p/nova-bug-triage-20230902 | 16:02 |
bauzas | ack, looking | 16:02 |
auniyal | https://bugs.launchpad.net/nova/+bug/2031635: | 16:02 |
auniyal | ack | 16:02 |
bauzas | I'm not expert, I can't tell | 16:03 |
auniyal | I'll paste the quick ones here | 16:04 |
auniyal | https://bugs.launchpad.net/nova/+bug/2033393: Xena: server name is not updated in guest xml after rename ? | 16:04 |
gmann | o/ | 16:04 |
gibi | so for 2031635 we need to check if https://review.opendev.org/c/openstack/nova/+/867788 is backported to stables or not | 16:04 |
auniyal | is it known ? does it does affect an operation | 16:05 |
bauzas | let's not discuss the resolution here | 16:05 |
auniyal | gibi, its not backported | 16:05 |
gibi | auniyal: then I guess we need to backport it | 16:06 |
gibi | as we need to support py3.8 | 16:06 |
auniyal | ack, | 16:06 |
bauzas | ok, resolution set, Valid then | 16:06 |
bauzas | we can look at the other ones next week | 16:06 |
dansmith | auniyal: the instance name one is invalid, I just updated | 16:07 |
auniyal | ack, thanks dansmith | 16:07 |
bauzas | ok, let's move on then | 16:07 |
bauzas | #topic Gate status | 16:08 |
bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:08 |
bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:08 |
bauzas | #info nova-emulation is still b0rked | 16:09 |
bauzas | chateaulav: are you around ? | 16:09 |
sean-k-mooney | oh really | 16:09 |
sean-k-mooney | i was not aware of that | 16:09 |
sean-k-mooney | whats broken | 16:09 |
sean-k-mooney | test_resize_volume_backed_server_confirm | 16:10 |
sean-k-mooney | so that might not be related to emulation | 16:10 |
sean-k-mooney | the other tests passed | 16:10 |
dansmith | yeah not sure why that would be emulation related | 16:10 |
dansmith | other than that it can be slow | 16:10 |
sean-k-mooney | ill quickly take a look async | 16:11 |
bauzas | yeah probably unrelated, but alas this is the only one which fails every week | 16:11 |
sean-k-mooney | no obvious error in the nova log | 16:11 |
gmann | and a few of the fixes are merged related to this test but yes it can fail on slow maybe | 16:11 |
bauzas | okay, so we'll see if next round it fails again | 16:12 |
bauzas | any other gate information to share ? | 16:13 |
dansmith | otherwise I think the gate is doing better than it has in a while | 16:14 |
dansmith | we still need volume test attention like we have for many years | 16:14 |
dansmith | but life isn't as terrible as it has been | 16:14 |
bauzas | glad to hear such good news | 16:15 |
bauzas | which weren't arriving just because of luck | 16:15 |
bauzas | so kudos to those who worked hard on chasing | 16:15 |
bauzas | moving on then | 16:16 |
bauzas | #topic Release Planning | 16:16 |
bauzas | #link https://releases.openstack.org/bobcat/schedule.html | 16:16 |
bauzas | #info Nova deadlines are set in the above schedule | 16:16 |
bauzas | #link https://etherpad.opendev.org/p/nova-bobcat-blueprint-status Etherpad for tracking blueprints status | 16:16 |
bauzas | #info 9 blueprints were implemented this cycle, 8 got deferred (5 of them were awaiting rebases from owner) | 16:16 |
bauzas | thanks folks who reviewed features | 16:17 |
sean-k-mooney | just on last thing on nova-emulation. we didn not pin that job to < 6 concurancy so maybe we need to drop that back to 4 | 16:17 |
bauzas | despite a very few of happy reviewers, we were able to merge a quite decent number of reviews | 16:17 |
bauzas | s/reviews/changes | 16:17 |
gmann | sean-k-mooney: but is it oom issue happening there? | 16:17 |
sean-k-mooney | i dont think so ill look after the meeting | 16:18 |
sean-k-mooney | i just assume since this is testing qemu with emulation of arm it might be slower | 16:18 |
sean-k-mooney | anyway we can go back to the release topic | 16:18 |
gmann | ack | 16:19 |
bauzas | sean-k-mooney: thanks | 16:19 |
bauzas | so, I was just saying thanks | 16:19 |
bauzas | we still have a couple of series that missed the target, but most of them were mostly due to merge conflicts or other states that aren't "shit, we missed to review that" | 16:20 |
sean-k-mooney | there was one that kind fo fell in that camp | 16:20 |
bauzas | we still have two long series that require proper attention next cycle if they're reproposed, as those series are very long and hard to review | 16:20 |
sean-k-mooney | the virtio-packed ring format | 16:20 |
bauzas | it was in merge conflict until last week | 16:21 |
bauzas | actually, until today | 16:21 |
sean-k-mooney | no it ws mergable a mount ago | 16:21 |
bauzas | so no, it was still on the camp "the owner has to do things" | 16:21 |
sean-k-mooney | and then went int merge conflcit in between | 16:21 |
bauzas | sure | 16:21 |
sean-k-mooney | i had tought we merged it | 16:22 |
bauzas | but I raised a couple of times the fact that we are requiring a correct state of a patch for being reviewed | 16:22 |
sean-k-mooney | it was kasyaps min version bump that caused the merge conflict | 16:22 |
bauzas | and it was correctly tracked in the etherpad as being 'in merge conflict' | 16:22 |
bauzas | sean-k-mooney: I bet no, since it was on merge conflict *before* we merged the min version bump | 16:22 |
bauzas | but I can doublecheck the timings based on the etherpad history | 16:23 |
sean-k-mooney | anyway im sad that missed but it is what it is | 16:23 |
dansmith | yeah I think it was in merge conflict before | 16:23 |
bauzas | again, that one was correctly tracked and marked as not reviewable yet | 16:23 |
sean-k-mooney | i had put review priorty +2 on it for what its worth | 16:23 |
sean-k-mooney | i think part of the miss there may have been pto related | 16:24 |
bauzas | so, I 'm against granting an exception for it (and even not considering the object version change, which is kinda huge to consider) | 16:24 |
sean-k-mooney | im not asking for one | 16:24 |
sean-k-mooney | but i i had realised it was not merged last week i would he pinned them to update it | 16:25 |
bauzas | we're providing weekly meeting updates for such concerns | 16:25 |
bauzas | I can't just ping every single owner to ask him or her to update | 16:25 |
bauzas | it was tracked, an action was awaited | 16:25 |
bauzas | anyway, like I said a couple of hours before, a reproposal is surely acceptable for 2024.1 and can be reviewed in 2 weeks, once master becomes Caracal | 16:26 |
sean-k-mooney | yep | 16:26 |
sean-k-mooney | we also shoudl have landed https://review.opendev.org/c/openstack/nova/+/860829 by the way | 16:27 |
sean-k-mooney | that is needed for the swithc to sqlalchme 2 | 16:28 |
bauzas | this isn't a feature, and according isn't impacted by FF | 16:28 |
bauzas | so it can be reviewed now until RC1 | 16:28 |
dansmith | it's also not release critical for 2023.2 AFAIK | 16:28 |
bauzas | and which gives me the right opportunity to jump to the next item | 16:28 |
bauzas | #info 1 week before RC1 | 16:28 |
bauzas | #link https://etherpad.opendev.org/p/nova-bobcat-rc-potential Etherpad for tracking RCs | 16:28 |
bauzas | if people would enjoy clicking on such link (exactly like they hopefully did for the feature tracking etherpad just before), | 16:29 |
bauzas | they'll see that there are a couple of paperwork-ish but important changes to deliver before RC1 | 16:29 |
bauzas | mostly for upgrade concerns (rpc aliases and object version tagging) | 16:30 |
bauzas | in that etherpad, people are fancy adding patches if they feel that require to be merged before RC1 | 16:30 |
bauzas | but, I'll ping people anyway asking for reviews | 16:32 |
bauzas | I haven't yet prepared a bobcat tag for bug reports, I'm just doing it now | 16:33 |
bauzas | there | 16:34 |
bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=bobcat-rc-potential feel free to tag any bug report with this tag in order to have it tracked for RC1 and post-RCs | 16:34 |
opendevreview | Jay Faulkner proposed openstack/nova master: [FUP] [ironic] Fix for mutable nova-cpu configs https://review.opendev.org/c/openstack/nova/+/893790 | 16:34 |
bauzas | moving on | 16:35 |
bauzas | #topic Review priorities | 16:35 |
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:35 |
bauzas | #info As a reminder, people eager to review changes can +1 to indicate their interest, +2 for asking cores to also review | 16:35 |
bauzas | #topic Stable Branches | 16:35 |
bauzas | elodilles: tell the good news | 16:35 |
elodilles | o7 | 16:35 |
elodilles | #info stable/train is marked as End of Life and branch is deleted | 16:35 |
bauzas | \o/ | 16:35 |
elodilles | #info all stable gates should be OK | 16:35 |
bauzas | \o/ | 16:36 |
elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:36 |
elodilles | please add there gate issues if you encounter one ^^^ | 16:36 |
elodilles | and that's all from me | 16:36 |
sean-k-mooney | elodilles: what was the result of testing os-vif stable | 16:36 |
bauzas | thanks elodilles | 16:36 |
sean-k-mooney | did the job fail? i.e. should i look at why or did it pass on your DNM patch | 16:37 |
elodilles | sean-k-mooney: let me check it quickly | 16:37 |
sean-k-mooney | no rush | 16:37 |
sean-k-mooney | if it failed i can try and take a look at why later | 16:37 |
bauzas | auniyal: you had other items to mention | 16:37 |
elodilles | sean-k-mooney: gate tests have passed | 16:37 |
sean-k-mooney | sweet | 16:37 |
auniyal | #info Please{10} *review* these backport patches of stable 2023.1, zed and yoga for next minor release | 16:38 |
auniyal | #info most of these already have one +2. | 16:38 |
auniyal | #link https://etherpad.opendev.org/p/release-liaison-PatchesToReview | 16:38 |
auniyal | thanks | 16:38 |
bauzas | cool, will try a round tomorrow | 16:38 |
bauzas | sounds to me it's important to deliver a 2023.1 release shortly | 16:39 |
bauzas | #topic Open discussion | 16:39 |
bauzas | nothing on the agenda, anything anyone ? | 16:39 |
bauzas | ok, looks not | 16:42 |
bauzas | thanks everyone | 16:42 |
bauzas | see you next week | 16:42 |
bauzas | #endmeeting | 16:42 |
opendevmeet | Meeting ended Tue Sep 5 16:42:12 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:42 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-05-16.00.html | 16:42 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-05-16.00.txt | 16:42 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-09-05-16.00.log.html | 16:42 |
elodilles | thanks o/ | 16:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!