| gibi | looking at the functional threding job results we are definitely in a better shape now. There is one thing to monitor. I see TIMED_OUT results still after our stabilization fixes landed. And the timeout happnes during the functional test execution. It seems like we have a potentially hanging test case somewhere | 08:28 |
|---|---|---|
| gibi | I'm not sure why the normal test case timeout does not hit and kill the test case | 08:29 |
| gibi | but given that native threads are not killable from outside I'm not that surprised | 08:29 |
| bauzas | excellent news, thanks gibi | 08:29 |
| stephenfin | sean-k-mooney: Uggla: Would I be able to get some eyes on https://review.opendev.org/c/openstack/nova/+/1002852 today? It's blocking the u-c bump of ksa. gibi has already taken a look (thanks 🙏) | 08:30 |
| sean-k-mooney | done | 08:34 |
| sean-k-mooney | gibi: ya i rechecekd a patch twice over the weekend and it timed out in the functional tests both times | 08:35 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | 08:39 |
| gibi | sean-k-mooney: I will take a look | 08:40 |
| sean-k-mooney | test_live_migrate_vifs_from_info_cache | 08:52 |
| sean-k-mooney | failed in the later run | 08:52 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/980679 | 08:52 |
| sean-k-mooney | but the job in general timesed out | 08:52 |
| sean-k-mooney | the proir run just timed out in general | 08:53 |
| sean-k-mooney | those were a day and ahalf apart so it may have been before some of the stablisation patches landed | 08:53 |
| sean-k-mooney | sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) database is locked | 08:54 |
| sean-k-mooney | i dont knwo if your per db lock tweak landed before that | 08:55 |
| sean-k-mooney | hum it shoudl have | 08:55 |
| stephenfin | sean-k-mooney++ ty | 08:55 |
| gibi | https://bugs.launchpad.net/nova/+bug/2165149 | 08:58 |
| gibi | there are new hits in check pipeline | 08:58 |
| gibi | it might or might not mean that the patches are not rebased yet | 08:58 |
| sean-k-mooney | they have not been rebvased but have been rechecked after the merge | 08:59 |
| opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | 08:59 |
| gibi | does the check pipeline rebases the patch before running the tests? I don't think so | 08:59 |
| sean-k-mooney | i clicked the rebase button in the ui we can see | 08:59 |
| gibi | cool | 09:00 |
| sean-k-mooney | gibi: am it does a speculitive merge with master yes | 09:00 |
| sean-k-mooney | so it wont rebase but it should have the same effect | 09:00 |
| gibi | even the check pipeline? not just the gate? | 09:00 |
| sean-k-mooney | correct | 09:01 |
| gibi | the the locking fix did not fully solved the issue reported in that bug | 09:01 |
| sean-k-mooney | ya so the fix | 09:01 |
| sean-k-mooney | is that we shoudl add database locked to the list of excation that oslo.deb treads as a db deadlock excption | 09:02 |
| sean-k-mooney | then our existing decorators will handel that properly | 09:02 |
| gibi | I'm not sure we want to change the production code due to functional test DB limitations | 09:03 |
| sean-k-mooney | my guess is unless we explcitly flush the db or add a small delay it might take a second for sqlachemy to finsih unlocking the db after the write | 09:03 |
| gibi | could be | 09:03 |
| gibi | sqlite also has some config to retry internall for hits | 09:03 |
| sean-k-mooney | well sqlite is used in production when you use bifrost or standalone ironic | 09:03 |
| gibi | /hits/this/ | 09:03 |
| gibi | they are broken then as our locking is only for functional test | 09:04 |
| sean-k-mooney | well yes but our lockign is for nova :) | 09:04 |
| gibi | ahh so they not using nova with sqlite | 09:04 |
| sean-k-mooney | i mean ironic proper not ironic virt driver | 09:04 |
| gibi | but then I'm not sure how this is relevant :) | 09:05 |
| sean-k-mooney | ya so if you use biforst to manage yoru hardware inventory it uses or can use sqlite as the db | 09:05 |
| sean-k-mooney | the same way that ironic suprot there own jsonrpc implation instead of rabbit for a lighter weight deployment if you want that | 09:05 |
| sean-k-mooney | oh just that the oslo.db change should benift them too | 09:06 |
| gibi | OK, so you say we change oslo.db's list of exceptions to retry on | 09:06 |
| gibi | :) | 09:06 |
| gibi | I still cannot really justify that from nova perspective and the ironic case is something I cannot reproduce | 09:07 |
| sean-k-mooney | i think that would be good in the long term yes but i think we shoudl be able to work aroudn this in the futnctionl tests | 09:07 |
| sean-k-mooney | its just somethign i looked into when sqlite was segfaulting in cybrog | 09:07 |
| sean-k-mooney | as a cleaner way then jsut hte locking | 09:08 |
| sean-k-mooney | but ya if there are other tuneable or we can specficly tell sqlachemy to flush/release the db or something that might be better | 09:10 |
| gibi | https://sqlite.org/c3ref/busy_timeout.html | 09:10 |
| gibi | this could be an SQLite native retry | 09:10 |
| gibi | https://sqlite.org/c3ref/busy_timeout.html | 09:11 |
| gibi | https://sqlite.org/pragma.html#pragma_busy_timeout | 09:11 |
| sean-k-mooney | """The SQLITE_LOCKED result code indicates that a write operation could not continue because of a conflict within the same database connection or a conflict with a different database connection that uses a shared cache. """ | 09:13 |
| sean-k-mooney | so the vs """The SQLITE_BUSY result code indicates that the database file could not be written (or in some cases read) because of concurrent activity by some other database connection, usually a database connection in a separate process. """ | 09:14 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [functional]Do not patch req-id generation https://review.opendev.org/c/openstack/nova/+/1003024 | 09:14 |
| sean-k-mooney | we are not using processes and we have a per db lock | 09:14 |
| sean-k-mooney | so im no t sure the busy tunabels will help | 09:15 |
| gibi | we use threads | 09:15 |
| sean-k-mooney | are we using the cache file | 09:15 |
| sean-k-mooney | i wonder if we are sharign the cache betwen dbs? | 09:15 |
| gibi | I think it is a separate file per DB but I can probably check by injecting a long see into our tests and see the dir structure | 09:17 |
| sean-k-mooney | ack | 09:17 |
| sean-k-mooney | i think we create a seprate WAL,cache and db fiel per database each in there own temp dir for the test | 09:20 |
| sean-k-mooney | but maybe only the db and wal file are separate | 09:21 |
| gibi | https://paste.openstack.org/show/b8jN0kKZZWRZVUQ11xDP/ I don't see cache files | 09:22 |
| sean-k-mooney | ya it looks like wal mode replaces it | 09:23 |
| sean-k-mooney | and shared_cache mdoe is the legacy version | 09:23 |
| sean-k-mooney | https://sqlite.org/wal.html | 09:23 |
| gibi | we do set WAL | 09:24 |
| sean-k-mooney | There is an additional quasi-persistent "-wal" file and "-shm" shared memory file associated with each database, | 09:24 |
| sean-k-mooney | i was confusing it with the shared memory file | 09:24 |
| gibi | but I don't see the WAL file either :) | 09:24 |
| gibi | maybe it is deleted when the connection closed | 09:25 |
| gibi | yepp | 09:27 |
| gibi | they are there | 09:27 |
| gibi | temporarily | 09:27 |
| gibi | nova-test-api-5dclxobp.sqlite-wal | 09:27 |
| gibi | nova-test-api-5dclxobp.sqlite-shm | 09:27 |
| gibi | but that also means it is a unique file per DB | 09:27 |
| gibi | now if two thread like nova-api and nova-conductor connects to the same api DB I assume they are sharing these | 09:28 |
| sean-k-mooney | ack well if you find something that helps let me knwo https://sqlite.org/pragma.html#pragma_locking_mode might be intersting | 09:28 |
| sean-k-mooney | gibi: they could but i thinkt hat is intended to be supproted | 09:28 |
| gibi | anyhow I will push a patch with a PRAGMA busy_timeout = milliseconds; | 09:29 |
| gibi | that is small and easy | 09:29 |
| gibi | and functional test specific | 09:29 |
| gibi | hm, maybe not. The WAL PRAGMA is persistent for the DB, but the busy_timeout is per connection so I cannot just add it to the same place :/ | 09:32 |
| gibi | looking at oslo.db retry :) | 09:32 |
| sean-k-mooney | the other thing you could try is set the lockign mode to exclusivne and back to normal in our currnet lock code https://sqlite.org/pragma.html#pragma_locking_mode but im not sure if that will really help or not | 09:36 |
| sean-k-mooney | i feel like that hsoudl not be required but ya im goign to go get coffee and step away for a bit | 09:37 |
| gibi | the hardness of this that I have no reproducer so whatever I try I have no proof it actually helps other than merging it and letting is run in the gate | 09:40 |
| gibi | so trying random things is expensive | 09:41 |
| gibi | the oslo.db retry code is pluged in a horrible way. I'm not even sure we use any retry at all by default | 09:42 |
| gibi | we have decorators @oslo_db_api.wrap_db_retry(max_retries=5, retry_on_deadlock=True) in some places | 09:43 |
| gibi | so we do retry in some cases | 09:43 |
| gibi | not all | 09:43 |
| gibi | so even if I change what to retry on I would need to decorate everything | 09:43 |
| gibi | that is uggly | 09:43 |
| gibi | OK, looking at differently. We have the write lock per DB file. That should serialize all the writes. If I assume it works well, then the only thing that can happen is that the SQLite DB lock is not released when our functional fixture lock is released (as you said at the start) so we could try to flush, close, whatever the connection to help with that DB level lock | 09:48 |
| opendevreview | Merged openstack/nova master: tests: Remove errant CONF mocks https://review.opendev.org/c/openstack/nova/+/1002852 | 10:12 |
| tkajinam | could anyone give 2nd +2 to https://review.opendev.org/c/openstack/nova/+/995370 and https://review.opendev.org/c/openstack/nova/+/995370 to reduce deprecation warnings ? | 10:15 |
| gibi | tkajinam: thanks I +Ad the https://review.opendev.org/c/openstack/nova/+/995370 but you linked that patch twice above so I assuem you have one more patch to look at | 10:52 |
| tkajinam | oh sorry I've bee struggling to make my copy-paste work correctly | 11:03 |
| tkajinam | gibi, https://review.opendev.org/c/openstack/nova/+/1002937 this is the 2nd one | 11:03 |
| tkajinam | gibi, and thanks a lot ! | 11:03 |
| tkajinam | oh wait I found a few remaining warnings | 11:05 |
| tkajinam | will address these in follow-up | 11:05 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_clas argument https://review.opendev.org/c/openstack/nova/+/1002937 | 11:20 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/nova/+/1002937 | 11:20 |
| gibi | thanks | 11:34 |
| opendevreview | Merged openstack/nova master: Replace deprecated remotable_classmethod https://review.opendev.org/c/openstack/nova/+/995370 | 12:25 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/nova/+/1002937 | 12:41 |
| clif | sean-k-mooney: is there anything I can do to help that perf patch merge? | 13:39 |
| sean-k-mooney | did it fail after the rebase? im on pto today so not really paying attaentio, gibi is lookign at some stablity impovment | 14:11 |
| sean-k-mooney | i think you just geting unlucky with the test failures | 14:12 |
| sean-k-mooney | the db locked error is oen of the knwo issues withthe func tests under threading mode currently | 14:12 |
| sean-k-mooney | and it was in an entirly differnt area of the code so not related to that patch | 14:13 |
| *** tobias-u1 is now known as tobias-urdin | 14:23 | |
| clif | gotcha | 14:40 |
| clif | sorry to ping you on your day off, enjoy | 14:41 |
| sean-k-mooney | its fien i would not be on irc if it bothered me | 14:41 |
| Uggla | Reminder: Upstream meeting in ~30mn. | 15:30 |
| Uggla | #startmeeting nova | 16:02 |
| opendevmeet | Meeting started Mon Aug 31 16:02:47 2026 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:02 |
| opendevmeet | The meeting name has been set to 'nova' | 16:02 |
| Uggla | Hello everyone | 16:02 |
| bauzas | o/ | 16:02 |
| gmaan | o/ | 16:03 |
| elodilles | o/ | 16:03 |
| lajoskatona | o/ | 16:04 |
| Uggla | Let's start | 16:05 |
| Uggla | #topic Bugs (stuck/critical) | 16:05 |
| Uggla | #info No Critical bug | 16:05 |
| Uggla | #topic Gate status | 16:05 |
| tkajinam | o/ | 16:05 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:05 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:06 |
| Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:06 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:06 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:06 |
| gmaan | we ,merged a lot of fixes last week, I will remove the bugs from that etherpad one we alrady fixed | 16:06 |
| Uggla | #info last week we had gate instability. Any progress? | 16:07 |
| Uggla | gmaan 👍 | 16:07 |
| tkajinam | I found a few strange errors this evening like dead workers or zombie process causing job failure, but I'm not sure if that appears consistently | 16:08 |
| gmaan | we merged almost all the fixes up for the frequent bugs happening but I dod not check today how gate is behaving | 16:08 |
| tkajinam | (these are not specific to nova but look like infra problems | 16:08 |
| gmaan | seems like, no change in CI pipeline | 16:08 |
| tkajinam | gmaan, https://review.opendev.org/c/openstack/nova/+/1002223 is the one which is supposed to fix unstable threading job, right ? | 16:09 |
| gmaan | tkajinam: ack, if you see them agian, please log the bug and we can check that if it happens more frequently | 16:09 |
| gmaan | tkajinam: yes, this is one of that. but there were 5-6 totals | 16:09 |
| tkajinam | gmaan, yeah or I'll report it to infra team, because that's a infra problem rather than a nova bug I believe. | 16:09 |
| tkajinam | ok. good | 16:09 |
| gmaan | cool, thanks | 16:09 |
| tkajinam | will be careful about threading unit tests this week | 16:09 |
| Uggla | I think we can move on | 16:10 |
| Uggla | #topic Release Planning | 16:11 |
| Uggla | #link https://releases.openstack.org/hibiscus/schedule.html | 16:11 |
| tkajinam | yup | 16:11 |
| Uggla | #info Nova deadlines are set in the above schedule | 16:11 |
| Uggla | #info Feature freeze was last week. | 16:11 |
| Uggla | #topic Review priorities | 16:12 |
| Uggla | #info Starting: https://etherpad.opendev.org/p/nova-2026.2-status#L16 interesting bugs to review. | 16:12 |
| Uggla | virtiofs cold migration needs focus too. I focused on testing it last week. | 16:13 |
| Uggla | #topic Stable Branches | 16:14 |
| * Uggla giving the mic to elodilles | 16:14 | |
| elodilles | thx | 16:14 |
| elodilles | #info stable gates should be OK | 16:14 |
| elodilles | though there were not so many patches against stable branches in the past week o:) | 16:14 |
| elodilles | anyway, if you see any stable gate issue, then please ping and/or: | 16:15 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:15 |
| elodilles | add here ^^^ | 16:15 |
| * elodilles gives back the mic to Uggla | 16:15 | |
| Uggla | thx elodilles | 16:17 |
| Uggla | skipping vmware topic fwiesel is still on PTO I guess | 16:17 |
| Uggla | #topic Kamil's news about eventlet removal | 16:18 |
| Uggla | sambork is not available today. But he left this info about eventlet: btw only update regarding eventlet is perf testing, so we're done there.Scaling down to 9 scheduler/conductor pods gave native threading a lower, peak-free, steady-state memory footprint (still marginally above eventlet), while CPU usage and timing stayed on par with eventlet | 16:19 |
| Uggla | and occasionally beat it thanks to native low-level thread scheduling. Setting executor_thread_pool_size to 32 for both the scheduler and conductor kept the queue depth at ~1, making those two the only default config changes required | 16:19 |
| gibi | + Last week we worked on stabilizing the functional-py313-threading job | 16:19 |
| Uggla | oh gibi, I thought you were not part of the meeting. My bad. | 16:20 |
| tkajinam | I like hearing such "the new model works better than the old model" news :-) | 16:20 |
| gibi | I think we eventually need to reopen https://bugs.launchpad.net/nova/+bug/2165149 as it might still happne | 16:20 |
| gibi | also I saw TIMED_OUTs from this job | 16:21 |
| gibi | that needs to be investiaget | 16:21 |
| gibi | investigated | 16:21 |
| Uggla | tkajinam ++ | 16:21 |
| tkajinam | gibi, is that what you discussed with sean today ? | 16:21 |
| gibi | yepp | 16:22 |
| gibi | or mostly we discussed the DB locked issue | 16:22 |
| gibi | not the TIMED_OUT one | 16:22 |
| tkajinam | ok | 16:22 |
| gibi | I have some local work on the former | 16:22 |
| gibi | but it is hard to reproduce the problem lcoally | 16:22 |
| gibi | locally | 16:23 |
| Uggla | anything else ? | 16:24 |
| Uggla | #topic Confidential computing, status of SNP and TDX features | 16:25 |
| Uggla | Not sure there are new stuffs on that topic. | 16:25 |
| antia | one question regarding TDX | 16:25 |
| antia | Should I make a patch for Glance? | 16:26 |
| tkajinam | antia, oh yes we have to update metadef | 16:26 |
| tkajinam | antia, I think you can find my change to add sev-snp and follow it | 16:26 |
| tkajinam | or I can do that if you want | 16:26 |
| antia | I can fix it after this | 16:26 |
| tkajinam | https://review.opendev.org/c/openstack/glance/+/995179 | 16:27 |
| antia | great! | 16:27 |
| tkajinam | you don't have to care a new release note file but modify the file added by that change | 16:27 |
| tkajinam | no news about sev-snp either. these features are in master so please try these and give your feedback :-) | 16:28 |
| Uggla | ok can we move on ? | 16:30 |
| Uggla | #topic Nova using openstack sdk for neutron | 16:31 |
| Uggla | lajoskatona something you'd like to sharE ? | 16:31 |
| lajoskatona | small things: SDK release is out | 16:31 |
| lajoskatona | so I can update the patch series to have fields=[....] like filtering with SDK also | 16:31 |
| lajoskatona | after the meeting I hope I can push some of the updated patches | 16:32 |
| lajoskatona | that's it for this topic from me | 16:32 |
| Uggla | thx lajoskatona | 16:32 |
| Uggla | #topic Bug scrubbing | 16:32 |
| Uggla | #info down to 68 (-1). | 16:32 |
| Uggla | #link https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:33 |
| Uggla | #link https://truc.uggla.fr/ to follow the trend. | 16:33 |
| Uggla | #info I need to skip this week. | 16:33 |
| Uggla | #info Next meeting (next week): [public] Upstream bug triage. Wednesday, Septembre 09th · 15:30 – 16:00 UTC. Video call link: meet.google.com/zjr-rxus-hzj | 16:33 |
| Uggla | #topic Open discussion | 16:33 |
| gibi | o/ one quick question | 16:33 |
| Uggla | no topic in the agenda for today. | 16:33 |
| Uggla | gibi please go ahead | 16:34 |
| gibi | when you said 18:13 < Uggla> virtiofs cold migration needs focus too. I focused on testing it last week. | 16:34 |
| gibi | what do you mean? | 16:34 |
| gibi | is there some post FF stuff to land there? | 16:34 |
| dansmith | an FFE was requested | 16:34 |
| dansmith | I thought that it was mostly good to go though after Uggla found and gouthamr fixed (right?) a ceph secret thing | 16:35 |
| gibi | ohh i missed the FFE | 16:35 |
| dansmith | still trying to catch up on the traffic since I was out but that was what I saw | 16:35 |
| sean-k-mooney | not sure if the micvoverison issue has been adressed. tl;dr we shoudl not have one for the migration as there is no api chagne | 16:36 |
| Uggla | yes absolutely a strange behavior with cepfs and migration that happens if you migrate vm depending of the order they created the secret. | 16:36 |
| dansmith | I commented on that | 16:36 |
| dansmith | I assume if we decide to drop it then it'll be trivial to do that | 16:36 |
| dansmith | so I wasn't really including it in "needs focus" :) | 16:36 |
| elodilles | (the FFE mail: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/GIYUU732R5VB6BNQONWPTHR6QF5S2FVT/ ) | 16:37 |
| sean-k-mooney | ack, i have been off since tursday evening so didnt follow up im back tomorrow | 16:37 |
| sean-k-mooney | that was my only real concern | 16:38 |
| gibi | cool, thanks for the summaries. :) | 16:38 |
| Uggla | I'm testing it again, to ensure the bug I found last week is corrected. | 16:39 |
| Uggla | I have also a couple of things remaining to test. But currently the feature looks quite solid. | 16:40 |
| Uggla | anything else you'd like to discuss ? | 16:42 |
| Uggla | if not I think we are done. | 16:43 |
| Uggla | Thanks for joining this meeting. Have a nice day/evening. | 16:44 |
| Uggla | #endmeeting | 16:44 |
| opendevmeet | Meeting ended Mon Aug 31 16:44:30 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:44 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-31-16.02.html | 16:44 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-31-16.02.txt | 16:44 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-31-16.02.log.html | 16:44 |
| tkajinam | thanks. | 16:44 |
| elodilles | thanks Uggla o/ | 16:44 |
| gibi | thanks Uggla | 16:44 |
| lajoskatona | o/ | 16:45 |
| tkajinam | just in case you have a few moments there are a few libvirt driver cleanups with one +2 . https://review.opendev.org/c/openstack/nova/+/997818 https://review.opendev.org/c/openstack/nova/+/995160 and https://review.opendev.org/c/openstack/nova/+/997878 | 16:49 |
| gmaan | gibi: this is the further refactoring (as dansmith commented on the original change) of thread pool executors. dansmith has initial review on that, waiting for you to have a look and I can update it further | 16:54 |
| gibi | gmaan: which one? :) | 16:57 |
| gibi | I guess https://review.opendev.org/c/openstack/nova/+/1001572 | 16:58 |
| gmaan | gibi: yeah this one :) https://review.opendev.org/c/openstack/nova/+/1001572 | 16:58 |
| gibi | I will try to get to it tomorrow | 16:58 |
| gibi | :) | 16:58 |
| gmaan | thanks | 16:58 |
| gmaan | monday morning, need more coffee | 16:59 |
| * gouthamr is out today - will need to add a highlight on “virtiofs” :D | 17:43 | |
| gouthamr | dansmith: welcome back | 17:43 |
| gouthamr | sean-k-mooney: dansmith : ack on the review comments; I’ll get to them as soon as I get to my laptop | 17:47 |
| gmaan | sean-k-mooney: as you pinged me on this, I think returning 400 is more suitable here, returning 409 need microversion https://review.opendev.org/c/openstack/nova/+/946223 | 18:12 |
| gmaan | i commented on it | 18:12 |
| sean-k-mooney | i would not really agree that all action apis are indivgual interfaces | 18:27 |
| sean-k-mooney | but i dont have a stong perfence | 18:27 |
| sean-k-mooney | its a single endpoing form my persecitie and the return code shoudl not really vary based on the request body once the request is valid | 18:28 |
| sean-k-mooney | but ya either is fine if you prefer the 400 that works for me | 18:28 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | 19:08 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | 19:08 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | 19:08 |
| opendevreview | Ashish Gupta proposed openstack/nova master: Fix init_host crash migration test under native threading https://review.opendev.org/c/openstack/nova/+/1001369 | 19:59 |
| gmaan | historically, nova action APIs has not been best interface but they are individual operations and interface which is why we have different set of return code per action APIs | 21:25 |
| opendevreview | Merged openstack/nova master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/nova/+/1002937 | 21:26 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Fix config option registration https://review.opendev.org/c/openstack/os-vif/+/1003173 | 22:25 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Fix config option registration https://review.opendev.org/c/openstack/os-vif/+/1003173 | 22:25 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (2/3) https://review.opendev.org/c/openstack/nova/+/1002657 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (3/3) https://review.opendev.org/c/openstack/nova/+/1002359 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1002658 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Tweak endpoint_override logic https://review.opendev.org/c/openstack/nova/+/1003174 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Use discover_version for microversion negotiation https://review.opendev.org/c/openstack/nova/+/1003175 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (1/3) https://review.opendev.org/c/openstack/nova/+/1003176 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Drop support for v2 password auth https://review.opendev.org/c/openstack/nova/+/1003177 | 22:29 |
| stephenfin | melwitt: I went down a rabbit hole 🤦 | 22:29 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Use discover_version for microversion negotiation https://review.opendev.org/c/openstack/nova/+/1003175 | 23:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (1/3) https://review.opendev.org/c/openstack/nova/+/1003176 | 23:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (2/3) https://review.opendev.org/c/openstack/nova/+/1002657 | 23:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: cinder: Replace legacy auth options (3/3) https://review.opendev.org/c/openstack/nova/+/1002359 | 23:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1002658 | 23:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Drop support for v2 password auth https://review.opendev.org/c/openstack/nova/+/1003177 | 23:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: WIP: cinder: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1003188 | 23:03 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!