Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-26
15:04:19 cdent bauzas: see line 51 of https://etherpad.openstack.org/p/placement-newton-leftovers
15:04:20 bauzas compared to API docs :)
15:04:23 mriedem there is a note in there that the get allocations gabbits can't sub the left side of the json path b/c of a bug
15:04:40 mriedem they are working on api-ref
15:04:50 cdent s/they/we/
15:04:57 cdent as in, all of us
15:05:13 mriedem us vs them
15:05:15 mriedem duh
15:05:31 cdent mriedem: are you the wolf or the dog today?
15:05:41 mriedem i meant, they as in 'people that aren't me'
15:05:42 mriedem :)
15:05:46 mriedem b/c i'm not being helpful
15:06:43 cdent the docs, although not yet published, are coming together nicely and look very nice and polished
15:08:21 cdent given sufficient time/energy/people it would be great to decompose the gabbi files into smaller pieces, been on the looking for volunteers todo list for months
15:10:41 openstackgerrit John Garbutt proposed openstack/nova master: Add description to policies in floating_ip files https://review.openstack.org/459791
15:12:11 efried mordred Just to be clear, for the endpoints-from-service-catalog stuff, you're suggesting changes in keystoneauth1 itself?
15:12:41 sean-k-mooney jaypipes: hi jay yes im familar with what at&t want to do we were talking about this at the PTG
15:12:47 openstackgerrit John Garbutt proposed openstack/nova master: Add description to policies in migrations.py https://review.openstack.org/459727
15:13:29 sean-k-mooney jaypipes: its been a while since i reviewd the code changes though so i need to see how its shaping up and what the apis will look like.
15:16:00 jaypipes sean-k-mooney: will you be in Boston?
15:16:19 sean-k-mooney i was not planning to be if i had a strong reason to go i could
15:16:56 jaypipes sean-k-mooney: heh, you know it's in <2 weeks right? :)
15:17:37 sean-k-mooney yep i know i do have a ticket for said event but others are going form my team just not me
15:18:04 sean-k-mooney i was not sure how much design was going to happen at the fourm now with the PTG
15:18:17 openstackgerrit Rafael Folco proposed openstack/nova master: Delete inventory has its own method DELETE https://review.openstack.org/460147
15:18:32 jaypipes sean-k-mooney: certainly less than the PTG. but if you were going to be there, I would propose meeting in person.
15:18:45 jaypipes sean-k-mooney: not a big deal though. we can always do a hangout.
15:18:57 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
15:20:06 openstackgerrit Kaitlin Farr proposed openstack/nova master: Parse algorithm from cipher for ephemeral disk encryption https://review.openstack.org/446072
15:20:35 sean-k-mooney yep it would have been good to sync on things but ya im always up for a hangout's chat too
15:21:37 sean-k-mooney jaypipes: AT&T originally were proposing this as a nova change and i redirected it to neutron at the ptg.
15:22:03 jaypipes sean-k-mooney: k
15:22:29 mordred efried: I am - I think we should add a method to keystoneauth which registers options needed by "adapters" (which is keystoneauth term for a session mounted on a service endpoint)
15:22:54 mordred efried: so basically the parameters you need to create an adapter are the same parameters you need to find an endpoint from the service catalog
15:23:30 mordred and if we add them there, then we can just re-use that method ot register the right opts in all the places we want to let nova users put in all of the different filters
15:23:39 mordred (similar to how we use register_session_opts now)
15:24:04 efried mordred I dig it. Can you point me to something in the keystone source that'll give me a hint as to how an "adapter" differs from a "session" or an "auth"?
15:24:17 mriedem swebster: see my comment in https://review.openstack.org/#/c/453938/2/nova/network/neutronv2/api.py after the fact - can you push a small change to update that code comment block for the migration = None case?
15:24:26 mordred one could also make the argument that an load_adapter_from_opts would be a nice equivilent method ot keystoneauth that we could use in the places we want just a plain keystoneuath REST interface
15:24:28 openstackgerrit Gábor Antal proposed openstack/nova master: [WIP] Transform instance.live_migration_rollback notification https://review.openstack.org/402124
15:24:33 mordred efried: absolutely! one sec ...
15:25:09 swebster mriedem: no problem, will do
15:25:37 mordred efried: so - if you start from here: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/__init__.py#L31-L32
15:25:46 efried mordred And next question: Where do I propose changes? review.openstack.org via https://git.openstack.org/openstack/keystoneauth ?
15:26:13 efried mordred Yuh, found that, and saw how it was being used by other groups (cinder, ironic).
15:26:18 mordred efried: you can see this: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/session.py
15:26:37 mordred so I believe what would be wanted is a similar keystoneauth1/loading/adapter.py
15:27:25 mordred the actual adapter code is here: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/adapter.py - which lists the different adapter constructor params - I'm not sure all of those need to be conf options though
15:28:16 mordred efried: I'd say for your purposes, https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/adapter.py#L32-L37 are the ones that want conf options
15:28:56 efried mordred Except service_name, which people thought was useless?
15:29:07 mordred https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/adapter.py#L38 is useful once you're actually calling get_endpoint - but I think that's a different nova-level thing not a conf option
15:29:43 mordred efried: yes - I do wish service_name would diaf, but I think also johnthetubaguy made a good point mentioning it - just because I don't like it doesn't mean that people out there may not have used it for something
15:30:18 mordred efried: for instance, in the infra account for rackspace public cloud, there actually used to be 2 entries for "compute" - because of history with pre-openstack rackspace
15:30:25 mordred and we had to use service_name to get it to pick the right one
15:30:44 mordred so if we're adding the conf_options to keystoneauth, we might as well be complete :)
15:30:59 efried mordred Fair enough.
15:31:31 efried mordred At a glance here, I would need to have the union of [session opts] with these adapter opts, right?
15:31:36 openstackgerrit Gábor Antal proposed openstack/nova master: [WIP] Transform instance.live_migration_rollback notification https://review.openstack.org/402124
15:31:36 mordred efried: fwiw, I recently wrote a spec: https://review.openstack.org/#/c/459405/ - which explains this entire thing in excruciating detail
15:31:51 efried ooo
15:31:54 mordred in case you've ever wanted to learn _WAY_ more about what's going on here than any human should know
15:31:58 efried That'll be helpful.
15:32:09 cdent "explains"
15:32:34 mordred efried: yah - each service grouping needs the session opts and the adapter opts in order to fully desscribe how to connect to a service
15:32:56 mordred efried: the sesion opts obvs go to the Session, and then the adapter opts are how the service endpoint is found
15:32:59 mordred cdent: :)
15:33:09 mordred cdent: ooh! you made reviews for me - I'm excited
15:33:22 cdent mordred: i'm afraid they are disappointingly short
15:34:01 swebster mriedem: do you recommend updating that existing review, or should I start a new branch/review for the comment change? Since the original change already got merged
15:34:04 efried mordred I believe I understand. This is just what I was looking for. Ima have questions for you later about os-client-config (once you get out of cdent-land)
15:34:24 cdent Yay! I have a land, how do I get there?
15:35:03 efried cdent Take the red pill
15:35:39 mriedem swebster: the existing review is merged
15:35:46 mriedem so it has to be new
15:35:51 jbernard mriedem: im not sure either
15:35:59 swebster mriedem: thx
15:36:02 mriedem jbernard: i pushed a rev of the d-g change with tracing
15:36:04 jbernard mriedem: ahh, you added a recheck, thanks
15:36:28 jbernard mriedem: something unrelated appears to have merged, the graph is not looking good this morning
15:37:48 mordred efried: cool - let's definitely chat later about the occ things - I'll ping when I'm done with the next round of updates on this spec
15:38:08 efried mordred Roger wilco, and thanks.
15:38:16 mordred cdent: one of your feedbacks brings up a thing that morgan had mentioned too and I came up with a good idea for making it better - so while short, they are helpful!
15:38:54 jbernard mriedem: http://logs.openstack.org/32/459832/1/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/85d7f13/console.html#_2017-04-26_14_11_03_609997
15:39:02 jbernard mriedem: looks like maybe it's being skipped over
15:39:24 mriedem artom: why not also setting capabilities for vmware and ironic drivers in https://review.openstack.org/#/c/415520/29 ?
15:40:05 efried Is it a done thing to refer to a nova blueprint from another project? (For these keystoneauth changes, I would be referring to https://blueprints.launchpad.net/nova/+spec/use-service-catalog-for-endpoints, which is listed under project nova)
15:40:21 efried Would the blueprint have to be tagged to be registered under keystone as well, and approved there?
15:40:23 mriedem jbernard: oh
15:40:24 mriedem {plugin-repo}
15:40:27 mriedem rather than ${plugin-repo}
15:40:53 artom mriedem, err, whoops. Not sure why that happened
15:41:09 mriedem artom: let me just -1 you right here real quick...
15:41:11 openstackgerrit Sylvain Bauza proposed openstack/nova master: WIP: Don't delete allocation if instance being scheduled https://review.openstack.org/460177
15:41:19 artom mriedem, fair
15:41:42 mriedem artom: although if the compute manager or whatever is checking that flag just does a .get('foo', False) that would be fine too
15:41:44 jbernard mriedem: i thought the $ was not needed in yaml
15:41:49 morgan mordred: ++
15:41:51 mriedem jbernard: i'm in bash
15:41:51 bauzas mriedem: POC above for the discussion we had https://review.openstack.org/460177
15:42:08 mriedem jbernard: i mean this is d-g which is bash

Earlier   Later