*** bobh has joined #openstack-sdks | 00:01 | |
mordred | woot! the f26 pypy seems to be working, as does the https://review.openstack.org/523938 tips fix | 00:12 |
---|---|---|
mordred | (and by seems to be working, I mean it passed) | 00:12 |
*** david-lyle has quit IRC | 00:15 | |
*** dayou has joined #openstack-sdks | 00:16 | |
adriant | thingee: 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 IRC | 00:22 | |
*** david-lyle has joined #openstack-sdks | 00:29 | |
openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly https://review.openstack.org/524418 | 00:33 |
adriant | mordred: ^ there, that should fix swift 404s being broken | 00:34 |
*** david-lyle has quit IRC | 00:34 | |
adriant | and it is unlikely to break anything since only swift appears to actually use HEAD | 00:35 |
adriant | hmmm, or at least called directly | 00:36 |
adriant | yeah, allow_head seems to only be applicable to swift :/ | 00:37 |
adriant | so I can see why no one found this bug before | 00:37 |
mordred | adriant: to my knowledge, swift is the only service that uses head at all | 00:38 |
adriant | yep, that would be why this was broken :P | 00:39 |
mordred | heh | 00:40 |
*** salv-orlando has joined #openstack-sdks | 00:42 | |
*** salv-orlando has quit IRC | 00:46 | |
adriant | mordred: and it appears part of this issue is actually fixed in resource2 | 00:49 |
mordred | woot! so the part of the plan which is "finish migrating to resource2" is a good one | 00:50 |
adriant | are we entirely dropping resource and moving to resource2 or is it being kept around for some form of backwards compat? | 00:51 |
openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly https://review.openstack.org/524418 | 01:00 |
*** salv-orlando has joined #openstack-sdks | 01:43 | |
*** salv-orlando has quit IRC | 01:48 | |
dtroyer | adriant: 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 released | 02:09 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 02:14 |
*** Anticime1 has joined #openstack-sdks | 02:18 | |
*** annp has joined #openstack-sdks | 02:20 | |
*** spredzy_ has joined #openstack-sdks | 02:20 | |
*** Dinesh__Bhor has joined #openstack-sdks | 02:21 | |
*** markvoelker has quit IRC | 02:22 | |
*** spredzy has quit IRC | 02:22 | |
*** fzdarsky|afk has quit IRC | 02:22 | |
*** Dinesh_Bhor has quit IRC | 02:22 | |
*** edleafe has quit IRC | 02:22 | |
*** Anticimex has quit IRC | 02:22 | |
*** spredzy_ is now known as spredzy | 02:22 | |
*** bobh has quit IRC | 02:23 | |
*** toabctl has quit IRC | 02:25 | |
*** edleafe has joined #openstack-sdks | 02:27 | |
*** markvoelker has joined #openstack-sdks | 02:27 | |
*** toabctl has joined #openstack-sdks | 02:27 | |
*** dave-mccowan has joined #openstack-sdks | 02:30 | |
*** ldl has joined #openstack-sdks | 02:32 | |
*** salv-orlando has joined #openstack-sdks | 02:44 | |
*** salv-orlando has quit IRC | 02:49 | |
*** dave-mccowan has quit IRC | 02:51 | |
*** gildub has joined #openstack-sdks | 03:01 | |
*** salv-orlando has joined #openstack-sdks | 03:45 | |
openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Fix Zuul jobs https://review.openstack.org/524353 | 03:45 |
dayou | Anyone 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 |
dayou | With the following I got "__init__() got an unexpected keyword argument 'token'" error | 03:49 |
*** salv-orlando has quit IRC | 03:49 | |
*** bobh has joined #openstack-sdks | 04:09 | |
SamYaple | mordred: meh. its only pypy | 04:12 |
* SamYaple hides | 04:12 | |
*** rajathagasthya has joined #openstack-sdks | 04:34 | |
*** bobh has quit IRC | 04:39 | |
*** ldl has quit IRC | 04:40 | |
*** ldl has joined #openstack-sdks | 04:42 | |
openstackgerrit | Gilles Dubreuil proposed openstack/api-wg master: Adds API-Schema guide https://review.openstack.org/524467 | 04:45 |
*** salv-orlando has joined #openstack-sdks | 04:46 | |
*** salv-orlando has quit IRC | 04:50 | |
*** ldl has quit IRC | 04:57 | |
*** reedip has quit IRC | 05:03 | |
*** ldl has joined #openstack-sdks | 05:07 | |
*** rajathagasthya has quit IRC | 05:08 | |
*** david-lyle has joined #openstack-sdks | 05:13 | |
*** shu-mutou-AWAY is now known as shu-mutou | 05:16 | |
*** gouthamr has quit IRC | 05:44 | |
*** salv-orlando has joined #openstack-sdks | 05:47 | |
*** salv-orlando has quit IRC | 05:51 | |
*** salv-orlando has joined #openstack-sdks | 06:14 | |
*** david-lyle has quit IRC | 06:30 | |
adriant | dayou: the issue is that it isn't 'auth_plugin' but 'auth_type' | 06:32 |
adriant | I ran into that issue using mordred's example as well :P | 06:32 |
adriant | it's assuming the password plugin, and getting confused | 06:33 |
*** ldl has quit IRC | 06:34 | |
dayou | adriant, thanks, that works, also I found user_domain_id can't be used also! | 06:50 |
*** reedip has joined #openstack-sdks | 07:02 | |
openstackgerrit | Andreas Jaeger proposed openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support https://review.openstack.org/524498 | 07:14 |
*** gildub has quit IRC | 07:19 | |
*** e0ne has joined #openstack-sdks | 07:27 | |
openstackgerrit | Merged openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | 07:40 |
*** ioggstream has joined #openstack-sdks | 07:45 | |
openstackgerrit | Andreas Jaeger proposed openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support https://review.openstack.org/524498 | 07:58 |
*** e0ne has quit IRC | 08:03 | |
*** slaweq has joined #openstack-sdks | 08:03 | |
*** salv-orlando has quit IRC | 08:06 | |
*** gildub has joined #openstack-sdks | 08:08 | |
*** gildub has quit IRC | 08:14 | |
*** gkadam has joined #openstack-sdks | 08:24 | |
*** salv-orlando has joined #openstack-sdks | 08:33 | |
*** ralonsoh has joined #openstack-sdks | 08:44 | |
*** shu-mutou is now known as shu-mutou-AWAY | 08:56 | |
*** jpich has joined #openstack-sdks | 08:58 | |
*** dmellado has joined #openstack-sdks | 09:07 | |
*** e0ne has joined #openstack-sdks | 09:16 | |
openstackgerrit | Édouard Thuleau proposed openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported https://review.openstack.org/523834 | 09:18 |
*** Dinesh__Bhor has quit IRC | 09:18 | |
*** Dinesh__Bhor has joined #openstack-sdks | 09:19 | |
*** dtantsur|afk is now known as dtantsur | 09:51 | |
*** markvoelker has quit IRC | 09:55 | |
*** openstackgerrit has quit IRC | 10:33 | |
*** LindaWang1 has joined #openstack-sdks | 10:34 | |
*** markvoelker has joined #openstack-sdks | 10:55 | |
*** LindaWang1 has quit IRC | 11:02 | |
*** Dinesh__Bhor has quit IRC | 11:30 | |
slaweq | dtroyer: hi, I respin work on https://review.openstack.org/#/c/515401/2 finally | 11:37 |
slaweq | and I have some doubt about Your comment and List command | 11:37 |
slaweq | there is already List command in OSC | 11:37 |
slaweq | dtroyer: can You check my comments there and tell me what You think about it? thx in advance | 11:37 |
*** LindaWang1 has joined #openstack-sdks | 11:41 | |
ioggstream | Hi @all! We opened the pycon Florence c4p https://www.pycon.it/en/call-for-proposals/ and your talks are very welcome! rcarrillocruz mordred | 11:41 |
*** LindaWang1 has quit IRC | 11:41 | |
rcarrillocruz | god | 11:41 |
rcarrillocruz | florence | 11:41 |
rcarrillocruz | when is that | 11:41 |
rcarrillocruz | ? | 11:41 |
rcarrillocruz | that'd be dead easy to convince my wife to go with me | 11:42 |
*** salv-orlando has quit IRC | 11:49 | |
*** salv-orlando has joined #openstack-sdks | 11:50 | |
ioggstream | rcarrillocruz: it's in April | 11:51 |
ioggstream | rcarrillocruz: 19TH 22TH APRIL 2018 | 11:51 |
ioggstream | the conference has historically the best catering of every pycon | 11:51 |
ioggstream | :D | 11:51 |
rcarrillocruz | welllll, if that is better than Bilbao, that must be awesome! (even though Bilbao was not technically pycon, but europython) | 11:52 |
ioggstream | rcarrillocruz: loOL | 11:53 |
ioggstream | rcarrillocruz: chianti wine, Italian cheese selections, pasta, ... | 11:53 |
* rcarrillocruz drools | 11:54 | |
ioggstream | and well, Firenze is stunning | 11:54 |
*** salv-orlando has quit IRC | 11:55 | |
*** thrash|g0ne is now known as thrash | 12:14 | |
*** ioggstream has quit IRC | 12:47 | |
briancurtin | adriant: 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 time | 12:52 |
*** cdent has joined #openstack-sdks | 12:59 | |
*** dtantsur is now known as dtantsur|brb | 13:14 | |
*** markvoelker has quit IRC | 13:25 | |
*** markvoelker has joined #openstack-sdks | 13:25 | |
*** ioggstream has joined #openstack-sdks | 13:34 | |
*** stephenfin is now known as finucannot | 13:40 | |
*** openstackgerrit has joined #openstack-sdks | 13:46 | |
openstackgerrit | Andreas Jaeger proposed openstack/osc-lib master: Avoid tox_install.sh for constraints support https://review.openstack.org/524604 | 13:46 |
*** salv-orlando has joined #openstack-sdks | 13:52 | |
-openstackstatus- NOTICE: gerrit has been restarted to get it back to its normal speed. | 13:52 | |
openstackgerrit | Andreas Jaeger proposed openstack/os-service-types master: Avoid tox_install.sh for constraints support https://review.openstack.org/524609 | 13:54 |
*** salv-orlando has quit IRC | 13:57 | |
*** bobh has joined #openstack-sdks | 14:00 | |
*** edleafe is now known as figleaf | 14:03 | |
*** salv-orlando has joined #openstack-sdks | 14:15 | |
*** chenybd_ has joined #openstack-sdks | 14:16 | |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 14:17 |
*** cdent is now known as dentures | 14:26 | |
*** dentures is now known as denture | 14:26 | |
*** denture is now known as cdent | 14:27 | |
mordred | ioggstream: awesome! I'm excited | 14:33 |
mordred | ioggstream: minor bug report: "PyCon Nove will be organized into four distinct tracks" ... but then only three are listed | 14:33 |
ioggstream | mordred: if we get enough proposal we could get an infra/devop track for openstack, ansible, datacenter | 14:35 |
*** jkilpatr has quit IRC | 14:35 | |
ioggstream | mordred: have you ever been in Florence? | 14:35 |
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 |
mordred | ioggstream: we liked it very much and definitely need to return for more :) | 14:37 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | 14:38 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483 | 14:38 |
ioggstream | what a bunch of patches! | 14:40 |
ioggstream | going python 3.x... | 14:40 |
*** slaweq has quit IRC | 14:41 | |
*** slaweq has joined #openstack-sdks | 14:41 | |
* mordred enjoys eating all of the build resources | 14:42 | |
openstackgerrit | Merged openstack/os-service-types master: Avoid tox_install.sh for constraints support https://review.openstack.org/524609 | 14:45 |
*** chenybd_ has quit IRC | 14:55 | |
*** chenybd_ has joined #openstack-sdks | 14:56 | |
*** d0ugal has quit IRC | 14:56 | |
*** chenybd_ has quit IRC | 15:00 | |
*** david-lyle has joined #openstack-sdks | 15:07 | |
*** gouthamr has joined #openstack-sdks | 15:11 | |
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:11 |
mordred | dtroyer: we could also just make another job specifically for cliff, it's not hard or anything | 15:12 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs https://review.openstack.org/524639 | 15:15 |
mordred | dtroyer: ^^ there's that, should you think it's a good idea | 15:16 |
*** jkilpatr has joined #openstack-sdks | 15:16 | |
*** honza has quit IRC | 15:17 | |
*** d0ugal has joined #openstack-sdks | 15:18 | |
dtroyer | mordred: +2 | 15:18 |
dtroyer | mordred: its taken me 3 tries but there were more things to fix in that job after getting the import right | 15:19 |
*** honza has joined #openstack-sdks | 15:20 | |
mordred | dtroyer: 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 Guest33545 | 15:20 | |
slaweq | dtroyer: hello, will You have time to check my answers to Your comments in https://review.openstack.org/#/c/515401/ ? | 15:22 |
slaweq | dtroyer: thx in advance | 15:22 |
thingee | os-service-types doesn't have branches, but reno wants branches https://review.openstack.org/#/c/520475/1 | 15:22 |
dtroyer | slaweq: it's on my list for today | 15:24 |
slaweq | dtroyer: thx a lot | 15:24 |
*** dtantsur|brb is now known as dtantsur | 15:27 | |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643 | 15:27 |
*** d0ugal has quit IRC | 15:29 | |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643 | 15:31 |
*** ianw has quit IRC | 15:34 | |
openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Remove reference to stable/pike from releasenotes https://review.openstack.org/524647 | 15:36 |
mordred | thingee, cmurphy: ^^ | 15:36 |
cmurphy | mordred: /home/colleen/dev/os-service-types/releasenotes/source/index.rst:5:toctree contains reference to nonexisting document u'master' | 15:38 |
cmurphy | rename pike.rst to master.rst i think | 15:38 |
mordred | gah | 15:38 |
mordred | tahnks | 15:39 |
openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Remove reference to stable/pike from releasenotes https://review.openstack.org/524647 | 15:39 |
dtroyer | dhellmann: left a copule of comments in that etherpad, overall looks like a good approach | 15:39 |
openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/520475 | 15:40 |
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 |
mordred | Shrews: I think you can just +W them - they've gone through the process once already | 15:40 |
Shrews | mordred: gah, is this the root? https://review.openstack.org/522974 it's the only one with a pypy fail | 15:42 |
* Shrews doesn't like the new gerrit's display of related changes | 15:42 | |
mordred | hrm. why does that havea pypy fail - the pypy issue should have been fixed ... | 15:42 |
mordred | Shrews: oh - my - that's an actual weird fail, rather than just a job issue ... | 15:43 |
Shrews | yay | 15:43 |
mordred | or, rather, it's just weird- but looking in to it | 15:44 |
Shrews | k. gotta step out for a coffee run real quick anyway | 15:45 |
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 |
Shrews | dtroyer: *nod* | 15:46 |
*** d0ugal has joined #openstack-sdks | 15:46 | |
*** Guest33545 is now known as honza | 15:48 | |
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:48 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974 | 15:49 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975 | 15:49 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976 | 15:49 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977 | 15:49 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978 | 15:49 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979 | 15:50 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980 | 15:50 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258 | 15:50 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | 15:50 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483 | 15:50 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack-tox-pypy jobs https://review.openstack.org/524651 | 15:50 |
*** david-lyle has quit IRC | 15:53 | |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add osc-tox-unit-tips jobs https://review.openstack.org/524656 | 15:55 |
openstackgerrit | Merged openstack/os-service-types master: Remove reference to stable/pike from releasenotes https://review.openstack.org/524647 | 15:57 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add shade and python-openstacksdk tips jobs https://review.openstack.org/524659 | 16:00 |
*** slaweq has quit IRC | 16:04 | |
*** slaweq has joined #openstack-sdks | 16:04 | |
openstackgerrit | Merged openstack/os-service-types master: Updated from global requirements https://review.openstack.org/520475 | 16:06 |
*** slaweq has quit IRC | 16:09 | |
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 |
amotoki | mordred: This is required to build cliff doc expectedly but I am okay if it is provisional. | 16:14 |
*** david-lyle has joined #openstack-sdks | 16:22 | |
mordred | amotoki: there's a previous patch which changes how it works to use code from in the tree | 16:27 |
mordred | amotoki: oh - wait- that was from you :) | 16:27 |
mordred | amotoki: I think that's a bug in my patch then - let me fix | 16:28 |
amotoki | mordred: thanks :) | 16:28 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs https://review.openstack.org/524639 | 16:28 |
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:28 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Avoid tox_install.sh for constraints support https://review.openstack.org/524671 | 16:28 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369 | 16:34 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643 | 16:34 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984 | 16:34 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370 | 16:34 |
mordred | amotoki: ^^ that should be happier I think | 16:35 |
amotoki | mordred: i think we need {toxinidir}/demoapp in [testenv:docs] deps too | 16:38 |
*** e0ne has quit IRC | 16:39 | |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add osc, shade and sdk tips jobs https://review.openstack.org/524656 | 16:39 |
*** slaweq has joined #openstack-sdks | 16:42 | |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add osc-tox-unit-tips jobs https://review.openstack.org/524676 | 16:43 |
mordred | amotoki: duh. of course. silly me :) | 16:43 |
mordred | amotoki: oh. wait. this is ... darn it | 16:45 |
mordred | this won't work at all ... let me mark that WIP for now | 16:45 |
openstackgerrit | Merged openstack/python-openstacksdk master: Avoid tox_install.sh for constraints support https://review.openstack.org/524498 | 16:46 |
*** slaweq has quit IRC | 16:46 | |
*** madorn has quit IRC | 16: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:47 |
*** madorn has joined #openstack-sdks | 16:48 | |
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 |
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:49 |
amotoki | mordred: honestly i am not sure cliffdemo is worth published to pypi | 16:50 |
amotoki | mordred: the current cliffdemo in PyPI is really old | 16:50 |
mordred | amotoki: it's already published there- that's how it was being installed before your patch | 16:51 |
*** slaweq_ has joined #openstack-sdks | 16:52 | |
amotoki | mordred: yes. it works with cliffdemo in PyPI, but cliffdemo itself is mainly for demo purpose (+ test for the sphinx directive) | 16:53 |
mordred | amotoki: totally. however, there is already precedent - openstack-release-test is in global-requirements for a similar reason | 16:53 |
amotoki | mordred: i am not sure what is the right solution. I would like to hear opinion of dhellmann | 16:53 |
mordred | amotoki: agree. definitely want his opinion :) | 16:54 |
mordred | amotoki, dhellmann: for sake of argument, here's a g-r patch adding it: https://review.openstack.org/524678 Add cliffdemo | 16:57 |
*** thrash is now known as thrash|biab | 17:05 | |
*** slaweq_ has quit IRC | 17:15 | |
openstackgerrit | Merged openstack/cliff master: Use in-tree cliffdemo app for docs build https://review.openstack.org/515016 | 17:22 |
*** rajathagasthya has joined #openstack-sdks | 17: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 |
mordred | dtroyer: and https://review.openstack.org/#/c/523483/6 is a non-cherry-pick that touches that section of the code | 17:32 |
*** tonythomas has joined #openstack-sdks | 17:43 | |
*** jpich has quit IRC | 17:51 | |
*** ragasthya has joined #openstack-sdks | 17:51 | |
*** ioggstream has quit IRC | 17:54 | |
*** rajathagasthya has quit IRC | 17:54 | |
*** dtantsur is now known as dtantsur|afk | 18:00 | |
*** david-lyle has quit IRC | 18:02 | |
*** david-lyle has joined #openstack-sdks | 18:04 | |
openstackgerrit | Merged openstack/python-openstacksdk master: Add pools attribute to load balancer heath monitor https://review.openstack.org/522962 | 18:10 |
*** rabel has joined #openstack-sdks | 18:21 | |
openstackgerrit | Doug Hellmann proposed openstack/cliff master: do not require installing demo app to build docs https://review.openstack.org/524695 | 18:22 |
*** gkadam has quit IRC | 18:23 | |
thingee | dtroyer, 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 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369 | 18:38 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Replace legacy tips jobs with shiny new versions https://review.openstack.org/524643 | 18:38 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984 | 18:38 |
openstackgerrit | Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370 | 18:38 |
mordred | dhellmann: ^^ reparented on your patch | 18:38 |
dhellmann | mordred : looking | 18:38 |
*** ragasthya has quit IRC | 18:39 | |
*** rabel has quit IRC | 18:51 | |
*** thrash|biab is now known as thrash | 19:04 | |
*** david-lyle has quit IRC | 19:05 | |
mordred | stevemar: responded to https://review.openstack.org/#/c/524670 - I can update it to move the branch exclusion down into the project tepmlate if you prefer | 19:06 |
*** david-lyle has joined #openstack-sdks | 19:21 | |
*** jkilpatr has quit IRC | 19:33 | |
*** dklyle has joined #openstack-sdks | 19:36 | |
*** david-lyle has quit IRC | 19:37 | |
openstackgerrit | Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370 | 19:37 |
mordred | dhellmann: zomg. ^^ passes!!! | 19:47 |
dhellmann | mordred : woot! | 19:56 |
mordred | dhellmann, 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 |
dhellmann | I think that's the point of running that job, yes | 19:57 |
*** dklyle has quit IRC | 19:57 | |
openstackgerrit | Merged openstack/python-openstackclient master: Add cliff and keystoneauth to tips jobs https://review.openstack.org/524639 | 19:58 |
dhellmann | it looks like we're still waiting for jobs to pass on my patch at the bottom of that stack | 19:58 |
mordred | cool. 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 cliff | 19:58 |
mordred | yah - the legacy devstack job is taking its sweet time | 19:59 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Fix SDK Connection creation alternative to Profile https://review.openstack.org/523938 | 20:01 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Rework Network client config for new SDK Connection https://review.openstack.org/524715 | 20:01 |
mordred | dhellmann, 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 |
mordred | ooh. that looks like a fix from dtroyer | 20:01 |
dtroyer | fwiw, the last run of my test timed out but I think I've got it all, ^^^ are the two that get this into production shape | 20:01 |
dtroyer | so if they look good and work they can merge | 20:02 |
mordred | woot | 20:02 |
dtroyer | I'm prepping the osc-lib part of that but I want this to merge first and put a little time on the compat code path | 20:02 |
mordred | dtroyer: ++ | 20:03 |
*** purplerbot has quit IRC | 20:09 | |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | 20:18 |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483 | 20:18 |
mordred | dtroyer: 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 reason | 20:38 |
*** ralonsoh has quit IRC | 20:50 | |
*** jkilpatr has joined #openstack-sdks | 20:51 | |
*** purplerbot has joined #openstack-sdks | 20:52 | |
dtroyer | mordred: 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-sdks | 21:09 | |
dtroyer | I've spent a good bit of time removing races from our tests, so that's also some of the trouble | 21:10 |
openstackgerrit | Merged openstack/python-openstacksdk master: Remove openstack-tox-pypy jobs https://review.openstack.org/524651 | 21:13 |
openstackgerrit | Merged openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974 | 21:13 |
openstackgerrit | Merged openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975 | 21:13 |
openstackgerrit | Merged openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976 | 21:13 |
*** slaweq has joined #openstack-sdks | 21:15 | |
dtroyer | mordred: so what did I do wrong? the functional-tips jobs are not running since some time this morning... | 21:18 |
dtroyer | the one that timed out earlier is the last one I've seen | 21:19 |
slaweq | dtroyer: hi again | 21:21 |
slaweq | dtroyer: thx for Your comment in https://review.openstack.org/#/c/515401 | 21:21 |
slaweq | I have one more question to You about this patch | 21:21 |
dtroyer | ok | 21:21 |
slaweq | can You see my last comment on it maybe? | 21:21 |
slaweq | thx in advance | 21:21 |
dtroyer | I would not add the —details option to the show command | 21:22 |
dtroyer | show commands already return everything they receive from the API | 21:22 |
slaweq | yes, I know | 21:23 |
dtroyer | it would only be doing additional queries for things in the resource fields (looking up image names for example) that we add to show, normally | 21:23 |
slaweq | but I was asking about list command | 21:23 |
dtroyer | the comment asks about quota show, did you mean list there? | 21:23 |
slaweq | but list command currently is IMHO returning quota per tenant | 21:23 |
slaweq | sorry, maybe I don't understand something still | 21:24 |
slaweq | so | 21:24 |
dtroyer | ok I may have misread that then | 21:24 |
dtroyer | in "change its behaviour" you man list? | 21:24 |
slaweq | I understood that You want to add "--detail" flag to list command instead of show (like it is currently proposed), right? | 21:24 |
dtroyer | s/man/mean/ | 21:24 |
slaweq | yes, I mean list | 21:24 |
dtroyer | yes | 21:24 |
slaweq | so, AFAIU list command currently lists "non default" quota for all tenants | 21:25 |
slaweq | I want to add call which will return quota values for one tenant only | 21:26 |
dtroyer | ok, 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 today | 21:26 |
dtroyer | does that make sense? | 21:26 |
dtroyer | normally, for one tenant, we would add —project for the filter, or something (I can't rememner exactly what) to use the tenant used for auth | 21:26 |
slaweq | so You propose to add some option (detail) to list command instead of show command | 21:27 |
dtroyer | yes | 21:27 |
slaweq | and this option will change behaviour of List command | 21:27 |
dtroyer | yes | 21:27 |
slaweq | and different data will be returned | 21:27 |
slaweq | ok, I understand now | 21:28 |
slaweq | I will try to do it this way then | 21:28 |
slaweq | thx | 21:28 |
dtroyer | by different, you mean they are still quotas, but filtered differently? | 21:28 |
slaweq | yes, still quota | 21:28 |
dtroyer | yes, that is it | 21:28 |
slaweq | but only for one tenant but with details about used/reserved/limit values | 21:28 |
dtroyer | how do you select the tenant? | 21:28 |
slaweq | instead of list of tenants and limits for each of them | 21:29 |
slaweq | probably it will be taken from context or set by project_id parameter | 21:29 |
dtroyer | ok, that should work | 21:29 |
slaweq | I will see how it's done now for "show" command | 21:29 |
dtroyer | it 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 awkward | 21:30 |
slaweq | ok, I will start doing that and I will see how it will go :) | 21:31 |
slaweq | but thx a lot for Your help on that | 21:31 |
dtroyer | thank you for staying with this! | 21:31 |
slaweq | no prob | 21:32 |
mordred | dtroyer: oh, like they are not even executing ... lemme look | 21:37 |
dtroyer | I'm not sure where to look for that | 21:38 |
*** htaccess has joined #openstack-sdks | 21:39 | |
htaccess | question did shade used to require openstacksdk and python-openstackclient but no longer does | 21:43 |
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 |
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:44 |
dtroyer | I don't have an older shade handy, is either of those listed in requirements.txt? | 21:45 |
dtroyer | they may be in test-requirements.txt to do testing | 21:45 |
* htaccess goes to look | 21:46 | |
htaccess | diff shade-1.25.0/requirements.txt shade-1.24.0/requirements.txt only shows six and futures version changes | 21:50 |
htaccess | are the packages in test-requirements.txt actually installed when you do a pip install? | 21:51 |
mordred | htaccess: openstacksdk and python-openstackclient have in the past come as transitive depends via python-ironicclient | 21:52 |
htaccess | mordred: that makes more sens | 21:52 |
htaccess | so its probably due to python-ironicclient not pulling them in any more | 21:53 |
mordred | yah. it looks like python-ironicclient finally removed the dep on python-openstackclient | 21:53 |
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 |
htaccess | thanks | 21:54 |
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 |
mordred | htaccess: woot! | 21:54 |
htaccess | :) | 21:54 |
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:54 |
htaccess | ah, interesting | 21:55 |
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:57 |
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 | 21:58 |
openstackgerrit | Carlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739 | 22:01 |
mordred | dtroyer: hrm. | 22:01 |
htaccess | ah ok, so in the future i will be able to install openstacksdk and get shade plus whats already in the sdk? | 22:02 |
mordred | htaccess: yup | 22:03 |
mordred | dtroyer: I've looped in jeblair to help me look :) | 22:09 |
dtroyer | kk | 22:10 |
openstackgerrit | Carlos Konstanski proposed openstack/python-openstackclient master: openstack subnet create fails when tags is None https://review.openstack.org/524739 | 22:17 |
*** figleaf is now known as edleafe | 22:48 | |
*** cdent has quit IRC | 22:48 | |
*** bobh has quit IRC | 22:49 | |
*** slaweq_ has joined #openstack-sdks | 22:51 | |
*** slaweq_ has quit IRC | 22:53 | |
*** slaweq_ has joined #openstack-sdks | 22:53 | |
*** slaweq_ has quit IRC | 22:54 | |
*** slaweq has quit IRC | 22:54 | |
*** slaweq_ has joined #openstack-sdks | 22:54 | |
*** slaweq_ is now known as slaweq | 22:55 | |
*** slaweq has quit IRC | 23:03 | |
*** slaweq has joined #openstack-sdks | 23:04 | |
*** slaweq has joined #openstack-sdks | 23:05 | |
openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | 23:09 |
*** dayou has quit IRC | 23:16 | |
*** slaweq has quit IRC | 23:21 | |
*** tonythomas has quit IRC | 23:33 | |
openstackgerrit | Merged openstack/python-openstackclient master: Make py27 and py35 versions and template of unit-tips jobs https://review.openstack.org/524670 | 23:33 |
*** salv-orlando has quit IRC | 23:37 | |
openstackgerrit | Merged openstack/python-openstackclient master: Avoid tox_install.sh for constraints support https://review.openstack.org/524671 | 23:37 |
*** salv-orlando has joined #openstack-sdks | 23:38 | |
*** salv-orlando has quit IRC | 23:42 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!