Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-06-18
10:29:36 sean-k-mooney gibi: i think nova-comptue with ironic/fake shoudl be a lot simpler
10:29:58 sean-k-mooney i cant say for vmware but libvirt driver is where i think most of the work will be
10:30:38 sean-k-mooney given it has all the extra logic for ceph and libvirt native thread and the ovsdb background connection ectra
10:31:21 sean-k-mooney the fake and ironic dirver have none of that so those might be able to prove out the core of the manager ectra first
10:31:38 sean-k-mooney but ya i can see it takeing more then 1 cycle
10:35:15 gibi sean-k-mooney: yeah it might make sense to do driver by driver. We can do an initial try with the fake driver very early
10:35:50 gibi I haven't looked at the ironic and other non libvirt drivers but I agree that they should be not too complex to handle.
10:36:10 gibi still libvirt driver is th most valuable
10:36:27 sean-k-mooney oh definetly
10:36:35 sean-k-mooney i was also talkign to JayF a littel
10:36:46 sean-k-mooney we deprecated peer_list
10:36:54 sean-k-mooney when we added the ironic shard support
10:37:00 sean-k-mooney but we have not deleted it yet
10:37:13 sean-k-mooney i think before we try to move the inronic driver we should nuke that code
10:37:48 sean-k-mooney because debuging moving compute nodes between compute serivce while converting to threaded is not something i want to wish on an enemy
10:38:00 sean-k-mooney let alone you or me for that matter
10:38:25 sean-k-mooney that is potirally something we could start on this cycle in adanvce if we had time
10:38:50 sean-k-mooney its pretty independet of the actual eventlet sereise and is mainly jsut removing code, docs and deps
10:49:25 gibi I'm not against removing code. I have no context about this actual removal but I'm happy to review the removal. Also if it seen as something that helps I can try to load context and do the removal myself but obvious reasons it would be easyer if the ironic folks propose the removal
10:55:24 sean-k-mooney gibi: no i have a lot on my plate right now in diffent context. some of those are going to hit spec freeze if they dont get reviewd so if that happens i might look at doing the removal
10:55:33 sean-k-mooney until we hit m2 i woudl not woryy about it
10:56:48 sean-k-mooney gibi: the wider context is we implemnge d ironic shardign supprot in caracal https://specs.openstack.org/openstack/nova-specs/specs/2024.1/implemented/ironic-shards.html so we are free to remove the old code which also allows us to remvoe taskflow as a dep in nova
10:59:52 gibi OK thanks for the pointr
12:30:02 gokhan__ Hi all, I have a question regarding attaching ISO images as CD-ROM devices to instances. I have an image in Glance with disk_format=iso and hw_cdrom_bus=ide set. I would like to attach it to a Windows instance as a CD-ROM so that Windows can see the contents (e.g. SQL Server setup files). I tried using --block-device with source_type=image, destination_type=local, device_type=cdrom, etc., but I get: "Mapping image to local is not supported".Is
12:30:02 gokhan__ there a supported way to attach a Glance ISO image as a CD-ROM device (non-bootable) to a running instance or at boot time? If not, is this a Nova limitation, or a backend/image format issue?Thanks in advance!
12:56:14 sean-k-mooney destination_type=local means do not use a cidner volume
12:56:36 sean-k-mooney and no there is not a way to do that today
12:57:36 sean-k-mooney wehn you create an instnace there is a way to add a cidner data volume created form an iso image in addtion too the boot disk, however there is no way to do it after the fact and its complex to do it a boot
12:58:34 sean-k-mooney currently there is no way to attch a cidner volume as a cdrom devicen so the only way to add an iso to a runnign vm is as a block device.
12:59:23 sean-k-mooney while its technically possibel to mount it in the guest with the right tools it wont work out of the box on say windows
13:00:18 sean-k-mooney gokhan__:you best bet would be to use rescue and boot from the iso do fix the issue and then unrescure to remove it
13:00:47 sean-k-mooney but if the iso is not a bootable image then thats not an option
13:01:26 sean-k-mooney in which case your only option would be ot use a normal cidner ovlume with a standar filesystem not and iso file system to sotre the data you want to preset to the runing guest
13:11:29 gokhan__ thanks sean-k-mooney , it seems the practical way is coply sql installation files to a cinder volume and using this. I tried also with virsh edit on compute node but it also didn't work.
14:59:04 Vii In OpenStack 2025.1, instance creation fails when there are two or more security groups with the same name in the same project. This occurs even if the security groups are not explicitly referenced during server creation (i.e., using only the default default security group).
14:59:10 Vii look: https://bugs.launchpad.net/nova/+bug/2114913
15:00:26 Vii creating an instance from horizon also fails
21:28:32 opendevreview Ghanshyam proposed openstack/nova master: Add Project Manager role context in unit tests https://review.opendev.org/c/openstack/nova/+/941056
#openstack-nova - 2025-06-19
09:41:03 jlejeune hello, I'm also impacted by that bug : https://bugs.launchpad.net/nova/+bug/2084238, do you know if someone begin to work on it ?
10:03:55 jlejeune I confirm that a request-id is set for new instances (created after migration to 2023.1) but that field stays at NULL for instances created before the migration
10:03:57 gibi jlejeune: I don't know about any work on that
10:04:30 jlejeune ok thanks, maybe it's something the migration tools should take care
10:05:04 jlejeune to fill random request-id in for allocated devices
10:12:50 gibi jlejeune: i don't have the time to dig into it right now. But I think this is a valid bug that needs to be fixed
14:10:44 opendevreview ribaudr proposed openstack/nova master: Reproducer for bug 2114951 https://review.opendev.org/c/openstack/nova/+/952894
14:10:44 opendevreview ribaudr proposed openstack/nova master: Fix bug 2114951 https://review.opendev.org/c/openstack/nova/+/952895
14:33:01 opendevreview Andriy Kurilin proposed openstack/nova master: Reuse 'detail:get_all_tenants' policy in server get api https://review.opendev.org/c/openstack/nova/+/952896
14:57:13 sean-k-mooney the request_id fucntionalty was added a very very long time ago
14:58:36 sean-k-mooney jlejeune: this feels like you skipped runnign an online data mighration
14:59:54 sean-k-mooney oh the one i was thinking of was a diffent fild https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L221
15:00:05 sean-k-mooney its the uuid on the device not the requst id
15:01:22 jlejeune sean-k-mooney: exactly
15:02:31 sean-k-mooney the request id should alwasy be set in recent version of openstack
15:03:13 sean-k-mooney it was added 11 years ago
15:03:54 elodilles sean-k-mooney: sorry for pinging you directly but you already reviewed the patch on master branch o:) could you please take a quick look/review on the stable/* backports of this patch? https://review.opendev.org/q/I56705bce8ee4354cd5cb1577a520c2d1c525f57b
15:03:59 jlejeune in recent version, yes it's filled, but not on stein release for example
15:04:31 sean-k-mooney jlejeune: it soudl have been there in stein too
15:04:49 jlejeune sean-k-mooney: hm
15:05:28 jlejeune it's not the case a t all
15:05:34 sean-k-mooney elodilles: sure
15:05:48 elodilles sean-k-mooney: o// thanks in advance!
15:09:13 sean-k-mooney the filed is nullable, when the reques is form a neuton port the request id is the neutron port uuid i belvie
15:09:17 jlejeune sean-k-mooney: indeed, the request_id field has been added 11 years ago
15:09:41 jlejeune I don't understand why it's null in my situation
15:09:42 sean-k-mooney when the request sorce is the flavor i belive it is expecte dto be null but i woudl have to dig itno this in detail
15:10:12 sean-k-mooney well there is request id and requeter_id
15:10:30 sean-k-mooney your speicificly asking about request.request_id)
15:11:39 sean-k-mooney the requester id is the one that hold the neutron port butalso backfil the request_id with a new uuid when we do that
15:13:37 sean-k-mooney jlejeune: is this a pci device related to a pci alis or a neutorn port
15:15:37 jlejeune to a pci alias
15:17:11 sean-k-mooney ack so the alisas was the older usage of this field as far as i recal which is whwere we woudl least expect to need ot back file
15:20:35 sean-k-mooney jlejeune: this is where it shoudl be generated https://github.com/openstack/nova/blob/64ca204c9cf497b0dcfff2d3a24b0dd795a57d1d/nova/pci/request.py#L261
15:20:56 sean-k-mooney https://github.com/openstack/nova/commit/ccab6fed463337c029459469c76e92af3b96fa06
15:28:36 jlejeune sean-k-mooney: ok, thanks for your commit, indeed I don't have backported it in my stein sources... that can explain
15:32:35 sean-k-mooney oh you did a feature backprot of the pic in placment code
15:32:49 sean-k-mooney ya this was new in zed
15:35:09 jlejeune ho there are a lot of missing commits: https://review.opendev.org/q/topic:%22bp/pci-device-tracking-in-placement%22
15:35:42 sean-k-mooney it was a non triival feature which is why we did nto backport it downstream in redhat
15:35:48 sean-k-mooney well not the only reaon
15:36:16 sean-k-mooney we are actully still n the process of fully graducatign it to full support in our antelope based product
15:36:49 sean-k-mooney while the final QE works is happeing its "tech preview" althoug it does work
15:45:15 sean-k-mooney gibi: stephenfin: if ye have time for a very short patch please review https://review.opendev.org/c/openstack/nova/+/952306 to enable the memballon optimisation when you have time
15:45:50 sean-k-mooney Uggla: and when you have time can you mark https://blueprints.launchpad.net/nova/+spec/automatic-memballoon-freeing as approved
15:46:26 Uggla sean-k-mooney sure.
15:54:09 opendevreview Merged openstack/nova stable/2025.1: [tool] Fix backport validator for non-SLURP https://review.opendev.org/c/openstack/nova/+/951968
16:07:13 Uggla sean-k-mooney I speak a bit too fast. I tracked your SLBP in the etherpad document as approved. But atm I can't set it in launchpad, I'm lacking rights to do it. So I'll set all approved SLBP as soon as I can.
16:11:21 gibi sean-k-mooney: is this memballon bp was approved? or you will bring it up for approval on the next nova meeting?
16:11:52 gibi my memories are vague
16:14:08 Uggla gibi, I approved it.
16:14:15 Uggla sorry I = we
16:18:49 gibi cool
16:18:51 gibi thanks
16:23:18 opendevreview Elod Illes proposed openstack/osc-placement master: DNM: gate health test https://review.opendev.org/c/openstack/osc-placement/+/952913
16:24:10 gibi sean-k-mooney: left some questions in the patch. The impl looks good btw.
16:27:39 opendevreview Elod Illes proposed openstack/python-novaclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-novaclient/+/952928
16:31:38 gibi sean-k-mooney: FYI I think this will be the first use of the round robin placement a_c strategy out in the field https://bugs.launchpad.net/nova/+bug/2114947
16:37:23 opendevreview Stephen Finucane proposed openstack/nova master: api: Add response body schemas for images APIs https://review.opendev.org/c/openstack/nova/+/952284
16:37:24 opendevreview Stephen Finucane proposed openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347
16:37:24 opendevreview Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348
16:37:25 opendevreview Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349

Earlier   Later