| Posted | Nick | Remark | |
|---|---|---|---|
| #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 APIs https://review.opendev.org/c/openstack/nova/+/924585 | |
| 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: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: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: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: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: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 console auth token APIs https://review.opendev.org/c/openstack/nova/+/924592 | |
| 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: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: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: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: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: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: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: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 | |
| 18:53:23 | sean-k-mooney | if we cant modify the filename but know what it is we can comptue the relitive name | |
| 18:53:42 | dansmith | the problem is that if we do that, we need to start copying it during migration | |
| 18:54:06 | sean-k-mooney | only if the image is deleted form glance | |
| 18:54:08 | dansmith | I also was hoping to make it a lot more obvious when we missed one | |
| 18:54:17 | sean-k-mooney | otherwise it would be download on the dest node instead of copying | |
| 18:54:18 | sean-k-mooney | but yes | |
| 18:54:51 | dansmith | right, you can see that hole right in front.. upload a bad image, boot, delete it, then migrate to exploit :) | |
| 18:54:56 | sean-k-mooney | am im just finsihing for today but ill sleep on it and take a look at the code tomrrow and see if anythign jump out at me | |
| 18:55:07 | dansmith | ack | |
| 18:55:41 | melwitt | dansmith: the fetch and disk image creation happen in Image.cache() which is called here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L11500 could you maybe return the file format from the fetch func and then pass it into create_image to tack on the extension to the name before writing it? | |
| 18:55:58 | dansmith | melwitt: I think that's only some of the time | |
| 18:56:11 | sean-k-mooney | the only other optiopn that jumps otu at me at the moment is to use the a singel file for all the format info or a db like pythons dbm or sqlite modules | |
| 18:56:16 | dansmith | for ami,ari,aki we fetch direct I think | |
| 18:56:19 | sean-k-mooney | which i assume you dont wnat to do at least right now | |
| 18:56:43 | dansmith | and I thought there were also some of those backend drivers that raise and cause us to take the fetch direct path again, like on boot from snapshot or unshelve or something? | |
| 18:56:55 | sean-k-mooney | i.e. you want this to be local to the compute node and obvious form simple inspection fo the filesytem without special tools | |
| 18:57:07 | dansmith | sean-k-mooney: multiple files that get deleted with their parent seems cleanest if we're going that route | |
| 18:57:16 | sean-k-mooney | and it also needs to work when /var/lib/nova is on nfs | |
| 18:57:20 | dansmith | there is also the concern... yeah NFS | |
| 18:57:28 | dansmith | multiple files make locking less reliable | |
| 18:59:07 | dansmith | melwitt: I was initially going to return format from fetch_as_raw (and add the lookup/return to fetch) yeah | |
| 18:59:23 | dansmith | however we'd need another move there in try_fetch_image_cache() which feels wrong, | |