| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-17 | |||
| 14:54:19 | sean-k-mooney | it will yes | |
| 14:54:27 | noonedeadpunk | so you are able only to reboot or shutdown? | |
| 14:54:37 | sean-k-mooney | but i might not check and just do a start but ya i normlally woud do hard-reboot | |
| 14:55:14 | sean-k-mooney | its less likely but if we are adressing this we proably should go throug all the statees and just make them consitent/intuitive | |
| 14:55:28 | noonedeadpunk | I mean that you can't start already active instance - you will get same Conflict exception iirc | |
| 14:55:44 | sean-k-mooney | ah | |
| 14:55:51 | sean-k-mooney | yes proably since its in the state you want | |
| 14:55:56 | noonedeadpunk | yeah | |
| 14:57:13 | noonedeadpunk | so powering-on would be really unfortunate co-incidence that will affect most likely only CI toolings or dunno... | |
| 14:58:05 | sean-k-mooney | ya its much less likely | |
| 14:58:20 | melwitt | gibi, stephenfin: heya, I've updated the --task-log archive patch to address gibi's comments https://review.opendev.org/c/openstack/nova/+/780395 | |
| 14:58:39 | sean-k-mooney | we can reason about it though and come to a logic conclution for what it shoudl do so we proably shoudl just cover it | |
| 15:15:10 | stephenfin | melwitt: trying to backport https://review.opendev.org/c/openstack/nova/+/602432 at the moment (it's hell) but I'll hit that again before EOD, hopefully | |
| 15:16:16 | melwitt | stephenfin: ok np, and good luck | |
| 15:24:15 | kashyap | stephenfin: sean-k-mooney: NUMA-related: you might find it interesting - libvirt upstream is wiring up "HMAT" - which defines the different latencies and bandwidths b/n NUMA nodes: | |
| 15:24:19 | kashyap | [quote] | |
| 15:24:21 | kashyap | "Links between NUMA nodes can have different latencies and bandwidths. This info is newly defined in ACPI 6.2 under Heterogeneous Memory Attribute Table (HMAT) table. Linux kernel learned how to report these values under sysfs and thus we can expose them in our capabilities XML. The sysfs interface is documented in kernel's Documentation/admin-guide/mm/numaperf.rst." | |
| 15:24:27 | kashyap | [/quote] | |
| 15:24:58 | kashyap | This is called "NUMA interconnects", apparently: https://listman.redhat.com/archives/libvir-list/2021-June/msg00268.html | |
| 15:28:12 | sean-k-mooney | ill take a look | |
| 15:46:21 | sean-k-mooney | kashyap: that could be useful yes | |
| 15:46:56 | kashyap | Yep; noted | |
| 16:42:10 | opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 16:42:11 | opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: fixup! libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/796891 | |
| 16:43:21 | opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 16:50:29 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796262 | |
| 17:04:02 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: fixtures: Handle binding of first port https://review.opendev.org/c/openstack/nova/+/796905 | |
| 17:04:03 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Neutron fixture: don't clobber profile and vif_details if empty https://review.opendev.org/c/openstack/nova/+/796906 | |
| 17:04:04 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/c/openstack/nova/+/796907 | |
| 17:04:05 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/796908 | |
| 17:04:07 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/796909 | |
| 17:06:36 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796259 | |
| 17:06:37 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/796260 | |
| 17:06:38 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: Trival Change: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/796912 | |
| 17:14:51 | lyarwood | melwitt: https://review.opendev.org/c/openstack/nova/+/796626 - Would you mind taking a look at this today if you get a chance, stephenfin is looking to split out the cherry-pick.sh script from the pep8 job. | |
| 17:16:08 | melwitt | sure | |
| 17:16:46 | lyarwood | thanks | |
| 17:20:37 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796262 | |
| 17:20:38 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: Detach is broken for multi-attached fs-based volumes https://review.opendev.org/c/openstack/nova/+/796263 | |
| 17:20:39 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/796264 | |
| 17:20:40 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: Trival Change: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/796929 | |
| 17:22:26 | melwitt | stephenfin: does this empty deps = do something? https://review.opendev.org/c/openstack/nova/+/796626/2/tox.ini#87 | |
| 17:22:46 | sean-k-mooney | melwitt: i beleive it prevent use installing any deps | |
| 17:22:56 | melwitt | ok | |
| 17:23:10 | sean-k-mooney | we just need bash so this shoud make it slightly faster | |
| 17:23:17 | sean-k-mooney | sicne we ware just looking at the commit message | |
| 17:24:25 | melwitt | thanks | |
| 18:19:20 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796935 | |
| 18:19:21 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: Detach is broken for multi-attached fs-based volumes https://review.opendev.org/c/openstack/nova/+/796936 | |
| 18:19:22 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: Handle unset 'connection_info' https://review.opendev.org/c/openstack/nova/+/796937 | |
| 18:19:23 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: Trival Change: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/796938 | |
| 18:19:25 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/796939 | |
| 21:42:28 | opendevreview | Merged openstack/nova master: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/796626 | |
| #openstack-nova - 2021-06-18 | |||
| 02:47:33 | opendevreview | HYSong proposed openstack/nova master: fix vm deleted incorrectly when restoring failed https://review.opendev.org/c/openstack/nova/+/796985 | |
| 03:49:22 | opendevreview | wangzhenmeng proposed openstack/nova master: Three CPU parameters, mode, model and vendor_id, are added to flavor, some guest os do not support new CPU, so you need to specify a specific CPU when starting. https://review.opendev.org/c/openstack/nova/+/796986 | |
| 07:01:01 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 07:01:03 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 07:01:06 | opendevreview | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 07:01:09 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 07:01:11 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 07:07:03 | yonglihe | alex_xu, addressed comments for 'smartnic support'. | |
| 08:49:24 | kevinz | Hi Nova, would be really appreciated if you can help to review this patch for live migrate on Arm64: https://review.opendev.org/c/openstack/nova/+/763928, the patch has been updated | |
| 08:50:54 | opendevreview | melanie witt proposed openstack/placement master: Microversion 1.37: API support for consumer types https://review.opendev.org/c/openstack/placement/+/679441 | |
| 08:50:59 | opendevreview | melanie witt proposed openstack/placement master: Switch ConsumerType to use an AttributeCache https://review.opendev.org/c/openstack/placement/+/679486 | |
| 08:52:26 | melwitt | gibi: just added a whackadoodle func test for the race scenario ^. I'm off tomorrow and next week so I just wanted to push this for now. still have to fix the rollback logic and address your other comments, will do that week after next | |
| 08:53:28 | melwitt | feel free to reapply your -1 to flag it | |
| 08:55:04 | gibi | melwitt: ack, thank. have a nice time off | |
| 08:55:30 | melwitt | thanks o/ | |
| 08:55:36 | gibi | o/ | |
| 09:15:30 | stephenfin | gibi: I've addressed your comments on https://review.opendev.org/c/openstack/nova/+/786292/ if you have time today | |
| 09:48:19 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/797039 | |
| 09:49:54 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/797040 | |
| 10:24:23 | stephenfin | Is it just me, or does building lower-constraints locally take, like, 5-10 minutes to run? | |
| 10:24:31 | stephenfin | guess there's a lot of dependency resolution going on | |
| 10:26:19 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/797050 | |
| 10:27:18 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: Move 'check-cherry-picks' test to gate, n-v check https://review.opendev.org/c/openstack/nova/+/797052 | |
| 10:27:23 | lyarwood | can't say I've tried recently but I can give it a go now | |
| 10:28:39 | stephenfin | yeah, it's still ongoing here 10 minutes later | |
| 10:29:13 | gibi | stephenfin: it is slow to me too | |
| 10:30:51 | stephenfin | aaaand it failed because it's trying to run with the default python3 version, so 3.9.5 on my host :-( | |
| 10:31:09 | opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 10:32:49 | lyarwood | stephenfin: lol it failed in like 30 seconds for me because of 3.9 | |
| 10:33:15 | lyarwood | stephenfin: building greenlet right? | |
| 10:33:18 | stephenfin | yup | |
| 10:33:26 | stephenfin | I guess you had the packages locally already | |
| 10:34:19 | lyarwood | right so it isn't pip for you at least | |
| 10:34:33 | lyarwood | ah well I guess it does that before downloading | |
| 10:34:37 | lyarwood | so maybe it is | |
| 10:34:58 | gibi | yup greenlet for me too | |
| 10:35:13 | gibi | it take 5 minutes on my laptop to fail | |
| 10:36:21 | lyarwood | with 3.8 it takes ~30 seconds before tests start running | |
| 10:36:42 | opendevreview | Stephen Finucane proposed openstack/nova master: tox: Encode specific Python versions https://review.opendev.org/c/openstack/nova/+/797054 | |
| 10:36:44 | lyarwood | with a fresh .tox folder but lots cached on the host | |
| 10:36:47 | stephenfin | lyarwood: gibi: ^ | |
| 10:37:07 | lyarwood | stephenfin: I'm sure I suggested this in the past but gmann had reasons not to do it | |
| 10:37:13 | stephenfin | it'll be nice to be able to use the 'functional' env (vs. 'functional-py38') again | |
| 10:37:20 | lyarwood | aye | |
| 10:37:52 | lyarwood | https://governance.openstack.org/tc/reference/runtimes/xena.html tbh 3.9 isn't listed as a supported runtime anyway so we should really cap | |
| 10:37:56 | stephenfin | I don't see any disadvantage other than the busywork aspect I've noted | |
| 10:38:38 | stephenfin | we use 'py3' as a default env in oslo land but the oslo libs are far simpler with fewer dependencies likely to cause issues on other python versions | |