| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-09 | |||
| 14:48:58 | sean-k-mooney | i guess it alreday can https://github.com/openstack/osc-placement/blob/master/osc_placement/resources/allocation_candidate.py | |
| 14:50:10 | gibi | it can today | |
| 14:50:19 | gibi | even with multiple requrest groups | |
| 14:50:38 | sean-k-mooney | ya i guess that is somehwat useful for debuging | |
| 14:50:52 | sean-k-mooney | you can get teh query form the sechduler log and try and recreate it i guess | |
| 14:51:22 | sean-k-mooney | well i guess its useful if you are using placment for things other then openstack or in a standalone mode | |
| 14:51:34 | sean-k-mooney | like the fidge/sandwich demos | |
| 14:54:52 | opendevreview | Balazs Gibizer proposed openstack/osc-placement master: Support microversion 1.39 https://review.opendev.org/c/openstack/osc-placement/+/828545 | |
| 14:55:59 | bauzas | gibi: sorry, just starting my day after ~4h of meetings | |
| 14:56:09 | bauzas | (lovely) | |
| 14:56:14 | gibi | bauzas: that sounds hard | |
| 14:56:18 | gibi | why soo many meetings? | |
| 14:58:40 | bauzas | internal needs (research tax credits discussion, documentation, mentoring...) | |
| 15:03:31 | gibi | all the fun stuff :) | |
| 15:03:34 | Uggla | bauzas, mentoring was not so long. ;) | |
| 15:03:56 | bauzas | ;) | |
| 15:04:13 | gibi | hups :) | |
| 15:19:15 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Revert instance's state when queued live migration is aborted https://review.opendev.org/c/openstack/nova/+/828570 | |
| 15:23:22 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova-specs master: Late Amendments to the Off-path Backends Spec https://review.opendev.org/c/openstack/nova-specs/+/828177 | |
| 15:32:36 | bauzas | melwitt: just saying, your whole series got trampled because of a wrong assert in the base patch | |
| 15:33:14 | bauzas | oh wait, no | |
| 15:33:17 | bauzas | weird | |
| 15:33:39 | bauzas | moving to another bp | |
| 15:45:45 | dansmith | bauzas: eh? | |
| 15:46:04 | bauzas | dansmith: maybe I was drunk | |
| 15:46:21 | bauzas | dansmith: but the base patch got a shitty number of failures and I just looked at the py38 one | |
| 15:46:37 | dansmith | the base patch was +1 from zuul last I looked | |
| 15:46:56 | bauzas | or maybe it's just a side effect of omicron and me just fooling because of the office room lockdown since 3 days | |
| 15:47:47 | bauzas | dansmith: talking of https://review.opendev.org/c/openstack/nova/+/712139/30 | |
| 15:48:09 | bauzas | and sorry, not the base patch, the first patch to be rebased from what I've seen, probably hence the confusion | |
| 15:48:16 | dansmith | oh that's far from the base patch | |
| 15:48:18 | dansmith | yeah | |
| 15:49:09 | dansmith | oh yeah I see the fails | |
| 15:50:10 | dansmith | the version I uploaded yesterday was +1 from zuul, so maybe her changes in the set after that had a problem | |
| 15:50:31 | dansmith | I'll check with her when she's around | |
| 16:02:40 | gibi | sean-k-mooney: I'm also +2 on the remote managed port series. Just to double check we don't have any open neutron dependency as the vnic type definition is merged. Am I correct? | |
| 16:04:04 | sean-k-mooney | yes and no | |
| 16:04:16 | sean-k-mooney | technially neutorn-lib has not been released with that change yet | |
| 16:04:21 | sean-k-mooney | but nova does not use it | |
| 16:04:31 | sean-k-mooney | im not sure if the neutron patches have merged dmitriis? | |
| 16:04:53 | sean-k-mooney | gibi: nova wont break without them but obviously it wont work until nueton merge the ml2/ovn change | |
| 16:05:13 | gibi | sean-k-mooney: OK then we have the same understanding | |
| 16:05:16 | melwitt | bauzas: thanks for the heads up, I haven't gotten a chance to look at the CI results yet. I uploaded that late last night | |
| 16:05:33 | bauzas | np | |
| 16:05:34 | dmitriis | sean-k-mooney: https://review.opendev.org/c/openstack/neutron-lib/+/828174 the lib patch is in | |
| 16:05:42 | gibi | the name of the new vnci type is fixed now so we can merge the nova side of that and at max neutron will not allow creating such port yet but that does not break nova | |
| 16:06:01 | sean-k-mooney | dmitriis: right but neutron uses neutron-lib form pypi | |
| 16:06:15 | sean-k-mooney | dmitriis: so until the release happens they cant use it in there unit tests | |
| 16:06:18 | dmitriis | sean-k-mooney: also waiting for this to land https://review.opendev.org/c/openstack/releases/+/828452 | |
| 16:06:24 | sean-k-mooney | yep | |
| 16:06:42 | sean-k-mooney | gibi: correct | |
| 16:06:49 | gibi | sean-k-mooney: cool | |
| 16:06:59 | gibi | then I pull the trigger on the series up until the doc | |
| 16:06:59 | sean-k-mooney | until the neutron change land there api validation will reject the vnic type | |
| 16:07:11 | sean-k-mooney | ack works for me | |
| 16:07:34 | sean-k-mooney | ok going to join donstream meeting | |
| 16:40:30 | opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 16:40:31 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 16:40:31 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 16:40:32 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 16:40:32 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 16:40:33 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 16:40:33 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 16:40:34 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 16:40:34 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 16:40:36 | opendevreview | melanie witt proposed openstack/nova master: Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 16:40:36 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 16:54:15 | opendevreview | Balazs Gibizer proposed openstack/osc-placement master: Support microversion 1.39 https://review.opendev.org/c/openstack/osc-placement/+/828545 | |
| 17:10:18 | gmann | gibi: replied - https://review.opendev.org/c/openstack/placement/+/826491/7/placement/util.py#426 | |
| 17:10:32 | gmann | it is same API behavior change that you also mentioned. !!!FOO is accepted as valid trait in code and there was no error at the down layer too so it is bad interface but success case for user. Now we are changing it to failure so break user. we should do this change in microversion only. any reason not to do as part of 1.39 ? | |
| 17:10:55 | gmann | we have lot of such cases in APIs in past and currently too | |
| 17:11:29 | gmann | main purpose of microversion was to improve the interface "not to be used incorrectly" without breaking the success case. | |
| 17:11:42 | gmann | * one of the purpose | |
| 17:33:23 | opendevreview | melanie witt proposed openstack/placement master: Make perfload jobs fail if write allocation fails https://review.opendev.org/c/openstack/placement/+/828438 | |
| 17:39:20 | bauzas | gmann: about the gate failure you mentioned yesterday at the nova meeting, do we have an open bug report against it ? | |
| 17:39:33 | bauzas | context : | |
| 17:39:35 | bauzas | https://meetings.opendev.org/meetings/nova/2022/nova.2022-02-08-16.00.log.html#l-112 | |
| 17:44:52 | gmann | bauzas: yes, this one https://bugs.launchpad.net/devstack/+bug/1960346 | |
| 17:46:42 | bauzas | gmann: thanks | |
| 18:00:07 | opendevreview | sean mooney proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | |
| 18:09:59 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Remove useless contributor/api-2 doc https://review.opendev.org/c/openstack/nova/+/828599 | |
| 18:09:59 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Split up notifications document https://review.opendev.org/c/openstack/nova/+/828598 | |
| 18:13:37 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Correct version for virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/828601 | |
| 18:13:39 | stephenfin | sean-k-mooney: Want to +2+W that ^ | |
| 18:13:49 | stephenfin | Rather than respin the base patch | |
| 18:14:09 | stephenfin | I'd do it myself but good to sanity check | |
| 18:15:49 | sean-k-mooney | sure i can take a look | |
| 18:15:56 | sean-k-mooney | so you restacked teh chain | |
| 18:16:12 | sean-k-mooney | did you read my comment as to why this was needed | |
| 18:16:44 | sean-k-mooney | discussing it with fungi and other one suggestion was to modify zuul or to comment when submit failes in this case | |
| 18:17:04 | sean-k-mooney | basically "this patch is in merge confclit please rebase" | |
| 18:18:45 | fungi | it's not technically in "merge conflict" so much as that it has an outdated parent change, and so gerrit will refuse to create a merge commit for it when asked | |
| 18:19:16 | sean-k-mooney | fungi: ya you are right the imporant bit though is "you need to do something" before this can merge | |
| 18:19:19 | fungi | its parent got a new revision and then was merged, but this change did not get rebased when its parent was updated | |
| 18:19:28 | fungi | so that's what needs to be done, yeah | |
| 18:20:19 | sean-k-mooney | fungi: is that technially a job failure fomment or an actul zuul change. e.g. is the submit implemented as a job or part of zuul when using the gerrit driver | |
| 18:21:19 | sean-k-mooney | basically im wondering is this a chagne to the project-config repo for the job/pipline definition or a change in zuul | |
| 18:21:19 | fungi | zuul would need to get a new feature in its gerrit source connection driver to report that error to the change | |
| 18:21:29 | sean-k-mooney | ack | |
| 18:21:34 | fungi | you can actually see it indicated in the gerrit webui if you're not colorblind (no joke), the way they decide to show it is by making the "(merged)" next to the parent change red instead of grey | |