| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-08 | |||
| 14:17:27 | artom | Weird, because looking at the libvirt driver code... | |
| 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:19 | artom | https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L8429 | |
| 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 | is special for shared vCPU then. | |
| 14:22:52 | artom | vCPUs of the instance NUMA cell, and the original ``InstanceNUMACell.cpuset`` | |
| 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: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: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | |
| 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: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:23 | opendevreview | melanie witt proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| 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: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: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: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: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 | |
| 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 | |
| 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 | sean-k-mooney | ok so i could either turn off the profileing or install it right | |
| 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:44:01 | sean-k-mooney | local with my ansible role stuff | |
| 14:44:19 | dansmith | yeah just set MYSQL_GATHER_PERFORMANCE=false | |
| 14:44:21 | dansmith | or whatever the variable is | |
| 14:44:30 | sean-k-mooney | i have not updted the verion of the devstack roles im using in a while so i can look into it | |
| 14:44:36 | sean-k-mooney | ok ill just disable it for now thanks | |
| 14:44:46 | dansmith | lib/databases/mysql: if [[ "$MYSQL_GATHER_PERFORMANCE" == "True" ]]; then | |
| 14:59:52 | sean-k-mooney | cool that worked | |
| 15:01:14 | sean-k-mooney | now i need to unstack and let ansible do it again to maksure it works end to end. | |
| 17:46:38 | erlon | hi nova folks, is there a policy for when tags are created for stable release branches? | |
| 18:50:50 | melwitt | erlon: the general policy is described here https://docs.openstack.org/project-team-guide/stable-branches.html#releases noting that branches in EM (Extended Maintenance) https://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance will no longer be released | |
| 18:52:09 | melwitt | releases are requested by proposing a patch to the openstack/releases repo https://releases.openstack.org/reference/using.html#requesting-a-release and such patches must be acked by the PTL and/or release liasion (bauzas and elodilles) | |