| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-07 | |||
| 17:10:59 | gmann | yes it was before and that is why it was not useful so we separated that out from check_str | |
| 17:11:10 | sean-k-mooney | ack | |
| 17:11:31 | sean-k-mooney | so i woudl not say its not use to have multiple systems | |
| 17:11:49 | sean-k-mooney | but we might not want to do it via the check sting | |
| 17:12:46 | sean-k-mooney | eventually i think it would to be abel to issue a token that was readonly on say neutron but could do nothing on glance | |
| 17:13:43 | sean-k-mooney | im not sure how to model that but that was the orginal usecase for system:all vs system:compute | |
| 17:14:44 | sean-k-mooney | when i say token i really am thinking of app creditials by the way | |
| 17:15:21 | sean-k-mooney | it woudl be nice ot be abel to generate an app credential that hand much more fine grained scope | |
| 17:15:40 | gmann | yeah, with no scope coupled in check_str we cannot have SYSTEM_READER until we enable scope by default and make it non-configurable | |
| 17:16:20 | sean-k-mooney | any way that proably a differnt topic then you wanted feedback on | |
| 17:17:03 | sean-k-mooney | one day it woudl benice if openstack coudl support the same aplication keys like funcationaltiy you get with github or other modern apis | |
| 17:20:35 | sean-k-mooney | https://docs.github.com/en/rest/overview/permissions-required-for-github-apps | |
| 17:21:01 | sean-k-mooney | they map all api endpoints endpoint to a permssiosn which is like a role | |
| 17:21:36 | sean-k-mooney | and then you create a topen with a set of permmison and either read or write capablity on each | |
| 17:22:10 | sean-k-mooney | some day it woudl benice if you coudl do the same with keystone app credetials or just in gneral | |
| 17:25:52 | gmann | yeah, may be good to have it separate as it can make things more complex to understand | |
| 17:26:29 | gmann | gibi_summit: bauzas I created a central etehrpad to get RBAC feedback from various forum/places. and linked it in nova etherpad also, please use that https://etherpad.opendev.org/p/rbac-operator-feedback | |
| 23:15:07 | opendevreview | Takashi Kajinami proposed openstack/placement master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/placement/+/845059 | |
| #openstack-nova - 2022-06-08 | |||
| 01:43:54 | opendevreview | Ghanshyam proposed openstack/placement master: Fix py310 unit test job https://review.opendev.org/c/openstack/placement/+/845074 | |
| 02:32:59 | opendevreview | Merged openstack/placement master: Update python testing as per zed cycle testing runtime https://review.opendev.org/c/openstack/placement/+/845059 | |
| 13:04:36 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host (compute_api part) https://review.opendev.org/c/openstack/nova/+/831507 | |
| 13:04:37 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host (REST API part) https://review.opendev.org/c/openstack/nova/+/845129 | |
| 13:44:54 | Uggla | sean-k-mooney, if you can find time to review ^ patches, it would be cool. | |
| 14:03:19 | artom | sean-k-mooney, instance.pcpuset is the *host* CPU IDs for that instance? | |
| 14:13:02 | sean-k-mooney | i think so i woudl have too double check | |
| 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 | |