Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-12-02
00:30:18 openstackgerrit Merged openstack/osc-lib master: Avoid tox_install.sh for constraints support https://review.openstack.org/524604
01:35:39 openstackgerrit Merged openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739
04:57:13 openstackgerrit zhangyifan proposed openstack/osc-lib master: parameter attr in def find can not be None https://review.openstack.org/524784
14:51:32 mordred dtroyer: the issue with the tips jobs actually exposed a zuul bug that got introduced recently
14:52:32 mordred dtroyer: jeblair has some patches up to fix it, but the osc patches will be also a good verification case
15:15:40 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
15:15:41 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
15:15:41 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
15:15:42 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
15:16:46 mordred TheJulia: ^^ rebased those last 4 for you
15:17:35 mordred TheJulia: also - I think we should perhaps add some of the bifrost jobs to shade - I'm not sure which ones would be the best choice
15:18:09 openstackgerrit Rosario Di Somma proposed openstack-infra/shade master: Merge OpenstackCloud and OperatorCloud files https://review.openstack.org/524877
15:20:16 TheJulia mordred: thanks, likely a good idea, our tinyIPA leveraging one should just be using cirros
15:21:03 mordred TheJulia: cool - I'll get a patch up fr that
15:21:16 mordred rods: did you catch my rebase in that one?
15:22:21 dtroyer mordred: oh, nice? :) so when they run again we're good?
15:23:23 mordred dtroyer: yup!
15:23:52 mordred dtroyer: I think we should be good to recheck them on monday once the fix is rolled out
15:24:11 dtroyer great, thanks
15:24:52 mordred dtroyer: yay for us - turns out the bug would have bitten the release team at release time, but luckily we tripped it in advance
15:25:04 rods mordred probably not :)
15:25:31 rods I'm gonna fix it
15:25:44 mordred rods: :) - I left 2 comments - but I think a rebase of that patch onto the latest version of the remove ironicclient patch and that patch should be golden!
15:38:45 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
15:38:45 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
15:38:46 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
15:38:46 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
15:38:47 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Add bifrost jobs https://review.openstack.org/524879
15:39:03 mordred TheJulia: ^^ there ... adds bifrost jobs and restacks the ectomy on top of it
15:47:24 openstackgerrit Merged openstack/python-openstacksdk master: Add osc-tox-unit-tips jobs https://review.openstack.org/524676
16:02:28 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
16:02:28 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
16:02:29 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
16:02:29 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
16:10:55 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Add bifrost jobs https://review.openstack.org/524879
16:10:55 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
16:10:56 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
16:10:56 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
16:10:57 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
18:21:39 openstackgerrit Andreas Jaeger proposed openstack/python-openstackclient master: Remove -U from pip install https://review.openstack.org/524887
18:21:41 openstackgerrit Andreas Jaeger proposed openstack/osc-lib master: Remove -U from pip install https://review.openstack.org/524888
18:28:12 openstackgerrit Andreas Jaeger proposed openstack/python-openstacksdk master: Remove -U from pip install https://review.openstack.org/524900
18:30:23 openstackgerrit Andreas Jaeger proposed openstack/os-service-types master: Remove -U from pip install https://review.openstack.org/524910
22:06:02 openstackgerrit Merged openstack/python-openstackclient master: Remove -U from pip install https://review.openstack.org/524887
22:12:40 openstackgerrit Merged openstack/osc-lib master: Remove -U from pip install https://review.openstack.org/524888
#openstack-sdks - 2017-12-03
16:17:08 mordred dtroyer: so - the functional-tips is running now, but it's failing on devstack check_libs_from_git for python-openstacksdk - I believe this is because the pip name for python-openstacksdk is now openstacksdk, but we refer to it as python-openstacksdk in devstack and LIBS_FROM_GIT - and I don't think there is any support for having a mapping like that
16:21:54 mordred dtroyer: if the name is going to be openstacksdk I kinda think we should go ahead and rename the git repo next time there is an infra rename window
16:22:46 mordred briancurtin: is there a reason we didn't rename the repo already? or just it didn't hurt anything so why bother? (which is an excellent approach I wish more people would take)
16:41:53 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add OSC functional tips jobs https://review.openstack.org/520812
16:42:14 mordred morning TheJulia
16:42:57 TheJulia I have coffee, and I'm not focused on trying to go to a party tonight... :)
16:43:00 mordred TheJulia: I rebased and did a few small fixes on your remaining remove-ironicclient stack ... only thing failing now are the bifrost jobs
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: Add bifrost jobs https://review.openstack.org/524879
16:52:10 openstackgerrit Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
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: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: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

Earlier   Later