| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-28 | |||
| 14:18:00 | bauzas | so, I left my comment but i need to get my kids from school, ttyl | |
| 14:33:07 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 14:57:05 | gibi | bauzas, sean-k-mooney: you were +2 on the Placement RP re-parenting spec, could you also look at the implementation https://review.opendev.org/c/openstack/placement/+/784020 ? | |
| 14:57:21 | bauzas | surelyu | |
| 14:57:24 | sean-k-mooney | yes i can do that | |
| 14:57:36 | bauzas | I tho have a meeting in 2 mins... so tomorrow morning, your dog :p | |
| 14:58:15 | gibi | bauzas: thanks :) | |
| 14:58:17 | gibi | sean-k-mooney: thanks | |
| 14:58:56 | gibi | also on placement side there is a quick small patch to add pps resources to os-resource-classes https://review.opendev.org/c/openstack/os-resource-classes/+/796591 | |
| 15:29:44 | bauzas | +2d on the last one | |
| 15:33:15 | gibi | bauzas: thanks | |
| 15:36:04 | opendevreview | Elod Illes proposed openstack/nova stable/pike: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/798345 | |
| 16:32:07 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Validate host belongs to availability zone https://review.opendev.org/c/openstack/nova/+/798145 | |
| 19:19:35 | opendevreview | Merged openstack/nova stable/ussuri: [CI] Fix gate by using zuulv3 live migration and grenade jobs https://review.opendev.org/c/openstack/nova/+/795432 | |
| 19:23:53 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 19:30:38 | ganso | gibi, bauzas: if you have a spare minute to please look at this backport that is ready for an extra +2 and +W: https://review.opendev.org/c/openstack/nova/+/796719 | |
| 20:36:07 | tosky | melwitt, elodilles: yay, now time to update the train backport of the legacy cleanup and the CI should be (almost) fine | |
| #openstack-nova - 2021-06-29 | |||
| 02:14:52 | opendevreview | Merged openstack/nova stable/ussuri: Remove broken legacy zuul jobs https://review.opendev.org/c/openstack/nova/+/795374 | |
| 07:03:48 | gibi | ganso: I have no +2 rights on stable branches | |
| 08:03:02 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 08:03:02 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 08:03:04 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | |
| 08:03:04 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | |
| 08:03:05 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 08:03:05 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 08:03:06 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 08:32:44 | lyarwood | sean-k-mooney: https://review.opendev.org/c/openstack/devstack/+/798514 btw | |
| 08:33:42 | lyarwood | https://bugs.launchpad.net/devstack/+bug/1933096 moved to devstack | |
| 08:35:43 | opendevreview | Lee Yarwood proposed openstack/nova master: zuul: Add CentOS 8 stream integrated compute tempest job to gate https://review.opendev.org/c/openstack/nova/+/797616 | |
| 08:42:16 | lyarwood | gibi: nova-live-migration doesn't look happy on master | |
| 08:43:11 | lyarwood | Call _is_port_status_active returns false in 60.000000 seconds | |
| 08:43:16 | lyarwood | test_live_migration_with_trunk | |
| 08:56:06 | opendevreview | Lee Yarwood proposed openstack/nova master: zuul: Skip test_live_migration_with_trunk until bug #1933954 is fixed https://review.opendev.org/c/openstack/nova/+/798580 | |
| 09:31:48 | stephenfin | sean-k-mooney: gibi: bauzas: When you're all around, I'd like to pick up discussion on that availability zone issue again (I got stuck in meetings after lunch yesterday) | |
| 09:32:24 | bauzas | stephenfin: I'm working on some A100 vGPU test... :( | |
| 09:32:38 | bauzas | urgent query from some PM | |
| 09:32:57 | stephenfin | ah, no worries, I guess I can keep it to the Gerrit review and let you respond async | |
| 09:33:28 | stephenfin | tl;dr: I still have concerns about recording null or the default AZ when the host doesn't belong to the availability zone | |
| 09:35:23 | stephenfin | I'd be okay with overwriting the requested AZ with the hosts AZ (with a warning) if we insist on not blocking the request | |
| 09:37:06 | lyarwood | https://bugs.launchpad.net/nova/+bug/1933954 and https://bugs.launchpad.net/nova/+bug/1933958 smell related to me if anyone with more neutron context has time to review | |
| 09:39:39 | bauzas | stephenfin: that's probably why we should only record None | |
| 09:39:48 | bauzas | and not the default AZ | |
| 09:40:09 | bauzas | stephenfin: so, letting the instance to be movable between AZs | |
| 09:41:12 | bauzas | stephenfin: if operators want the instance to *not* be movable between AZs, they should use both --az and --host | |
| 09:46:23 | bauzas | stephenfin: replied on PS1 https://review.opendev.org/c/openstack/nova/+/798145 | |
| 09:48:27 | rohit02 | hi team on any openstack ussuri we are getting error while launching multiattach volume booted instance on horizon"Multiattach volumes are only supported starting with compute API version 2.60. (HTTP 400) (Request-ID: req-fd275b1c-c827-460f-9005-ff9f3d02dbb6)" | |
| 09:50:32 | rohit02 | is a known issue for multiattach volume type for horizon? is there any fix for this issue | |
| 09:53:44 | lyarwood | Odd, no idea why Horizon wouldn't be using 2.latest tbh | |
| 09:54:01 | lyarwood | is it configurable in Horizon? | |
| 09:57:03 | stephenfin | lyarwood: Yeah, they look related. Looking at the logs for the first one (https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8f7/771362/28/check/nova-live-migration/8f76ccd/compute1/logs/screen-n-cpu.txt) I see: | |
| 09:57:12 | stephenfin | "Neutron is not new enough to perform early destination host port binding activation. Port bindings will be updated later." | |
| 09:58:01 | stephenfin | which only happens if neutron doesn't support port bindings, apparently https://github.com/openstack/nova/blob/master/nova/network/neutron.py#L2860-L2868 | |
| 09:58:43 | lyarwood | cool cool, I don't see anything obvious in nova or neutron that might be causing this | |
| 09:58:57 | lyarwood | guess it might be in devstack or tempest (for the job definitions) | |
| 10:00:10 | stephenfin | I'm looking at recent devstack changes as we speak | |
| 10:00:15 | stephenfin | I asked on #openstack-neutron too | |
| 10:01:52 | lyarwood | ta | |
| 10:03:12 | stephenfin | Merge "[ML2] Change way how list of supported API extensions is made" | |
| 10:03:14 | stephenfin | that looks relevant | |
| 10:03:17 | stephenfin | (from neutron) | |
| 10:05:50 | lyarwood | gah I was looking at author and not the commit date | |
| 10:07:36 | stephenfin | git config :q | |
| 10:07:39 | stephenfin | whoops | |
| 10:08:38 | stephenfin | git config format.pretty fuller # <-- if you don't have it, very helpful | |
| 10:08:53 | lyarwood | wonder if that works with tig | |
| 10:09:57 | lyarwood | nope, nvm I'll just engage my brain next time | |
| 10:10:20 | stephenfin | ||
| 10:10:23 | stephenfin | lyarwood: fyi ^ | |
| 10:10:34 | stephenfin | (from #openstack-neutron) | |
| 10:10:40 | lyarwood | Coolio | |
| 10:15:28 | opendevreview | Stephen Finucane proposed openstack/nova master: DNM: Testing ML2 extension aliases fix https://review.opendev.org/c/openstack/nova/+/798635 | |
| 10:18:21 | opendevreview | Stephen Finucane proposed openstack/nova master: DNM: Testing ML2 extension aliases fix https://review.opendev.org/c/openstack/nova/+/798635 | |
| 10:31:50 | kashyap | lyarwood: Hi, is this soft-lockup w/ CirrOS consistently reproducible? - https://bugs.launchpad.net/nova/+bug/1931702 | |
| 10:33:00 | lyarwood | kashyap: hard to say, I had to hack dumping the console log into a few runs of the job before we disabled certain tests that always failed, I've not had time to rerun things with that change reverted yet | |
| 10:34:16 | kashyap | Hm; I saw your comment on the console log. | |
| 10:34:45 | lyarwood | kashyap: https://review.opendev.org/c/openstack/tempest/+/794757 and https://review.opendev.org/c/openstack/nova/+/795997 FWIW | |
| 10:38:07 | kashyap | lyarwood: Thanks for the rework; and the skip makes sense for now. To guess a non-OpenStack reproducer from the test: | |
| 10:39:13 | kashyap | lyarwood: Live-migrating a guest, its disk, with an additional file-based disk should do it? | |
| 10:39:34 | lyarwood | kashyap: yeah pretty much | |
| 10:40:02 | kashyap | lyarwood: Okay; thanks. I can also ask the virt QE to add a test to this effect to this suite. As this is a common path for OpenStack | |
| 11:13:44 | gibi | kashyap: do you want to talk about https://blueprints.launchpad.net/nova/+spec/virtio-as-default-display-device on the todays Nova meeting? | |
| 11:23:34 | kashyap | gibi: Oh, hi. Yes, that'd be good | |
| 11:23:50 | gibi | then I will add it to the agenda | |
| 11:23:51 | kashyap | gibi: Thanks for priming my memory | |
| 11:26:19 | gibi | no problem | |
| 11:43:20 | gibi | stephenfin: I'm here until the top of the hour to talk about the availability zone check. Alternatively I'm here from 15:30 CEST till the nova meeting | |
| 11:47:28 | gibi | stephenfin: bauzas replyed in the review and I think that was how I understood the the original agreement. so --az az:host should be translated to an internal behavior to match the --host host behavior and log a warning if that host is not in the az | |
| 11:49:27 | kashyap | gibi: Afraid, I might not be there for the entire meeting, as I have to run for an errand; I see it's 18:00 CET (1600 UTC) | |
| 11:50:11 | gibi | kashyap: I can move the topic a earlier in the agend, what is your cut of time_ | |
| 11:50:14 | gibi | ? | |
| 11:50:46 | kashyap | I can be around the first 20 mins | |
| 11:50:53 | kashyap | Thank you, as usual for adjusting. I feel guilty :D | |
| 11:51:30 | gibi | kashyap: OK, I will move it. Don't worry | |
| 11:52:58 | kashyap | Thx! | |
| 12:53:03 | bauzas | gibi: again, even if we do the same, the main difference between the az hack and the --host value is that for the former, we don't verify the AZ | |
| 12:53:44 | gibi | bauzas: but for the --host we also not verify any az as az was not even provided | |
| 12:53:45 | bauzas | that's why I would like to continue to not verify it, but saying that the instance couldn't be stuck to a specific AZ | |
| 12:54:01 | bauzas | gibi: ah, correct indeed | |
| 12:54:03 | gibi | that work sof me | |
| 12:54:08 | gibi | works for me | |