Posted	Nick	Remark
#openstack-sdks - 2017-12-01
21:31:34	slaweq	ok, I will start doing that and I will see how it will go :)
21:31:43	slaweq	but thx a lot for Your help on that
21:31:48	dtroyer	thank you for staying with this!
21:32:02	slaweq	no prob
21:37:35	mordred	dtroyer: oh, like they are not even executing ... lemme look
21:38:38	dtroyer	I'm not sure where to look for that
21:43:29	htaccess	question did shade used to require openstacksdk and python-openstackclient but no longer does
21:44:26	htaccess	it looks to me that in 1.24.0 and earlier shade was pulling in openstacksdk and python-openstackclient but not longer does with 1.25.0
21:44:48	dtroyer	before the merge, shade did not require the SDK or OSC, it originally used the project python libs, then brought al of that internal.
21:45:06	dtroyer	I don't have an older shade handy, is either of those listed in requirements.txt?
21:45:16	dtroyer	they may be in test-requirements.txt to do testing
21:50:57	htaccess	diff shade-1.25.0/requirements.txt shade-1.24.0/requirements.txt only shows six and futures version changes
21:51:26	htaccess	are the packages in test-requirements.txt actually installed when you do a pip install?
21:52:38	mordred	htaccess: openstacksdk and python-openstackclient have in the past come as transitive depends via python-ironicclient
21:52:49	htaccess	mordred: that makes more sens
21:53:36	htaccess	so its probably due to python-ironicclient not pulling them in any more
21:53:54	mordred	yah. it looks like python-ironicclient finally removed the dep on python-openstackclient
21:54:14	htaccess	anyway it was silly to be relying on them being pulled in ill actaully put them in my requirements.txt if they are requirments :)
21:54:17	htaccess	thanks
21:54:21	mordred	although we're also about to drop our depend on python-ironicclient ... so the transitive depend wouldhave cone away as of 1.26 anyway
21:54:30	mordred	htaccess: woot!
21:54:33	htaccess	:)
21:54:53	mordred	htaccess: now - in the not-too-distant future shade will grow a depend on openstacksdk ... but that's unlikely before january I'd guess
21:55:18	htaccess	ah, interesting
21:57:33	mordred	htaccess: tl;dr - we merged the shade code in to sdk, and once we're happy with the shape of that, we'll turn shade the library into a thin backwards-compatibility shim on top of the shade-derived functionality in sdk
21:58:12	mordred	but I want to make sure we're happy with how it's working sdk-side before we start mucking with consuming it from shade
22:01:01	openstackgerrit	Carlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739
22:01:23	mordred	dtroyer: hrm.
22:02:02	htaccess	ah ok, so in the future i will be able to install openstacksdk and get shade plus whats already in the sdk?
22:03:25	mordred	htaccess: yup
22:09:38	mordred	dtroyer: I've looped in jeblair to help me look :)
22:10:01	dtroyer	kk
22:17:28	openstackgerrit	Carlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739
23:09:54	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259
23:33:57	openstackgerrit	Merged openstack/python-openstackclient master: Make py27 and py35 versions and template of unit-tips jobs https://review.openstack.org/524670
23:37:46	openstackgerrit	Merged openstack/python-openstackclient master: Avoid tox_install.sh for constraints support https://review.openstack.org/524671
#openstack-sdks - 2017-12-02
00:04:19	openstackgerrit	Merged openstack/cliff master: add support for legacy command name translation https://review.openstack.org/522380
00:04:20	openstackgerrit	Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/520389
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: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988
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: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: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
15:38:45	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
15:38:46	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
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: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: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
16:02:28	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
16:02:29	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989
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:10:55	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986
16:10:55	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: Add bifrost jobs https://review.openstack.org/524879
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:56	openstackgerrit	Monty Taylor proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987
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

Earlier   Later