| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-27 | |||
| 15:27:22 | dansmith | bauzas: that's a connector object though | |
| 15:27:31 | bauzas | instead of using what looks to me an internal brick API | |
| 15:27:59 | bauzas | dansmith: doesn't it doing the same thing ? | |
| 15:28:01 | sean-k-mooney | bauzas that need us to constuct a connector which whil we could proably fake normlay need connection info form cinder | |
| 15:28:02 | dansmith | sean-k-mooney: okay that's fine, but.. don't we call brick things all over the place? are they all wrapped in privsep? | |
| 15:28:12 | bauzas | ah | |
| 15:28:14 | sean-k-mooney | dansmith: no | |
| 15:28:15 | dansmith | bauzas: we don't have a connector object though.. and we wouldn't here | |
| 15:28:30 | dansmith | sean-k-mooney: which no? :) | |
| 15:28:35 | sean-k-mooney | i was punting on that i just wanted a commnt to be added to know we need to clean this up in the future | |
| 15:28:41 | bauzas | I wish we had then a more abstract API to consume | |
| 15:29:04 | bauzas | but I'm cool with merging it as it is, provided we document it may break :) | |
| 15:29:05 | dansmith | sean-k-mooney: comment works :) | |
| 15:29:14 | bauzas | heh, jinxed | |
| 15:29:20 | sean-k-mooney | i was ok if not supper happy with multipath_running = linuxscsi.LinuxSCSI.is_multipath_running ... | |
| 15:29:36 | sean-k-mooney | a comment woudl be nice i just didnt want to ask for a respin for that | |
| 15:29:39 | dansmith | bauzas: I don't think that's a private brick api | |
| 15:29:41 | sean-k-mooney | on its own | |
| 15:30:03 | dansmith | bauzas: it's underscored in the connector usage, but it's api is not underscored | |
| 15:30:17 | bauzas | you're right | |
| 15:30:44 | bauzas | anyway, I'm cool with shipping as it is, hence my +1 but I'd appreciate if we could comment out that call | |
| 15:30:49 | bauzas | comment s/out// | |
| 15:31:31 | bauzas | (english and all your adverbs) | |
| 15:35:13 | sean-k-mooney | tkajinam: so tl;dr dansmith was right and if you adress there intial comemnts i think we are all ok with your patch | |
| 15:35:14 | dansmith | sean-k-mooney: just to circle back.. I'm not sure InvalidConfiguration should really inherit from Invalid, but raising Invalid itself defaults to "400 Bad Request" because that's really for HTTP Invalid.. I know he's overriding the message, | |
| 15:35:22 | dansmith | but it's just not what Invalid should be used for, IMHO | |
| 15:36:21 | sean-k-mooney | your right i was orgianly considerign if it shoudl be a RuntimeError but decied not to suggest that | |
| 15:36:40 | sean-k-mooney | partly because i was nto sure RuntimeError woudl abort startup | |
| 15:37:19 | dansmith | yeah, RuntimeError is interesting, but I tend to think that's more useful for things like "This shouldn't happen" .. programming errors more than something the operator could have done | |
| 15:37:28 | dansmith | anyway, it's all a bit pedantic anyway | |
| 15:38:28 | sean-k-mooney | to me that is what the second test really is, the config is valid but the runtime depency is not met. so either this should be some kind of custome excpetion or we use `InvalidConfiguration` adn agree that you were right before you confince me otherwise :) | |
| 15:38:42 | sean-k-mooney | s/confince/convince/ | |
| 15:54:49 | masahito | hi nova folks, could you review the API bug fix? https://review.opendev.org/c/openstack/nova/+/939658 It needs +W now. | |
| 16:47:59 | jizaymes | processed that correctly and added the ubuntu user ssh authorized key, but somehow the functionality within nova is now not working. no clear log events in the neutron-ovn-metadata service logs or nova logs. Recommendations? | |
| 16:47:59 | jizaymes | Hello. I made the mistake of trying to rebuild my linux images for openstack (2024.2) to add qemu-agent baked in -- that worked but couldn't SSH in. I then reverted my images to the vanilla ubuntu cloud images (tried 22 and 24, no customization.) In all cases now, cloud-init sets the IP and dns servers, etc, but no longer injects the ssh key for the ubuntu user. I ran the image separately on a kvm box with a cloud-init .iso I made and it | |
| 16:58:57 | opendevreview | Doug Szumski proposed openstack/nova master: Stop corrupting ephemeral volumes during cold migration https://review.opendev.org/c/openstack/nova/+/940900 | |
| 17:08:04 | jizaymes | seems to match this thread, but I am not sure what their resolution was https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/XPHA7B5D6SIZZPLXT7CLLXX5OHYPTYSW/#XPHA7B5D6SIZZPLXT7CLLXX5OHYPTYSW | |
| 17:13:41 | dougszu | sean-k-mooney: I've had a go at removing the ephemeral volume backing file in the above patch - it doesn't seem too bad. Any thoughts when you've got a moment would be great | |
| 17:21:19 | sean-k-mooney | dougszu: skiming it, ack on updating to also ensure new vms dont have backing files for ephmeeral, (it sould also be done for swap) today is feature freeze (upstream and a seocnd one downstream) so im kind of maxed out on reviews currently | |
| 17:21:45 | sean-k-mooney | dougszu: i can try and take a look at this next week dougszu feel free to ping me about it | |
| 17:22:04 | dougszu | many thanks! will do | |
| 17:22:54 | sean-k-mooney | live migration is harder so one appoch would be to do that in a diffent patch. | |
| 17:23:19 | sean-k-mooney | focus this on new vms (inculding unshelve) and cold migrate/rezise | |
| 17:23:30 | sean-k-mooney | if we flattened images on hard reboot | |
| 17:23:47 | sean-k-mooney | or agent restart that might also be an option | |
| 17:24:26 | sean-k-mooney | everngly all vms would not have backign files for ephmeral and swap and so live migrate woudl eventurally resolve | |
| 17:24:37 | sean-k-mooney | thats kind of a cop out but | |
| 17:24:46 | sean-k-mooney | incremental progress is still better then none | |
| 17:24:51 | dougszu | yeah - I've tested that, it's a path at least | |
| 17:24:59 | dougszu | thanks for the pointers | |
| 17:50:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 17:50:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Adjust validation helpers for a single-method future https://review.opendev.org/c/openstack/nova/+/936365 | |
| 17:50:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 17:50:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only check minimum API version https://review.opendev.org/c/openstack/nova/+/940325 | |
| 17:50:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 17:50:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 17:50:20 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 17:50:20 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 17:50:21 | opendevreview | Stephen Finucane proposed openstack/nova master: api: project/tenant and user IDs are not UUIDs https://review.opendev.org/c/openstack/nova/+/940369 | |
| 17:50:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 17:50:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 17:50:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 17:50:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 17:50:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 17:50:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 17:50:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for keypairs APIs https://review.opendev.org/c/openstack/nova/+/940115 | |
| 17:50:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remote consoles https://review.opendev.org/c/openstack/nova/+/940114 | |
| 17:50:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server topology API https://review.opendev.org/c/openstack/nova/+/940300 | |
| 17:50:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for image metadata APIs https://review.opendev.org/c/openstack/nova/+/940299 | |
| 17:50:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server external events API https://review.opendev.org/c/openstack/nova/+/940508 | |
| 17:50:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server diagnostics API https://review.opendev.org/c/openstack/nova/+/940506 | |
| 18:06:40 | bauzas | gmann: https://review.opendev.org/c/openstack/nova/+/942875 is a bugfix, so it won't be impacted by the FF | |
| 18:09:37 | gmann | bauzas: yeah, bug fix for documentation only and not in the code. At least it is to match the code change (done intentionally or unintentionally ) with documentation unless anyone object not to have any change in update server API response which is not discussed in the spec or original implementation. | |
| 18:10:25 | gmann | bauzas: but we want to change update server response also which is done in the original code change and in my change I am matching the documentation (api-ref, tests, releasenotes etc) accordingly | |
| 18:11:01 | gmann | bauzas: ditto for the https://review.opendev.org/c/openstack/nova/+/942875/2 also | |
| 18:11:26 | gmann | I found it when reviewing the 2.100 microversion https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:11:58 | gmann | also fixing the tempest test/schema also for those microversion https://review.opendev.org/c/openstack/tempest/+/942870 | |
| 18:12:39 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.96 for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942875 | |
| 18:12:52 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.98 doc/tests for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942878 | |
| 18:15:04 | bauzas | nova-cores, I eventually give an extra 3 days (until Thursday) for merging implementation that already got one +2 | |
| 18:15:32 | bauzas | please see https://etherpad.opendev.org/p/nova-2025.1-status for the current state of exceptions | |
| 18:17:06 | gibi | until Tuesday or Thursday? | |
| 18:17:46 | gmann | yeah, I am also holding this to approve in case FFE things here https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:18:04 | gmann | bauzas: can you write the date also along with Tuesday/day so that it will be clear | |
| 18:18:25 | gmann | I saw this and not clear which tuesday https://etherpad.opendev.org/p/nova-2025.1-status#L73 | |
| 18:18:45 | gmann | sorry this one https://etherpad.opendev.org/p/nova-2025.1-status#L84 | |
| 18:18:56 | gmann | 'show-scheduler-hints-in-server-details' | |
| 18:31:33 | gmann | bauzas: added these two microversion doc bug fixes detail in etherpad under 'Feature Complete With Follow Up Pending' https://etherpad.opendev.org/p/nova-2025.1-status | |
| 18:35:28 | sean-k-mooney | gmann: feature freeze is today | |
| 18:35:32 | sean-k-mooney | not tuesday | |
| 18:36:12 | sean-k-mooney | which is why i was hopign we could proceed with https://review.opendev.org/c/openstack/nova/+/938604 | |
| 18:36:20 | gmann | sean-k-mooney ok its today right not next thursday? | |
| 18:36:46 | sean-k-mooney | ill check again | |
| 18:36:50 | sean-k-mooney | maybe im off by a week | |
| 18:36:58 | gmann | days are confusing there, having date will be more clear | |
| 18:37:05 | sean-k-mooney | https://releases.openstack.org/epoxy/schedule.html#epoxy-3-milestone | |
| 18:37:12 | sean-k-mooney | February 27, 2025 i | |
| 18:37:15 | sean-k-mooney | so today | |
| 18:37:16 | gmann | I asked bauzas just to be more clear | |
| 18:37:23 | stephenfin | Uggla: fyi, you probably want to respin https://review.opendev.org/c/openstack/openstacksdk/+/880056 soon enough if that's intended for this release | |