| opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | 01:15 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | 01:15 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 01:15 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 01:15 |
| *** mhen_ is now known as mhen | 01:19 | |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 02:01 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 02:01 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 02:31 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 02:31 |
| opendevreview | Friedrich Hiekel proposed openstack/nova master: Fix nova-scheduler placement error https://review.opendev.org/c/openstack/nova/+/951936 | 07:00 |
| opendevreview | Friedrich Hiekel proposed openstack/nova master: Fix nova-scheduler placement error https://review.opendev.org/c/openstack/nova/+/951936 | 07:46 |
| *** ykarel_ is now known as ykarel | 08:02 | |
| opendevreview | Florian proposed openstack/nova master: Add check for PCIe devices attach limit for volume and ports https://review.opendev.org/c/openstack/nova/+/955584 | 11:02 |
| opendevreview | Florian proposed openstack/nova master: Add check for PCIe devices attach limit for volume and ports https://review.opendev.org/c/openstack/nova/+/955584 | 11:04 |
| *** ykarel_ is now known as ykarel | 11:12 | |
| *** ykarel_ is now known as ykarel | 11:48 | |
| opendevreview | Florian proposed openstack/nova master: Add check for PCIe devices attach limit for volume and ports https://review.opendev.org/c/openstack/nova/+/955584 | 12:53 |
| opendevreview | Wesley Hershberger proposed openstack/nova stable/2024.2: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955310 | 15:27 |
| opendevreview | Wesley Hershberger proposed openstack/nova stable/2024.1: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955305 | 15:30 |
| opendevreview | Wesley Hershberger proposed openstack/nova stable/2024.2: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955310 | 16:38 |
| opendevreview | Wesley Hershberger proposed openstack/nova stable/2024.1: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955305 | 16:41 |
| opendevreview | Wesley Hershberger proposed openstack/nova stable/2024.1: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955305 | 16:42 |
| sean-k-mooney | dansmith: gibi i mentioned this a few day ago but before the end fo the release we probly need to add a license file to nova-contibe and decied if that shoudl be refactored into a actul python package or not | 17:00 |
| sean-k-mooney | the latter topic does not nessisarly need to be decied now | 17:00 |
| sean-k-mooney | but we should have a license file in the repo ideally before rc1 and figure out what the release model for that repo is ectra | 17:01 |
| sean-k-mooney | at the moment i dont think its possibel to use the release repo to actully do releases of it | 17:02 |
| dansmith | license file is fine, I think we should mark it as non-released (like os-test-images) | 17:02 |
| gibi | yeah I'm OK to add a licence file and make it non-released | 17:02 |
| sean-k-mooney | i dont really have a stong opion currently | 17:03 |
| sean-k-mooney | it might be nice to release it eventually but i kind fo wanted use to have a dicussion about what it should and shoudl not contains and docuemnt that somehere like in a readme file | 17:03 |
| sean-k-mooney | but we dont need to have that dicusssion now | 17:04 |
| melwitt | dansmith, gibi, sean-k-mooney: I expect this question will need context load time so no worries to answer right away but in the vTPM patches I have up for review, I did not do a service version bump. the reason is because I could not find how it is needed, | 19:11 |
| melwitt | new instances require compute traits that old computes will not have, so attempts to live migrate from new to old will fail with NoValidHost if there are no eligible new computes to go to | 19:12 |
| melwitt | were you thinking maybe of rejecting in the API if the min service version in the deployment is still old? to not allow live migration at all even to new computes when mid-upgrade? if so, then I do understand that | 19:13 |
| sean-k-mooney | i need to load context but it may have been related to how the traits usage is implemented | 19:14 |
| melwitt | I might be missing something totally obvious but so far I was not understanding the requirement | 19:14 |
| sean-k-mooney | im not sure we need a comptue service version bump | 19:14 |
| sean-k-mooney | but if you requet say user mode vs one of the other modes you need to be sure the host that the vm land on understands the new iamge metadata | 19:15 |
| sean-k-mooney | if we are doign that via traits uncondtionaly | 19:15 |
| sean-k-mooney | then we are prorbaly fine as you said | 19:15 |
| melwitt | yeah I kind of wondered if the service version bump was a discussion before the traits were decided on, so something like that. so I wanted to talk and find out if there is something I'm missing | 19:15 |
| melwitt | *or something like that | 19:15 |
| sean-k-mooney | i would honestly need to reload context by reading the spec | 19:16 |
| melwitt | yeah, np. I wasn't sure when to ask this because I know it is complicated | 19:16 |
| sean-k-mooney | i mean if i had recently read your patchs askign here woudl be fine but i have not looked at the vtpm stuff this cycle so this has all been paged out of my workign memory | 19:17 |
| melwitt | for now I'm working on more func tests and maybe some more whitebox coverage and maybe some plain tempest coverage | 19:17 |
| sean-k-mooney | melwitt: so based on yoru current undersdainding yoru not seeign a need for a bump is that correct | 19:18 |
| melwitt | yep 100% understand | 19:18 |
| melwitt | sean-k-mooney: that is my belief yes but I am aware I might be missing understanding | 19:18 |
| sean-k-mooney | i wonder if maybe it was related to the image show changes? | 19:19 |
| sean-k-mooney | did we ant to make sure tha tyou could see the policy or something | 19:19 |
| sean-k-mooney | that in epoxy so at this poitn we woudl not need it | 19:19 |
| melwitt | oh, hm, that's an angle I didn't think of | 19:19 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!