| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-24 | |||
| 15:18:44 | bauzas | and having a better returned exception | |
| 15:19:20 | bauzas | like "heh, we don't accept your parameters for the new feature" doesn't really explain *why* the exception was returned | |
| 15:19:20 | sean-k-mooney | we can explictly say use a microversion before X (2.95?) to evacuate until upgades are complete | |
| 15:19:34 | bauzas | sean-k-mooney: true, that's what I'm asking | |
| 15:19:53 | sean-k-mooney | this is also an adming only api so the impact is much less | |
| 15:20:12 | bauzas | again, I've said it : it's admin-only hence why I'm OK | |
| 15:20:16 | sean-k-mooney | but we still need to tell them about it so ya | |
| 15:20:34 | bauzas | if this was an enduser API, then definitely -1 and asking for another design | |
| 15:20:48 | bauzas | because endusers don't know the environment was upgraded | |
| 15:21:00 | sean-k-mooney | im not sure i would agree | |
| 15:21:02 | bauzas | and actually, even API admins couldn't be knowing itr | |
| 15:21:21 | sean-k-mooney | but i would say if that how you felt giving that feedback now instead of in the spec review is unfortunet | |
| 15:21:22 | bauzas | sometimes, environment operators and API admins are different people | |
| 15:21:47 | bauzas | sean-k-mooney: again, I'm not against the desing | |
| 15:21:51 | bauzas | we discussed it at the PTG | |
| 15:22:00 | bauzas | and the spec was there, agreed | |
| 15:22:07 | sean-k-mooney | we did and the spec had 4 +2s too | |
| 15:22:19 | bauzas | I'm just explaining that this could have been different if the API was enduser, that's it | |
| 15:22:43 | bauzas | and see "could" | |
| 15:22:49 | bauzas | not saying I would disagree | |
| 15:22:55 | sean-k-mooney | we have microversion to allow this so we could but i dont think we should even for an end user api | |
| 15:23:18 | sean-k-mooney | anyway lets work on the release note and excetion message so | |
| 15:23:19 | bauzas | anyway, I think we both agree : it needs a better explanation | |
| 15:23:22 | sean-k-mooney | thanks for taking a look | |
| 15:23:27 | sean-k-mooney | yep | |
| 15:25:18 | dansmith | no, it's already +2 | |
| 15:25:21 | dansmith | https://review.opendev.org/c/openstack/nova/+/871612 | |
| 15:25:34 | dansmith | whoops | |
| 15:25:51 | sean-k-mooney | differnt patch :) | |
| 15:26:04 | dansmith | wrong window, sorry | |
| 15:26:23 | sean-k-mooney | shall i send that through the gate given what it is | |
| 15:26:33 | dansmith | I would | |
| 15:27:07 | sean-k-mooney | on its way | |
| 15:28:04 | sean-k-mooney | brb | |
| 15:53:55 | bauzas | as a reminder, nova meeting in 7 mins | |
| 16:00:18 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:18 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:18 | opendevmeet | Meeting started Tue Jan 24 16:00:18 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:18 | bauzas | #startmeeting nova | |
| 16:00:24 | bauzas | hey everyone | |
| 16:01:02 | dansmith | o/ | |
| 16:01:35 | gibi | o/ | |
| 16:01:58 | bauzas | I guess we can start, as we can make it quick | |
| 16:02:14 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:15 | elodilles | o/ | |
| 16:02:24 | bauzas | #info One Critical bug | |
| 16:02:38 | bauzas | same than last week | |
| 16:02:47 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2002951 | |
| 16:03:05 | bauzas | AFAICT, gibi's skip patch is now merged, so we can put it to High | |
| 16:03:16 | bauzas | fine ? | |
| 16:03:28 | bauzas | dansmith's series is on the fly | |
| 16:03:36 | gibi | fine | |
| 16:03:58 | bauzas | #link https://review.opendev.org/c/openstack/tempest/+/870974 skipping now the failing test | |
| 16:04:32 | bauzas | #link https://review.opendev.org/c/openstack/tempest/+/871000 proposal for fixing the test issue | |
| 16:04:45 | bauzas | dansmith: can I review it ? looks so | |
| 16:05:12 | dansmith | bauzas: you can review whatever you want :) | |
| 16:05:17 | bauzas | excellenbt | |
| 16:05:20 | dansmith | but yeah need gmann and kopecmartin to hit that I think | |
| 16:05:31 | bauzas | yup, can't +2 tempest | |
| 16:05:38 | dansmith | oh, | |
| 16:05:46 | dansmith | I need to unskip so he can see, I'll do that after this | |
| 16:06:00 | bauzas | ok, I can hold then | |
| 16:06:06 | bauzas | moving on | |
| 16:07:25 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 26 new untriaged bugs (-1 since the last meeting) | |
| 16:07:36 | bauzas | thanks gibi for the triage, any bug you wanted to raise ? | |
| 16:07:56 | gibi | bauzas: I triaged ~5 bugs, nothing serious popped up | |
| 16:08:00 | bauzas | cool | |
| 16:08:40 | bauzas | I think we can easily put the number down when I look at the open ones | |
| 16:09:08 | bauzas | gibi: no worries | |
| 16:09:20 | bauzas | easy transition : | |
| 16:09:22 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:09:30 | bauzas | melwitt: are you around ? | |
| 16:09:34 | bauzas | she's the next in the roster | |
| 16:10:02 | bauzas | looks not | |
| 16:10:11 | bauzas | no worries, I'll try to reach her later | |
| 16:10:27 | bauzas | #topic Gate status | |
| 16:10:33 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:10:45 | bauzas | so the gate is still a bit funky | |
| 16:10:49 | dansmith | yeah | |
| 16:10:57 | dansmith | lots of sporadic fails still | |
| 16:11:05 | bauzas | https://bugs.launchpad.net/nova/+bug/2002782 needs help | |
| 16:11:36 | bauzas | I'll try to add more logs in some DNM to try to see whether we have more explanations | |
| 16:11:50 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 16:12:27 | bauzas | anything people wanna discuss about it ? | |
| 16:12:39 | gibi | I have no extra input to it | |
| 16:12:54 | bauzas | gibi: I've seen you updated the bug report, thanks | |
| 16:12:58 | gibi | no worries | |
| 16:13:03 | gibi | that was part of my triage round | |
| 16:13:22 | bauzas | I just triaged it to Confirmed/High fwiw | |
| 16:13:53 | gibi | ack | |
| 16:14:01 | gibi | I did not want to triage my one bug | |
| 16:14:04 | Uggla | o/ | |
| 16:14:06 | bauzas | https://zuul.openstack.org/builds?job_name=nova-tox-functional&job_name=nova-tox-functional-py310&project=openstack%2Fnova&skip=0 | |
| 16:14:13 | bauzas | this is wedgy | |
| 16:14:29 | bauzas | but at least the gate isn't hold | |
| 16:14:49 | bauzas | (assuming we have true positives in that list) | |
| 16:15:10 | gibi | filtering that to the gate pipeline is a bit better | |
| 16:15:27 | bauzas | true, it avoids the true positives | |
| 16:15:29 | gibi | but that does not show patches that are good but fail on check | |
| 16:15:36 | gibi | so meh | |