| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-16 | |||
| 09:33:01 | sean-k-mooney | we will hold merging until ga | |
| 09:33:07 | bauzas | correct | |
| 09:33:12 | sean-k-mooney | sicne it not an rc2 bug | |
| 09:33:13 | bauzas | we can prepare the zed backport | |
| 09:33:18 | bauzas | but I'll -2 it | |
| 09:33:21 | sean-k-mooney | hopefully we wont need an rc2 | |
| 09:33:35 | bauzas | in general, we accept backports, we just hold them with -2 | |
| 10:02:06 | opendevreview | OpenStack Release Bot proposed openstack/nova stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/nova/+/858081 | |
| 10:02:12 | 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:19 | opendevreview | OpenStack Release Bot proposed openstack/nova master: Update master for stable/zed https://review.opendev.org/c/openstack/nova/+/858084 | |
| 10:02:26 | 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:51:51 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 10:56:06 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 11:04:11 | 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:23:46 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 11:42:07 | opendevreview | Elod Illes proposed openstack/nova stable/zed: [stable-only] Update .gitreview for stable/zed https://review.opendev.org/c/openstack/nova/+/858081 | |
| 11:43:11 | 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:50 | 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:50:28 | 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:58:00 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 12:32:21 | 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:44:01 | 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:47:56 | opendevreview | Merged openstack/nova master: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858013 | |
| 13:05:12 | 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:06:21 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 15:49:09 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 15:59:27 | sean-k-mooney | auniyal: we should proably keep the old comment about it really beign a sting not a int | |
| 15:59:45 | sean-k-mooney | although i think stephenfin wrote that so we shoudl get tehre in put ^ | |
| 16:00:18 | stephenfin | The flavors doc? Don't think so | |
| 16:00:25 | sean-k-mooney | the api ref | |
| 16:00:28 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/858094/6/api-ref/source/parameters.yaml#3135 | |
| 16:00:44 | sean-k-mooney | it sound like something you might right | |
| 16:00:50 | sean-k-mooney | *write | |
| 16:01:59 | sean-k-mooney | stephenfin: auniyal is tring to update the docs to capture the constiats on the falvor id string | |
| 16:02:14 | stephenfin | I'm going to guess mriedem... | |
| 16:02:35 | 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:03:01 | stephenfin | Called it! commit f7d9e82a206, authored by Matt Riedemann | |
| 16:05:50 | sean-k-mooney | so looking at the api i dont see where this is being validated | |
| 16:07:24 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/compute/flavors.py#L64 | |
| 16:07:50 | sean-k-mooney | it just seams to pas directly to the object | |
| 19:21:41 | opendevreview | Merged openstack/placement stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/placement/+/857898 | |
| 19:21:43 | opendevreview | Merged openstack/placement stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/placement/+/857899 | |
| 19:21:47 | opendevreview | Merged openstack/nova stable/zed: [stable-only] Update .gitreview for stable/zed https://review.opendev.org/c/openstack/nova/+/858081 | |
| 19:21:54 | opendevreview | Merged openstack/nova stable/zed: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/nova/+/858083 | |
| 19:56:49 | 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:41 | 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 | |
| #openstack-nova - 2022-09-17 | |||
| 03:40:42 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/858232 | |
| 11:02:37 | opendevreview | Thomas Goirand proposed openstack/nova master: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858236 | |
| 15:09:05 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2023.1 Antelope https://review.opendev.org/c/openstack/nova/+/858238 | |
| 16:01:51 | gibi | zigo: thanks for noticing the broken os.traits dependencies. | |
| 16:02:01 | gibi | bauzas: we need an RC2 due to that ^^ | |
| 17:34:16 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 17:41:48 | opendevreview | Amit Uniyal proposed openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| #openstack-nova - 2022-09-18 | |||
| 13:32:08 | opendevreview | Dylan McCulloch proposed openstack/nova master: Limit vgpu inventory to specified device_addresses https://review.opendev.org/c/openstack/nova/+/858012 | |
| #openstack-nova - 2022-09-19 | |||
| 01:11:08 | opendevreview | Takashi Natsume proposed openstack/placement master: Move implemented specs for Xena and Yoga release https://review.opendev.org/c/openstack/placement/+/853730 | |
| 01:11:40 | opendevreview | Takashi Natsume proposed openstack/nova master: Add a hacking rule for the setDaemon method https://review.opendev.org/c/openstack/nova/+/854653 | |
| 01:12:05 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove unnecessary testing code https://review.opendev.org/c/openstack/python-novaclient/+/852949 | |
| 08:10:09 | bauzas | gibi: zigo: ack for https://review.opendev.org/c/openstack/nova/+/858236 but we need a bug report in order to make it RC2 | |
| 08:10:45 | zigo | :) | |
| 08:11:12 | bauzas | zigo: thanks for spotting the bug | |
| 08:11:52 | zigo | This one was easy to find, the error was quite explicit (no such trait or something...). | |
| 08:12:36 | gibi | my personal goal is implement bugs that at least clearly bugs :D | |
| 08:13:11 | bauzas | zigo: can you create the bug report ? | |
| 08:13:50 | zigo | ok | |
| 08:16:19 | zigo | bauzas: https://bugs.launchpad.net/nova/+bug/1990121 | |
| 08:16:26 | zigo | Not sure if I have enough rights to make it RC. | |
| 08:16:37 | zigo | Should I ammend the patch review to include the Bug: ? | |
| 08:17:25 | zigo | I mean: | |
| 08:17:26 | zigo | Closes-Bug: #1990121 | |
| 08:32:31 | bauzas | zigo: I can do the paperwork | |
| 08:32:44 | bauzas | zigo: in a meeting now | |
| 08:47:54 | opendevreview | Sylvain Bauza proposed openstack/nova master: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858236 | |
| 08:48:14 | bauzas | zigo: gibi: ^ | |
| 08:48:39 | gibi | +2 | |
| 08:54:53 | bauzas | zigo: gibi: let's wait for the master patch to be merged but then could you please backport to stable/zed ? | |
| 09:04:43 | gibi | if zigo can push the backport then I'm happy to look at it, and if he has no time then I can push the backport then just need to ping an extra core to get the second +2 | |
| 09:22:58 | zigo | I'll do it when it passes master, thanks guys. | |
| 09:26:10 | gibi | ack | |
| 09:35:12 | opendevreview | Andre Aranha proposed openstack/nova master: Replace Centos 8 jobs for Centos 9 https://review.opendev.org/c/openstack/nova/+/858272 | |
| 09:41:21 | opendevreview | Merged openstack/nova master: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858236 | |
| 09:57:29 | opendevreview | Thomas Goirand proposed openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858280 | |
| 09:57:44 | zigo | bauzas: gibi: There you go. | |
| 10:00:50 | jkulik | Hi. Is there a way to have quota for custom resources? | |
| 10:07:53 | gibi | jkulik: unified limits should support custom resources https://specs.openstack.org/openstack/nova-specs/specs/yoga/implemented/unified-limits-nova.html | |
| 10:08:30 | jkulik | thank you | |
| 10:09:43 | gibi | jkulik: melwitt knows the details about it | |
| 10:10:31 | jkulik | thank you again. having a pointer in some directions is already great. having someone to ask even better :) | |
| 11:09:06 | opendevreview | Artem Goncharov proposed openstack/nova stable/yoga: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858025 | |
| 11:10:21 | 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:10:52 | bauzas | zigo: your backport doesn't have the hash in the commit msg, sorry | |
| 12:11:27 | bauzas | zigo: can you please provide it ? | |
| 12:18:49 | zigo | bauzas: The "Cherry-pick from" thingy? | |
| 12:19:40 | sean-k-mooney | we have a scprit that checks the commit shas in the cherry-picked form lines | |
| 12:19:44 | sean-k-mooney | to ensure they are meged | |
| 12:21:13 | sean-k-mooney | we also check that if the cherry-picked form line is not there that the patch has stable-only or something like that | |
| 12:21:34 | opendevreview | Thomas Goirand proposed openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858308 | |
| 12:21:39 | sean-k-mooney | zigo: so ya https://review.opendev.org/c/openstack/nova/+/858280 need to be recherrypicked for that | |
| 12:22:54 | zigo | wrong change-id on the above. | |
| 12:23:00 | zigo | redoing it... | |
| 12:24:02 | zigo | Am I missing a param in "git cherry-pick" so it get the correct line? :/ | |