| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-04-30 | |||
| 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 | bauzas | #startmeeting nova | |
| 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 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:12 | opendevmeet | The meeting name has been set to '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 | auniyal | o/ | |
| 16:00:41 | elodilles | 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 | opendevreview | Merged openstack/nova master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/nova/+/917741 | |
| 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: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 | |
| 16:05:56 | tkajinam | requirements-check job is currently broken because old excludes were cleaned up in global requirements file | |
| 16:06:36 | tkajinam | because these excludes are no longer maintained in the global requirements repo, I think it makes sense to remove these from individual repo | |
| 16:06:38 | sean-k-mooney | yep it sound like we are missing somethign on the requirement repo to have caught that breakage | |
| 16:06:53 | bauzas | correct | |
| 16:07:03 | bauzas | and I'm afraid to bump our minimums just because of that | |
| 16:07:39 | sean-k-mooney | so i think its a bug if we do nto bump | |
| 16:07:59 | bauzas | we don't have jobs testing our minimums, right? | |
| 16:08:10 | bauzas | we had that before but we removed it AFAICR | |
| 16:08:10 | sean-k-mooney | im not really happy about removing know broken verison and not bumping to a version above them as our minium | |
| 16:08:20 | sean-k-mooney | correct | |
| 16:08:24 | sean-k-mooney | we dont test our miniums | |
| 16:08:39 | sean-k-mooney | so i would like to rasied them above any version that was listed as know to have issues | |
| 16:08:44 | bauzas | so there is no clear guarantee about the minimums | |
| 16:08:46 | sean-k-mooney | we will still use latest in the jobs | |
| 16:09:03 | bauzas | the problem is simple : we are on a non-SLURP release | |