| opendevreview | Amit Uniyal proposed openstack/nova stable/victoria: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/854980 | 06:24 |
|---|---|---|
| opendevreview | Dylan McCulloch proposed openstack/nova master: Limit vgpu inventory to specified device_addresses https://review.opendev.org/c/openstack/nova/+/858012 | 06:38 |
| bauzas | good morning | 08:06 |
| bauzas | elodilles: could you please respin https://review.opendev.org/c/openstack/releases/+/857149 based on the hash I mentioned ? | 08:07 |
| elodilles | bauzas: yepp, will do in a minute | 08:09 |
| bauzas | thanks | 08:09 |
| gibi | o/ | 08:10 |
| gibi | happy stable/zed branc day! | 08:10 |
| elodilles | gibi: not there yet... :D | 08:11 |
| gibi | we will have it today, wont we? | 08:11 |
| elodilles | that's the plan :) | 08:11 |
| elodilles | bauzas: there you go: https://review.opendev.org/c/openstack/releases/+/857149 | 08:11 |
| gibi | let me know if help i sneeded | 08:11 |
| bauzas | elodilles: approved | 08:12 |
| bauzas | gibi: we're still on zed, as the RC1 patch wasn't yet updated | 08:12 |
| gibi | ack | 08:13 |
| bauzas | now this is done, I approved, we need the release team to merge it so we will branch | 08:13 |
| bauzas | just checking whether placement is branched | 08:13 |
| elodilles | let's wait until the validator jobs finish | 08:13 |
| bauzas | np | 08:14 |
| bauzas | and yeah, placement is now branched https://review.opendev.org/c/openstack/releases/+/857149 | 08:14 |
| gibi | \o/ | 08:14 |
| bauzas | gibi: while you're there, core needed here https://review.opendev.org/c/openstack/osc-placement/+/856786 | 08:14 |
| opendevreview | ribaudr proposed openstack/nova master: Default Nova persistent objects without soft delete. https://review.opendev.org/c/openstack/nova/+/854355 | 08:14 |
| gibi | I need to bake a cake during the weekend | 08:14 |
| gibi | and dedicate it to zed | 08:15 |
| bauzas | (and the next change too) | 08:15 |
| gibi | on it... | 08:15 |
| bauzas | gibi: hope you won't eat an antelope | 08:15 |
| bauzas | and even if we said zed for the release, this series is definitely 'zombie' for me :) | 08:16 |
| elodilles | :) | 08:19 |
| gibi | hm, I dont have a good cake recepie for antelope | 08:19 |
| gibi | ;) | 08:19 |
| opendevreview | Vladislav Belogrudov proposed openstack/nova master: Nova instance snapshot should wait for volumes https://review.opendev.org/c/openstack/nova/+/848638 | 08:20 |
| opendevreview | Merged openstack/osc-placement master: Update master for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856786 | 08:28 |
| opendevreview | Pierre Riteau proposed openstack/nova master: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858013 | 08:40 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/xena: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858024 | 09:20 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/yoga: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858025 | 09:21 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/xena: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858024 | 09:21 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858050 | 09:22 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/victoria: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858051 | 09:23 |
| stephenfin | gibi: bauzas: Either of you want to blitz through those super simple doc backports? I'd do it only for me having proposed them | 09:24 |
| stephenfin | We missed it when sean-k-mooney removed the filters way back in Victoria | 09:24 |
| bauzas | stephenfin: I can give it a shoot | 09:29 |
| stephenfin | ta | 09:29 |
| bauzas | stephenfin: techically, we have our ass in the middle of two chairs | 09:30 |
| sean-k-mooney | oh which ones | 09:31 |
| bauzas | stephenfin: master is still on zed, but we plan to branch | 09:31 |
| sean-k-mooney | ah | 09:31 |
| bauzas | stephenfin: so https://review.opendev.org/c/openstack/nova/+/858013 will be in antelope | 09:31 |
| bauzas | because the current hash is for another change | 09:31 |
| bauzas | https://review.opendev.org/c/openstack/releases/+/857149 | 09:31 |
| bauzas | stephenfin: ^ | 09:31 |
| sean-k-mooney | ya so we will need anouther backport | 09:32 |
| bauzas | so I can approve for the master patch, but we'll need to rebase on zed once it's open | 09:32 |
| sean-k-mooney | we should have cut the stable branch last night | 09:32 |
| bauzas | and this won't be until we GA zed | 09:32 |
| bauzas | sean-k-mooney: this wouldn't have changed the problem | 09:32 |
| sean-k-mooney | actully no | 09:32 |
| sean-k-mooney | it can be rebased now but | 09:32 |
| bauzas | given we branch on a specific hash | 09:32 |
| sean-k-mooney | we will hold merging until ga | 09:33 |
| bauzas | correct | 09:33 |
| sean-k-mooney | sicne it not an rc2 bug | 09:33 |
| bauzas | we can prepare the zed backport | 09:33 |
| bauzas | but I'll -2 it | 09:33 |
| sean-k-mooney | hopefully we wont need an rc2 | 09:33 |
| bauzas | in general, we accept backports, we just hold them with -2 | 09:33 |
| opendevreview | OpenStack Release Bot proposed openstack/nova stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/nova/+/858081 | 10:02 |
| opendevreview | OpenStack Release Bot proposed openstack/nova stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/nova/+/858083 | 10:02 |
| opendevreview | OpenStack Release Bot proposed openstack/nova master: Update master for stable/zed https://review.opendev.org/c/openstack/nova/+/858084 | 10:02 |
| opendevreview | OpenStack Release Bot proposed openstack/nova master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/nova/+/858087 | 10:02 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 10:51 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 10:56 |
| opendevreview | Rajesh Tailor proposed openstack/os-vif master: Move mtu update request into ovsdb transaction https://review.opendev.org/c/openstack/os-vif/+/857868 | 11:04 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 11:23 |
| opendevreview | Elod Illes proposed openstack/nova stable/zed: [stable-only] Update .gitreview for stable/zed https://review.opendev.org/c/openstack/nova/+/858081 | 11:42 |
| opendevreview | Elod Illes proposed openstack/nova stable/zed: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/nova/+/858083 | 11:43 |
| opendevreview | Elod Illes proposed openstack/nova stable/zed: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/nova/+/858083 | 11:43 |
| opendevreview | Rajesh Tailor proposed openstack/os-vif master: Move mtu update request into ovsdb transaction https://review.opendev.org/c/openstack/os-vif/+/857868 | 11:50 |
| *** dasm|off is now known as dasm | 11:57 | |
| opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 11:58 |
| opendevreview | Rajesh Tailor proposed openstack/os-vif master: Move mtu update request into ovsdb transaction https://review.opendev.org/c/openstack/os-vif/+/857868 | 12:32 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: extend_volume of libvirt/volume/fc should not use device_path https://review.opendev.org/c/openstack/nova/+/858129 | 12:44 |
| opendevreview | Merged openstack/nova master: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858013 | 12:47 |
| opendevreview | Mikhail Stolyarov proposed openstack/nova master: extend_volume of libvirt/volume/fc should not use device_path https://review.opendev.org/c/openstack/nova/+/858129 | 13:05 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 13:06 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 15:49 |
| sean-k-mooney | auniyal: we should proably keep the old comment about it really beign a sting not a int | 15:59 |
| sean-k-mooney | although i think stephenfin wrote that so we shoudl get tehre in put ^ | 15:59 |
| stephenfin | The flavors doc? Don't think so | 16:00 |
| sean-k-mooney | the api ref | 16:00 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/858094/6/api-ref/source/parameters.yaml#3135 | 16:00 |
| sean-k-mooney | it sound like something you might right | 16:00 |
| sean-k-mooney | *write | 16:00 |
| sean-k-mooney | stephenfin: auniyal is tring to update the docs to capture the constiats on the falvor id string | 16:01 |
| stephenfin | I'm going to guess mriedem... | 16:02 |
| * stephenfin looks | 16:02 | |
| sean-k-mooney | its technially not defied anywhere but its normally a uuid and can be a number but its just a string today | 16:02 |
| stephenfin | Called it! commit f7d9e82a206, authored by Matt Riedemann | 16:03 |
| sean-k-mooney | so looking at the api i dont see where this is being validated | 16:05 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/compute/flavors.py#L64 | 16:07 |
| sean-k-mooney | it just seams to pas directly to the object | 16:07 |
| opendevreview | Merged openstack/placement stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/placement/+/857898 | 19:21 |
| opendevreview | Merged openstack/placement stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/placement/+/857899 | 19:21 |
| opendevreview | Merged openstack/nova stable/zed: [stable-only] Update .gitreview for stable/zed https://review.opendev.org/c/openstack/nova/+/858081 | 19:21 |
| opendevreview | Merged openstack/nova stable/zed: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/nova/+/858083 | 19:21 |
| opendevreview | melanie witt proposed openstack/nova stable/wallaby: Add --task-log option to nova-manage db archive_deleted_rows https://review.opendev.org/c/openstack/nova/+/802766 | 19:56 |
| opendevreview | melanie witt proposed openstack/nova stable/wallaby: Add --task-log option to nova-manage db archive_deleted_rows https://review.opendev.org/c/openstack/nova/+/802766 | 20:06 |
| *** tosky is now known as Guest765 | 21:33 | |
| *** tosky_ is now known as tosky | 21:33 | |
| *** dasm is now known as dasm|of | 22:43 | |
| *** dasm|of is now known as dasm|off | 22:43 | |
| *** dasm|off is now known as Guest772 | 22:44 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!