Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-12-04
04:06:16 openstackgerrit Julia Kreger proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
12:35:44 openstackgerrit Daniel Mellado proposed openstack/python-openstacksdk master: Add tag support to create_stack https://review.openstack.org/525174
13:50:41 openstackgerrit Daniel Mellado proposed openstack/python-openstacksdk master: Add tag support to create_stack https://review.openstack.org/525174
15:24:45 openstackgerrit Daniel Mellado proposed openstack/python-openstacksdk master: Add tag support to create_stack https://review.openstack.org/525174
15:48:16 openstackgerrit Merged openstack/cliff master: Fix codec error when format=csv https://review.openstack.org/508760
15:53:14 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: [WIP] Add support for get details of Quota https://review.openstack.org/515401
15:59:58 mordred TheJulia: I think in general that's a good idea
16:10:50 mordred TheJulia: I wonder - we do a thing in the adapter where we retry retriable connection errors automatically - I wonder if we could combine the two approaches at some point - but I think that's fine for now
16:11:22 mordred TheJulia: I left a few comments inline, otherwise looks great - also +3 to the stack otherwise \o/
16:18:28 Shrews mordred: TheJulia: there is a 'retrying' python lib i've found pretty flexible and useful. no need to recode such logic if that works for us
16:18:52 Shrews https://pypi.python.org/pypi/retrying
16:19:22 openstackgerrit Merged openstack-infra/shade master: Add bifrost jobs https://review.openstack.org/524879
16:36:39 openstackgerrit Merged openstack-infra/shade master: Add helper to wait until baremetal locks clear https://review.openstack.org/524999
17:00:25 openstackgerrit Merged openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
17:00:26 openstackgerrit Merged openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
17:01:07 openstackgerrit Merged openstack/os-service-types master: Remove -U from pip install https://review.openstack.org/524910
17:01:22 openstackgerrit Merged openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
17:08:16 openstackgerrit Merged openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
17:10:27 mordred TheJulia, Shrews: ^^ \o/
17:11:54 mordred TheJulia: with ironicclient gone, I'd like to cut another release - do you want to get https://review.openstack.org/#/c/525037 in before we do?
17:13:38 TheJulia mordred: I would kind of like to, but I'm in NHO today/tomorrow, but I'll likely hack on it later tonight more.
17:14:06 TheJulia Shrews: Yeah, I was totally trying to avoid pulling in another library and still keep it super simple when I wrote it
17:14:11 mordred TheJulia: kk.
17:15:04 TheJulia mordred: I was kind of thinking in the adapter, but I also structurally don't know that area well enough to have confidence to hack on it... at least not without going and reloading all that from swap/git into my brain :)
17:15:37 TheJulia That and I was looking through and seeing some very explicit response handling and thinking "maybe we need that level of granularity"
17:15:46 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove shade-functional-devstack-legacy https://review.openstack.org/525258
17:17:26 mordred TheJulia: yah - also, it changes up a little bit in the sdk merge too ... so I think keeping it separate for now until we understand how all the adapter mush shakes out it likely for the best
17:18:28 TheJulia yeah, that was kind of also where the entire thought process went into my head, that it might be too "big" of a change
17:19:03 mordred ++
17:24:54 mordred #success Finished the shade transition to pure REST away from client libs
17:24:56 openstackstatus mordred: Added success to Success page
17:34:55 openstackgerrit Merged openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643
17:34:56 openstackgerrit Merged openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984
17:35:23 openstackgerrit Rosario Di Somma proposed openstack-infra/shade master: Merge OpenstackCloud and OperatorCloud files https://review.openstack.org/524877
17:39:59 openstackgerrit Merged openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977
17:40:00 openstackgerrit Merged openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978
17:40:01 openstackgerrit Merged openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979
17:40:01 openstackgerrit Merged openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980
17:40:02 openstackgerrit Merged openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258
19:02:25 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Allow grant_role to select users outside default domain https://review.openstack.org/525278
19:02:26 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Implement availability_zone_hints for networks and routers. https://review.openstack.org/525279
19:02:26 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove python-ironicclient https://review.openstack.org/525280
19:02:54 mordred TheJulia: ^^ cherry-pick of the ironic stack, but squashed into one commit
19:02:59 mordred Shrews: ^^
19:03:35 mordred Shrews: also, you might find https://review.openstack.org/#/c/524877 interesting
19:18:36 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: USe the correct package name in LIBS_FROM_GIT https://review.openstack.org/525283
19:25:23 mordred dtroyer: is that all it takes? did I make that too complicated then?
19:25:54 dtroyer I tried that locally and DevStack ran
19:26:04 mordred neat
19:26:31 mordred dtroyer: I made this stack: https://review.openstack.org/#/c/524991 - to make it more complex - but more than happy to abandon if it's just flat not needed :)
19:27:57 dtroyer Let's see if my test is accurate. I can see times when not having the mapping may be an issue, I do think that the names matching was an assumption from a while back
19:29:15 dtroyer mordred: also, for this particular case, I think ultimately renaming the repo is the best answer
19:29:48 mordred dtroyer: ok. cool. I can add that to the TDL
20:14:39 Shrews mordred: ugh
20:15:56 Shrews mordred: you ready for that to merge? will likely affect anything else outstanding
20:16:23 mordred Shrews: yah - TheJulia's stuff has landed which is the only complex outstanding stuff
20:16:49 Shrews away it goes
20:17:19 openstackgerrit Merged openstack/os-api-ref master: Fix Zuul jobs https://review.openstack.org/524353
20:19:15 mordred Shrews: boom
20:20:02 Shrews that makes git history fun
20:20:25 Shrews but we did it before... should make it an annual event!
20:30:40 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/525301
20:32:30 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Implement availability_zone_hints for networks and routers. https://review.openstack.org/525279
20:32:30 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove python-ironicclient https://review.openstack.org/525280
20:58:35 slaweq__ mordred: hi
20:58:42 slaweq__ mordred: I have quick question to You
20:58:53 mordred slaweq__: heya!
20:59:07 slaweq__ mordred: now, when shade is merged with OpenstackSDK, what is status of patches for shade?
20:59:31 slaweq__ should we still review them? or should them be abandoned and proposed to sdk repo?
21:00:17 mordred I think for now we still review them - at least until we make shade into a compatability layer on top of sdk
21:00:32 slaweq__ ok
21:00:48 mordred slaweq__: I think that might take a little while to accomplish - in the meantime, I've just been cherry-picking patches over to sdk after they land in shade
21:01:07 openstackgerrit Merged openstack-infra/shade master: Merge OpenstackCloud and OperatorCloud files https://review.openstack.org/524877
21:01:08 openstackgerrit Merged openstack-infra/shade master: Remove shade-functional-devstack-legacy https://review.openstack.org/525258
21:01:12 slaweq__ ok, that's now clear for me
21:01:14 slaweq__ thx
21:01:30 mordred excellent
21:07:14 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/520481
21:26:59 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Make functional-tips job voting https://review.openstack.org/525283
21:35:42 slaweq__ dtroyer: hello
21:36:09 slaweq__ dtroyer: can You maybe take a look at https://review.openstack.org/#/c/515401/ and tell me if it's something what You wanted more or less?
21:36:34 slaweq__ it's not finished yet but I wan't to hear Your opinion about it
21:36:37 slaweq__ thx in advance
21:36:51 dtroyer slaweq__: sure, I'll try to get to it today yet
21:37:01 slaweq__ dtroyer: thx a lot
22:08:51 openstackgerrit Merged openstack/python-openstackclient master: Remove a bunch of things we promised to remove in 2H 2017 https://review.openstack.org/524399
#openstack-sdks - 2017-12-05
00:10:25 SamYaple oh yea mordred. the keystone shade salt module/states merged
00:10:32 SamYaple glance and neutron will merge soon
00:19:29 mordred \o/
00:22:03 SamYaple i think im goign to not touch the nova onewith a 10 foot pole lol
00:22:08 SamYaple ill leave that to someone else
00:22:14 SamYaple ill do cinder though
01:17:31 openstackgerrit Julia Kreger proposed openstack-infra/shade master: Add retry logic mechanism https://review.openstack.org/525037
01:21:20 TheJulia mordred: ^^^ should take care of the most likely places where a retry will be needed due to a occasional lock. I didn't write a super low level unit test for the wrapper since the rest calls largely cover it. Let me know what you guys would prefer to see, and I'll toss it up as a follow-up, or maybe do it during lunch tomorrow if I have brain
01:56:19 SamYaple TheJulia: really really good work on python-irnoicclient. You have made me happier than you know
01:56:37 SamYaple when im feeling down i just think about that and smile
01:56:59 SamYaple (smile because i didnt have to do it heh)
02:56:47 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/525368
03:26:23 openstackgerrit OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/520472
03:26:42 openstackgerrit OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/525388

Earlier   Later