| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-19 | |||
| 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 | |
| 18:47:02 | sean-k-mooney | so im not really sure how to look up the glance iamge | |
| 18:47:33 | dansmith | sean-k-mooney: right, that's a problem for the day-2 case, but I haven't even gotten there... the day zero case is hard enough | |
| 18:47:50 | dansmith | like "assume a blank compute node, how do we *start* recording and enforcing formats" | |
| 18:47:55 | dansmith | figure out the migration of existing stuff later | |
| 18:48:01 | melwitt | sean-k-mooney: yeah it's a fingerprint not the image id (name in the cache) | |
| 18:48:10 | dansmith | right | |
| 18:48:48 | sean-k-mooney | ack i honestly have not looked at this closely enough to say of the top of my head. i was hoping we woudl have some of the image metadata stored like the format in the instastance_system_metadata or somethign like that | |
| 18:49:12 | melwitt | oh yeah, we do have all the image properties stored in instance sysmeta | |
| 18:49:14 | sean-k-mooney | that or the bdms. ill need to load contex tbefore ill be of any help | |
| 18:49:20 | melwitt | not sure if disk_format is always part of that | |
| 18:49:25 | dansmith | I think that's too weak, because we won't have that for things like a rescue image | |
| 18:49:36 | melwitt | yeah, true won't have for rescue | |
| 18:49:44 | dansmith | like, if we're going to make this holistic, it has to be more than just the root disk of an instance during boot | |
| 18:50:09 | dansmith | it would be bad if you could pollute the image cache by doing a rescue to a bad image and then a subsequent normal boot, etc | |
| 18:50:32 | sean-k-mooney | dansmith: well we will likely need to modify the image cache api to pass the format but im not really sure form where other then when we are downloadign the image | |
| 18:50:42 | dansmith | and, we won't know which images in the cache were/are for rescue or anything.. I think we have to get to a point where it's something we can rely on, else we will always have the legacy path | |
| 18:50:51 | dansmith | sean-k-mooney: right, that | |
| 18:50:55 | dansmith | is my point though, | |
| 18:51:00 | dansmith | we | |
| 18:51:10 | dansmith | have already decided what the dest filename will be, before we even fetch it | |
| 18:51:20 | dansmith | and changing that reaches *deep* backwards into the driver | |
| 18:51:35 | dansmith | it's sort of ... insane | |
| 18:51:50 | sean-k-mooney | ok then the alternitive is to write a new info file becied the orgianl with the format as the content | |
| 18:51:55 | melwitt | yeah, it is very disjointed | |
| 18:52:24 | sean-k-mooney | as ub _base/backing_file _base/backing_file.info | |
| 18:52:31 | sean-k-mooney | *as in | |