| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-01 | |||
| 17:08:32 | gmann | perfect, it should work now. will keep eyes on it | |
| 17:08:43 | dansmith | thanks gmann ;) | |
| 17:08:47 | sean-k-mooney | its slightly overkill but ya it will | |
| 17:09:16 | sean-k-mooney | the sub node only has cinder volume and not swift bits | |
| 17:09:25 | sean-k-mooney | but it does not hurt to be explcit | |
| 17:09:31 | dansmith | it clearly had c-bak | |
| 17:09:50 | sean-k-mooney | really i tought that was only on the contoler | |
| 17:09:57 | dansmith | that was the failure | |
| 17:10:11 | sean-k-mooney | in anycase it does not hurt to disabel them all explictly | |
| 17:10:23 | dansmith | c-bak was trying to start and didn't find the cinder database because it wasn't setup on the main node | |
| 17:10:38 | sean-k-mooney | ack we might have moved that at some point | |
| 17:11:53 | gmann | yeah, we do not enable swift in subnode by default but never know which child job doing might be doing/will do that so disable explicitly is better | |
| 17:12:59 | sean-k-mooney | ok im going to go mostly afk. i will skim the logs later tonight but otherwise ill cat to ye on tuesday? its a long weekend in the US right | |
| 17:13:06 | dansmith | yup | |
| 17:13:12 | sean-k-mooney | cool well enjoy | |
| 17:13:26 | dansmith | thanks, o/ | |
| 17:14:06 | gmann | sean-k-mooney: o/ enjoy weekend | |
| 17:33:01 | dansmith | auniyal: devstack finished on the subnode this time | |
| 17:33:23 | auniyal | ack | |
| 17:34:33 | auniyal | job "openstacksdk-functional-devstack" is still running | |
| 17:35:07 | dansmith | I meant on the top patch to change the job | |
| 17:35:30 | dansmith | the one where we had to add the c-bak settings to the subnode | |
| 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 | 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:25 | gmann | as the nova-live-migration job) tests with the OVS network backend " | |
| 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: Refactor volumeAttachment for regressions https://review.opendev.org/c/openstack/nova/+/893585 | |
| 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: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 volumeAttachment for regressions https://review.opendev.org/c/openstack/nova/+/893585 | |
| 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: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 | |