| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-16 | |||
| 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? :/ | |
| 12:24:02 | sean-k-mooney | bauzas: that proably means we need an rc2 right | |
| 12:24:15 | sean-k-mooney | -x or -X | |
| 12:24:23 | sean-k-mooney | i cant remeber which one | |
| 12:24:35 | sean-k-mooney | i normally use git review -X i think | |
| 12:24:47 | opendevreview | Thomas Goirand proposed openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858280 | |
| 12:25:08 | zigo | Should be good this time. Please forgive-me, I don't do this often. | |
| 12:25:16 | sean-k-mooney | zigo: -x for git -X for git review | |
| 12:25:28 | sean-k-mooney | no worries | |
| 12:26:57 | sean-k-mooney | zigo: +2 | |
| 12:28:59 | sean-k-mooney | bauzas: can you create the A seriese and set it as the active one https://launchpad.net/nova | |
| 12:29:16 | sean-k-mooney | our launchpad bug trackign is kind of broken currently | |
| 12:29:47 | sean-k-mooney | the bot is crrently setin the bug as tracked in zed and fix released when it merges in master | |
| 12:29:50 | sean-k-mooney | which is nolonger correct | |
| 12:47:25 | sean-k-mooney | stephenfin: while your reviewign here a pretty trivial fix https://review.opendev.org/c/openstack/nova/+/850152 | |
| 12:57:35 | gibi | bauzas, elodilles: do we need a requirement freeze exception for this https://review.opendev.org/c/openstack/nova/+/858280 ? | |
| 13:05:39 | elodilles | gibi: i don't think an RFE is needed | |
| 13:06:07 | elodilles | requirements repository contains the upper constraints | |
| 13:16:01 | gibi | OK so it is enought to approve this fromnova | |
| 13:16:04 | gibi | cool then | |
| 13:16:14 | gibi | I'm putting the +A then | |
| 13:17:48 | elodilles | maybe one question though | |
| 13:18:05 | elodilles | whether we need the 2.9 as minimum o:) | |
| 13:19:01 | gibi | probably we can fly with 2.8 now but it does not cause any trouble to go with the release latest 2.9 | |
| 13:22:41 | elodilles | indeed. i've added a comment to the patch. | |
| 13:24:03 | gibi | I noted this in the master patch | |
| 13:27:28 | bauzas | zigo: sorry I was afk | |
| 13:27:39 | bauzas | I meant you missed to add -x to cherry-pick parameter | |
| 13:27:48 | elodilles | gibi: ack | |
| 13:28:35 | bauzas | sean-k-mooney: sure, I'll create it | |
| 13:42:16 | opendevreview | Merged openstack/placement master: Fix typos https://review.opendev.org/c/openstack/placement/+/848636 | |
| 13:42:21 | opendevreview | Merged openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | |
| 13:45:28 | opendevreview | Merged openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858280 | |
| 13:48:26 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 13:48:26 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 14:08:18 | bauzas | gibi: sean-k-mooney: elodilles: fwiw, I'll hold RC2 until Thursday | |
| 14:08:45 | bauzas | elodilles: want me to propose the release patch ? | |
| 14:09:10 | bauzas | or is this done by the rel mgmt team automatically ? | |
| 15:15:28 | elodilles | bauzas: the RC2 is not done automatically, so the best is if it is done by nova team | |
| 15:15:35 | elodilles | bauzas: but i can do it :) | |
| 15:15:37 | bauzas | ack | |
| 15:15:41 | bauzas | can do | |
| 15:17:15 | elodilles | bauzas: fwiw relmgmt team likes to hold the RC2 patches till Thursday to avoid the need for a lot of RCs (in case more RC critical fixes arrives during the week) | |
| 15:18:23 | elodilles | bauzas: thanks, just let me know when the release patch is ready for review | |