Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-05-20
16:35:43 Uggla elodilles, the floor is yours.
16:35:59 elodilles ack, thanks, not so much things this time
16:36:06 elodilles #info stable branches (stable/2025.1 and stable/2024.*) seem to be in OK state
16:36:11 elodilles #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci
16:36:17 elodilles that's all from me
16:36:27 elodilles Uggla: back to you
16:36:37 Uggla thx elodilles
16:36:52 elodilles np
16:37:10 Uggla #topic vmwareapi 3rd-party CI efforts Highlights
16:37:23 Uggla fwiesel, anything to say ?
16:37:25 fwiesel Hi, no updates from my side.
16:37:35 Uggla ok
16:37:44 Uggla #topic Gibi's news about eventlet removal.
16:37:50 Uggla #link Series: https://gibizer.github.io/categories/eventlet/
16:38:32 Uggla gibi, do you want to say something ? I think you are chasing for reviews.
16:38:45 gibi yepp
16:38:56 gibi the nova-scheduler series is ready for core review
16:39:10 gibi (I've added it to the status etherpad as well)
16:39:28 gibi the series starts here https://review.opendev.org/c/openstack/nova/+/947966
16:39:56 gibi the first 11 patches are ready to go, the last two depends on oslo.service to land and release the threading backend support
16:40:26 gibi that is it
16:40:56 Uggla cool thx gibi
16:41:05 Uggla #topic Open discussion
16:41:19 Uggla I can see topic on the agenda.
16:41:27 Uggla s/can/can't/
16:42:01 Uggla so unless if we have points, maybe we can try to triage bugs.
16:43:20 Uggla #topic Bug scrubbing
16:43:27 Uggla #link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L4
16:44:01 Uggla First one : https://bugs.launchpad.net/nova/+bug/2110222
16:45:07 Uggla Nova server image uploaded to S3 uses singlepart instead of multipart upload
16:45:26 gibi that feels like a feature request
16:45:37 Uggla yep I was about to say the same.
16:46:34 Uggla it is more an optimization request.
16:47:11 Uggla So I could probably set the bug to invalid and ask for a blueprint.
16:47:13 gibi it is a small feature even a specless one if we can assume that we can pass the size to the glance client at uploat
16:48:28 Uggla ok I'll do that for this one.
16:48:37 Uggla Next one: https://bugs.launchpad.net/nova/+bug/2109727 - removing an host from an aggregate should be accepted even if the host no longer exists
16:48:54 Uggla I think it is a valid one as opened by Sylvain
16:50:09 Uggla bauzas, ^
16:51:11 gibi I hope bauzas could push a functional reproducer for that
16:51:41 bauzas sorry was taking a break
16:52:02 bauzas not sure if I have time for the reproducer but I can try
16:52:39 sean-k-mooney bauzas: i guess that only matteres if your going to work on the bug
16:52:40 Uggla bauzas, at least this is something valid.
16:52:49 sean-k-mooney if not then the peroson who picks it up can
16:54:08 Uggla So I'll move the bug to valid.
16:54:16 Uggla Next one : https://bugs.launchpad.net/nova/+bug/2108974 - Keypairs lost during cross-cell resize in instance_extra
16:54:26 opendevreview Merged openstack/nova master: Revert "Support glance's new location API" https://review.opendev.org/c/openstack/nova/+/950336
16:55:42 gibi it has reproducer steps so somebody should run them and if reproducible mark it valid
16:56:29 Uggla Is resizing cross-cell supported ?
16:56:53 dansmith yeah
16:57:06 dansmith it uses shelve on the backend
16:57:22 dansmith and it has to copy between databases, so it might be missing something
16:58:46 gibi The config drive associated with the server, if there is one, will be re-generated on the destination host in the target cell. Therefore if the server was created with personality files they will be lost. However, this is no worse than evacuating a server that had a config drive when the source and destination compute host are not on shared storage or when shelve offloading and unshelving a server
16:58:52 gibi with a config drive. If necessary, the resized server can be rebuilt to regain the personality files.
16:59:07 gibi so maybe such config drive regen is the problem
16:59:25 gibi (the above is from https://docs.openstack.org/nova/latest/admin/configuration/cross-cell-resize.html#limitations )
16:59:59 dansmith ah yep
17:01:04 gibi the bug says it is lost form metadata response as well
17:01:29 Uggla yep I understand it is not only config drive.
17:01:49 gibi and it also points to `select keypairs from dst_cell.instance_extra ` I did not know we have a copy of a keypair there
17:02:01 gibi but it says it is in the source cell
17:02:09 gibi so that is probably not copied to the dest cell DB
17:02:16 gibi I vote that this is valid
17:02:47 Uggla I can try to reproduce, but I'm unsure how to easily deploy a multi cell env.
17:03:31 gibi we have multicell job we just need to see if we can enable some tempest there to catch this
17:04:59 sean-k-mooney thats in the sepc for what its porth
17:05:29 sean-k-mooney at least im 99% sure we called out that any files injected in the config drive would be lost with a cross cell resize
17:06:14 sean-k-mooney https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/cross-cell-resize.html#known-issues
17:06:16 sean-k-mooney limiation 3
17:06:30 sean-k-mooney Servers created with personality files, commonly known as file injection, that are resized across cells will lose the personality files since they are not persisted in the database
17:06:42 gibi sean-k-mooney: keypairs are not personality files
17:06:52 gibi afaik
17:06:56 dansmith correct
17:07:06 dansmith but they mentioned instance_extra
17:07:20 dansmith so if there's something we're not copying there and then regen'ing the configdrive...
17:07:44 sean-k-mooney oh it the keypari
17:07:55 sean-k-mooney ok that shoudl be copied and regenerated
17:08:13 sean-k-mooney i actully otught keyparis were in teh api db
17:08:33 sean-k-mooney not the cell db but i guess we embded them when we first create teh instance in the cell db?
17:08:40 dansmith keypairs themselves are I think (maybe) but the instance's attachment to them is in the main one of course
17:08:50 sean-k-mooney ack
17:09:49 sean-k-mooney im wondering if we could see this in ci. i think this is valid in anycase
17:10:12 sean-k-mooney we could create a repodcuer functional tests. in thoery tempest coudl detect this also
17:10:26 sean-k-mooney we just need to curl the metadata after the resize
17:10:32 sean-k-mooney or check the config drive i guess
17:10:49 sean-k-mooney presumable it workign now because the keypari is already in the authorized keys
17:11:24 Uggla sean-k-mooney, I'd like to try to do a reproducer.
17:11:30 gibi yepp we should be able to catch this in tempest
17:12:16 sean-k-mooney the workaround for now is just pass the keypari again on rebuild i guess
17:12:23 Uggla So if you all agree I will set this bug to valid and I'll try to do a reproducer. But I may need your help.
17:12:38 sean-k-mooney ack, sure shout if you hit issues
17:13:25 Uggla I will also update the bug with the workaround and remind limitation.
17:14:02 Uggla We are overtime, so I think we can wrap for today.
17:14:20 sean-k-mooney im not sure if we have multi cell functional tests today so that could be "fun" to make work
17:14:55 sean-k-mooney reproducing it in tempest will be simpler in this specific case.
17:15:10 sean-k-mooney oh we do https://github.com/openstack/nova/blob/master/nova/tests/functional/test_cross_cell_migrate.py
17:15:29 dansmith matt riedemann did that work, so I was *sure* there was functional testing for it :D
17:16:07 sean-k-mooney ack. getting diffent config to work in the fucntionla tests is still a bit fo a pain so i was unsure if we did or not
17:17:15 Uggla anything else ?

Earlier   Later