| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-10 | |||
| 11:43:41 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | |
| 11:56:06 | gibi | Uggla: sean-k-mooney: elodilles: I've sent the question about pyroute2 req rollback to ML and proposed the req patch. https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/FFYMQTJPJREQZ2P32HGS5HE2NLTXHVUC/ and https://review.opendev.org/c/openstack/requirements/+/960366 | |
| 12:05:06 | zigo | Your thoughts ? | |
| 12:05:06 | zigo | So I wonder, would the team agree for a feature to disallow spawning an instance with boot from volume, if it is with a flavor that has a system disk? Of course, that would be only an option (or a policy ?) and we could keep the usual old (permissive) behavior of the Nova API. | |
| 12:05:06 | zigo | However, there's a problem when customers are doing a boot from volume, and select a flavor that includes a system disk. In this case, they get billed for something they will never be able to use. | |
| 12:05:06 | zigo | At my company, we bill instances depending on their resources. ie: if you want a 20GB system disk, you get billed (linearly) for it. Nothing special here. | |
| 12:05:06 | zigo | Hi there! | |
| 12:12:10 | elodilles | gibi: ACK, thanks o/ let's see if we get an answer for req folks | |
| 12:12:19 | elodilles | s/for/from | |
| 12:43:05 | sean-k-mooney | zigo: just reading back | |
| 12:44:13 | sean-k-mooney | zigo: what do you mean by 20GB system disk, do you mean a flavor with nova provisioned storage i.e. root_GB and/or ephmeral_GB >0 | |
| 12:45:34 | sean-k-mooney | zigo: i advocated for blockign BFV instnace form using flavor with root_gb >0 when we prevented non boot form volume instance using flavors with root_gb=0 | |
| 12:45:57 | sean-k-mooney | zigo: in the past it was consier too large of a upgrade impact for exisign clouds to make that change | |
| 12:46:15 | sean-k-mooney | zigo: i dont think we can jsut block it. but what we coudl do is add a policy rule for it | |
| 12:46:39 | sean-k-mooney | we have a policy rule for allowing non BFV guest to use falvoar with 0 root_gb | |
| 12:47:17 | sean-k-mooney | we could add one for BFV with flavors that are not 0 | |
| 12:47:38 | sean-k-mooney | and then you could configure that to @ or a specific rule other then member | |
| 12:48:12 | sean-k-mooney | i dont recally if '@' is the sentenil for no one or everyoen but we have sental values for both | |
| 12:48:20 | sean-k-mooney | zigo: would somethign like that work for you | |
| 12:49:02 | sean-k-mooney | zigo: one thing that we should not break is bfv with non-zero ephmeral_gb | |
| 12:49:56 | sean-k-mooney | that is often sued to have the root disk run form say ceph while having an addtion ephmeral disk that uses fast local storage, i.e. for ectd or fast local scratch space | |
| 12:50:20 | zigo | "we could add one for BFV with flavors that are not 0" <--- That's what I would like, yes. | |
| 12:51:08 | zigo | Can it be done with an existing deployment already ? | |
| 12:51:09 | sean-k-mooney | gmaan is on pto but i think that woudl be consitent with our exisitng policies | |
| 12:51:47 | sean-k-mooney | yes so the policy rule change woudl jsut prevent you creatin new instnace in that state or resizing to that state | |
| 12:52:02 | sean-k-mooney | so existing workload woudl contiue to work | |
| 12:52:41 | sean-k-mooney | but if folks use heat/tereform ectra to automate deployment then it woudl break there automation | |
| 12:52:42 | zigo | Ok, I'll search how to do it then. | |
| 12:53:03 | sean-k-mooney | i.e. if they refence specifc flavors | |
| 12:53:05 | sean-k-mooney | oh | |
| 12:53:11 | sean-k-mooney | your asking if it cna be done today | |
| 12:53:17 | sean-k-mooney | zigo: sorry no | |
| 12:53:34 | sean-k-mooney | zigo: ill doubel check but i belive we dont have the policy rule today | |
| 12:54:30 | zigo | Ok. | |
| 12:54:39 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/policies/servers.py#L288-L312 | |
| 12:55:04 | sean-k-mooney | that the existing one we have which does not do what you want but it prevent using an iamge with a 0 disk flavor | |
| 12:55:19 | sean-k-mooney | we need to add the inverse | |
| 12:55:44 | sean-k-mooney | zigo: the related config option is https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.max_local_block_devices | |
| 12:56:29 | sean-k-mooney | what you could do modify the flavor that have local disk with a forbiden trait | |
| 12:57:03 | sean-k-mooney | then you could set DEFAULT.max_local_block_devices=0 on a subset of hosts adn use provider.yaml or the api to add a trait to those hosts | |
| 12:57:19 | sean-k-mooney | you could also do thie with the aggrate flavor extra specs filter | |
| 12:57:42 | sean-k-mooney | that woudl turn the combinaiton yuou want to block into a no valid host | |
| 12:57:56 | sean-k-mooney | honelsy however that is not a good solution for an existing cloud | |
| 12:58:15 | sean-k-mooney | that really intended fo diskless blade servers | |
| 12:58:29 | zigo | Thanks a lot, that's helpful. | |
| 12:58:40 | sean-k-mooney | zigo: there is one other non nova thing you coudl consider | |
| 12:58:54 | sean-k-mooney | currently i guess your doing flavor based billing | |
| 12:59:09 | sean-k-mooney | actully it might now work | |
| 12:59:25 | sean-k-mooney | i was going ot say you could look at the allocation in palcemetn, | |
| 12:59:27 | zigo | Yes we are doing flavor based billing. | |
| 12:59:41 | zigo | And today, a customer complained he was billed for a system disk he's not able to use. | |
| 13:00:02 | sean-k-mooney | i belive ofr BFV guest the allcoation for the instnace will not have disk as part of it for BFV guests | |
| 13:00:16 | zigo | It's his fault, but for a definitive fix, we should just forbid doing that. | |
| 13:01:02 | sean-k-mooney | well that debatable nova supprot this but yes you need to acount for that in the billing system as a result | |
| 13:02:48 | sean-k-mooney | anyway nova zeros out the root disk storage form the flavor for bfv guest in palcement so if you wanted too you shoudl be able ot sue that to bill more acurratly until a new policy rule is intorduced | |
| 13:03:20 | sean-k-mooney | zigo: would you mind crating a blueprint for this or adding a ptg topic so we have somethign to dicuss and track this request | |
| 13:04:13 | sean-k-mooney | i dont know if other woudl want a small spec for this but we can dicuss that in the irc meeting once we have a blueprint. | |
| 13:28:01 | zigo | sean-k-mooney: Will do, thanks for your input ! | |
| 14:36:23 | dansmith | gibi: so on the the s-g parallelize thing.. in fixing the last few tests, I think I've realized that the accounting for the "batches" test(s) was so off because of the sequential processing | |
| 14:36:36 | dansmith | sort of a "yeah, duh" moment | |
| 14:38:55 | opendevreview | Merged openstack/nova master: Fix bug 2114951 https://review.opendev.org/c/openstack/nova/+/952895 | |
| 14:48:06 | gibi | dansmith: :) | |
| 14:50:33 | dansmith | the downside is that the behavior becomes a little less deterministic (as expected) so I'm having to change assertions a bit | |
| 15:29:30 | sean-k-mooney | gibi: so lookign at https://docs.openstack.org/project-team-guide/release-management.html#milestone-3-feature-freeze the requirements freeze starts at m3 and is unfozent on master after the stable branch of the requirement repos created at https://docs.openstack.org/project-team-guide/release-management.html#release-2 | |
| 15:31:22 | sean-k-mooney | gibi: so taht woudl imply its too late to rolback pyroute2 however i have no idea regarding your other question fo can this happen on stable/2025.2, my gut feeling is no becuase it could break released packages based on rc1 or the ga release, or at the very least cause the pyroute2 package to need a downgrade | |
| 15:32:11 | sean-k-mooney | with that said im not sure so thanks for sending that email again | |
| 16:14:09 | gibi | yeah I feel we need to roll forward once we have a pyroute2 fix both on master and stable/2025.2 I hope rolling forward will be OK on stable | |
| 16:32:56 | opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Drop GLANCE_STANDALONE https://review.opendev.org/c/openstack/nova/+/960404 | |
| 17:00:33 | sean-k-mooney | gibi: my understanding is we are not allowed to raise minversion but we can increase max | |
| 17:01:04 | sean-k-mooney | so it shoudl be ok as long as theyre is no breaking change | |
| 17:01:41 | sean-k-mooney | its not something i have seen us do often however so im not very familar with the critia that is applied | |
| 18:30:20 | opendevreview | sean mooney proposed openstack/nova stable/2025.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960409 | |
| 18:34:11 | opendevreview | sean mooney proposed openstack/nova stable/2024.2: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960410 | |
| 18:35:00 | opendevreview | sean mooney proposed openstack/nova stable/2024.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960411 | |
| 21:12:24 | opendevreview | Cyril Roelandt proposed openstack/nova master: DNM: Drop GLANCE_STANDALONE https://review.opendev.org/c/openstack/nova/+/960404 | |
| #openstack-nova - 2025-09-11 | |||
| 09:03:17 | opendevreview | Merged openstack/nova master: Add Flamingo prelude section https://review.opendev.org/c/openstack/nova/+/959188 | |
| 09:32:32 | opendevreview | Stephen Finucane proposed openstack/nova master: DNM: Test patch removing glance standalone https://review.opendev.org/c/openstack/nova/+/960446 | |
| 12:32:38 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960507 | |
| 12:32:40 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960508 | |
| 12:32:41 | opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960509 | |
| 12:34:15 | opendevreview | Thomas Goirand proposed openstack/nova-specs master: Restrict booting from volume with disk https://review.opendev.org/c/openstack/nova-specs/+/960513 | |
| 12:34:38 | zigo | sean-k-mooney: It's the first time I'm writing a spec, I hope it's ok... | |
| 13:09:42 | jlejeune | sean-k-mooney: gibi: Hello, I recently resumed my work on bug #2044235. You asked me to create a regression test to confirm the bug, but I'm having trouble reproducing it. When I raise the messagingTimeout exception from the ‘check_can_live_migrate_source’ method, it seems like this exception is caught by the fake transport driver, causing my test to fail. When I try with any other exception, | |
| 13:09:48 | jlejeune | my test works fine. Have you ever seen this behavior before? | |
| 13:11:50 | sean-k-mooney | zigo: you removed section instead of jsut say None so that is why pep8 is failing | |
| 13:12:08 | zigo | Ok, will re-add it. | |
| 13:12:43 | sean-k-mooney | zigo: ill take a look later but thansk for creating one. we can discuss in the spec what the default for the policy shoudl be | |
| 13:12:58 | sean-k-mooney | my guess is member but we may decied to change it in the cycle after to admin | |
| 13:15:16 | sean-k-mooney | zigo: the approch we normally take is add a new policy maintianing the same beahvior then update it after, however that in sope to dicuss in the spec | |
| 13:15:31 | sean-k-mooney | basiclly do we have an upgrade impact or not | |
| 13:16:00 | opendevreview | Thomas Goirand proposed openstack/nova-specs master: Restrict booting from volume with disk https://review.opendev.org/c/openstack/nova-specs/+/960513 | |
| 13:16:11 | sean-k-mooney | my expection is if we changed it to admin it would break tempest tests as we dont use speical falvors with 0 disk to test BFV in general | |
| 13:16:32 | sean-k-mooney | or at least i dont think we do that exclusively | |
| 14:14:33 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | |
| 14:31:36 | sean-k-mooney | jlejeune: sorry i am in a few converstaion at once. i see you pushed the tst so i can try and reveiw that patch and play with it loccally | |
| 14:33:00 | jlejeune | sean-k-mooney: no worry, I saw that you're in discussion with zigo | |
| 14:33:26 | sean-k-mooney | and on slack and oslo and watcher channel | |
| 14:33:42 | jlejeune | :) | |
| 14:34:09 | jlejeune | you're such a busy man :) | |
| 14:34:33 | sean-k-mooney | that or i have the attention span fo a magpie | |