| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-01 | |||
| 17:03:24 | dansmith | sean-k-mooney: I've got https://review.opendev.org/c/openstack/neutron/+/893538 for that verification | |
| 17:03:52 | sean-k-mooney | cool well i was just checking back to see if there were any issues | |
| 17:03:57 | opendevreview | Amit Uniyal proposed openstack/nova master: only attempt to clean up dangling bdm if cinder is installed https://review.opendev.org/c/openstack/nova/+/893502 | |
| 17:03:58 | opendevreview | Amit Uniyal proposed openstack/nova master: zuul update https://review.opendev.org/c/openstack/nova/+/893540 | |
| 17:04:27 | sean-k-mooney | auniyal: why did you rebase those? | |
| 17:04:35 | dansmith | sean-k-mooney: to fix the commit message | |
| 17:04:51 | dansmith | sean-k-mooney: if we can figure out why it's not working in devstack, I'll fast-approve that job change or get gmann to help me, it just doesn't work as you had it | |
| 17:05:09 | dansmith | since it was part of the approved initial patch | |
| 17:05:13 | sean-k-mooney | it can wait till monday so if its not workign ill look at it then | |
| 17:05:23 | dansmith | ack | |
| 17:05:30 | gmann | dansmith: sean-k-mooney auniyal this is multinode job and we need to disable the cinder in subnode also | |
| 17:05:42 | dansmith | ahh | |
| 17:05:44 | sean-k-mooney | oh right | |
| 17:05:47 | gmann | neutron might be having example of those | |
| 17:06:05 | gmann | like this one https://github.com/openstack/neutron/blob/ed6023c3473585b4af6980404e7fe36ea152f980/zuul.d/tempest-multinode.yaml#L151 | |
| 17:06:18 | sean-k-mooney | its trivial | |
| 17:06:19 | dansmith | yeah | |
| 17:06:26 | sean-k-mooney | it just need to be in teh group vars section | |
| 17:06:27 | dansmith | I'll fix it | |
| 17:07:32 | opendevreview | Dan Smith proposed openstack/nova master: zuul update https://review.opendev.org/c/openstack/nova/+/893540 | |
| 17:08:09 | opendevreview | Dan Smith proposed openstack/nova master: Make our nova-ovs-hybrid-plug job omit cinder https://review.opendev.org/c/openstack/nova/+/893540 | |
| 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 | 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 | |