| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-08 | |||
| 14:13:36 | sean-k-mooney | for a pinned instance that is | |
| 14:14:04 | sean-k-mooney | and it would be in the instance numa toploloy object | |
| 14:14:31 | sean-k-mooney | here https://github.com/openstack/nova/blob/master/nova/objects/instance_numa.py#L75= | |
| 14:14:37 | sean-k-mooney | not directly on the instance object | |
| 14:15:39 | sean-k-mooney | cpu_pinning_raw i think is the dict that map logicl to host cpu | |
| 14:16:03 | sean-k-mooney | https://github.com/openstack/nova/commit/867d4471013bf6a70cd3e9e809daf80ea358df92 | |
| 14:16:46 | sean-k-mooney | so ya this is related to mixed cpus | |
| 14:16:59 | sean-k-mooney | so i think this is actully the logical guest cpus | |
| 14:17:04 | sean-k-mooney | not the host ones | |
| 14:17:15 | sean-k-mooney | its tracking which cpus are pinned | |
| 14:17:27 | artom | Weird, because looking at the libvirt driver code... | |
| 14:18:19 | artom | https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L8429 | |
| 14:18:19 | sean-k-mooney | it might be the host ones i just have not looke at this lately so dont have the context loaded right now | |
| 14:18:35 | artom | That looks like it's binary-AND'ig with the host available dedicated CPUs | |
| 14:20:14 | sean-k-mooney | this is in _get_host_numa_topology | |
| 14:20:27 | sean-k-mooney | so this is not using the instance numa toplogy | |
| 14:20:49 | sean-k-mooney | we are just using the same variable name i thikn | |
| 14:21:11 | artom | Ah right | |
| 14:22:52 | artom | OK, I guess the spec lays that to rest: Add the ``pcpuset`` field in ``InstanceNUMACell`` object to track the dedicated | |
| 14:22:52 | artom | vCPUs of the instance NUMA cell, and the original ``InstanceNUMACell.cpuset`` | |
| 14:22:52 | artom | is special for shared vCPU then. | |
| 14:23:21 | sean-k-mooney | yes | |
| 14:23:38 | sean-k-mooney | so this is trackign the bitmas of pinned vs floating cpus in the mixed case | |
| 14:24:03 | sean-k-mooney | if cpu policy = shared or dediceted only one of them wil have data | |
| 14:24:08 | sean-k-mooney | for mixed both do | |
| 14:24:19 | sean-k-mooney | and then the cpu_pinning_raw dict | |
| 14:24:27 | sean-k-mooney | maps the locgical cpu to a host cpu | |
| 16:44:12 | opendevreview | Sven Kieske proposed openstack/nova stable/yoga: libvirt: Add a workaround to skip compareCPU() on destination https://review.opendev.org/c/openstack/nova/+/845045 | |
| 16:53:53 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 18:57:41 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: DNM: Reproduction trial for bug 1975490 https://review.opendev.org/c/openstack/nova/+/845168 | |
| 21:33:07 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843959 | |
| 21:33:08 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843960 | |
| 21:33:09 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Make connection_info returned by CinderFixture unique per attachment https://review.opendev.org/c/openstack/nova/+/844744 | |
| 21:33:10 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 21:33:11 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| #openstack-nova - 2022-06-09 | |||
| 00:43:12 | opendevreview | Wenping Song proposed openstack/nova master: Fix the exception message for duplicate pgpu addresses https://review.opendev.org/c/openstack/nova/+/844716 | |
| 03:45:42 | opendevreview | Wenping Song proposed openstack/nova master: Fix the exception message for duplicate pgpu addresses https://review.opendev.org/c/openstack/nova/+/844716 | |
| 06:52:40 | opendevreview | Wenping Song proposed openstack/nova master: Add ut for one vgpu type with mulitple pci addresses https://review.opendev.org/c/openstack/nova/+/845227 | |
| 08:39:58 | bauzas | eek, both PTG and berlin feedback session etherpad got trampled by a chinese automatic translation tool | |
| 08:45:17 | opendevreview | Jorhson Deng proposed openstack/nova master: Clear the UPDATING_PASSWORD when compute restarted https://review.opendev.org/c/openstack/nova/+/845242 | |
| 11:25:25 | opendevreview | Pierre Riteau proposed openstack/nova master: Change TooOldComputeService upgrade check code to failure https://review.opendev.org/c/openstack/nova/+/845262 | |
| 12:57:35 | opendevreview | Merged openstack/placement stable/wallaby: Add periodic-stable-jobs template https://review.opendev.org/c/openstack/placement/+/843174 | |
| 13:24:43 | viks__ | Hi, i'm trying to have a check if the disk associated with the running instance is corrupt due to some reason using `qemu-img` with the below: | |
| 13:24:50 | viks__ | https://www.irccloud.com/pastebin/IZbgf13U/ | |
| 13:25:09 | viks__ | However if i copy the disk file to another location, then the `qemu-img` command works: | |
| 13:25:16 | viks__ | https://www.irccloud.com/pastebin/e55nPBeC/ | |
| 13:25:39 | viks__ | So is there any way i can do this check without copying to another location? | |
| 13:31:38 | mnaser | viks__: sounds like a vm is up and using that same file | |
| 13:31:49 | mnaser | also what you're doing is not supported :) | |
| 13:43:38 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 13:43:39 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 13:51:42 | viks__ | mnaser: Thanks... is there any other way i can check for disk corruption dynamically for live instances? | |
| 15:54:21 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 15:54:22 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 18:03:11 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 18:03:12 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 18:53:37 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 18:53:38 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 18:56:47 | opendevreview | Pierre Riteau proposed openstack/nova master: Change TooOldComputeService upgrade check code to failure https://review.opendev.org/c/openstack/nova/+/845262 | |
| 21:24:14 | opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/victoria: [stable-only] libvirt: UEFI: pc: skip OVMF_CODE.secboot.fd if possible https://review.opendev.org/c/openstack/nova/+/828979 | |
| 21:30:22 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 21:30:23 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/845300 | |
| #openstack-nova - 2022-06-10 | |||
| 00:45:55 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 00:45:56 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 04:53:06 | opendevreview | Steve Baker proposed openstack/nova master: Align ironic driver with libvirt secure boot enable https://review.opendev.org/c/openstack/nova/+/844243 | |
| 05:44:21 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | |
| 05:44:22 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | |
| 05:44:22 | opendevreview | melanie witt proposed openstack/nova master: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | |
| 05:44:23 | opendevreview | melanie witt proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| 05:44:23 | opendevreview | melanie witt proposed openstack/nova master: scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | |
| 05:44:24 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add DriverImageBlockDevice to block_device_info https://review.opendev.org/c/openstack/nova/+/826527 | |
| 05:44:24 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to image and ephemeral disks https://review.opendev.org/c/openstack/nova/+/826528 | |
| 05:44:25 | opendevreview | melanie witt proposed openstack/nova master: virt: Add block_device_info helper to find encrypted disks https://review.opendev.org/c/openstack/nova/+/826529 | |
| 05:44:25 | opendevreview | melanie witt proposed openstack/nova master: blockinfo: Add encryption details to the disk_info mappings when provided https://review.opendev.org/c/openstack/nova/+/772272 | |
| 05:44:27 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add disk_info_mapping as an optional attribute of Image https://review.opendev.org/c/openstack/nova/+/826530 | |
| 05:44:27 | opendevreview | melanie witt proposed openstack/nova master: privsep: Move qemu-img create calls under nova.privsep.qemu https://review.opendev.org/c/openstack/nova/+/826750 | |
| 07:12:25 | admin1 | hi | |
| 07:12:43 | admin1 | how are limits added on existing flavors activated ? | |
| 07:59:30 | jamespage_ | tobias-urdin: hiya - I noticed your comment about re-creating mdevs prior to nova-compute startup on https://review.opendev.org/c/openstack/nova/+/838976 and was interested in what you might have todo that | |
| 08:00:12 | jamespage_ | I can see that the way forward is to use persistent mdevs in libvirt but I'm trying to figure out a sane way to dealing with host reboots with attached vgpus for older releases of openstack + libvirt | |
| 08:26:13 | tobias-urdin | jamespage_: i can drop you what we have, it's not pretty but maybe it can help you forward | |
| 08:26:40 | jamespage_ | tobias-urdin: yes please :) | |
| 08:27:24 | tobias-urdin | we have systemd unit Type=oneshot that has Before=nova-compute.service that executes this https://paste.opendev.org/show/bwReCtkRa4eiIYa64AqN/ | |
| 08:27:32 | tobias-urdin | this is the remediate-dev code https://paste.opendev.org/show/bW91TyoqQCbFcPSTC3yT/ | |
| 08:29:16 | tobias-urdin | there is some edge cases in that, and a little nasty undefine so you might want read through it and verify it for your use-case | |
| 08:31:42 | jamespage_ | tobias-urdin: ack thanks - I'll have a good review | |
| 09:05:28 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove return from rpc cast https://review.opendev.org/c/openstack/nova/+/845335 | |
| 09:48:53 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 09:48:54 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 11:45:33 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove return from rpc cast https://review.opendev.org/c/openstack/nova/+/845335 | |
| 13:28:47 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: func: Add live migration rollback volume attachment tests https://review.opendev.org/c/openstack/nova/+/844745 | |
| 13:28:48 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 14:37:49 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844746 | |
| 14:42:42 | sean-k-mooney | dansmith: have you seen this before https://paste.opendev.org/show/bqi5a04gFzt0NE9SbHRV/ | |
| 14:43:13 | sean-k-mooney | is this related to the perf stuff you were doing by anychance or is this unrelated | |
| 14:43:21 | dansmith | sean-k-mooney: well, during debugging, but that means we didn't install the plugin in early devstack | |
| 14:43:25 | dansmith | sean-k-mooney: yeah it's related | |
| 14:43:46 | dansmith | sean-k-mooney: we shouldn't configure it to use the plugin if we didn't install it.. is this in a job or local? | |
| 14:43:46 | sean-k-mooney | ok so i could either turn off the profileing or install it right | |
| 14:44:01 | sean-k-mooney | local with my ansible role stuff | |