Friday, 2017-12-01

*** bobh has joined #openstack-sdks00:01
mordredwoot! the f26 pypy seems to be working, as does the https://review.openstack.org/523938 tips fix00:12
mordred(and by seems to be working, I mean it passed)00:12
*** david-lyle has quit IRC00:15
*** dayou has joined #openstack-sdks00:16
adriantthingee: I'm trying to shift some code from swift client to the SDK, and that was one issue I found.00:19
*** sdague has quit IRC00:22
*** david-lyle has joined #openstack-sdks00:29
openstackgerritAdrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly  https://review.openstack.org/52441800:33
adriantmordred: ^ there, that should fix swift 404s being broken00:34
*** david-lyle has quit IRC00:34
adriantand it is unlikely to break anything since only swift appears to actually use HEAD00:35
adrianthmmm, or at least called directly00:36
adriantyeah, allow_head seems to only be applicable to swift :/00:37
adriantso I can see why no one found this bug before00:37
mordredadriant: to my knowledge, swift is the only service that uses head at all00:38
adriantyep, that would be why this was broken :P00:39
mordredheh00:40
*** salv-orlando has joined #openstack-sdks00:42
*** salv-orlando has quit IRC00:46
adriantmordred: and it appears part of this issue is actually fixed in resource200:49
mordredwoot! so the part of the plan which is "finish migrating to resource2" is a good one00:50
adriantare we entirely dropping resource and moving to resource2 or is it being kept around for some form of backwards compat?00:51
openstackgerritAdrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly  https://review.openstack.org/52441801:00
*** salv-orlando has joined #openstack-sdks01:43
*** salv-orlando has quit IRC01:48
dtroyeradriant: IIRC finishing that was one of the things holding up a 1.0 release.  OSC's network code has a good bit of compat in it for that changeover that we're not removing until 1.0 is released02:09
openstackgerritDean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations  https://review.openstack.org/52393802:14
*** Anticime1 has joined #openstack-sdks02:18
*** annp has joined #openstack-sdks02:20
*** spredzy_ has joined #openstack-sdks02:20
*** Dinesh__Bhor has joined #openstack-sdks02:21
*** markvoelker has quit IRC02:22
*** spredzy has quit IRC02:22
*** fzdarsky|afk has quit IRC02:22
*** Dinesh_Bhor has quit IRC02:22
*** edleafe has quit IRC02:22
*** Anticimex has quit IRC02:22
*** spredzy_ is now known as spredzy02:22
*** bobh has quit IRC02:23
*** toabctl has quit IRC02:25
*** edleafe has joined #openstack-sdks02:27
*** markvoelker has joined #openstack-sdks02:27
*** toabctl has joined #openstack-sdks02:27
*** dave-mccowan has joined #openstack-sdks02:30
*** ldl has joined #openstack-sdks02:32
*** salv-orlando has joined #openstack-sdks02:44
*** salv-orlando has quit IRC02:49
*** dave-mccowan has quit IRC02:51
*** gildub has joined #openstack-sdks03:01
*** salv-orlando has joined #openstack-sdks03:45
openstackgerritAndreas Jaeger proposed openstack/os-api-ref master: Fix Zuul jobs  https://review.openstack.org/52435303:45
dayouAnyone know about the correct parameter to feed os-client-config get_one_cloud for token authentication tyep?03:48
dayou    cloud_config = occ.OpenStackConfig(load_yaml_config=False).get_one_cloud(03:48
dayou        verify=not insecure,03:48
dayou        cacert=cacert,03:48
dayou        region_name=request.user.services_region,03:48
dayou        auth_plugin='token',03:48
dayou        auth=dict(03:48
dayou            project_id=request.user.project_id,03:49
dayou            project_domain_id=request.user.domain_id,03:49
dayou            user_domain_id=request.user.domain_id,03:49
dayou            token=request.user.token.unscoped_token,03:49
dayou            auth_url=request.user.endpoint),03:49
dayou        app_name='octavia-dashboard',03:49
dayou        app_version=octavia_dashboard.__version__)03:49
dayouWith the following I got "__init__() got an unexpected keyword argument 'token'" error03:49
*** salv-orlando has quit IRC03:49
*** bobh has joined #openstack-sdks04:09
SamYaplemordred: meh. its only pypy04:12
* SamYaple hides04:12
*** rajathagasthya has joined #openstack-sdks04:34
*** bobh has quit IRC04:39
*** ldl has quit IRC04:40
*** ldl has joined #openstack-sdks04:42
openstackgerritGilles Dubreuil proposed openstack/api-wg master: Adds API-Schema guide  https://review.openstack.org/52446704:45
*** salv-orlando has joined #openstack-sdks04:46
*** salv-orlando has quit IRC04:50
*** ldl has quit IRC04:57
*** reedip has quit IRC05:03
*** ldl has joined #openstack-sdks05:07
*** rajathagasthya has quit IRC05:08
*** david-lyle has joined #openstack-sdks05:13
*** shu-mutou-AWAY is now known as shu-mutou05:16
*** gouthamr has quit IRC05:44
*** salv-orlando has joined #openstack-sdks05:47
*** salv-orlando has quit IRC05:51
*** salv-orlando has joined #openstack-sdks06:14
*** david-lyle has quit IRC06:30
adriantdayou: the issue is that it isn't 'auth_plugin' but 'auth_type'06:32
adriantI ran into that issue using mordred's example as well :P06:32
adriantit's assuming the password plugin, and getting confused06:33
*** ldl has quit IRC06:34
dayouadriant, thanks, that works, also I found user_domain_id can't be used also!06:50
*** reedip has joined #openstack-sdks07:02
openstackgerritAndreas Jaeger proposed openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52449807:14
*** gildub has quit IRC07:19
*** e0ne has joined #openstack-sdks07:27
openstackgerritMerged openstack/python-openstacksdk master: Fix py35 and pypy tox env  https://review.openstack.org/52338307:40
*** ioggstream has joined #openstack-sdks07:45
openstackgerritAndreas Jaeger proposed openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52449807:58
*** e0ne has quit IRC08:03
*** slaweq has joined #openstack-sdks08:03
*** salv-orlando has quit IRC08:06
*** gildub has joined #openstack-sdks08:08
*** gildub has quit IRC08:14
*** gkadam has joined #openstack-sdks08:24
*** salv-orlando has joined #openstack-sdks08:33
*** ralonsoh has joined #openstack-sdks08:44
*** shu-mutou is now known as shu-mutou-AWAY08:56
*** jpich has joined #openstack-sdks08:58
*** dmellado has joined #openstack-sdks09:07
*** e0ne has joined #openstack-sdks09:16
openstackgerritÉdouard Thuleau proposed openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported  https://review.openstack.org/52383409:18
*** Dinesh__Bhor has quit IRC09:18
*** Dinesh__Bhor has joined #openstack-sdks09:19
*** dtantsur|afk is now known as dtantsur09:51
*** markvoelker has quit IRC09:55
*** openstackgerrit has quit IRC10:33
*** LindaWang1 has joined #openstack-sdks10:34
*** markvoelker has joined #openstack-sdks10:55
*** LindaWang1 has quit IRC11:02
*** Dinesh__Bhor has quit IRC11:30
slaweqdtroyer: hi, I respin work on https://review.openstack.org/#/c/515401/2 finally11:37
slaweqand I have some doubt about Your comment and List command11:37
slaweqthere is already List command in OSC11:37
slaweqdtroyer: can You check my comments there and tell me what You think about it? thx in advance11:37
*** LindaWang1 has joined #openstack-sdks11:41
ioggstreamHi @all!  We opened the pycon Florence c4p https://www.pycon.it/en/call-for-proposals/ and your talks are very welcome!  rcarrillocruz mordred11:41
*** LindaWang1 has quit IRC11:41
rcarrillocruzgod11:41
rcarrillocruzflorence11:41
rcarrillocruzwhen is that11:41
rcarrillocruz?11:41
rcarrillocruzthat'd be dead easy to convince my wife to go with me11:42
*** salv-orlando has quit IRC11:49
*** salv-orlando has joined #openstack-sdks11:50
ioggstreamrcarrillocruz: it's in April11:51
ioggstreamrcarrillocruz: 19TH 22TH APRIL 201811:51
ioggstreamthe conference has historically  the best catering of every pycon11:51
ioggstream:D11:51
rcarrillocruzwelllll, if that is better than Bilbao, that must be awesome! (even though Bilbao was not technically pycon, but europython)11:52
ioggstreamrcarrillocruz: loOL11:53
ioggstreamrcarrillocruz: chianti wine, Italian cheese selections, pasta, ...11:53
* rcarrillocruz drools11:54
ioggstreamand well, Firenze is stunning11:54
*** salv-orlando has quit IRC11:55
*** thrash|g0ne is now known as thrash12:14
*** ioggstream has quit IRC12:47
briancurtinadriant: resource is going away once the one or two things that use it have moved over to resource2, then s/resource2/resource everywhere. the dual version was just a way to transition things over one at a time, though 95% of the work took 5% of the time, and this remaining 5% has just sat there for a very long time12:52
*** cdent has joined #openstack-sdks12:59
*** dtantsur is now known as dtantsur|brb13:14
*** markvoelker has quit IRC13:25
*** markvoelker has joined #openstack-sdks13:25
*** ioggstream has joined #openstack-sdks13:34
*** stephenfin is now known as finucannot13:40
*** openstackgerrit has joined #openstack-sdks13:46
openstackgerritAndreas Jaeger proposed openstack/osc-lib master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52460413:46
*** salv-orlando has joined #openstack-sdks13:52
-openstackstatus- NOTICE: gerrit has been restarted to get it back to its normal speed.13:52
openstackgerritAndreas Jaeger proposed openstack/os-service-types master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52460913:54
*** salv-orlando has quit IRC13:57
*** bobh has joined #openstack-sdks14:00
*** edleafe is now known as figleaf14:03
*** salv-orlando has joined #openstack-sdks14:15
*** chenybd_ has joined #openstack-sdks14:16
openstackgerritDean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations  https://review.openstack.org/52393814:17
*** cdent is now known as dentures14:26
*** dentures is now known as denture14:26
*** denture is now known as cdent14:27
mordredioggstream: awesome! I'm excited14:33
mordredioggstream: minor bug report: "PyCon Nove will be organized into four distinct tracks" ... but then only three are listed14:33
ioggstreammordred: if we get enough proposal we could get an infra/devop track for openstack, ansible, datacenter14:35
*** jkilpatr has quit IRC14:35
ioggstreammordred: have you ever been in Florence?14:35
mordredioggstream: I have! my wife and I went there laster year after the OpenStack Ops Meetup in Milan - but only for one night14:37
mordredioggstream: we liked it very much and definitely need to return for more :)14:37
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images  https://review.openstack.org/52297414:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects  https://review.openstack.org/52297514:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests  https://review.openstack.org/52297614:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars  https://review.openstack.org/52297714:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute  https://review.openstack.org/52297814:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery  https://review.openstack.org/52297914:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks.  https://review.openstack.org/52298014:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator  https://review.openstack.org/52325814:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud  https://review.openstack.org/52325914:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client  https://review.openstack.org/52348314:38
ioggstreamwhat a bunch of patches!14:40
ioggstreamgoing python 3.x...14:40
*** slaweq has quit IRC14:41
*** slaweq has joined #openstack-sdks14:41
* mordred enjoys eating all of the build resources14:42
openstackgerritMerged openstack/os-service-types master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52460914:45
*** chenybd_ has quit IRC14:55
*** chenybd_ has joined #openstack-sdks14:56
*** d0ugal has quit IRC14:56
*** chenybd_ has quit IRC15:00
*** david-lyle has joined #openstack-sdks15:07
*** gouthamr has joined #openstack-sdks15:11
mordreddtroyer: 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-tips15:11
mordreddtroyer: we could also just make another job specifically for cliff, it's not hard or anything15:12
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs  https://review.openstack.org/52463915:15
mordreddtroyer: ^^ there's that, should you think it's a good idea15:16
*** jkilpatr has joined #openstack-sdks15:16
*** honza has quit IRC15:17
*** d0ugal has joined #openstack-sdks15:18
dtroyermordred: +215:18
dtroyermordred: its taken me 3 tries but there were more things to fix in that job after getting the import right15:19
*** honza has joined #openstack-sdks15:20
mordreddtroyer: woot. well -that tells me the job is worthwhile and that it'll be very good to have in place :)15:20
*** honza is now known as Guest3354515:20
slaweqdtroyer: hello, will You have time to check my answers to Your comments in https://review.openstack.org/#/c/515401/ ?15:22
slaweqdtroyer: thx in advance15:22
thingeeos-service-types doesn't have branches, but reno wants branches https://review.openstack.org/#/c/520475/115:22
dtroyerslaweq: it's on my list for today15:24
slaweqdtroyer: thx a lot15:24
*** dtantsur|brb is now known as dtantsur15:27
openstackgerritMonty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions  https://review.openstack.org/52464315:27
*** d0ugal has quit IRC15:29
openstackgerritMonty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions  https://review.openstack.org/52464315:31
*** ianw has quit IRC15:34
openstackgerritMonty Taylor proposed openstack/os-service-types master: Remove reference to stable/pike from releasenotes  https://review.openstack.org/52464715:36
mordredthingee, cmurphy: ^^15:36
cmurphymordred: /home/colleen/dev/os-service-types/releasenotes/source/index.rst:5:toctree contains reference to nonexisting document u'master'15:38
cmurphyrename pike.rst to master.rst i think15:38
mordredgah15:38
mordredtahnks15:39
openstackgerritMonty Taylor proposed openstack/os-service-types master: Remove reference to stable/pike from releasenotes  https://review.openstack.org/52464715:39
dtroyerdhellmann: left a copule of comments in that etherpad, overall looks like a good approach15:39
openstackgerritMonty Taylor proposed openstack/os-service-types master: Updated from global requirements  https://review.openstack.org/52047515:40
Shrewswould anyone else like to review mordred's merge-shade stack, or shall i just +W them? they're pretty much just cherry-picks from shade15:40
mordredShrews: I think you can just +W them - they've gone through the process once already15:40
Shrewsmordred: gah, is this the root? https://review.openstack.org/522974  it's the only one with a pypy fail15:42
* Shrews doesn't like the new gerrit's display of related changes15:42
mordredhrm. why does that havea pypy fail - the pypy issue should have been fixed ...15:42
mordredShrews: oh - my - that's an actual weird fail, rather than just a job issue ...15:43
Shrewsyay15:43
mordredor, rather, it's just weird- but looking in to it15:44
Shrewsk. gotta step out for a coffee run real quick anyway15:45
dtroyerShrews: 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
Shrewsdtroyer: *nod*15:46
*** d0ugal has joined #openstack-sdks15:46
*** Guest33545 is now known as honza15:48
mordreddtroyer, 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 unused15:48
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images  https://review.openstack.org/52297415:49
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects  https://review.openstack.org/52297515:49
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests  https://review.openstack.org/52297615:49
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars  https://review.openstack.org/52297715:49
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute  https://review.openstack.org/52297815:49
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery  https://review.openstack.org/52297915:50
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks.  https://review.openstack.org/52298015:50
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator  https://review.openstack.org/52325815:50
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud  https://review.openstack.org/52325915:50
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client  https://review.openstack.org/52348315:50
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Remove openstack-tox-pypy jobs  https://review.openstack.org/52465115:50
*** david-lyle has quit IRC15:53
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add osc-tox-unit-tips jobs  https://review.openstack.org/52465615:55
openstackgerritMerged openstack/os-service-types master: Remove reference to stable/pike from releasenotes  https://review.openstack.org/52464715:57
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add shade and python-openstacksdk tips jobs  https://review.openstack.org/52465916:00
*** slaweq has quit IRC16:04
*** slaweq has joined #openstack-sdks16:04
openstackgerritMerged openstack/os-service-types master: Updated from global requirements  https://review.openstack.org/52047516:06
*** slaweq has quit IRC16:09
amotokimordred: one question on https://review.openstack.org/#/c/522369/3/tox.ini : do you drop '{toxinidir}/demoapp' from docs requirements temporarily?16:14
amotokimordred:  This is required to build cliff doc expectedly but I am okay if it is provisional.16:14
*** david-lyle has joined #openstack-sdks16:22
mordredamotoki: there's a previous patch which changes how it works to use code from in the tree16:27
mordredamotoki: oh - wait- that was from you :)16:27
mordredamotoki: I think that's a bug in my patch then - let me fix16:28
amotokimordred: thanks :)16:28
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs  https://review.openstack.org/52463916:28
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Make py27 and py35 versions and template of unit-tips jobs  https://review.openstack.org/52467016:28
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52467116:28
openstackgerritMonty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt  https://review.openstack.org/52236916:34
openstackgerritMonty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions  https://review.openstack.org/52464316:34
openstackgerritMonty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox  https://review.openstack.org/52398416:34
openstackgerritMonty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs  https://review.openstack.org/52237016:34
mordredamotoki: ^^ that should be happier I think16:35
amotokimordred: i think we need {toxinidir}/demoapp in [testenv:docs] deps too16:38
*** e0ne has quit IRC16:39
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add osc, shade and sdk tips jobs  https://review.openstack.org/52465616:39
*** slaweq has joined #openstack-sdks16:42
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Add osc-tox-unit-tips jobs  https://review.openstack.org/52467616:43
mordredamotoki: duh. of course. silly me :)16:43
mordredamotoki: oh. wait. this is ... darn it16:45
mordredthis won't work at all ... let me mark that WIP for now16:45
openstackgerritMerged openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52449816:46
*** slaweq has quit IRC16:46
*** madorn has quit IRC16:46
amotokimordred: 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 on16:47
*** madorn has joined #openstack-sdks16:48
mordredamotoki: 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 testing16:49
mordredamotoki: otherwise I think we're going to wind up getting too clever all just to avoid adding one line to global-requirements16:49
amotokimordred: honestly i am not sure cliffdemo is worth published to pypi16:50
amotokimordred: the current cliffdemo in PyPI is really old16:50
mordredamotoki: it's already published there- that's how it was being installed before your patch16:51
*** slaweq_ has joined #openstack-sdks16:52
amotokimordred: yes. it works with cliffdemo in PyPI, but cliffdemo itself is mainly for demo purpose (+ test for the sphinx directive)16:53
mordredamotoki: totally. however, there is already precedent - openstack-release-test is in global-requirements for a similar reason16:53
amotokimordred: i am not sure what is the right solution. I would like to hear opinion of dhellmann16:53
mordredamotoki: agree. definitely want his opinion :)16:54
mordredamotoki, dhellmann: for sake of argument, here's a g-r patch adding it: https://review.openstack.org/524678 Add cliffdemo16:57
*** thrash is now known as thrash|biab17:05
*** slaweq_ has quit IRC17:15
openstackgerritMerged openstack/cliff master: Use in-tree cliffdemo app for docs build  https://review.openstack.org/51501617:22
*** rajathagasthya has joined #openstack-sdks17:22
mordreddtroyer: 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 patches17:32
mordreddtroyer: and https://review.openstack.org/#/c/523483/6 is a non-cherry-pick that touches that section of the code17:32
*** tonythomas has joined #openstack-sdks17:43
*** jpich has quit IRC17:51
*** ragasthya has joined #openstack-sdks17:51
*** ioggstream has quit IRC17:54
*** rajathagasthya has quit IRC17:54
*** dtantsur is now known as dtantsur|afk18:00
*** david-lyle has quit IRC18:02
*** david-lyle has joined #openstack-sdks18:04
openstackgerritMerged openstack/python-openstacksdk master: Add pools attribute to load balancer heath monitor  https://review.openstack.org/52296218:10
*** rabel has joined #openstack-sdks18:21
openstackgerritDoug Hellmann proposed openstack/cliff master: do not require installing demo app to build docs  https://review.openstack.org/52469518:22
*** gkadam has quit IRC18:23
thingeedtroyer, mordred with https://review.openstack.org/#/c/523834/3 and https://review.openstack.org/#/c/516701/7/openstackclient/network/common.py extensions being proposed in osc and python-openstacksdk, perhaps we can have some common extension checking code for the existing cases, but stop accept new cases?18:32
openstackgerritMonty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt  https://review.openstack.org/52236918:38
openstackgerritMonty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions  https://review.openstack.org/52464318:38
openstackgerritMonty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox  https://review.openstack.org/52398418:38
openstackgerritMonty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs  https://review.openstack.org/52237018:38
mordreddhellmann: ^^ reparented on your patch18:38
dhellmannmordred : looking18:38
*** ragasthya has quit IRC18:39
*** rabel has quit IRC18:51
*** thrash|biab is now known as thrash19:04
*** david-lyle has quit IRC19:05
mordredstevemar: responded to https://review.openstack.org/#/c/524670 - I can update it to move the branch exclusion down into the project tepmlate if you prefer19:06
*** david-lyle has joined #openstack-sdks19:21
*** jkilpatr has quit IRC19:33
*** dklyle has joined #openstack-sdks19:36
*** david-lyle has quit IRC19:37
openstackgerritMonty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs  https://review.openstack.org/52237019:37
mordreddhellmann: zomg. ^^ passes!!!19:47
dhellmannmordred : woot!19:56
mordreddhellmann, dtroyer: cliff runs legacy-tempest-dsvm-neutron-src ... I think (once it's solid) that osc-functional-devstack-tips would be a good replacement for that - since hte point would be to make sure that patches to cliff don't break devstack by breaking osc, right?19:56
dhellmannI think that's the point of running that job, yes19:57
*** dklyle has quit IRC19:57
openstackgerritMerged openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs  https://review.openstack.org/52463919:58
dhellmannit looks like we're still waiting for jobs to pass on my patch at the bottom of that stack19:58
mordredcool. once dtroyer has the -tips job fixed I can put up a patch to switch that over, and we'll be done with legacy jobs across osc, sdk and cliff19:58
mordredyah - the legacy devstack job is taking its sweet time19:59
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Fix SDK Connection creation alternative to Profile  https://review.openstack.org/52393820:01
openstackgerritDean Troyer proposed openstack/python-openstackclient master: Rework Network client config for new SDK Connection  https://review.openstack.org/52471520:01
mordreddhellmann, dtroyer: also, fwiw, I put up a patch to keystoneauth to run osc, sdk and shade functional tests on keystoneauth changes - so I think we're actually getting into a really good place in terms of cross-testing these base-level pieces \o/ https://review.openstack.org/#/c/524656/20:01
mordredooh. that looks like a fix from dtroyer20:01
dtroyerfwiw, the last run of my test timed out but I think I've got it all, ^^^ are the two that get this into production shape20:01
dtroyerso if they look good and work they can merge20:02
mordredwoot20:02
dtroyerI'm prepping the osc-lib part of that but I want this to merge first and put a little time on the compat code path20:02
mordreddtroyer: ++20:03
*** purplerbot has quit IRC20:09
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud  https://review.openstack.org/52325920:18
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client  https://review.openstack.org/52348320:18
mordreddtroyer: wow. https://review.openstack.org/#/c/524670 has failed functional tests twice - but neither time has it had anything to do with the patch, and the two times were not for the same reason20:38
*** ralonsoh has quit IRC20:50
*** jkilpatr has joined #openstack-sdks20:51
*** purplerbot has joined #openstack-sdks20:52
dtroyermordred: yup, it had been behaving for a bit, aggregates is one of the usual problem children.  volumes had been a big one for a while, Cinder team sorted out at least some of that but I've noticed those returning again too.21:09
*** ianw has joined #openstack-sdks21:09
dtroyerI've spent a good bit of time removing races from our tests, so that's also some of the trouble21:10
openstackgerritMerged openstack/python-openstacksdk master: Remove openstack-tox-pypy jobs  https://review.openstack.org/52465121:13
openstackgerritMerged openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images  https://review.openstack.org/52297421:13
openstackgerritMerged openstack/python-openstacksdk master: Add method to cleanup autocreated image objects  https://review.openstack.org/52297521:13
openstackgerritMerged openstack/python-openstacksdk master: Complete move of baremetal machine tests  https://review.openstack.org/52297621:13
*** slaweq has joined #openstack-sdks21:15
dtroyermordred: so what did I do wrong?  the functional-tips jobs are not running since some time this morning...21:18
dtroyerthe one that timed out earlier is the last one I've seen21:19
slaweqdtroyer: hi again21:21
slaweqdtroyer: thx for Your comment in https://review.openstack.org/#/c/51540121:21
slaweqI have one more question to You about this patch21:21
dtroyerok21:21
slaweqcan You see my last comment on it maybe?21:21
slaweqthx in advance21:21
dtroyerI would not add the —details option to the show command21:22
dtroyershow commands already return everything they receive from the API21:22
slaweqyes, I know21:23
dtroyerit would only be doing additional queries for things in the resource fields (looking up image names for example) that we add to show, normally21:23
slaweqbut I was asking about list command21:23
dtroyerthe comment asks about quota show, did you mean list there?21:23
slaweqbut list command currently is IMHO returning quota per tenant21:23
slaweqsorry, maybe I don't understand something still21:24
slaweqso21:24
dtroyerok I may have misread that then21:24
dtroyerin "change its behaviour" you man list?21:24
slaweqI understood that You want to add "--detail" flag to list command instead of show (like it is currently proposed), right?21:24
dtroyers/man/mean/21:24
slaweqyes, I mean list21:24
dtroyeryes21:24
slaweqso, AFAIU list command currently lists "non default" quota for all tenants21:25
slaweqI want to add call which will return quota values for one tenant only21:26
dtroyerok, then I get it.  and yes, sometimes we do change things about how list commands based on options.  I should probably think about it a bit deeper to be sure, but we may want to add some options to do those sorts of selections, and leave the defaults to get what we have today21:26
dtroyerdoes that make sense?21:26
dtroyernormally, for one tenant, we would add —project for the filter, or something (I can't rememner exactly what) to use the tenant used for auth21:26
slaweqso You propose to add some option (detail) to list command instead of show command21:27
dtroyeryes21:27
slaweqand this option will change behaviour of List command21:27
dtroyeryes21:27
slaweqand different data will be returned21:27
slaweqok, I understand now21:28
slaweqI will try to do it this way then21:28
slaweqthx21:28
dtroyerby different, you mean they are still quotas, but filtered differently?21:28
slaweqyes, still quota21:28
dtroyeryes, that is it21:28
slaweqbut only for one tenant but with details about used/reserved/limit values21:28
dtroyerhow do you select the tenant?21:28
slaweqinstead of list of tenants and limits for each of them21:29
slaweqprobably it will be taken from context or set by project_id parameter21:29
dtroyerok, that should work21:29
slaweqI will see how it's done now for "show" command21:29
dtroyerit may make sense to re-think the entire quota command set, we can make breaking changes for osc 4.0 if this is getting too awkward21:30
slaweqok, I will start doing that and I will see how it will go :)21:31
slaweqbut thx a lot for Your help on that21:31
dtroyerthank you for staying with this!21:31
slaweqno prob21:32
mordreddtroyer: oh, like they are not even executing ... lemme look21:37
dtroyerI'm not sure where to look for that21:38
*** htaccess has joined #openstack-sdks21:39
htaccessquestion did shade used to require openstacksdk and python-openstackclient but no longer does21:43
htaccessit 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.021:44
dtroyerbefore the merge, shade did not require the SDK or OSC, it originally used the project python libs, then brought al of that internal.21:44
dtroyerI don't have an older shade handy, is either of those listed in requirements.txt?21:45
dtroyerthey may be in test-requirements.txt to do testing21:45
* htaccess goes to look21:46
htaccessdiff shade-1.25.0/requirements.txt  shade-1.24.0/requirements.txt only shows six and futures version changes21:50
htaccessare the packages in test-requirements.txt actually installed when you do a pip install?21:51
mordredhtaccess: openstacksdk and python-openstackclient have in the past come as transitive depends via python-ironicclient21:52
htaccessmordred: that makes more sens21:52
htaccessso its probably due to python-ironicclient not pulling them in any more21:53
mordredyah. it looks like python-ironicclient finally removed the dep on python-openstackclient21:53
htaccessanyway 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
htaccessthanks21:54
mordredalthough we're also about to drop our depend on python-ironicclient ... so the transitive depend wouldhave cone away as of 1.26 anyway21:54
mordredhtaccess: woot!21:54
htaccess:)21:54
mordredhtaccess: now - in the not-too-distant future shade will grow a depend on openstacksdk ... but that's unlikely before january I'd guess21:54
htaccessah, interesting21:55
mordredhtaccess: 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 sdk21:57
mordredbut I want to make sure we're happy with how it's working sdk-side before we start mucking with consuming it from shade21:58
openstackgerritCarlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None  https://review.openstack.org/52473922:01
mordreddtroyer: hrm.22:01
htaccessah ok, so in the future i will be able to install openstacksdk and get shade plus whats already in the sdk?22:02
mordredhtaccess: yup22:03
mordreddtroyer: I've looped in jeblair to help me look :)22:09
dtroyerkk22:10
openstackgerritCarlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None  https://review.openstack.org/52473922:17
*** figleaf is now known as edleafe22:48
*** cdent has quit IRC22:48
*** bobh has quit IRC22:49
*** slaweq_ has joined #openstack-sdks22:51
*** slaweq_ has quit IRC22:53
*** slaweq_ has joined #openstack-sdks22:53
*** slaweq_ has quit IRC22:54
*** slaweq has quit IRC22:54
*** slaweq_ has joined #openstack-sdks22:54
*** slaweq_ is now known as slaweq22:55
*** slaweq has quit IRC23:03
*** slaweq has joined #openstack-sdks23:04
*** slaweq has joined #openstack-sdks23:05
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud  https://review.openstack.org/52325923:09
*** dayou has quit IRC23:16
*** slaweq has quit IRC23:21
*** tonythomas has quit IRC23:33
openstackgerritMerged openstack/python-openstackclient master: Make py27 and py35 versions and template of unit-tips jobs  https://review.openstack.org/52467023:33
*** salv-orlando has quit IRC23:37
openstackgerritMerged openstack/python-openstackclient master: Avoid tox_install.sh for constraints support  https://review.openstack.org/52467123:37
*** salv-orlando has joined #openstack-sdks23:38
*** salv-orlando has quit IRC23:42

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!