| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-01 | |||
| 17:37:07 | auniyal | .zuul patch where `c-bak: false` is set | |
| 17:42:21 | dansmith | yeah | |
| 17:43:58 | auniyal | in here https://zuul.openstack.org/status#893540 - nova-ovs-hybrid-plug: failing and openstacksdk-functional-devstack: in-progress | |
| 17:44:06 | auniyal | unit and functional tests are passed | |
| 17:44:30 | auniyal | okay you meant deployment finished without issue | |
| 17:44:35 | auniyal | right ? | |
| 17:50:27 | elodilles | hi, nova team! fyi, train-eol patch has merged & the branches are deleted: https://paste.opendev.org/show/bVfeofByKRN0ZrNX779o/ | |
| 17:50:37 | auniyal | this is not a regular thing right - https://zuul.opendev.org/t/openstack/build/475dab49cd33437a96ba81437779b875/log/controller/logs/screen-n-cpu.txt#14515 | |
| 17:50:47 | dansmith | auniyal: yeah | |
| 17:52:09 | gmann | elodilles: thanks | |
| 17:52:17 | elodilles | gmann: np | |
| 21:05:16 | dansmith | gmann: this is a different failure: https://zuul.opendev.org/t/openstack/build/475dab49cd33437a96ba81437779b875 | |
| 21:05:35 | dansmith | the one we were trying to resolve was where it tried to start c-bak on the subnode and failed | |
| 21:06:00 | dansmith | oh, but, it's running an evacuate test on a bfv instance which clearly won't work, I see | |
| 21:06:09 | dansmith | so we need to remove that post action or whatever | |
| 21:06:43 | gmann | dansmith: I mean the failure it had before for evacuate test | |
| 21:07:03 | dansmith | okay gotcha | |
| 21:07:18 | gmann | yeah, or separate the test with cinder availability or so? | |
| 21:09:43 | opendevreview | Merged openstack/nova master: only attempt to clean up dangling bdm if cinder is installed https://review.opendev.org/c/openstack/nova/+/893502 | |
| 21:12:46 | dansmith | gmann: I don't think there's any reason we should be running that post test on the ovs job, I think it's just inherited and wasting time | |
| 21:13:47 | dansmith | I'll leave that to sean on monday | |
| 21:14:25 | gmann | as the nova-live-migration job) tests with the OVS network backend " | |
| 21:14:25 | gmann | dansmith: not sure, but it job definition explicitly says about it "Run move operations, reboot, and evacuation (via the same post-run hook | |
| 21:14:54 | gmann | I think we can avoid running bfv evacuate test if cinder disabled. | |
| 21:18:17 | dansmith | yep, that's what I meant and said in my comment | |
| 21:23:09 | gmann | yeah | |
| #openstack-nova - 2023-09-02 | |||
| 02:42:23 | opendevreview | Merged openstack/nova master: Add nova-manage ironic-compute-node-move https://review.opendev.org/c/openstack/nova/+/886989 | |
| 04:40:27 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | |
| 08:26:45 | opendevreview | Merged openstack/nova master: Make compute node rebalance safter https://review.opendev.org/c/openstack/nova/+/887151 | |
| 10:02:24 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds server show test method https://review.opendev.org/c/openstack/nova/+/893584 | |
| 10:02:25 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactored test_cross_cell_migrate https://review.opendev.org/c/openstack/nova/+/893586 | |
| 10:02:25 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactor volumeAttachment for regressions https://review.opendev.org/c/openstack/nova/+/893585 | |
| 10:02:26 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactor functional tests https://review.opendev.org/c/openstack/nova/+/893587 | |
| #openstack-nova - 2023-09-03 | |||
| 04:01:29 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | |
| 16:21:35 | frickler | gmann: https://review.opendev.org/c/openstack/nova/+/891228 can be abandoned I guess (/me doing open patch cleanup) | |
| 16:24:09 | gmann | frickler: done | |
| 16:25:05 | frickler | gmann: oh, that was fast ;) thx | |
| 16:25:25 | gmann | :) | |
| #openstack-nova - 2023-09-04 | |||
| 07:15:47 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds server show test method https://review.opendev.org/c/openstack/nova/+/893584 | |
| 07:15:48 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactor vol attachment api call for rest of the tests https://review.opendev.org/c/openstack/nova/+/893587 | |
| 07:15:48 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactor volumeAttachment for regressions https://review.opendev.org/c/openstack/nova/+/893585 | |
| 07:58:26 | frickler | I wonder whether it was really a good idea to bump libvirt requirements in the last minute before requirements freeze. this is affecting a lot of projects that still have tests running on Ubuntu 20.04. yes, they should have migrated their testing a long time ago, but it seems they didn't have the capacity to do so, and forcing this onto them now so shortly before the end of the cycle ... was this | |
| 07:58:32 | frickler | really necessary? | |
| 08:00:02 | bauzas | frickler: you're doing a point, we did hit the ground hard last cycle due to a very last dependency bump | |
| 08:01:45 | bauzas | https://governance.openstack.org/tc/reference/runtimes/2023.2.html | |
| 08:02:47 | frickler | I know that doc, but it doesn't say that projects must be incompatible to everything else | |
| 08:02:59 | bauzas | so, yeah, basically, those projects were lagging, but maybe the bump is quite unappropriate at this period in time of a release | |
| 08:03:27 | frickler | yes, doing the bump at the beginning of the cycle, I would have had no objections at all | |
| 08:03:31 | bauzas | frickler: I thought we had a long conversation last cycle with the TC about PTIs | |
| 08:06:00 | bauzas | this time, we were clear on the fact that Focal was out of support for a long time | |
| 08:06:20 | bauzas | but yeah, we lagged merging this patch | |
| 08:06:40 | bauzas | at least, projects have 1.5 weeks to remove their focal jobs before RC1... | |
| 08:06:47 | bauzas | (half-kidding) | |
| 09:09:51 | sahid | o/ | |
| 09:10:31 | sahid | quick question guys, is there a way to restrict the usage of a AZ for certain projects or users? | |
| 09:46:07 | admin1 | sahid, you can create an aggregate and filer it by tenant_id and add AggregateMultiTenancyIsolation for nova | |
| 09:51:43 | sean-k-mooney | frickler: it was ment to done last cycle and was intened to be done at milestone one this cycle | |
| 09:52:52 | sean-k-mooney | frickler: so while not ideal im pretty storongly agasint not bumping it again. it has been weel over due and we are just bumping it to what we actully use in ci. | |
| 09:54:42 | sean-k-mooney | i dont think it would have been reasonable to wait to calacal for a bump that was ment to be in antilope when it got punted form antelope because it was beign done late in the cycle. | |
| 09:56:35 | sahid | admin1: interesting point thank you | |
| 09:56:59 | sean-k-mooney | sahid: there is a way to do that wiht placement too instead of a filter | |
| 09:57:01 | sahid | if a situation where I have several tenant | |
| 09:57:14 | sahid | sean-k-mooney: how do you see that? | |
| 09:57:23 | sean-k-mooney | https://docs.openstack.org/nova/latest/admin/aggregates.html#tenant-isolation-with-placement | |
| 09:57:50 | sean-k-mooney | it works more or less thet same but placement enforces it instead of a post filter | |
| 09:58:25 | sean-k-mooney | which is imporant if you return a small set of canidates form placement like cern does (they limit to <20 allocation candiates) | |
| 10:01:04 | sahid | perfect, thanks a lot for your help ! | |
| 10:07:44 | sahid | sean-k-mooney: if I set a AZ for the aggregate that should avoid to manually set all the hosts related, right? | |
| 10:08:30 | admin1 | i think that is the idea from the examples | |
| 10:09:10 | sean-k-mooney | sahid: so often we tell people to not set any addtional metadta on the host aggreate that holds the AZ metadata | |
| 10:09:26 | sean-k-mooney | but you can and in this case that is proably what you want to do | |
| 10:09:43 | sean-k-mooney | if you want to resrict an az to a tenant and not have them ever get out of sync | |
| 10:10:21 | sahid | i'm glad it's exactly what I was needed :-) | |
| 10:10:26 | sahid | thanks you admin1, sean-k-mooney | |
| 12:49:06 | elodilles | bauzas auniyal : hi, i've updated the hash in the yoga release patch, please review it and then we can merge it: https://review.opendev.org/c/openstack/releases/+/889517 | |
| 12:49:17 | bauzas | cool cool | |
| 12:53:28 | elodilles | thanks o/ | |
| 13:25:43 | opendevreview | Merged openstack/nova master: [alembic] Alembic operations require keywords only arguments https://review.opendev.org/c/openstack/nova/+/883344 | |
| 13:49:05 | auniyal | thanks elodilles | |
| 13:52:14 | elodilles | np :) | |
| 14:24:31 | sean-k-mooney | hi folks what should i be setting oc_header too in vars.yaml? | |
| 14:26:19 | sean-k-mooney | oh wrong window... | |
| #openstack-nova - 2023-09-05 | |||
| 03:10:04 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | |
| 08:38:09 | bauzas | sean-k-mooney: can you review https://review.opendev.org/c/openstack/nova/+/893462 ? | |
| 08:49:55 | sean-k-mooney | i need to leave fo a doctors approintment soon but ill take a look at it today | |
| 08:57:11 | opendevreview | Merged openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | |
| 09:58:06 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 10:10:32 | plibeau | hello guys, I need your help on this change https://review.opendev.org/c/openstack/nova/+/861172 | |
| 12:21:41 | 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:27:26 | sean-k-mooney | dvo-plv: unfortunetlly we are past feature freeze fo this cycle | |
| 12:28:26 | 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:29:18 | sean-k-mooney | bauzas: unless we want to consider a FFE for ^ | |
| 12:29:28 | sean-k-mooney | i was +2 on it a month ago | |
| 12:29:42 | sean-k-mooney | it would have been nice to land that but we are now out of time | |
| 12:30:25 | 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:49 | 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:31:55 | 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:39:19 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 13:05:39 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 14:20:22 | opendevreview | Sylvain Bauza proposed openstack/nova master: doc: mark the maximum microversion for 2023.2 Bobcat https://review.opendev.org/c/openstack/nova/+/893742 | |