Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-05-16
12:58:01 mordred your commit message makes me happy
12:58:36 dtantsur follow-up patches will make you even happier :)
13:02:11 dtantsur mordred: something in openstacksdk confuses me. we have connection.update_machine (for example) and connection.baremetal.update_node
13:02:22 mordred dtantsur: those all look great! there's a stack of sdk patches that should put us in a good position to support what you need from microversions for ironic
13:02:24 dtantsur which one is supposed to be used? connection.baremetal does not have many eatures
13:02:29 dtantsur cool!
13:02:52 mordred dtantsur: yah - so - connection.update_machine is the shade method that TheJulia wrote
13:03:26 mordred connection.baremetal is the support for ironic that was in the sdk before the merge - we still have work to do to merge the two of them into a single piece of sanity
13:04:14 dtantsur mordred: and which one will be the final?
13:04:19 mordred but if there is a method directly on connection, I'd use it - it'll be the same code that backs up the ansible modules/bifrost
13:04:59 dtantsur sure, it's just confusing. and I don't want to start relying on something that is pending deprecation
13:05:31 mordred yah - you should be safe with the connection methods- they're not going to get deprecated
13:06:18 mordred it'll either remain different or might become just a convenience wrapper around connection.baremetal - but it won't go away
13:07:34 dtantsur neat! while we're here: are you open to adding things like "deploy_machine", "provide_machine", etc? essentially, convenience wrappers around node_set_provision_state, similar to inspect_machine?
13:07:43 mordred absolutely!
13:09:41 dtantsur okay, I may be able to carve some time for these
13:10:03 dtantsur actually, I'd rename node_set_provision_state to something, because consistency
13:10:08 mordred woot!
13:10:51 mordred machine_set_provision_state perhaps?
13:12:12 mordred or, honestly, I think set_machine_provision_state is more in keeping with the verb_noun pattern
13:17:51 dtantsur yeah, I like the latter more
13:30:32 frickler dtantsur: mordred: this might be related to the ca issue, but also doesn't have a solution yet https://storyboard.openstack.org/#!/story/1756723
13:31:12 dtantsur not wow
13:34:01 mordred frickler: that's awesome
13:35:13 mordred frickler: so - I think that's going to be an issue with parameter exchange between osc and sdk ... and should be a thing we can fix as part of the rework of how the config is processed and objects get created in osc 4
13:42:08 frickler mordred: so when should this "osc 4" thing happen? I'm having a hard time already finding reviewers for small patches to current OSC
13:44:23 mordred frickler: I think it's a thing dtroyer has already started in on ... are you going to be in vancouver next week?
13:45:26 frickler nope
13:46:34 mordred darn. well - after I finish debugging this morning's sdk patch, lemme see if I can spot an easier path to fixing yourbug
13:47:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
13:47:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049
15:16:35 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Collect timing information for API calls https://review.openstack.org/568877
15:16:36 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add oslo.config option for split-loggers https://review.openstack.org/568878
15:30:58 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Return the endpoint_override from EndpointData https://review.openstack.org/491947
15:56:46 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Stop using pbr autodoc https://review.openstack.org/568889
16:20:56 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Allow registering additional plugin loaders https://review.openstack.org/333126
17:33:13 openstackgerrit Colleen Murphy proposed openstack/keystoneauth master: [DNM] See if latest pbr is causing our docs problems https://review.openstack.org/568909
20:44:43 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Collect timing information for API calls https://review.openstack.org/568877
20:44:44 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add oslo.config option for split-loggers https://review.openstack.org/568878
20:45:07 openstackgerrit Merged openstack/keystoneauth master: Fix logging of encoded headers https://review.openstack.org/568365
21:52:28 openstackgerrit Merged openstack/keystoneauth master: Return the endpoint_override from EndpointData https://review.openstack.org/491947
#openstack-sdks - 2018-05-17
09:58:54 openstackgerrit Stephen Breyer-Menke proposed openstack/openstacksdk master: Support add/remove user to/from group https://review.openstack.org/569080
10:17:01 openstackgerrit Stephen Breyer-Menke proposed openstack/openstacksdk master: Support add/remove user to/from group https://review.openstack.org/569082
12:04:33 openstackgerrit Stephen Breyer-Menke proposed openstack/openstacksdk master: Support add/remove user to/from group https://review.openstack.org/569097
15:56:20 gtema mordred: what is the default python version for shade-ansible tests? Is it the one from system? In openstacksdk tox it is set by default for python2, but not in shade. There is an issue in Ansible (https://github.com/ansible/ansible/issues/39472), which makes keypair role fail for me with python 3.6
16:02:26 mordred gtema: yeah - we're not setting it explicitly in the tox config. we should definitely fix that module with the fix in that issue ... do you have any interest in making a PR to ansible and I can approve it? (otherwise I can get it up there)
16:10:30 gtema mordred: making it currently. Will be in couple of minutes
16:18:32 gtema mordred: https://github.com/ansible/ansible/pull/40341
16:32:45 edleafe elmiko: too late - the session was just announced: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130617.html
16:36:03 elmiko edleafe: thanks for the heads up, and no worries
16:36:21 elmiko i'm gonna see if i can just stay till thursday
16:42:52 elmiko cdent edleafe dtantsur, ready for review: https://etherpad.openstack.org/p/api-sig-newsletter
16:43:54 elmiko how did i drop bof from there... i had it in originally
16:44:54 elmiko ++ good additions
16:45:23 edleafe elmiko: done
16:45:42 elmiko thanks edleafe !
16:47:56 cdent not much to add beyond what edleafe has said. what's there is fine, but if you want to make the second session sound more exciting you could order it a bit differently: it is _explicitly_ about how to drop old functionality
16:48:46 cdent elmiko: ^
16:48:58 edleafe cdent: you should add wording to that effect then
16:49:15 cdent i'm not sure how to say it. thus bouncing the idea here
16:49:21 elmiko i changed it a little, see if that works
16:50:17 dtantsur "clarify what it means when we bump the minimum microversion" ambitiously :)
16:50:46 dtantsur otherwise LGTM
16:51:06 elmiko i like the reference point cdent
16:51:23 elmiko dtantsur: haha, totally
16:51:55 elmiko ooh, even more ambitious edleafe
16:52:03 edleafe asbestos jackets are required
16:52:10 elmiko hahaha, indeed!
16:52:30 edleafe I changed my color. Blue is cdent
16:52:36 elmiko oh, whoops
16:52:40 elmiko then good show cdent
16:52:40 edleafe our colors were too close
16:53:05 cdent I just took what I added from the session description. I'm done.
16:53:09 elmiko can we add (abestos jackets recommended) at the end?
16:53:45 elmiko thanks for the help all, gonna publish
16:53:51 cdent rumor has it all the firefighting will happen over beer beforehand
16:53:58 elmiko +1
16:53:59 edleafe nah, let 'em figure it out themselves!
16:54:04 cdent so I phoned ahead to the vancouver breweries to prepare them for mordred and mugsie
16:54:08 elmiko edleafe: haha
16:54:16 cdent dtantsur: it was mordred's idea
16:54:23 cdent (partly)
16:54:50 dtantsur that's a plan: collect all people who want to bump the minimum version in one room aaaaand <invent your even ending>
16:55:01 dtantsur s/even/evil/
16:55:07 elmiko sent
16:55:29 mugsie well, there is one person who wants to `rm -rf microversions` going to be in that room :P
16:56:18 dtantsur oh, I"m going to miss an epic battle :)
16:57:07 edleafe Will we all have little flags to show our allegiances?
16:57:27 mugsie that is a great idea :)
17:00:04 dtantsur I have love-hate relationships with microversions
17:08:32 cdent I have no idea what flag to fly.
17:22:44 elmiko cdent edleafe, ok, i'm good for wednesday too now =)
17:23:20 cdent like I said:
17:23:50 elmiko hahaha
17:25:13 edleafe how does that even work?
17:25:26 edleafe I change a flight, and it's several hundred $$
17:25:50 elmiko they let me change for free, maybe status on delta?
17:32:29 cdent didn't you buy your flight late? In which case it would be changeable anyway?
17:32:44 elmiko good point, i was still in some sort of grace period
18:16:50 mordred dtantsur|afk: I'm all for bumping min microversions - I'm just arguing that doing so already has a name and a well-understood and supported system. it's called "bumping the major version" and we support it everywhere with no additional effort :)

Earlier   Later