| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-19 | |||
| 10:23:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924427 | |
| 10:23:20 | bauzas | sean-k-mooney: gibi: patch updated | |
| 10:23:34 | bauzas | once merged, I'll provide the backports to Antelope | |
| 10:37:33 | sean-k-mooney | the previous results were green and the change is trivial so +2 | |
| 10:45:34 | gibi | bauzas: thanks +A | |
| 11:32:32 | bauzas | tkajinam: cores, fwiw os-traits release patch https://review.opendev.org/c/openstack/releases/+/924492 | |
| 11:56:17 | stephenfin | gmann: Lemme know when you're around. I'm not sure I understand what you mean in https://review.opendev.org/c/openstack/nova/+/915738 | |
| 11:56:29 | opendevreview | Stefan Hoffmann proposed openstack/nova master: wait for ovn network at migration https://review.opendev.org/c/openstack/nova/+/924500 | |
| 12:00:25 | songwenping_ | sean-k-mooney: hi, the cmd 'openstack server migration show f4078cee-71c0-4a47-a0e6-76b67ab625fa 845cca1c-ec99-43bd-a7e8-ce588c0e1dbf' raise exception 'In-progress live migration ddf01c0f-d0f3-469e-a320-ebcb6732420e is not found for server f4078cee-71c0-4a47-a0e6-76b67ab625fa.' | |
| 12:00:37 | songwenping_ | the migration status is failed | |
| 12:01:07 | songwenping_ | so we cannot get the detail of failed migration? | |
| 12:01:54 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Clarify '[api] response_validation help' text https://review.opendev.org/c/openstack/nova/+/924501 | |
| 13:48:16 | atmark | hello, we have bunch of windows VMs that are boot from volume and got impacted by crowdstrike incident today. As per doc, `Rescuing a volume-backed instance is not supported with this mode`. Is there any method to rescue an bfv instance? | |
| 14:13:07 | sean-k-mooney | atmark: it depend on the release | |
| 14:15:19 | sean-k-mooney | rescuse with boot form voluem was added in ussuri https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-bfv-instance-rescue.html | |
| 14:22:43 | atmark | sean-k-mooney: thought it wasn't supported. i'm on xena | |
| 14:26:23 | sean-k-mooney | you need to use this with the stable rescue feature i think but its been supproted for several years now | |
| 14:32:57 | atmark | ok, i can't get it to work. I uploaded with cirros image with this property `--property hw_rescue_device=cdrom` | |
| 14:33:57 | atmark | then I tried `openstack --os-compute-api-version 2.87 server rescue --image e0b042a3-ad6a-4f6e-ad5d-f6f150b5287e a11283a0-55bb-46cb-b0be-b8c3df79a6cf` | |
| 14:34:19 | atmark | it returns ` cannot be rescued: Cannot rescue a volume-backed instance HTTP 404` | |
| 14:34:30 | atmark | s/404/400 | |
| 14:34:49 | sean-k-mooney | you need an newer api micorversion and you need to set other image properites on the image | |
| 14:49:27 | opendevreview | Merged openstack/nova master: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924427 | |
| 14:53:54 | sean-k-mooney | atmark: https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#maximum-in-ussuri-and-victoria | |
| 14:54:04 | sean-k-mooney | you need micro version 2.87 | |
| 14:55:10 | sean-k-mooney | and you should set hw_rescue_bus i recommend hw_rescue_bus=usb | |
| 14:56:22 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2024.1: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924514 | |
| 14:57:54 | sean-k-mooney | atmark: these are the tempest test in case that helps https://github.com/openstack/tempest/blob/1af21705c53bc9911ea467eaeee2bc12489a43ed/tempest/api/compute/servers/test_server_rescue.py#L259-L310 | |
| 15:05:21 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2023.2: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924517 | |
| 15:08:37 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2023.1: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924518 | |
| 15:32:58 | gibi | stephenfin: what would be the way to get an openstack sdk client not based on the service user in the config but based on the user token from a nova context? I tried naivly https://paste.opendev.org/show/bnuA5WLIZvlJGR4cSHUt/ but it fails with `Jul 19 15:29:53 gibi-devstack-aio-jammy devstack@n-api.service[1814794]: ERROR nova.api.openstack.wsgi openstack.exceptions.NotSupported: The | |
| 15:33:04 | gibi | shared-file-system service for :None exists but does not have any supported versions. | |
| 15:33:07 | gibi | ` | |
| 15:34:45 | gibi | for the nova - manila integration we try to use the sdk to talk to manila but we want to make sure that the user who request attaching a manial share from nova has the creds to read the share in the manila api | |
| 16:57:47 | atmark | sean-k-mooney: Thanks. Got it working now. I set hw_rescue_device=cdrom and hw_rescue_bus=ide for ISO. Upstream doc https://docs.openstack.org/nova/latest/user/rescue.html#stable-device-instance-rescue says `or` instead of `and` https://docs.openstack.org/nova/latest/user/rescue.html#stable-device-instance-rescue | |
| #openstack-nova - 2024-07-20 | |||
| 00:49:29 | gmann | stephenfin: +2 now, sorry I missed to check that you added empty schema only for those GET Action APIs which is not actually query schema but can be counted as action schema only so all good https://review.opendev.org/c/openstack/nova/+/915738/6/nova/api/openstack/compute/schemas/hosts.py | |
| 04:40:27 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 04:40:28 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | |
| 04:40:28 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | |
| 04:40:29 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | |
| 04:40:29 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | |
| 04:56:39 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 04:56:40 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | |
| 04:56:40 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | |
| 04:56:41 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | |
| 04:56:41 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | |
| 08:50:34 | opendevreview | Merged openstack/nova master: api: Add remaining missing query parameter schema https://review.opendev.org/c/openstack/nova/+/915738 | |
| 09:13:13 | opendevreview | Merged openstack/nova master: tests: Ensure all APIs have a request query schema https://review.opendev.org/c/openstack/nova/+/915739 | |
| #openstack-nova - 2024-07-21 | |||
| 13:14:22 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| 13:20:52 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| 13:26:52 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| 15:05:36 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Follow up for "libvirt: AMD SEV-ES support" https://review.opendev.org/c/openstack/nova-specs/+/924563 | |
| #openstack-nova - 2024-07-22 | |||
| 05:25:49 | frickler | stephenfin: how much of https://review.opendev.org/q/topic:%22remove-wsgi_scripts%22 do you think can we get still done this cycle? I have little hope for cinder, but placement could be fine if someone reviews https://review.opendev.org/c/openstack/placement/+/919569 | |
| 07:16:41 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Correct info about volume-backed server rebuild https://review.opendev.org/c/openstack/nova/+/924574 | |
| 09:40:33 | opendevreview | Sylvain Bauza proposed openstack/nova stable/2024.1: cpu: Only check governor type on online cores https://review.opendev.org/c/openstack/nova/+/924514 | |
| 10:53:48 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for host aggregate actions API https://review.opendev.org/c/openstack/nova/+/924584 | |
| 10:53:48 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for host aggregate APIs https://review.opendev.org/c/openstack/nova/+/924585 | |
| 10:53:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for extensions API https://review.opendev.org/c/openstack/nova/+/924586 | |
| 10:53:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for versions APIs https://review.opendev.org/c/openstack/nova/+/924587 | |
| 10:53:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | |
| 10:53:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for port interface APIs https://review.opendev.org/c/openstack/nova/+/924589 | |
| 10:53:51 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for availability zone APIs https://review.opendev.org/c/openstack/nova/+/924590 | |
| 10:53:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for bare metal node APIs https://review.opendev.org/c/openstack/nova/+/924591 | |
| 10:53:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for console auth token APIs https://review.opendev.org/c/openstack/nova/+/924592 | |
| 10:53:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor access API https://review.opendev.org/c/openstack/nova/+/924593 | |
| 10:53:54 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor extra specs APIs https://review.opendev.org/c/openstack/nova/+/924594 | |
| 10:53:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavors APIs https://review.opendev.org/c/openstack/nova/+/924595 | |
| 10:53:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP pool APIs https://review.opendev.org/c/openstack/nova/+/924596 | |
| 10:53:58 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add contributor docs for response body validation https://review.opendev.org/c/openstack/nova/+/924597 | |
| 10:53:58 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 10:54:00 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: tests: Temporarily ignore missing schemas for removed APIs https://review.opendev.org/c/openstack/nova/+/924599 | |
| 10:54:49 | stephenfin | frickler: There's nothing technically challenging about any of them: it's simply a case of getting them reviewed and merged | |
| 10:57:21 | stephenfin | gibi: You could try calling 'connect_as' on the connection object? That doesn't accept a context object (yet) so you'll need to set the various fields manually but | |
| 10:58:21 | stephenfin | ...that shouldn't be too difficult to do | |
| 10:58:38 | stephenfin | gtema might have a better idea on #openstack-sdks | |
| 11:04:14 | sean-k-mooney | frickler: stephenfin i reviewd the placement patch only the first of the two is requried and can merge this cycle | |
| 11:33:20 | gibi | stephenfin: thanks I will try and jump over to the sdk channel if needed. | |
| 18:41:13 | dansmith | melwitt: have you given any thought to how we might record/enforce the disk_format for the things in our _base directory? | |
| 18:41:30 | dansmith | with the oslo stuff simmering I'm trying to figure out what the simplest path to do that is | |
| 18:41:42 | dansmith | and I'm pretty mired down in all the crazy indirection | |
| 18:42:18 | dansmith | at this point I'm kinda thinking that making imagecache.get_cache_fname() ping glance for every call to get disk_format will be the easiest, but that's sort of insane | |
| 18:42:51 | dansmith | I was thinking doing it in fetch(_to_raw)() would be doable, but the depth from which we pass the destination fname is pretty deep | |
| 18:42:59 | dansmith | (or sean-k-mooney ^) | |
| 18:43:58 | melwitt | dansmith: did you change your mind about the "add a file extension to the name" idea or find some issue about it? | |
| 18:44:20 | dansmith | melwitt: no, that's what I'm trying to implement, but it's not as easy as I was hoping | |
| 18:45:04 | dansmith | I guess maybe doing the "database on disk | |
| 18:45:09 | dansmith | might help with the overhead of pinging glance, | |
| 18:45:17 | sean-k-mooney | so for exitng images in the cache that are used for backign files we cant rename them as it would break runing domian | |
| 18:45:24 | dansmith | but we need to make sure we clean that up, and consult it for existing/copied images, etc | |
| 18:45:32 | dansmith | sean-k-mooney: right I know | |
| 18:45:33 | melwitt | dansmith: oh, I see. you could ping for disk_format the first time and then cache it maybe? | |
| 18:45:36 | sean-k-mooney | i was thinkign we would need to check all of the in use images in init host | |
| 18:45:45 | sean-k-mooney | and create a hard/soft link with the extention | |
| 18:45:46 | dansmith | melwitt: sure, but that makes it (much) heavier | |
| 18:46:10 | dansmith | sean-k-mooney: yeah that's how we could handle the existing ones, | |
| 18:46:22 | dansmith | I'm just looking at how to make the first-run case even work | |
| 18:46:40 | dansmith | because we sort of pass the target fname from deep inside libvirt driver, | |
| 18:46:43 | sean-k-mooney | the problem with that is i think the file name is a md5 of the uuid + the caasity | |
| 18:46:48 | sean-k-mooney | althoguh i coudl be wrong | |
| 18:46:51 | dansmith | and when we do, we have recorded only the image id and not the type, etc | |