| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-05 | |||
| 08:48:22 | sean-k-mooney[m] | its too late to pull that into our next downstream release but it would still be nice to get that fixed before the end of the year if we can | |
| 08:50:04 | sean-k-mooney[m] | melwitt: oh i see you have some comments since i last reviewd ill read over them now | |
| 09:03:30 | sean-k-mooney[m] | ok dropped back to +1 based on your comments | |
| 09:45:36 | sean-k-mooney[m] | bauzas: can you review this functional repoducer test quickly. gibi and i are +2 on the actual fix https://review.opendev.org/c/openstack/nova/+/891290/1 gibi helped write the repoducer so we need another core to review | |
| 10:11:12 | opendevreview | Merged openstack/nova stable/2023.1: Do not manage CPU0's state https://review.opendev.org/c/openstack/nova/+/901567 | |
| 10:31:29 | opendevreview | Merged openstack/nova stable/2023.1: Allow enabling cpu_power_management with 0 dedicated CPUs https://review.opendev.org/c/openstack/nova/+/901660 | |
| 10:36:37 | bauzas | sean-k-mooney: sorry I was off due to some issue with my car, I'm back now | |
| 10:40:16 | stephenfin | JayF: If it's okay, I'd rather someone else take that. I think it's reasonably complete | |
| 10:45:46 | sean-k-mooney | bauzas: no worries | |
| 10:46:37 | bauzas | I needed to go to the policy just because the folk that hitted my car with his skis wasn't replied to me :( | |
| 10:46:49 | bauzas | replying* | |
| 10:47:01 | bauzas | police* | |
| 10:58:08 | bauzas | shit, I need to remember how to run a specific stestr worker | |
| 11:01:04 | bauzas | yeah found $ stestr last --subunit | subunit-filter -s --xfail --with-tag=worker-3 | subunit-ls > slave-3.list | |
| 11:09:58 | bauzas | hah | |
| 11:10:19 | bauzas | once you have a list of tests from the same worker, stestr run --analyze-isolation does the job for you | |
| 11:10:21 | bauzas | naïce | |
| 11:10:21 | bauzas | naïce | |
| 11:10:49 | bauzas | https://paste.opendev.org/show/bDpwRC0njzsbM1PmxTKr/ | |
| 11:10:54 | bauzas | \o/ | |
| 11:11:31 | bauzas | not sure auniyal6 and Uggla know about this ^ | |
| 11:15:24 | dvo-plv | Hello, sean-k-mooney gibi your comments for glance were resolved and zull verification pass successful | |
| 11:15:25 | dvo-plv | https://review.opendev.org/c/openstack/glance/+/882133 | |
| 11:21:47 | elodilles | bauzas: fyi, i need to take off the rest of the day, so i'm almost certain that i won't be around at the nova meeting :S i'll update the stable release patches when i'll be back so that we can release them finally | |
| 11:23:49 | bauzas | ++ | |
| 11:24:26 | sean-k-mooney | dvo-plv: cool just bare in mind gibi and i dont have core rights on glance | |
| 11:24:45 | sean-k-mooney | but ill re review and add my +1 so they know nova are fine with the patch | |
| 11:25:55 | dvo-plv | okay, thanks, I saw Abhishek in the reviewers, but I need one more person, whom I should to ask for review ? | |
| 11:26:41 | sean-k-mooney | i added dansmith to the review list but im not sure if they are on PTO for the rest of they year yet. i think tehy were here this week but not sure | |
| 11:27:25 | dvo-plv | I see, thank you | |
| 11:38:04 | opendevreview | Merged openstack/nova master: Remove workaround for old python-ironicclient https://review.opendev.org/c/openstack/nova/+/823247 | |
| 11:38:24 | opendevreview | Merged openstack/nova master: Add retry on attachment_update https://review.opendev.org/c/openstack/nova/+/901815 | |
| 11:43:37 | opendevreview | Merged openstack/nova master: [libvirt]Add migration_inbound_addr https://review.opendev.org/c/openstack/nova/+/900203 | |
| 11:43:44 | opendevreview | Merged openstack/nova master: Reproduce bug #2025480 in a functional test https://review.opendev.org/c/openstack/nova/+/891290 | |
| 12:10:35 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/c/openstack/nova/+/659691 | |
| 12:10:35 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | |
| 12:10:36 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 12:10:36 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | |
| 12:10:37 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | |
| 12:10:37 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | |
| 12:10:38 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining network operations https://review.opendev.org/c/openstack/nova/+/867794 | |
| 12:10:38 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining ironic driver calls https://review.opendev.org/c/openstack/nova/+/867795 | |
| 12:10:39 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for deprecated baremetal proxy API https://review.opendev.org/c/openstack/nova/+/867796 | |
| 12:10:40 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | |
| 12:10:40 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | |
| 12:10:42 | opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |
| 12:10:43 | stephenfin | JayF: Actually, it was a trivial enough rebase | |
| 12:10:54 | stephenfin | I thought it would have rotted more | |
| 12:29:07 | stephenfin | sean-k-mooney: Could you look at https://review.opendev.org/c/openstack/nova/+/896227/ and https://review.opendev.org/c/openstack/nova/+/896228/ again, when you get a chance? | |
| 12:29:35 | sean-k-mooney | i just got the email pings so yes | |
| 12:31:15 | stephenfin | ta | |
| 12:38:38 | sean-k-mooney | done, im happy with both of those and all the commits below | |
| 12:56:31 | opendevreview | Merged openstack/nova master: Do not untrack resources of a server being unshelved https://review.opendev.org/c/openstack/nova/+/890406 | |
| 12:56:39 | opendevreview | Merged openstack/nova master: doc: Remove crud from conf.py file https://review.opendev.org/c/openstack/nova/+/896224 | |
| 14:48:20 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove nova.wsgi module https://review.opendev.org/c/openstack/nova/+/902686 | |
| 14:48:20 | opendevreview | Stephen Finucane proposed openstack/nova master: Add new nova.wsgi module https://review.opendev.org/c/openstack/nova/+/902687 | |
| 14:48:21 | opendevreview | Stephen Finucane proposed openstack/nova master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/nova/+/902688 | |
| 14:48:27 | stephenfin | sean-k-mooney: I think that's where we are now ^ | |
| 14:51:48 | sean-k-mooney | i started looking at makeing pbr work but i didnt have time to actully fix the wsgi_sripts functionatly | |
| 14:52:15 | sean-k-mooney | but im ok with also takeign the approch of avoiding the extion if others are | |
| 14:52:21 | sean-k-mooney | ill try an review them again shortly | |
| 14:52:57 | sean-k-mooney | i know rougly where to make the change just havnt figureoud out exactly how too. | |
| 14:53:57 | opendevreview | Merged openstack/nova master: pre-commit: Update plugin versions https://review.opendev.org/c/openstack/nova/+/896226 | |
| 14:54:17 | sahid | o/ quick question guys | |
| 14:54:22 | sahid | regarding https://review.opendev.org/c/openstack/nova/+/896512 | |
| 14:54:51 | sahid | I'm glad that may consider it | |
| 14:54:56 | sahid | just, naything else is needed? | |
| 14:55:25 | opendevreview | Merged openstack/nova master: pre-commit: Use native hacking pre-commit hook https://review.opendev.org/c/openstack/nova/+/897005 | |
| 14:56:39 | sean-k-mooney | sahid: procedurelly jsut need to be approved in the team meeting later | |
| 14:56:59 | sean-k-mooney | i have withdrawn my objections so im fine with tracking this as a specless blueprint | |
| 15:06:12 | jkulik | on the topic of WSGI: there's this bug open https://bugs.launchpad.net/nova/+bug/1930406 which talks about 2 things: 1) dropping the rpc-call to nova-compute on volume-attach to reserve_block_device_name() and 2) running Nova with threads instead of eventlet - where could I look for the state/updates on these topics? | |
| 15:11:58 | sean-k-mooney | jkulik: im not sure that anyone is working on them currecnly i will remove lees name form it as they nolonger work on openstack | |
| 15:12:41 | sean-k-mooney | threads>1 is not somethign we are likely to supprot going forward | |
| 15:13:02 | sean-k-mooney | at least not in the short term | |
| 15:13:36 | sean-k-mooney | we may howeveer do that indirecly fi we end up removing our use of eventlets entirly for the nova-api when run under wsgi | |
| 15:13:53 | sean-k-mooney | i know melwitt had partly looked at changign to using treadpools at one point | |
| 15:13:57 | sean-k-mooney | i cant recall if we merged that | |
| 15:14:27 | jkulik | ok. maybe I can find a review-request for that. I'll take a look | |
| 15:14:31 | jkulik | thank you | |
| 15:16:23 | sahid | sean-k-mooney: ack thank you | |
| 15:16:32 | sean-k-mooney | jkulik: https://review.opendev.org/c/openstack/nova/+/650172 | |
| 15:17:03 | sahid | dansmith: regarding your reviews on it, thanks a lot. There are some that I'm not sure about, I have kept them as unrosolved, if you can have a look | |
| 15:17:27 | sean-k-mooney | jkulik: i dont reacall if there was anything preventing that form movign forword other then the fact i and dansmith have been busy and melwitt hasnt pinged use to review | |
| 15:17:31 | sean-k-mooney | but the last patch passed ci | |
| 15:17:45 | sean-k-mooney | so perhaps we can move forward with that. | |
| 15:18:28 | jkulik | sounds good. I'll read through it, too. | |
| 15:19:43 | bauzas | gibi: given my next internal meeting, I can't really do the upstream meeting | |
| 15:19:49 | sean-k-mooney | i just rebased https://review.opendev.org/c/openstack/devstack/+/893302 to get new results too | |
| 15:20:01 | bauzas | gibi: could you please chair it ? | |
| 15:28:40 | jkulik | sean-k-mooney: FYI, I found an abandoned spec https://review.opendev.org/c/openstack/nova-specs/+/765097 for "Remove device from attach_volume request response" | |
| 15:30:06 | sean-k-mooney | jkulik: ya so this feel off our radar since we dint have anyoen working on it | |
| 15:30:31 | jkulik | yeah, I'll put it into the bug report to maybe re-propose it later on | |
| 15:55:04 | bauzas | man the global CONF object is killing me | |
| 15:56:14 | bauzas | if some test runs before some other, then a group is persistent between them | |
| 16:00:53 | bauzas | #startmeeting nova | |
| 16:00:53 | opendevmeet | Meeting started Tue Dec 5 16:00:53 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:53 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:53 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:04 | bauzas | as I said earlier, I have a lot of things to do now | |
| 16:01:20 | bauzas | so, please, let's do a very short meeting (like 10 mins) | |