Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-02-09
14:39:47 sean-k-mooney for consumer types
14:40:14 sean-k-mooney so assuming you add 1.39 support for any traits we shoudl be ok form that perspective
14:42:15 gibi I don't find the patch but my env defaults to 1.29 not 1.38
14:42:28 sean-k-mooney https://github.com/openstack/osc-placement/commits/master
14:42:35 sean-k-mooney i was just looking at the recent patches
14:43:11 sean-k-mooney ah yes
14:43:13 sean-k-mooney MAX_VERSION_NO_GAP = '1.29'
14:43:17 sean-k-mooney https://github.com/openstack/osc-placement/blob/281ace20df8bd01f4cb8ae29c39d80e754a9d0a3/osc_placement/version.py#L57
14:43:43 gibi yepp, so that gets picked when nothing is provided in the CLI
14:44:16 sean-k-mooney yes ok so we would need to close the gap to have a new micorvesion auto micked
14:44:22 sean-k-mooney so 30-36
14:44:23 gibi yes
14:45:03 sean-k-mooney ok well i would hope we could fix it with out a microverion but if we do need one then i guess that can improve the ux
14:47:19 sean-k-mooney 1.30 - ‘/reshaper’, 1.31 - ‘in_tree’ 1.32 - Support forbidden aggregates 1.33 - Support string request group suffixes 1.34 - Request group mappings in allocation candidates¶ 1.35 - Support ‘root_required’ and 1.36 - Support ‘same_subtree’
14:47:56 sean-k-mooney so ingnoring 1.30 which we proably shoudl add as unused
14:48:05 sean-k-mooney the rest seam to be allocation canidate related
14:48:18 sean-k-mooney do we want osc-placemetn to be able to get allcoation candiates
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 sean-k-mooney until the neutron change land there api validation will reject the vnic type
16:06:59 gibi then I pull the trigger on the series up until the doc
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: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142
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:32 opendevreview melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707
16:40:32 opendevreview melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749
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:33 opendevreview melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180
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: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:36 opendevreview melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271
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: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: Split up notifications document https://review.opendev.org/c/openstack/nova/+/828598
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:13:37 opendevreview Stephen Finucane proposed openstack/nova master: doc: Correct version for virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/828601

Earlier   Later