| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-04-30 | |||
| 10:21:29 | opendevreview | Takashi Kajinami proposed openstack/os-resource-classes master: Remove old excludes https://review.opendev.org/c/openstack/os-resource-classes/+/917614 | |
| 11:30:14 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add regression test case for bug 1978573 https://review.opendev.org/c/openstack/nova/+/914653 | |
| 11:53:28 | opendevreview | Merged openstack/os-traits master: Remove old excludes https://review.opendev.org/c/openstack/os-traits/+/917613 | |
| 12:13:14 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Remove old excludes https://review.opendev.org/c/openstack/os-vif/+/917611 | |
| 12:44:29 | opendevreview | Takashi Kajinami proposed openstack/python-novaclient master: Remove old excludes https://review.opendev.org/c/openstack/python-novaclient/+/917685 | |
| 12:48:34 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix KeyError on assisted snapshot call https://review.opendev.org/c/openstack/nova/+/900783 | |
| 12:54:34 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add regression test case for bug 1978573 https://review.opendev.org/c/openstack/nova/+/914653 | |
| 12:54:35 | opendevreview | Rajesh Tailor proposed openstack/nova master: Restore original AZ when unshelve fails https://review.opendev.org/c/openstack/nova/+/911108 | |
| 13:52:51 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: DNM: Run requirement-check jobs https://review.opendev.org/c/openstack/os-vif/+/917707 | |
| 13:53:33 | tkajinam | ^^^ sean-k-mooney I think this shows how the requirement-check job fails with the current u-c.txt file which was cleaned up. | |
| 13:58:27 | sean-k-mooney | tkajinam: im not sayign we shoudl not update them | |
| 13:58:39 | sean-k-mooney | im saying that the update you propoes is not correct as it allows know broken version | |
| 13:58:50 | sean-k-mooney | so we either need to fix the requiremetn repo | |
| 13:59:03 | sean-k-mooney | or you need to increase the min version above the blocked version | |
| 13:59:44 | tkajinam | I can bump the min version but I doubt the current list excludes all of known bad versions really | |
| 14:00:17 | sean-k-mooney | sure but allowing know broken version i would consider to be a medium severity bug | |
| 14:01:08 | sean-k-mooney | it would be a high severity bug if pip tied to install a previously blocked version and it cuase dissues | |
| 14:01:30 | tkajinam | I understand that | |
| 14:01:51 | tkajinam | my expectation is that pip would install the "latest" good version so it may not attempt to install such ancient versions | |
| 14:02:13 | sean-k-mooney | sure and my epecation is project do not allow pip to install know bad versions | |
| 14:02:23 | tkajinam | so this is more like a policy decision. I'll leave it for a while in case someone else chime in with a different thought | |
| 14:02:34 | sean-k-mooney | by default it wont but it should not be possibel even if you pass a constratigs file | |
| 14:03:01 | tkajinam | yeah | |
| 14:03:02 | tkajinam | but will update it to bump min unless I get additional feedback | |
| 14:04:05 | tkajinam | Stephen merged patches in a few other nova repos so I can push min bumps as follow-up if we agree with the direction | |
| 14:04:33 | sean-k-mooney | or we revert them | |
| 14:04:53 | tkajinam | we can't revert because requiremet-check job fails on reverts | |
| 14:05:09 | tkajinam | unless we "fix" the upper-constraint file | |
| 14:05:31 | sean-k-mooney | ok well lets fix them quickly then | |
| 14:05:39 | sean-k-mooney | and let not merge any more | |
| 14:09:00 | sean-k-mooney | i think os-traits is the only one where it merged | |
| 14:10:24 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Remove old excludes https://review.opendev.org/c/openstack/os-vif/+/917611 | |
| 14:12:34 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Bump min versions to exclude known bad versions https://review.opendev.org/c/openstack/os-traits/+/917713 | |
| 14:13:06 | tkajinam | ^^^ this is the only nova repo which gets that change merged | |
| 14:13:10 | tkajinam | I'll update the other open ones | |
| 14:14:29 | tkajinam | maybe I can bring the topic to the meeting today before I update all | |
| 14:29:17 | sean-k-mooney | sure but it likely should also be disucssed with the requirements project | |
| 14:29:48 | sean-k-mooney | im +2 on both by the way | |
| 14:37:08 | tkajinam | let's see how others think. but I hope we can move on with project decisions now and follow up with the global policy discussion, because the job failure may block any change which updates requirements | |
| 15:00:55 | opendevreview | OpenStack Release Bot proposed openstack/nova master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/nova/+/917741 | |
| 15:01:23 | opendevreview | OpenStack Release Bot proposed openstack/os-vif master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/os-vif/+/917743 | |
| 15:01:50 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/osc-placement/+/917745 | |
| 15:02:19 | opendevreview | OpenStack Release Bot proposed openstack/placement master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/placement/+/917747 | |
| 15:02:48 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-novaclient/+/917749 | |
| 15:27:50 | elodilles | if any nova core wants easy-win patch reviews then these bot-proposed, generated patches need to merge ASAP ^^^ o:) | |
| 15:37:02 | gibi | on it | |
| 15:39:59 | gibi | I think I hit all of them | |
| 15:40:01 | elodilles | thx o/ | |
| 15:40:51 | gibi | thanks for eoling zed :) | |
| 15:41:17 | gibi | I know onlu unmaintained it, but for me that is eol :D | |
| 15:43:22 | elodilles | :D yeah, i remember :) | |
| 15:47:45 | bauzas | cool thanks | |
| 15:48:01 | bauzas | sorry guys, I was working hard on a devstack issue, eventually I'm all done \o/ | |
| 15:48:35 | bauzas | fwiw, centos9 has some iptables rule that doesn't accept to forward any IP address :) | |
| 15:48:51 | bauzas | folks, nova meeting now in 12 mins | |
| 15:49:05 | elodilles | no worries, gibi +2+W'd everything ;) | |
| 15:49:50 | opendevreview | Merged openstack/os-vif master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/os-vif/+/917743 | |
| 15:50:28 | opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-novaclient/+/917749 | |
| 16:00:12 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:12 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:12 | opendevmeet | Meeting started Tue Apr 30 16:00:12 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:12 | bauzas | #startmeeting nova | |
| 16:00:27 | bauzas | hey folks | |
| 16:00:31 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:00:34 | Uggla_ | o/ | |
| 16:00:37 | tkajinam | o/ | |
| 16:00:39 | gibi | o/ | |
| 16:00:41 | elodilles | o/ | |
| 16:00:41 | auniyal | o/ | |
| 16:00:51 | fwiesel | \o | |
| 16:00:57 | bauzas | cool, let's start | |
| 16:01:06 | bauzas | #topic Bugs (stuck/critical) | |
| 16:01:12 | bauzas | #info No Critical bug | |
| 16:01:19 | 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:23 | bauzas | any bug to report ? | |
| 16:01:51 | bauzas | looks not | |
| 16:01:56 | bauzas | moving on? | |
| 16:02:19 | bauzas | #topic Gate status | |
| 16:02:24 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:02:29 | bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:02:34 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | |
| 16:02:34 | opendevreview | Merged openstack/nova master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/nova/+/917741 | |
| 16:02:37 | opendevreview | Merged openstack/placement master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/placement/+/917747 | |
| 16:02:41 | bauzas | all greens | |
| 16:02:54 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:03:01 | bauzas | #info Please try to provide meaningful comment when you recheck | |
| 16:03:15 | bauzas | anything about some gate failure you just found ? | |
| 16:03:40 | tkajinam | I noticed that requirements-check job is broken now but I'll talk about that later when we talk about review priorities | |
| 16:03:48 | bauzas | ack | |
| 16:04:07 | bauzas | #topic Release Planning | |
| 16:04:13 | bauzas | #link https://releases.openstack.org/dalmatian/schedule.html | |
| 16:04:16 | bauzas | #info Dalmatian-1 in 2 weeks | |
| 16:04:22 | bauzas | #action bauzas to add the nova deadlines in the Dalmatian schedule page | |
| 16:04:36 | bauzas | I'll do this tomorrow (as I'm done with my devstack problems :) ) | |
| 16:04:50 | bauzas | #topic Review priorities | |
| 16:04:55 | bauzas | #link https://etherpad.opendev.org/p/nova-dalmatian-status | |
| 16:05:04 | bauzas | tkajinam: shoot | |
| 16:05:19 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix KeyError on assisted snapshot call https://review.opendev.org/c/openstack/nova/+/900783 | |
| 16:05:33 | tkajinam | I've added summary to L52 | |
| 16:05:38 | bauzas | I see you added it indeed | |