| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-04 | |||
| 14:15:22 | bauzas | this is just for closing a coverage gap we had | |
| 14:15:29 | bauzas | (and why rebuild was broken) | |
| 14:15:33 | sean-k-mooney | ok cool so you approvced the zed code fix | |
| 14:15:44 | sean-k-mooney | so we just need to wait for that to merge then update the sha | |
| 14:15:47 | bauzas | yup | |
| 14:15:50 | sean-k-mooney | and LarsErikP then we can do the release | |
| 14:16:22 | LarsErikP | great! | |
| 14:16:28 | bauzas | sean-k-mooney: if you want a fun puzzle, you'll like that https://review.opendev.org/c/openstack/nova/+/902084/comments/4e8056fe_b4b6a789 | |
| 14:16:46 | LarsErikP | and then I have to wait for canonical and the SRU of course.. fun times :P | |
| 14:18:11 | sean-k-mooney | LarsErikP well they can do release of patches without use doign the release | |
| 14:18:24 | sean-k-mooney | so they may do an async/hotfix for you if your one of there customers | |
| 14:18:29 | LarsErikP | they can.. but will they? :P | |
| 14:18:35 | LarsErikP | I'm not paying :P | |
| 14:18:46 | sean-k-mooney | heheh then proably not | |
| 14:18:53 | sean-k-mooney | hopefully this will get resovled this week | |
| 14:18:55 | LarsErikP | *sad trumpet noises* | |
| 14:19:12 | sean-k-mooney | if it does not keep letting us knwo until we get it done | |
| 14:19:39 | LarsErikP | will do =) | |
| 14:20:27 | LarsErikP | thanks for all the help! We found it a bit.. surprising that the bug was reintroduced when we did the upgrade.. :P | |
| 14:21:33 | obrest | Luckily we have a dedicated production-environment! So that we can fail safely in test :P | |
| 14:40:01 | sean-k-mooney | LarsErikP: normally the yoga release sould not have happend until after all the rest | |
| 14:40:28 | sean-k-mooney | its movign from stable/yoga to unmaintained/yoga and that was really ment ot have happend back in septemeber | |
| 14:40:46 | LarsErikP | aha.. | |
| 14:41:10 | sean-k-mooney | well octopber i guess it was ment to have happend after the 2023.2 release was officlaly made | |
| 14:55:29 | bauzas | sean-k-mooney: do you think we should have a minor release (.y) for https://review.opendev.org/c/openstack/releases/+/899603 ? (due to the hyverv experimental backport) | |
| 15:00:10 | sean-k-mooney | we could i guess | |
| 15:00:17 | sean-k-mooney | im not sure its required but we can | |
| 15:01:03 | sean-k-mooney | it does not actully disable the driver or anythign it just adds the warnign on start up | |
| 15:01:29 | sean-k-mooney | if we took the neutron approch of not loadign experimntal drivers without an explict opt in it woudl be another matter | |
| 15:04:06 | bauzas | sean-k-mooney: I added a comment so the release folks will tell us whether we need it | |
| 15:04:15 | sean-k-mooney | cool | |
| 15:32:45 | gibi | sean-k-mooney: could you check the two backports of power management fixes https://review.opendev.org/c/openstack/nova/+/901567 https://review.opendev.org/c/openstack/nova/+/901660/2 (both has +2 from elodilles already) | |
| 15:57:28 | opendevreview | Merged openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | |
| 17:14:23 | opendevreview | Elod Illes proposed openstack/nova stable/zed: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/900307 | |
| 17:14:24 | opendevreview | Elod Illes proposed openstack/nova stable/zed: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/900341 | |
| 17:14:25 | opendevreview | Elod Illes proposed openstack/nova stable/zed: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/900342 | |
| 17:18:29 | elodilles | bauzas: sorry, i did not realize that the functional test failure was valid, please review the updated test & that my thinking is correct ^^^ | |
| 23:14:08 | JayF | bauzas: for https://review.opendev.org/c/openstack/nova/+/659691 and other patches related to the SDK migration for ironic<>nova driver, is there a place you'd like me to link these, or some way you'd like me to tag them, once Ironic cores are happy w/em? | |
| 23:14:53 | JayF | bauzas: we're trying hard to land as many of these cleanups as possible before sharding this time :D | |
| 23:23:14 | JayF | stephenfin: https://review.opendev.org/c/openstack/nova/+/867799 (and the chain) needs a rebase; I was going to review, get this tested in Ironic CI, and try to help land this -- do you want to rebase this or would you like me to? | |
| #openstack-nova - 2023-12-05 | |||
| 03:37:48 | opendevreview | melanie witt proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | |
| 07:25:01 | opendevreview | melanie witt proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | |
| 08:47:42 | sean-k-mooney[m] | gibi: melwitt can either of ye take a look at https://review.opendev.org/c/openstack/nova/+/883682 | |
| 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 | |