Posted	Nick	Remark
#openstack-sdks - 2017-12-01
14:35:58	ioggstream	mordred: have you ever been in Florence?
14:37:18	mordred	ioggstream: I have! my wife and I went there laster year after the OpenStack Ops Meetup in Milan - but only for one night
14:37:36	mordred	ioggstream: we liked it very much and definitely need to return for more :)
14:38:11	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974
14:38:12	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976
14:38:12	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975
14:38:13	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978
14:38:13	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977
14:38:14	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980
14:38:14	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979
14:38:15	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259
14:38:15	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258
14:38:16	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483
14:40:05	ioggstream	what a bunch of patches!
14:40:59	ioggstream	going python 3.x...
14:45:50	openstackgerrit	Merged openstack/os-service-types master: Avoid tox_install.sh for constraints support https://review.openstack.org/524609
15:11:50	mordred	dtroyer: what do you think about adding cliff to the required-projects in osc-tox-unit-tips ? I believe that would let us replace cliff's legacy-cliff-tox-openstackclient-tip job with just using osc-tox-unit-tips
15:12:15	mordred	dtroyer: we could also just make another job specifically for cliff, it's not hard or anything
15:15:52	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs https://review.openstack.org/524639
15:16:06	mordred	dtroyer: ^^ there's that, should you think it's a good idea
15:18:51	dtroyer	mordred: +2
15:19:51	dtroyer	mordred: its taken me 3 tries but there were more things to fix in that job after getting the import right
15:20:45	mordred	dtroyer: woot. well -that tells me the job is worthwhile and that it'll be very good to have in place :)
15:22:26	slaweq	dtroyer: hello, will You have time to check my answers to Your comments in https://review.openstack.org/#/c/515401/ ?
15:22:35	slaweq	dtroyer: thx in advance
15:22:41	thingee	os-service-types doesn't have branches, but reno wants branches https://review.openstack.org/#/c/520475/1
15:24:38	dtroyer	slaweq: it's on my list for today
15:24:44	slaweq	dtroyer: thx a lot
15:27:24	openstackgerrit	Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643
15:31:11	openstackgerrit	Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643
15:36:51	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Remove reference to stable/pike from releasenotes https://review.openstack.org/524647
15:36:58	mordred	thingee, cmurphy: ^^
15:38:20	cmurphy	mordred: /home/colleen/dev/os-service-types/releasenotes/source/index.rst:5:toctree contains reference to nonexisting document u'master'
15:38:28	cmurphy	rename pike.rst to master.rst i think
15:38:58	mordred	gah
15:39:00	mordred	tahnks
15:39:17	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Remove reference to stable/pike from releasenotes https://review.openstack.org/524647
15:39:45	dtroyer	dhellmann: left a copule of comments in that etherpad, overall looks like a good approach
15:40:02	openstackgerrit	Monty Taylor proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/520475
15:40:32	Shrews	would anyone else like to review mordred's merge-shade stack, or shall i just +W them? they're pretty much just cherry-picks from shade
15:40:59	mordred	Shrews: I think you can just +W them - they've gone through the process once already
15:42:18	Shrews	mordred: gah, is this the root? https://review.openstack.org/522974 it's the only one with a pypy fail
15:42:51	mordred	hrm. why does that havea pypy fail - the pypy issue should have been fixed ...
15:43:31	mordred	Shrews: oh - my - that's an actual weird fail, rather than just a job issue ...
15:43:44	Shrews	yay
15:44:48	mordred	or, rather, it's just weird- but looking in to it
15:45:18	Shrews	k. gotta step out for a coffee run real quick anyway
15:46:01	dtroyer	Shrews: FWIW I'm OK with you just +W on shade backports. I'm trying to pay more attention to the o-c-c stuff.
15:46:19	Shrews	dtroyer: *nod*
15:48:47	mordred	dtroyer, Shrews: ok. that error is something weird and deep in subunit ... I think I'm ready to give up on it - this is way too much effort for a thing that's pretty much unused
15:49:57	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974
15:49:58	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976
15:49:58	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975
15:49:59	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978
15:49:59	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977
15:50:00	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980
15:50:00	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979
15:50:01	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259
15:50:01	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258
15:50:02	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack-tox-pypy jobs https://review.openstack.org/524651
15:50:02	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483
15:55:55	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add osc-tox-unit-tips jobs https://review.openstack.org/524656
15:57:17	openstackgerrit	Merged openstack/os-service-types master: Remove reference to stable/pike from releasenotes https://review.openstack.org/524647
16:00:46	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add shade and python-openstacksdk tips jobs https://review.openstack.org/524659
16:06:50	openstackgerrit	Merged openstack/os-service-types master: Updated from global requirements https://review.openstack.org/520475
16:14:02	amotoki	mordred: one question on https://review.openstack.org/#/c/522369/3/tox.ini : do you drop '{toxinidir}/demoapp' from docs requirements temporarily?
16:14:04	amotoki	mordred: This is required to build cliff doc expectedly but I am okay if it is provisional.
16:27:37	mordred	amotoki: there's a previous patch which changes how it works to use code from in the tree
16:27:57	mordred	amotoki: oh - wait- that was from you :)
16:28:29	mordred	amotoki: I think that's a bug in my patch then - let me fix
16:28:41	amotoki	mordred: thanks :)
16:28:47	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs https://review.openstack.org/524639
16:28:48	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Avoid tox_install.sh for constraints support https://review.openstack.org/524671
16:28:48	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Make py27 and py35 versions and template of unit-tips jobs https://review.openstack.org/524670
16:34:52	openstackgerrit	Monty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369
16:34:53	openstackgerrit	Monty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984
16:34:53	openstackgerrit	Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643
16:34:54	openstackgerrit	Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370
16:35:04	mordred	amotoki: ^^ that should be happier I think
16:38:34	amotoki	mordred: i think we need {toxinidir}/demoapp in [testenv:docs] deps too
16:39:32	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: Add osc, shade and sdk tips jobs https://review.openstack.org/524656
16:43:44	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add osc-tox-unit-tips jobs https://review.openstack.org/524676
16:43:58	mordred	amotoki: duh. of course. silly me :)
16:45:16	mordred	amotoki: oh. wait. this is ... darn it
16:45:44	mordred	this won't work at all ... let me mark that WIP for now
16:46:11	openstackgerrit	Merged openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support https://review.openstack.org/524498
16:47:46	amotoki	mordred: it is a bit tricky. we installs cliffdemo from local mainly for docs and testing the sphinx directive. it might need more work for docs build with venv you are working on
16:49:32	mordred	amotoki: yah - that's the issue - tox isn't involved in the new docs build jobs, so specifying the requirement there is an issue- I kinda thing we should revert the local cliffdemo patch and go back to installing it from pypi like before - and just add cliffdemo to global-requirements since it *is* a requirement that cliff has for both docs and testing
16:49:55	mordred	amotoki: otherwise I think we're going to wind up getting too clever all just to avoid adding one line to global-requirements
16:50:32	amotoki	mordred: honestly i am not sure cliffdemo is worth published to pypi
16:50:55	amotoki	mordred: the current cliffdemo in PyPI is really old
16:51:04	mordred	amotoki: it's already published there- that's how it was being installed before your patch
16:53:18	amotoki	mordred: yes. it works with cliffdemo in PyPI, but cliffdemo itself is mainly for demo purpose (+ test for the sphinx directive)
16:53:38	mordred	amotoki: totally. however, there is already precedent - openstack-release-test is in global-requirements for a similar reason
16:53:48	amotoki	mordred: i am not sure what is the right solution. I would like to hear opinion of dhellmann
16:54:16	mordred	amotoki: agree. definitely want his opinion :)
16:57:00	mordred	amotoki, dhellmann: for sake of argument, here's a g-r patch adding it: https://review.openstack.org/524678 Add cliffdemo
17:22:24	openstackgerrit	Merged openstack/cliff master: Use in-tree cliffdemo app for docs build https://review.openstack.org/515016
17:32:22	mordred	dtroyer: https://review.openstack.org/#/c/522977 https://review.openstack.org/#/c/522978 https://review.openstack.org/#/c/522979 and https://review.openstack.org/#/c/522980 are cherry-picks of occ patches
17:32:44	mordred	dtroyer: and https://review.openstack.org/#/c/523483/6 is a non-cherry-pick that touches that section of the code

Earlier   Later