| 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 |
|---|---|---|
| opendevreview | Takashi Natsume proposed openstack/nova master: Add a hacking rule for the setDaemon method https://review.opendev.org/c/openstack/nova/+/854653 | 01:11 |
| opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove unnecessary testing code https://review.opendev.org/c/openstack/python-novaclient/+/852949 | 01:12 |
| 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 |
| zigo | :) | 08:10 |
| bauzas | zigo: thanks for spotting the bug | 08:11 |
| zigo | This one was easy to find, the error was quite explicit (no such trait or something...). | 08:11 |
| gibi | my personal goal is implement bugs that at least clearly bugs :D | 08:12 |
| bauzas | zigo: can you create the bug report ? | 08:13 |
| zigo | ok | 08:13 |
| zigo | bauzas: https://bugs.launchpad.net/nova/+bug/1990121 | 08:16 |
| zigo | Not sure if I have enough rights to make it RC. | 08:16 |
| zigo | Should I ammend the patch review to include the Bug: ? | 08:16 |
| zigo | I mean: | 08:17 |
| zigo | Closes-Bug: #1990121 | 08:17 |
| bauzas | zigo: I can do the paperwork | 08:32 |
| bauzas | zigo: in a meeting now | 08:32 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858236 | 08:47 |
| bauzas | zigo: gibi: ^ | 08:48 |
| gibi | +2 | 08:48 |
| bauzas | zigo: gibi: let's wait for the master patch to be merged but then could you please backport to stable/zed ? | 08:54 |
| 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:04 |
| zigo | I'll do it when it passes master, thanks guys. | 09:22 |
| gibi | ack | 09:26 |
| opendevreview | Andre Aranha proposed openstack/nova master: Replace Centos 8 jobs for Centos 9 https://review.opendev.org/c/openstack/nova/+/858272 | 09:35 |
| opendevreview | Merged openstack/nova master: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858236 | 09:41 |
| opendevreview | Thomas Goirand proposed openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858280 | 09:57 |
| zigo | bauzas: gibi: There you go. | 09:57 |
| jkulik | Hi. Is there a way to have quota for custom resources? | 10:00 |
| gibi | jkulik: unified limits should support custom resources https://specs.openstack.org/openstack/nova-specs/specs/yoga/implemented/unified-limits-nova.html | 10:07 |
| jkulik | thank you | 10:08 |
| gibi | jkulik: melwitt knows the details about it | 10:09 |
| jkulik | thank you again. having a pointer in some directions is already great. having someone to ask even better :) | 10:10 |
| opendevreview | Artem Goncharov proposed openstack/nova stable/yoga: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858025 | 11:09 |
| 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:10 |
| bauzas | zigo: your backport doesn't have the hash in the commit msg, sorry | 12:10 |
| bauzas | zigo: can you please provide it ? | 12:11 |
| zigo | bauzas: The "Cherry-pick from" thingy? | 12:18 |
| sean-k-mooney | we have a scprit that checks the commit shas in the cherry-picked form lines | 12:19 |
| sean-k-mooney | to ensure they are meged | 12:19 |
| 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 |
| opendevreview | Thomas Goirand proposed openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858308 | 12:21 |
| sean-k-mooney | zigo: so ya https://review.opendev.org/c/openstack/nova/+/858280 need to be recherrypicked for that | 12:21 |
| zigo | wrong change-id on the above. | 12:22 |
| zigo | redoing it... | 12:23 |
| zigo | Am I missing a param in "git cherry-pick" so it get the correct line? :/ | 12:24 |
| sean-k-mooney | bauzas: that proably means we need an rc2 right | 12:24 |
| sean-k-mooney | -x or -X | 12:24 |
| sean-k-mooney | i cant remeber which one | 12:24 |
| sean-k-mooney | i normally use git review -X i think | 12:24 |
| opendevreview | Thomas Goirand proposed openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858280 | 12:24 |
| zigo | Should be good this time. Please forgive-me, I don't do this often. | 12:25 |
| sean-k-mooney | zigo: -x for git -X for git review | 12:25 |
| sean-k-mooney | no worries | 12:25 |
| sean-k-mooney | zigo: +2 | 12:26 |
| sean-k-mooney | bauzas: can you create the A seriese and set it as the active one https://launchpad.net/nova | 12:28 |
| sean-k-mooney | our launchpad bug trackign is kind of broken currently | 12:29 |
| sean-k-mooney | the bot is crrently setin the bug as tracked in zed and fix released when it merges in master | 12:29 |
| sean-k-mooney | which is nolonger correct | 12:29 |
| sean-k-mooney | stephenfin: while your reviewign here a pretty trivial fix https://review.opendev.org/c/openstack/nova/+/850152 | 12:47 |
| gibi | bauzas, elodilles: do we need a requirement freeze exception for this https://review.opendev.org/c/openstack/nova/+/858280 ? | 12:57 |
| elodilles | gibi: i don't think an RFE is needed | 13:05 |
| elodilles | requirements repository contains the upper constraints | 13:06 |
| gibi | OK so it is enought to approve this fromnova | 13:16 |
| gibi | cool then | 13:16 |
| gibi | I'm putting the +A then | 13:16 |
| elodilles | maybe one question though | 13:17 |
| elodilles | whether we need the 2.9 as minimum o:) | 13:18 |
| 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:19 |
| elodilles | indeed. i've added a comment to the patch. | 13:22 |
| gibi | I noted this in the master patch | 13:24 |
| bauzas | zigo: sorry I was afk | 13:27 |
| bauzas | I meant you missed to add -x to cherry-pick parameter | 13:27 |
| elodilles | gibi: ack | 13:27 |
| bauzas | sean-k-mooney: sure, I'll create it | 13:28 |
| *** Guest772 is now known as dasm | 13:30 | |
| -opendevstatus- NOTICE: As of the weekend, Zuul only supports queue declarations at the project level; if expected jobs aren't running, see this announcement: https://lists.opendev.org/pipermail/service-announce/2022-September/000044.html | 13:38 | |
| opendevreview | Merged openstack/placement master: Fix typos https://review.opendev.org/c/openstack/placement/+/848636 | 13:42 |
| opendevreview | Merged openstack/nova master: Updated flavors doc https://review.opendev.org/c/openstack/nova/+/858094 | 13:42 |
| opendevreview | Merged openstack/nova stable/zed: requires os-traits >= 2.9.0 https://review.opendev.org/c/openstack/nova/+/858280 | 13:45 |
| 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 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | 13:48 |
| bauzas | gibi: sean-k-mooney: elodilles: fwiw, I'll hold RC2 until Thursday | 14:08 |
| bauzas | elodilles: want me to propose the release patch ? | 14:08 |
| bauzas | or is this done by the rel mgmt team automatically ? | 14:09 |
| elodilles | bauzas: the RC2 is not done automatically, so the best is if it is done by nova team | 15:15 |
| elodilles | bauzas: but i can do it :) | 15:15 |
| bauzas | ack | 15:15 |
| bauzas | can do | 15: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:17 |
| elodilles | bauzas: thanks, just let me know when the release patch is ready for review | 15:18 |
| elodilles | or simply add me as reviewer... | 15:18 |
| *** EugenMayer8 is now known as EugenMayer | 17:33 | |
| opendevreview | Artem Goncharov proposed openstack/nova stable/yoga: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858025 | 17:33 |
| opendevreview | Merged openstack/nova master: Move uwsgi-gmr reno to the proper place https://review.opendev.org/c/openstack/nova/+/850152 | 18:06 |
| *** dasm is now known as dasm|off | 22:00 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!