| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-12 | |||
| 07:35:00 | bauzas | gibi: sean-k-mooney: I'll actually be off from tonight to next week (I'll also take Friday) | |
| 07:38:40 | gibi | bauzas: thanks for moving you stuff top of Uggla's that is nice selflessness | |
| 07:39:12 | gibi | bauzas: have a nice PTO, do you have someting on your PTL table we should keep in mind while you are away? | |
| 07:39:37 | bauzas | gibi: nothing in my mind, I'll abandon the yoga open specs next week then | |
| 07:40:04 | bauzas | we're on yoga-2 on Thursday | |
| 07:40:13 | bauzas | July-14 | |
| 07:40:23 | bauzas | that will mean we won't accept new specs | |
| 07:40:33 | bauzas | but actually, we don't have a lot of them for zed | |
| 07:42:10 | gibi | OK, so I guess you will do the official freeze mail when you are back on Monday. That is KO | |
| 07:42:13 | gibi | OK | |
| 07:42:29 | bauzas | yup | |
| 07:42:45 | bauzas | unless you wanna use the axe | |
| 07:43:16 | gibi | nope | |
| 07:43:22 | gibi | the axe is yours :) | |
| 07:43:43 | gibi | and as you said we don't have much open | |
| 07:43:51 | gibi | so there is no need for the axe | |
| 07:45:47 | bauzas | gibi: yeah, in my email, I'll clarify the situation https://review.opendev.org/q/project:openstack/nova-specs+status:open+file:%255Especs/zed/.* | |
| 07:46:08 | bauzas | about ironic's discussion, this won't need to be hold by the deadline | |
| 07:46:37 | bauzas | so only artom's spec is impacted... unless he's able to revive it before the deadline | |
| 07:47:25 | bauzas | I don't know if sean-k-mooney had wheels for https://review.opendev.org/c/openstack/nova-specs/+/821419 | |
| 07:49:51 | bauzas | gibi: about the API changes, those are stacking | |
| 07:49:57 | bauzas | with the same API microversion | |
| 07:50:12 | bauzas | I'm not really worried yet | |
| 07:50:40 | bauzas | but maybe next week, I'd propose some etherpad for trying to organize series between them | |
| 07:50:59 | bauzas | like, Uggla would take the 2.91 as he's close to be merged | |
| 07:51:17 | bauzas | mine would take 2.92 as this is a quite self-contained change | |
| 07:51:35 | bauzas | and we would debate on other patches for 2.93 and others | |
| 07:51:52 | bauzas | idea being that owners of those patches would have time in advance to rebase | |
| 08:03:51 | opendevreview | Sylvain Bauza proposed openstack/nova master: zuul: Put Centos9 Stream job periodic-weekly and experimental https://review.opendev.org/c/openstack/nova/+/849463 | |
| 08:03:58 | bauzas | gibi: sean-k-mooney: ^ | |
| 08:05:24 | bauzas | Uggla: good morning | |
| 08:06:00 | gibi | bauzas: thanks for the summary above. I agree with the plans | |
| 08:06:54 | Uggla | bauzas, o/ | |
| 08:06:55 | gibi | I'm +2 on the centos9 patch | |
| 08:06:59 | gibi | Uggla: o/ | |
| 08:07:27 | bauzas | Uggla: as I mentioned above, I'll rebase my keypair generation API change on top of your unshelve API patch | |
| 08:08:08 | bauzas | gibi: do you think we actually need to rebase all our branches ? Can't I just write my patch saying "this is 2.92" ? | |
| 08:08:44 | bauzas | of course, I would get a merge conflict because gerrit wouldn't be able to rebase the rest api microversion list doc | |
| 08:09:03 | bauzas | but this would waaaaay simplify the merge conflict resolution | |
| 08:10:30 | Uggla | bauzas, regarding unshelve have you entered your comments ? | |
| 08:10:41 | bauzas | Uggla: not yet, that's my next move | |
| 08:11:07 | bauzas | Uggla: I have a direct interest in merging your branch | |
| 08:11:55 | gibi | bauzas: I'm not sure you can verify your code if it is on 2.92 without 2.91 existing | |
| 08:12:13 | gibi | but other than that I'm OK to have 2.92 haning off 2.90 with a merge conflict | |
| 08:12:31 | Uggla | bauzas, ok I'll wait then I will fix gibi and yours quickly so you could probably merge. | |
| 08:12:45 | gibi | I'm here so I can quickly re-review | |
| 08:14:24 | bauzas | gibi: yeah the tests will probably fail | |
| 08:14:47 | bauzas | but I see this as a security layer in case of a distracted core reviewer | |
| 08:15:13 | bauzas | people can work on the the latest microversion, wait for Zuul +1ing | |
| 08:15:30 | bauzas | and then modifying their patches with a placeholder microversion | |
| 08:15:44 | bauzas | Zuul would say no, but we'd have evidence this was working before | |
| 08:16:20 | bauzas | and a merge resolution would solve it quickier once the concurrent patch merges | |
| 08:16:44 | bauzas | I think I'm fool enough to test it on my series | |
| 08:17:00 | bauzas | once Zuul blesses my last revision | |
| 08:17:36 | gibi | bauzas: yeah if you want then you can test this on the keypair series | |
| 08:23:06 | opendevreview | Amit Uniyal proposed openstack/nova master: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/849104 | |
| 09:33:34 | bauzas | Uggla: -1 on https://review.opendev.org/c/openstack/nova/+/831507 due to missing UTs on nova.compute.api | |
| 09:34:05 | bauzas | you wrote excellent conditionals (kudos to gibi and you) but you don't verify them :) | |
| 09:34:35 | bauzas | also, please help poor reviewers by not reindenting tests, that doesn't help to see the bone of the change :) | |
| 09:34:37 | gibi | bauzas: there is a bunch of functional coverage that I felt enough | |
| 09:35:25 | bauzas | gibi: yeah but we already have UTs for az | |
| 09:35:46 | gibi | ack, I'm not against having exta UTs too, just stated why I ' | |
| 09:35:47 | bauzas | and the functests are done on the latter patch | |
| 09:35:49 | gibi | why I'm OK as is | |
| 09:40:07 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds check, if admin has set compute service down https://review.opendev.org/c/openstack/nova/+/848886 | |
| 09:42:00 | Uggla | bauzas, you mean the conditionals with host and az ? There are fully tested with functional tests. Am I missing something ? | |
| 09:44:13 | bauzas | Uggla: you test them on the functests in https://review.opendev.org/c/openstack/nova/+/845897/4/nova/tests/functional/test_servers.py | |
| 09:44:20 | bauzas | which is the latter patch | |
| 09:44:41 | bauzas | Uggla: but you also touch https://review.opendev.org/c/openstack/nova/+/831507/17/nova/tests/unit/compute/test_shelve.py in the compute patch | |
| 09:44:54 | bauzas | you're actually just reindenting a few calls | |
| 09:45:15 | bauzas | but you could also test the host param in some other tests | |
| 09:46:48 | opendevreview | Merged openstack/nova master: Catch an exception in power off procedure https://review.opendev.org/c/openstack/nova/+/817176 | |
| 09:46:56 | opendevreview | Merged openstack/nova master: Optimize _local_delete calls by compute unit tests https://review.opendev.org/c/openstack/nova/+/844285 | |
| 09:47:47 | sean-k-mooney | bauzas: ill review your api removal patch shortly. i spend a lot of time reviewing this morning before going up to the office so im getting a little burnt out by it but i can do one or two more | |
| 09:48:01 | sean-k-mooney | bauzas: the centos 9 patch is on its way to merging | |
| 09:49:16 | sean-k-mooney | bauzas: gibi im going to leave the unshleve to host serise to ye. ping me if needed but since ye had open comments on them ill let ye take lead on teh review of that | |
| 09:49:38 | gibi | sean-k-mooney: ack, make sense | |
| 09:49:41 | sean-k-mooney | bauzas: regarding the external power manamgment i have not had time to look at it but its on my todo list for today | |
| 09:50:16 | gibi | sean-k-mooney: my only concern is that bauzas is off the rest of this week and I'd like to merge the unshelve | |
| 09:50:41 | sean-k-mooney | gibi: i can review just proably not today | |
| 09:50:58 | gibi | sean-k-mooney: superb, thanks | |
| 09:51:04 | bauzas | thanks | |
| 09:51:06 | sean-k-mooney | if ye agree on the path forward ill review when Uggla respins the patch to adress your comments | |
| 10:05:05 | bauzas | sean-k-mooney: gibi: we deserve to be humble with Uggla https://review.opendev.org/c/openstack/os-traits/+/832769 | |
| 10:07:26 | gibi | Uggla, bauzas: I'm -1 on https://review.opendev.org/c/openstack/os-traits/+/832769 | |
| 10:08:17 | bauzas | gibi: excellent point | |
| 10:08:27 | gibi | easy to fix :) | |
| 10:08:52 | opendevreview | Manuel Bentele proposed openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 10:11:24 | opendevreview | Manuel Bentele proposed openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 10:14:15 | frickler | sean-k-mooney: wow, you really get me wondering now why gerrit is sending me mails about a nova patch, which it usually doesn't. finding out that I reviewed it 5 years ago was ... interesting ;) | |
| 10:15:47 | opendevreview | Manuel Bentele proposed openstack/nova-specs master: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova-specs/+/849488 | |
| 10:56:45 | sean-k-mooney | frickler: i have a dashboard that i sometimes use when i want to find patches to review | |
| 10:57:06 | sean-k-mooney | i went througyh some of hte small ones this moringin then looked for ones with one +2 that were not in merge conflict | |
| 10:57:17 | sean-k-mooney | then looked at a few form my normal todo list | |
| 10:57:24 | sean-k-mooney | so ya some of those were old | |
| 10:58:11 | sean-k-mooney | https://review.opendev.org/dashboard/?foreach=%28+project%3Aopenstack%2Fnova+OR%0Aproject%3Aopenstack%2Fpython-novaclient+OR%0Aproject%3Aopenstack%2Fnova-specs+OR%0Aproject%3Aopenstack%2Fos-vif+OR%0Aproject%3Aopenstack%2Fos-traits+%29%0Astatus%3Aopen%0ANOT+owner%3Aself%0ANOT+label%3AWorkflow%3C%3D-1%0Alabel%3AVerified%3E%3D1%2Czuul%0ANOT+reviewedby%3Aself%0Abranch%3Amaster&tit | |
| 10:58:14 | sean-k-mooney | le=Nova+Review+Inbox&Small+patches=%28project%3Aopenstack%2Fnova+OR+project%3Aopenstack%2Fpython-novaclient+OR+project%3Aopenstack%2Fos-vif+OR+project%3Aopenstack%2Fos-traits%29+NOT+label%3ACode-Review%3E%3D2%2Cself+NOT+label%3ACode-Review%3C%3D-1%2Cnova-core+NOT+message%3A%22DNM%22+delta%3A%3C%3D10&Needs+final+%2B2=%28project%3Aopenstack%2Fnova+OR+project%3Aopenstack%2Fpython | |
| 10:58:16 | sean-k-mooney | -novaclient+OR+project%3Aopenstack%2Fos-vif+OR+project%3Aopenstack%2Fos-traits%29+NOT+label%3ACode-Review%3E%3D2%2Cself+label%3ACode-Review%3E%3D2+limit%3A50&Bug+fix%2C+Passed+Zuul%2C+No+Negative+Feedback=NOT+label%3ACode-Review%3E%3D2%2Cself+NOT+label%3ACode-Review%3C%3D-1%2Cnova-core+message%3A%22bug%3A+%22+limit%3A50&Wayward+Changes+%28Changes+with+no+code+review+in+the+las | |
| 10:58:18 | sean-k-mooney | t+two+days%29=NOT+label%3ACode-Review%3C%3D-1+NOT+label%3ACode-Review%3E%3D1+age%3A2d+limit%3A50&Needs+feedback+%28Changes+older+than+5+days+that+have+not+been+reviewed+by+anyone%29=NOT+label%3ACode-Review%3C%3D-1+NOT+label%3ACode-Review%3E%3D1+age%3A5d+limit%3A50&Passed+Zuul%2C+No+Negative+Feedback=NOT+label%3ACode-Review%3E%3D2+NOT+label%3ACode-Review%3C%3D-1+limit%3A50&Need | |
| 10:58:20 | sean-k-mooney | s+revisit+%28You+were+a+reviewer+but+haven%27t+voted+in+the+current+revision%29=reviewer%3Aself+limit%3A50&Specs=project%3Aopenstack%2Fnova-specs+status%3Aopen+limit%3A20 | |