| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-19 | |||
| 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 | |
| 15:18:34 | elodilles | or simply add me as reviewer... | |
| 17:33:34 | opendevreview | Artem Goncharov proposed openstack/nova stable/yoga: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858025 | |
| 18:06:15 | opendevreview | Merged openstack/nova master: Move uwsgi-gmr reno to the proper place https://review.opendev.org/c/openstack/nova/+/850152 | |
| #openstack-nova - 2022-09-20 | |||
| 07:24:20 | gibi | bauzas: ack | |
| 07:24:46 | bauzas | ack of what ? :) | |
| 07:32:07 | gibi | 16:08 < bauzas> gibi: sean-k-mooney: elodilles: fwiw, I'll hold RC2 until Thursday | |
| 07:32:21 | bauzas | heh ok | |
| 07:32:22 | gibi | that was the last thing you pinged me with :) | |
| 07:32:36 | bauzas | I need to prepare the release rc2 patch | |
| 07:32:47 | bauzas | but now I have some internal tool I need to run locally :D | |
| 07:33:05 | bauzas | joys of flask apps | |
| 07:42:55 | gibi | dont we have a container image published somewhere just to run it locally? | |
| 07:49:01 | bauzas | gibi: my yearly compliance training tells me I shouldn't leak any internal tooling in an upstream chan that's logged :p | |
| 08:04:59 | gibi | then dont :) | |
| 08:27:32 | opendevreview | junbo proposed openstack/nova master: Test free memory if ram is not overcommit. https://review.opendev.org/c/openstack/nova/+/858495 | |
| 08:30:07 | sean-k-mooney[m] | but but but what if we want too | |
| 08:30:20 | sean-k-mooney[m] | stupid rules | |
| 08:31:41 | sean-k-mooney[m] | ill be starting work in a bit but assuming my laptop has not disconnected i have that tool running locally on my laptop and i pinged you with the url yesterday bauzas if you just want to use my copy | |
| 09:44:24 | bauzas | I'll leave the Zanata patch up for reviews but if nobody looks at it until tonight, I'll fast-approve it as we always usually do https://review.opendev.org/c/openstack/nova/+/858232 | |
| 09:44:53 | bauzas | tl;dr: updated translations for the Zed release | |
| 09:45:33 | sean-k-mooney | hume the have a en_GB one i can review that at least | |
| 09:45:51 | sean-k-mooney | im wondering if they actully change anything | |
| 09:46:33 | sean-k-mooney | so far no | |
| 09:47:06 | bauzas | no, in general this is chinese | |
| 09:47:16 | bauzas | I updated one french translation iirc | |
| 09:49:00 | bauzas | oh, actually, this is purely french :) | |
| 09:51:20 | sean-k-mooney | the updates in the english one just moved words to differnt lines there is actully no change | |
| 09:51:27 | sean-k-mooney | but anyway i +2w'd it | |
| 09:51:47 | sean-k-mooney | i knwo the actual review of this happens else where so was just wondering what was changed | |
| 09:53:13 | sean-k-mooney | i still kind of wish that this was done in a sperat repo so that project teams did not have to basically blinly apporve text changes to our repos without the ablity to ask for the content to be updated | |