Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-12-03
16:43:18 mordred TheJulia: coffee == good
16:43:29 TheJulia Yeah, I was just looking at that :\
16:44:34 mordred TheJulia: I'm wondering if because it's a legacy job it's not set up to deal with being triggered by a project that isn't bifrost
16:45:13 mordred yah. that's it
16:45:30 mordred TheJulia: the job in its current form can'thandle being triggered by not-bifrost
16:45:38 TheJulia yeah, that would be it :(
16:51:00 openstackgerrit Merged openstack/python-openstacksdk master: Remove -U from pip install https://review.openstack.org/524900
16:51:24 mordred TheJulia: I'm going to add a depends-on to the bifrost patch to the add-bifrost-jobs patch to see if it fixes it
16:51:54 TheJulia okay, I'm still about an hour from having my current path done and tests written for it
16:52:03 mordred coolio
16:52:10 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
16:52:10 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Add bifrost jobs https://review.openstack.org/524879
16:52:11 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
16:52:11 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
16:52:12 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
18:33:02 openstackgerrit Julia Kreger proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
18:33:03 openstackgerrit Julia Kreger proposed openstack-infra/shade master: Add helper to wait until baremetal locks clear https://review.openstack.org/524999
22:38:07 openstackgerrit Julia Kreger proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
23:39:21 mordred TheJulia: woot! it passed!
23:39:31 TheJulia indeed :)
#openstack-sdks - 2017-12-04
01:25:42 openstackgerrit Merged openstack/cliff master: do not require installing demo app to build docs https://review.openstack.org/524695
01:39:03 openstackgerrit Merged openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369
03:49:53 openstackgerrit Julia Kreger proposed openstack-infra/shade master: WIP: Add retry logic mechanism https://review.openstack.org/525037
03:50:36 TheJulia mordred: Take a look at ^^^ and lmk what you think.
04:05:57 openstackgerrit Julia Kreger proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
04:06:08 openstackgerrit Julia Kreger proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
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: Added nat_source flag for networks. https://review.openstack.org/522980
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: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: Remove python-ironicclient https://review.openstack.org/525280
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: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: Remove python-ironicclient https://review.openstack.org/525280
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: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

Earlier   Later