| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-04 | |||
| 12:51:41 | sean-k-mooney | technially i just added all the video moeles that were not already supported when i did that change not just virtio but you could be able to just follow the same steps | |
| 12:55:22 | kashyap | sean-k-mooney: So, here we go, the specless-bp: https://blueprints.launchpad.net/nova/+spec/add-bochs-display-device | |
| 12:55:40 | sean-k-mooney | kashyap: do you need to add a test for uefi fo this model type | |
| 12:55:54 | sean-k-mooney | e.g. hw_fireware_type=uefi | |
| 12:56:24 | sean-k-mooney | we proably should assuming it wont work with a bios boot | |
| 12:57:05 | kashyap | sean-k-mooney: For BIOS boot, "this device might be useable as well" is what the QEMU dev writes | |
| 12:57:14 | sean-k-mooney | kashyap: ack on the blueprint that hsould be more or less enough we can disucss it at the meeting on tueday | |
| 12:57:23 | kashyap | Okido | |
| 12:57:36 | sean-k-mooney | kashyap: ok so we dont need to put a gard in place at the api | |
| 12:58:02 | sean-k-mooney | in valdiate_falvor_image_nostatus | |
| 12:58:03 | kashyap | sean-k-mooney: BTW, a silly question: what are the hashes based on, when bumping the ImageMetaPros? | |
| 12:58:45 | sean-k-mooney | they are computed based in the filed defintions of the the object by olso.versioned.objects | |
| 12:58:45 | kashyap | (When bumping it in unit/test_objects.py, i.e.) | |
| 12:59:01 | kashyap | I see, based on the string length, looks like md5 | |
| 12:59:17 | sean-k-mooney | that or sha1 proably | |
| 13:01:07 | sean-k-mooney | they are just used in tests so we can change the hash algoritiom freely provide we update the tests | |
| 13:01:22 | kashyap | sean-k-mooney: md5() for sure :-) -- | |
| 13:01:25 | kashyap | fingerprint = '%s-%s' % (obj_class.VERSION, md5( | |
| 13:01:31 | kashyap | (Source: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fixture.py#L274) | |
| 13:02:10 | sean-k-mooney | ah ya it is md5 | |
| 13:02:34 | sean-k-mooney | and it has usedforsecurity=Fals | |
| 13:02:45 | sean-k-mooney | so no issue with fips | |
| 13:05:01 | kashyap | sean-k-mooney: BTW, what would be be the full expanded trait here, for video models, when an operator configures it? | |
| 13:05:31 | sean-k-mooney | COMPUTE_GRAPHICS_MODEL_BOCHS | |
| 13:05:48 | sean-k-mooney | its in COMPUTE since its an atribute of what the hypervior can vituralise | |
| 13:06:04 | sean-k-mooney | HW_ is used for descibing atributes of the phsyical hardware | |
| 13:06:45 | kashyap | I see | |
| 13:06:57 | sean-k-mooney | so just add MODEL_BOCHS here https://github.com/openstack/os-traits/blob/master/os_traits/compute/graphics.py#L19 | |
| 13:07:19 | sean-k-mooney | like https://github.com/openstack/os-traits/commit/95668fdd59bf526277874ca50c4800ebd3aafb26 | |
| 13:08:46 | kashyap | sean-k-mooney: Yep; done. I recall also reworking the layout of CPU-related traits, which was more confusing before :D | |
| 13:09:50 | sean-k-mooney | common x86 ones are here https://github.com/openstack/os-traits/blob/master/os_traits/hw/cpu/x86/__init__.py | |
| 13:10:14 | sean-k-mooney | but we have vendor specific ones in there own module https://github.com/openstack/os-traits/blob/master/os_traits/hw/cpu/x86/intel.py https://github.com/openstack/os-traits/blob/master/os_traits/hw/cpu/x86/amd.py | |
| 13:11:29 | sean-k-mooney | im not sure why https://github.com/openstack/os-traits/blob/master/os_traits/hw/cpu/amd.py is here though | |
| 13:11:33 | kashyap | sean-k-mooney: Right; it was before haphazard; I reworked it in this commit: https://github.com/openstack/os-traits/commit/93d2ff782f | |
| 13:11:48 | sean-k-mooney | unless amd plan to support SEV on non x86 | |
| 13:12:21 | sean-k-mooney | oh right it dperecated | |
| 13:12:34 | kashyap | Yep; see the ASCII "diagram" in the commit message :) | |
| 13:13:37 | sean-k-mooney | so to not break the placemnt prefilter you will need to both have a depends on the os-triats patch and we will have to do an os traits release | |
| 13:14:00 | sean-k-mooney | kashyap: the tempest jobs will pull in the os-traits patch correctly | |
| 13:14:06 | kashyap | I see | |
| 13:14:08 | sean-k-mooney | but the upstream tox jobs wont | |
| 13:14:19 | sean-k-mooney | because of how things are installed int eh venv | |
| 13:14:31 | kashyap | sean-k-mooney: So is this the correct config for operator? — trait:COMPUTE_GRAPHICS_MODEL_BOCHS=true | |
| 13:14:46 | sean-k-mooney | no | |
| 13:14:55 | kashyap | Do correct | |
| 13:15:03 | sean-k-mooney | in the image they set hw_video_model=bochs | |
| 13:15:23 | sean-k-mooney | and the prefilter will generate trait:COMPUTE_GRAPHICS_MODEL_BOCHS=reuqired | |
| 13:15:28 | sean-k-mooney | in the placemnet query | |
| 13:15:55 | sean-k-mooney | kashyap: traits never change how an xml is generated directly | |
| 13:16:21 | kashyap | sean-k-mooney: Ah, right; I first wrote "required", and then wrote 'true' | |
| 13:16:43 | sean-k-mooney | adding trait:COMPUTE_GRAPHICS_MODEL_BOCHS=reuqired to the flavor for example woudl jsut ensure it laned on a host with supprot for it but we would still default to cirrus graphics | |
| 13:17:18 | opendevreview | Kashyap Chamarthy proposed openstack/os-traits master: Add a trait for the "bochs" display device https://review.opendev.org/c/openstack/os-traits/+/794807 | |
| 13:39:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 13:39:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 13:39:48 | kashyap | sean-k-mooney: By notification object, you mean this one, right? notification_samples/common_payloads/ImageMetaPropsPayload.json | |
| 13:40:14 | sean-k-mooney | well that the api sample | |
| 13:42:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 13:42:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 13:42:53 | sean-k-mooney | i mean this https://github.com/openstack/nova/blob/master/nova/notifications/objects/image.py#L22 | |
| 13:43:13 | sean-k-mooney | well actully https://github.com/openstack/nova/blob/master/nova/notifications/objects/image.py#L107 proably | |
| 13:43:40 | sean-k-mooney | you will need to bump this to 1.8 https://github.com/openstack/nova/blob/master/nova/notifications/objects/image.py#L127 | |
| 13:43:47 | sean-k-mooney | and add the version comment | |
| 13:46:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 13:49:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 13:51:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 13:53:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 13:55:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 14:19:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 14:20:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 14:21:31 | opendevreview | Victor Coutellier proposed openstack/nova master: Allow configuration of direct-snapshot feature https://review.opendev.org/c/openstack/nova/+/794837 | |
| 15:24:34 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Add generic mdevs to Nova https://review.opendev.org/c/openstack/nova-specs/+/792796 | |
| 15:52:10 | opendevreview | Merged openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 15:52:10 | opendevmeet | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to sean mooney (sean-k-mooney) | |
| 15:52:43 | opendevreview | Merged openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153 | |
| 16:28:51 | opendevreview | Ghanshyam proposed openstack/nova master: DNM: testing https://review.opendev.org/c/openstack/nova/+/794863 | |
| 17:00:12 | ganso | Looks like the stable/wallaby grenade CI job is misconfigured: https://6ba5c0d21853f43b6382-b0014becc5640974ea321328f0a307e3.ssl.cf2.rackcdn.com/794328/1/check/nova-grenade-multinode/79f8e0a/job-output.txt | |
| 18:45:37 | melwitt | ganso: we are working on it https://review.opendev.org/c/openstack/nova/+/794345 | |
| 19:02:47 | ganso | melwitt: thanks! | |
| 19:34:25 | opendevreview | Ade Lee proposed openstack/nova master: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/790519 | |
| #openstack-nova - 2021-06-05 | |||
| 22:53:28 | opendevreview | Ghanshyam proposed openstack/nova master: DNM: testing https://review.opendev.org/c/openstack/nova/+/794863 | |
| #openstack-nova - 2021-06-06 | |||
| 16:11:59 | opendevreview | Takashi Natsume proposed openstack/placement master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/placement/+/791971 | |
| 16:13:26 | opendevreview | Takashi Natsume proposed openstack/osc-placement master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/osc-placement/+/791972 | |
| #openstack-nova - 2021-06-07 | |||
| 00:31:31 | opendevreview | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 04:29:16 | opendevreview | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 07:24:41 | opendevreview | fionali proposed openstack/nova master: manager: Reduce unnecessary calls https://review.opendev.org/c/openstack/nova/+/795027 | |
| 07:49:23 | bauzas | good morning Nova | |
| 07:56:37 | lyarwood | Morning | |
| 08:00:17 | yonglihe | good morning | |
| 08:32:32 | swp20 | gibi, bauzas: morning. as Yumeng has changed company, she has no time working on the vgpu feature, and i am responsible for this work. please review the reproposed spec: https://review.opendev.org/c/openstack/nova-specs/+/780452 | |
| 08:33:07 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Do not destroy volume secrets during _hard_reboot https://review.opendev.org/c/openstack/nova/+/793463 | |
| 08:33:32 | swp20 | by the way, do you have a vgpu account to download virtual GPU software for Tesla V100, and could i use it? | |
| 08:41:21 | bauzas | swp20: ack, thanks and I'll do ;) | |
| 08:44:49 | swp20 | bauzas:thanks, could you please provide me some virtual GPU software for Tesla V100? | |
| 08:45:21 | bauzas | swp20: sorry, I can't :( | |
| 08:45:23 | swp20 | my account has expired, | |
| 08:45:44 | bauzas | swp20: nVidia is a partner for RedHat so we can use some software but I can't pass it to another company | |
| 08:45:46 | bauzas | :( | |
| 08:46:13 | swp20 | bauzas: ack, thanks. | |
| 08:46:27 | bauzas | swp20: again, apologies | |
| 08:46:57 | bauzas | swp20: but if I remember correctly, you could use some testing softawre | |