| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-17 | |||
| 14:32:41 | sean-k-mooney | before completing the soft delete action correct | |
| 14:33:16 | sean-k-mooney | e.g. when the compute comes back up it shoudl see the vm was evacuated | |
| 14:33:21 | sean-k-mooney | or if it was jsut restored | |
| 14:33:32 | gibi | there was no evacuation in this sequence | |
| 14:33:34 | sean-k-mooney | tehn it would see that its been restored in the db | |
| 14:33:46 | sean-k-mooney | so it would need to hanel that | |
| 14:34:48 | gibi | today restore sets the power state back to running. the db only restore would set it to shutoff? | |
| 14:36:13 | gibi | anyhow I have to run | |
| 14:36:35 | gibi | my days are soo random, I don't feel productive | |
| 14:37:35 | noonedeadpunk | https://bugs.launchpad.net/nova/+bug/1932326 | |
| 14:46:50 | sean-k-mooney | noonedeadpunk: thanks | |
| 14:47:21 | sean-k-mooney | noonedeadpunk: just going to triage this quickly how impactful is to you production wise | |
| 14:47:47 | sean-k-mooney | im leaning towards medium or low since there is no data lose but there is a workload outage | |
| 14:48:02 | sean-k-mooney | noonedeadpunk: i.e. you can just fix it by starting the vm again | |
| 14:48:45 | noonedeadpunk | I'd say it's closer to medium I guess, because as for public cloud provider it's hard to explain why customer VM wents down in a day after previous outage | |
| 14:49:00 | noonedeadpunk | and you can start it when you own vm or monitor it | |
| 14:49:52 | noonedeadpunk | but if it's not your VM it's hard to even know that it went down | |
| 14:50:17 | sean-k-mooney | yep | |
| 14:50:19 | noonedeadpunk | As current workaround we will probably attempt to flush queue for compute that went down... | |
| 14:50:33 | noonedeadpunk | but it's so nasty imo | |
| 14:50:35 | sean-k-mooney | more then likely it will cause the custoemr to notice a failure of the vm restore it and file a ticket | |
| 14:50:48 | noonedeadpunk | that's exactly what has happened :) | |
| 14:51:37 | noonedeadpunk | that's pretty much a corner case though as well | |
| 14:51:55 | sean-k-mooney | noonedeadpunk: so if we allow evac in the powering-off state restoring it to shutdown would make sense right | |
| 14:51:59 | sean-k-mooney | rather then active | |
| 14:52:11 | noonedeadpunk | yes, totally | |
| 14:52:29 | noonedeadpunk | and powering-on to active :) | |
| 14:52:54 | sean-k-mooney | we had disscced doing that for vms in suspend and pause so including powering-off in that list i think is consitent | |
| 14:52:57 | noonedeadpunk | (but it's harder to imagine happening) | |
| 14:53:24 | sean-k-mooney | powering-on to active would also make sense | |
| 14:53:32 | sean-k-mooney | well i dont know | |
| 14:53:55 | sean-k-mooney | if i was a custoemr and my vm sudenly stoped working i might do a start to see if that fixes it | |
| 14:54:12 | noonedeadpunk | I think it will appear as active | |
| 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 | |