| *** sdague has quit IRC | 00:01 | |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369 | 00:03 |
|---|---|---|
| openstackgerrit | Monty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984 | 00:03 |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370 | 00:03 |
| *** yuanying_ has joined #openstack-sdks | 00:04 | |
| *** yuanying has quit IRC | 00:05 | |
| *** edmondsw has quit IRC | 00:15 | |
| *** salv-orlando has joined #openstack-sdks | 00:23 | |
| *** salv-orlando has quit IRC | 00:27 | |
| adriant | mordred: fantastic! :) | 00:40 |
| adriant | mordred: and yeah, I don't think making clouds.yaml harder to use is right, just that, we should make it equally easy to use either. :) | 00:41 |
| adriant | mordred: because for the longest time I always felt that the openstackSDK was tailored more towards clouds.yaml stuff rather than : "I'm an API service which needs to talk to another service." which is more my primary use case. | 00:42 |
| adriant | while the python-<service>clients always felt far better to just throw a session at, and then use. | 00:43 |
| *** tonythomas has quit IRC | 01:06 | |
| TheJulia | SamYaple: Wait... is there a real whiskey chicken thing?!? | 01:20 |
| *** salv-orlando has joined #openstack-sdks | 01:24 | |
| *** edmondsw has joined #openstack-sdks | 01:24 | |
| *** yuanying_ has quit IRC | 01:29 | |
| *** salv-orlando has quit IRC | 01:29 | |
| *** edmondsw has quit IRC | 01:29 | |
| *** yuanying has joined #openstack-sdks | 01:29 | |
| *** yuanying_ has joined #openstack-sdks | 01:33 | |
| openstackgerrit | Merged openstack-infra/shade master: de-client-ify baremetal get_machine https://review.openstack.org/499774 | 01:33 |
| *** yuanying has quit IRC | 01:37 | |
| adriant | mordred: I'm going to be opening a lot of issues for the sdk over the next while as I find things that don't quite work as expected, or are missing. Should I be doing that here: https://storyboard.openstack.org/#!/project/760 ? | 01:44 |
| adriant | I mean, now that shade == openstacksdk that makes sense, but just curious. | 01:44 |
| adriant | the launchpad page seem ancient so I assume bugs aren't tracked there. | 01:44 |
| openstackgerrit | Merged openstack-infra/shade master: Catch attrbute error for other APIs https://review.openstack.org/499775 | 01:48 |
| *** annp has joined #openstack-sdks | 01:56 | |
| *** ldl has joined #openstack-sdks | 02:17 | |
| *** dave-mccowan has quit IRC | 02:18 | |
| *** salv-orlando has joined #openstack-sdks | 02:24 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Release note cleanup https://review.openstack.org/524004 | 02:25 |
| *** salv-orlando has quit IRC | 02:29 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 02:30 | |
| *** fzdarsky_ has joined #openstack-sdks | 02:39 | |
| *** fzdarsky|afk has quit IRC | 02:41 | |
| *** dave-mccowan has joined #openstack-sdks | 02:46 | |
| *** thrash is now known as thrash|g0ne | 02:57 | |
| *** edmondsw has joined #openstack-sdks | 03:13 | |
| *** edmondsw has quit IRC | 03:17 | |
| *** bobh has quit IRC | 03:23 | |
| *** salv-orlando has joined #openstack-sdks | 03:25 | |
| *** salv-orlando has quit IRC | 03:30 | |
| *** bobh has joined #openstack-sdks | 03:31 | |
| openstackgerrit | Hengqing Hu proposed openstack/python-openstacksdk master: Add pools attribute to load balancer heath monitor https://review.openstack.org/522962 | 03:33 |
| *** d0ugal_ has joined #openstack-sdks | 03:50 | |
| *** d0ugal has quit IRC | 03:52 | |
| *** e0ne has joined #openstack-sdks | 03:55 | |
| *** prg3 has joined #openstack-sdks | 03:57 | |
| *** bobh has quit IRC | 03:58 | |
| *** e0ne has quit IRC | 04:00 | |
| *** dave-mccowan has quit IRC | 04:07 | |
| *** e0ne has joined #openstack-sdks | 04:11 | |
| *** e0ne has quit IRC | 04:16 | |
| *** e0ne has joined #openstack-sdks | 04:23 | |
| *** salv-orlando has joined #openstack-sdks | 04:26 | |
| *** e0ne has quit IRC | 04:28 | |
| *** salv-orlando has quit IRC | 04:30 | |
| *** e0ne has joined #openstack-sdks | 04:39 | |
| *** ldl has quit IRC | 04:44 | |
| *** e0ne has quit IRC | 04:44 | |
| *** e0ne has joined #openstack-sdks | 04:50 | |
| SamYaple | TheJulia: you can put whiskey in anything if you try hard enough | 04:57 |
| TheJulia | SamYaple: Indeed, although I think most people will prefer a glass of some sort. | 04:57 |
| *** e0ne has quit IRC | 05:03 | |
| *** prg3 has quit IRC | 05:06 | |
| *** e0ne has joined #openstack-sdks | 05:07 | |
| *** prg3 has joined #openstack-sdks | 05:09 | |
| *** e0ne has quit IRC | 05:12 | |
| *** e0ne has joined #openstack-sdks | 05:17 | |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Stop osSDK mangling Swift metadata keys https://review.openstack.org/524066 | 05:22 |
| *** e0ne has quit IRC | 05:22 | |
| adriant | mordred: potentially awkward, but even with yaml=False, calling 'get_one_cloud' without a cloud name, it will look in local OS_* envvars. Which might be good to also disable/avoid as well as yaml checking. | 05:25 |
| adriant | not that it really is likely to crop up much, but still a little weird because of how I was testing stuff. I didn't realise I had set any vars, and it seemed to pick them up and yell at me for providing domain name for token auth. | 05:26 |
| *** salv-orlando has joined #openstack-sdks | 05:27 | |
| *** e0ne has joined #openstack-sdks | 05:30 | |
| *** salv-orlando has quit IRC | 05:32 | |
| adriant | hmmmm... anyone here who has played with swift support in the openstackSDK? | 05:34 |
| adriant | I'm trying to do HEAD on a container to see if it exists before I try creating it. | 05:34 |
| adriant | but well... `x = conn.object_store.get_container_metadata('not_a_real_container')` returns a valid resource object | 05:35 |
| *** e0ne has quit IRC | 05:35 | |
| adriant | it's not a 'real' container | 05:35 |
| adriant | but it has some attrs | 05:35 |
| adriant | I can't for the life of me find a value to say "this is not a real container" | 05:36 |
| adriant | which is broken | 05:36 |
| adriant | like, pretty broken | 05:36 |
| adriant | swiftclient sucks, but at least it throws a 404 type error | 05:36 |
| *** e0ne has joined #openstack-sdks | 05:38 | |
| adriant | `x = conn.compute.get_server("not a real server")` does throw an error | 05:38 |
| adriant | `x = conn.object_store.get_object_metadata("not a real container", "fake object")` also returns a not none value :/ | 05:40 |
| adriant | while `x = conn.object_store.get_object("not a real container", "fake object")` returns: '<html><h1>Not Found</h1><p>The resource could not be found.</p></html>' | 05:41 |
| adriant | wow, object_store in the SDK is broken | 05:41 |
| adriant | I think this is a good time to call it quits for today... | 05:42 |
| *** e0ne has quit IRC | 05:43 | |
| *** e0ne has joined #openstack-sdks | 05:46 | |
| *** ldl has joined #openstack-sdks | 05:50 | |
| *** e0ne has quit IRC | 05:51 | |
| *** e0ne has joined #openstack-sdks | 05:57 | |
| *** e0ne has quit IRC | 06:02 | |
| *** e0ne has joined #openstack-sdks | 06:04 | |
| *** toabctl has joined #openstack-sdks | 06:05 | |
| *** e0ne has quit IRC | 06:09 | |
| *** salv-orlando has joined #openstack-sdks | 06:12 | |
| *** e0ne has joined #openstack-sdks | 06:15 | |
| *** e0ne has quit IRC | 06:20 | |
| *** e0ne has joined #openstack-sdks | 06:25 | |
| *** e0ne has quit IRC | 06:29 | |
| *** prg3 has quit IRC | 06:46 | |
| *** edmondsw has joined #openstack-sdks | 06:49 | |
| *** edmondsw has quit IRC | 06:54 | |
| *** gouthamr has quit IRC | 07:20 | |
| *** gkadam_ has quit IRC | 07:24 | |
| *** Jeffrey4l has joined #openstack-sdks | 07:34 | |
| *** markvoelker has quit IRC | 07:37 | |
| *** ralonsoh has joined #openstack-sdks | 07:53 | |
| *** ioggstream has joined #openstack-sdks | 08:01 | |
| *** slaweq has joined #openstack-sdks | 08:06 | |
| *** LindaWang1 has joined #openstack-sdks | 08:15 | |
| *** e0ne has joined #openstack-sdks | 08:18 | |
| *** ScrewTSW has joined #openstack-sdks | 08:20 | |
| *** slaweq has quit IRC | 08:27 | |
| *** slaweq has joined #openstack-sdks | 08:31 | |
| ldl | can I through the server id to get the server image snapshots? | 08:35 |
| *** edmondsw has joined #openstack-sdks | 08:38 | |
| *** markvoelker has joined #openstack-sdks | 08:38 | |
| *** slaweq has quit IRC | 08:41 | |
| *** edmondsw has quit IRC | 08:42 | |
| *** slaweq has joined #openstack-sdks | 08:44 | |
| *** gkadam_ has joined #openstack-sdks | 08:46 | |
| *** jpich has joined #openstack-sdks | 08:55 | |
| *** fzdarsky_ is now known as fzdarsky | 09:06 | |
| *** tonythomas has joined #openstack-sdks | 09:13 | |
| *** d0ugal_ has quit IRC | 09:17 | |
| *** d0ugal has joined #openstack-sdks | 09:18 | |
| *** ScrewTSW has quit IRC | 09:18 | |
| *** cdent has joined #openstack-sdks | 09:29 | |
| *** spredzy has joined #openstack-sdks | 09:35 | |
| *** e0ne has quit IRC | 09:40 | |
| *** purplerbot has quit IRC | 09:42 | |
| *** LindaWang1 has quit IRC | 09:59 | |
| *** e0ne has joined #openstack-sdks | 10:01 | |
| *** purplerbot has joined #openstack-sdks | 10:03 | |
| *** Kevin_Zheng has quit IRC | 10:09 | |
| *** dgonzalez has quit IRC | 10:09 | |
| *** dtantsur|afk has quit IRC | 10:09 | |
| *** dtantsur has joined #openstack-sdks | 10:15 | |
| *** Kevin_Zheng has joined #openstack-sdks | 10:15 | |
| *** dgonzalez has joined #openstack-sdks | 10:15 | |
| *** dtantsur has quit IRC | 10:15 | |
| *** dtantsur has joined #openstack-sdks | 10:15 | |
| *** annp has quit IRC | 10:23 | |
| *** ldl has quit IRC | 10:24 | |
| *** salv-orlando has quit IRC | 10:36 | |
| openstackgerrit | Édouard Thuleau proposed openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported https://review.openstack.org/523834 | 10:43 |
| *** ldl has joined #openstack-sdks | 10:46 | |
| *** salv-orlando has joined #openstack-sdks | 10:48 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 10:55 | |
| *** sdague has joined #openstack-sdks | 11:02 | |
| *** cdent has quit IRC | 11:04 | |
| *** jlk has quit IRC | 11:12 | |
| *** jlk has joined #openstack-sdks | 11:12 | |
| *** jlk has quit IRC | 11:13 | |
| *** jlk has joined #openstack-sdks | 11:13 | |
| *** salv-orlando has quit IRC | 11:18 | |
| *** salv-orlando has joined #openstack-sdks | 11:32 | |
| *** cdent has joined #openstack-sdks | 11:35 | |
| *** jkilpatr has quit IRC | 11:37 | |
| *** baffle has quit IRC | 11:45 | |
| *** jkilpatr has joined #openstack-sdks | 11:56 | |
| *** dave-mccowan has joined #openstack-sdks | 12:06 | |
| *** thrash|g0ne is now known as thrash | 12:08 | |
| *** edmondsw has joined #openstack-sdks | 12:14 | |
| *** jkilpatr has quit IRC | 12:14 | |
| *** jkilpatr has joined #openstack-sdks | 12:17 | |
| *** edmondsw has quit IRC | 12:20 | |
| *** bobh has joined #openstack-sdks | 12:26 | |
| *** ScrewTSW has joined #openstack-sdks | 12:28 | |
| *** bobh has quit IRC | 12:30 | |
| *** cdent has quit IRC | 12:46 | |
| *** thrash is now known as thrash|biab | 12:51 | |
| *** chenybd_ has joined #openstack-sdks | 12:51 | |
| *** LindaWang1 has joined #openstack-sdks | 13:06 | |
| *** dtantsur is now known as dtantsur|brb | 13:13 | |
| *** edmondsw has joined #openstack-sdks | 13:18 | |
| *** thrash|biab is now known as thrash | 13:23 | |
| *** markvoelker has quit IRC | 13:23 | |
| *** markvoelker has joined #openstack-sdks | 13:24 | |
| *** LindaWang1 has quit IRC | 13:46 | |
| *** ldl has quit IRC | 13:54 | |
| *** d0ugal has quit IRC | 14:03 | |
| *** LindaWang1 has joined #openstack-sdks | 14:06 | |
| *** cdent has joined #openstack-sdks | 14:06 | |
| *** chenybd_ has quit IRC | 14:15 | |
| *** chenybd_ has joined #openstack-sdks | 14:15 | |
| *** chenybd_ has joined #openstack-sdks | 14:16 | |
| *** d0ugal has joined #openstack-sdks | 14:17 | |
| *** chenybd__ has joined #openstack-sdks | 14:26 | |
| *** chenybd_ has quit IRC | 14:29 | |
| mordred | adriant: morning! so ... | 14:31 |
| mordred | adriant: good point re: yaml=False - I think what we want is a "make me one of these and only use the kwargs I'm passing, not other stuff | 14:31 |
| mordred | adriant: for the other thing, good to know - I know the shade layer for swift objects is good and also support for the 'use head to check if it exists' use case | 14:37 |
| mordred | adriant: as we work through rationalizing the shade and sdk layers, we'll make sure to fix that SDK brokenness you just reported (thanks for that btw) | 14:38 |
| mordred | adriant: hopefully we'll at least have a $something you can use related to the new-world-order soonish | 14:38 |
| *** chenybd_ has joined #openstack-sdks | 14:39 | |
| *** e0ne has quit IRC | 14:39 | |
| *** e0ne has joined #openstack-sdks | 14:40 | |
| *** chenybd__ has quit IRC | 14:41 | |
| *** dtantsur|brb is now known as dtantsur | 14:42 | |
| mordred | dhellmann: morning mr. "I know literally every python module that exists" ... I think I remember there being some sort of module that would let you execute example code from docs as part of tests ... do you know if that's a thing or if I'm just dreaming it? | 14:42 |
| cmurphy | mordred: i've heard of that, i think it's this https://docs.python.org/2/library/doctest.html | 14:44 |
| mordred | cmurphy: ah! yes - that's what I was thinking of, thanks! | 14:45 |
| mordred | cmurphy: I wrote some example docs yesterday, but it turns out the examples I wrote don't work ... made me think testing examples in docs would be a good idea | 14:46 |
| mordred | although that module isn't *quite* what I was thinking - maybe I could either add support for the usage pattern I was considering, or use it as inspiration to write a thing | 14:47 |
| mordred | basically very similar to https://docs.python.org/2/library/doctest.html#simple-usage-checking-examples-in-a-text-file - but finding .. code-block: python sections in rst files, and not requiring them to be interactive sessions | 14:48 |
| mordred | it's also possible that in this particular case it needs to be specific to shade/sdk codebase, as I probably want a requests_mock thing set up ... | 14:49 |
| *** bobh has joined #openstack-sdks | 14:52 | |
| mordred | rcarrillocruz, Shrews, jlk: https://github.com/ansible/ansible/pull/20974 has been updated (the rebase on that SUCKED) | 14:53 |
| *** chenybd_ has quit IRC | 14:53 | |
| *** chenybd_ has joined #openstack-sdks | 14:54 | |
| *** chenybd_ has quit IRC | 14:58 | |
| Shrews | mordred: seems to have failed tests still | 15:02 |
| *** marst has joined #openstack-sdks | 15:17 | |
| *** LindaWang1 has quit IRC | 15:18 | |
| *** prg3 has joined #openstack-sdks | 15:20 | |
| *** LindaWang1 has joined #openstack-sdks | 15:22 | |
| mordred | yah - I'll fix that next | 15:24 |
| mordred | Shrews: sake of argument, what do you think about this: https://gist.github.com/ce8b1c90cf972bfc104e3eb38ae74762 as an alternative approach? | 15:24 |
| *** gouthamr has joined #openstack-sdks | 15:26 | |
| *** prg3 has quit IRC | 15:34 | |
| Shrews | mordred: hmm, i sort of like that | 15:34 |
| mordred | Shrews: cool, me too ... jlk, rcarrillocruz, SamYaple: ^^ what about y'all ? | 15:35 |
| rcarrillocruz | think latter is cleaner | 15:36 |
| -openstackstatus- NOTICE: if you receieved a result of "RETRY_LIMIT" after 14:15 UTC, it was likely due to an error since corrected. please "recheck" | 15:38 | |
| *** nikhil has joined #openstack-sdks | 15:42 | |
| *** thrash is now known as thrash|biab | 15:48 | |
| *** slaweq has quit IRC | 15:52 | |
| thingee | mordred: where are these roles defined again? https://review.openstack.org/#/c/523993 | 15:52 |
| *** tonythomas has quit IRC | 15:52 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 15:53 |
| cdent | api-sig meeting in #openstack-meeting-3 in a few minutes | 15:54 |
| mordred | thingee: in that case, tox-siblings is/was defined in openstack-infra/zuul-jobs | 15:59 |
| thingee | was? | 15:59 |
| mordred | thingee: yah - the patch above is part of a sequence that removes the tox-siblings role - we combined its functionality into the main tox role | 15:59 |
| thingee | got it | 15:59 |
| thingee | right | 16:00 |
| mordred | thingee: but we needed to remove use of it first - that patch chain is a fun DAG of depends-on changes :) | 16:00 |
| mordred | I sohuld have left needed-by comments on everything to make it easier to follow :) | 16:00 |
| mordred | thingee: here's teh final patch: https://review.openstack.org/#/c/523996 | 16:00 |
| mordred | thingee: that depends on all of the changes to remove the use of the role, which all in turn depends-on the change to move the functionality from tox-siblings into tox | 16:01 |
| SamYaple | mordred: yea seems ok. im not too fussy as long as its consistent | 16:02 |
| thingee | mordred: I can't even find ensure-tox role being defined | 16:03 |
| mordred | thingee: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/ensure-tox | 16:06 |
| mordred | thingee: we have a todo-list item to have zuul generate a global documentation of all the jobs and roles | 16:06 |
| thingee | mordred: ah ha, that's why I didn't find it. | 16:06 |
| mordred | thingee: we have them for each jobs/roles repo: https://docs.openstack.org/infra/zuul-jobs/ ... but since zuul knows about ALL of the jobs and roles, we figurehaving something like zuul.openstack.org/docs that has an index of all the jobs and roles, links to where they're defined, and their documentation would be *very* helpful | 16:07 |
| thingee | mordred: are you just looking for an auto generate approach? | 16:08 |
| *** LindaWang1 has quit IRC | 16:08 | |
| mordred | thingee: we've also talked about adding support for some explicit 'import' statements in .zuul.yaml files - along with namespacing - so you could say "import git.openstack.org/openstack-infra/zuul-jobs as zuul_jobs and then refer to things like zuul-jobs.tox-pep8 ... *handwave* | 16:08 |
| mordred | thingee: kind of - there are two sides to the coin - one is an index of jobs - that's already very close to being done in the new web dashboard ... | 16:09 |
| mordred | thingee: the second part is generating the docs - we're using sphinx in a post job right now, but if we want zuul to just have an always up to date dynamic listing of the docs, we might not want to delegate that to sphinx and may instead want to just read the rst ourselves? that part is still undefined | 16:10 |
| mordred | thingee: fwiw, https://review.openstack.org/#/q/topic:zuul-web+(status:open+OR+status:merged) is the series tristanC has been working on related to adding a web dashboard | 16:11 |
| thingee | mordred: ok, I went through your series for removal in other project's pre.yaml files. | 16:12 |
| thingee | only +1 | 16:13 |
| thingee | s :) | 16:13 |
| *** ScrewTSW has quit IRC | 16:15 | |
| *** thrash|biab is now known as thrash | 16:15 | |
| mordred | thingee: woot! thanks | 16:16 |
| rcarrillocruz | mordred: so, talking to evrardjp at summit that, well, our os_* modules lacking integration tests was less than ideal. I was thinking of adding tests on our ansible zuul, but if we are going to test them at openstack-infra, can I just start writing them? | 16:31 |
| rcarrillocruz | my understanding is that we already get webhooks from ansible/ansible | 16:32 |
| rcarrillocruz | so we could potentially just start ? | 16:32 |
| mordred | rcarrillocruz: yes- the plan is to start running the shade-ansible-functional-devstack jobs on them | 16:32 |
| rcarrillocruz | is that on hold till zuul v3, or just pending | 16:33 |
| mordred | rcarrillocruz: we are getting the webhooks already, but we're waiting just a little longer - we're still having a few scale issues with the gh app and missing events | 16:33 |
| rcarrillocruz | if pending, i'm happy to get the ball running | 16:33 |
| rcarrillocruz | AH | 16:33 |
| rcarrillocruz | the rate limiting i assume? | 16:33 |
| mordred | rcarrillocruz: it's unclear - we need to dig in a little and see | 16:33 |
| rcarrillocruz | k, when that's sorted i'm happy to help on that, just ping me | 16:34 |
| mordred | rcarrillocruz: we should *probably* get this finished nad merged too: https://review.openstack.org/#/c/504267/ | 16:34 |
| rcarrillocruz | also, OSA folks seem interested on testing against ansible devel | 16:34 |
| *** cdent has quit IRC | 16:34 | |
| rcarrillocruz | and they bundle a few modules due to lack of os_* functionality | 16:35 |
| rcarrillocruz | which i'm hoping to tackle soon | 16:35 |
| mordred | rcarrillocruz: yah - I think we'll get several takers for doing cross-testing once it's there ... but I mostly want to make sure the gh/webhook stuff is stable enough so that github integration doesn't give people an impression that zuul is flaky - since for several of them it'll be their first impression | 16:35 |
| rcarrillocruz | like, they have os_magnum_container | 16:35 |
| mordred | ah - fun | 16:35 |
| rcarrillocruz | and some keystone stuff, cos for some reason upstream lacks some stuff | 16:35 |
| mordred | rcarrillocruz: jeez - we're going to have to figure out a good name for that- os_magnum_container isn't it ... | 16:35 |
| rcarrillocruz | lol | 16:36 |
| rcarrillocruz | yeah, iirc 'container' was taken by swift :S | 16:36 |
| mordred | rcarrillocruz: oh - the keystone stuff ... we ran in to problems trying to figure out how to deal with the v2/v3 differences in the ansible module - and I think at this point we should just maybe have an os_keystone_endpoint module that is v3 only ... nobody should be setting up new v2 endpoints | 16:37 |
| mordred | rcarrillocruz: yup | 16:37 |
| mordred | rcarrillocruz: so we need an answer for that for shade and for ansible | 16:37 |
| mordred | rcarrillocruz: the only magnum function we have in shade currently is cluster_template | 16:37 |
| rcarrillocruz | yeah, i think that's my fault | 16:38 |
| rcarrillocruz | i started putting magnum stuff on shade | 16:38 |
| rcarrillocruz | then ELOSTJOB | 16:38 |
| rcarrillocruz | and you renamed it to cluster_template | 16:38 |
| rcarrillocruz | meaning | 16:38 |
| rcarrillocruz | is a halfway work, i never completed putting magnum on it | 16:39 |
| *** cdent has joined #openstack-sdks | 16:39 | |
| rcarrillocruz | i committed to evrardjp to look at their stuff, put missing stuff on shade | 16:39 |
| rcarrillocruz | and mark https://github.com/ansible/ansible/issues/24197 as complete | 16:39 |
| mordred | ++ | 16:40 |
| mordred | well- we have magnum enabled jobs at least, so that part is done :) | 16:40 |
| *** e0ne has quit IRC | 16:43 | |
| dhellmann | dtroyer : I have some ideas for improving cliff performance. I'm going to ask for a volunteer on the ML, but wanted to run the notes past you first in case you've already started thinking along the same lines. https://etherpad.openstack.org/p/mFsAgTZggf | 16:48 |
| dtroyer | dhellmann: cool, I'll have a look after lunch | 16:51 |
| dhellmann | dtroyer : thanks | 16:54 |
| *** d0ugal has quit IRC | 16:55 | |
| dtroyer | mordred: as a follow-up to https://review.openstack.org/#/c/523520/ (shade->OSSDK rename) I proposed https://review.openstack.org/524249 to put o-c-c next to shade | 16:59 |
| *** thrash is now known as thrash|biab | 17:01 | |
| openstackgerrit | Doug Hellmann proposed openstack/cliff master: add support for legacy command name translation https://review.openstack.org/522380 | 17:01 |
| *** dtantsur is now known as dtantsur|afk | 17:03 | |
| cdent | edleafe, elmiko, dtantsur|afk : https://etherpad.openstack.org/p/api-sig-newsletter | 17:07 |
| *** d0ugal has joined #openstack-sdks | 17:09 | |
| *** kmalloc has quit IRC | 17:11 | |
| edleafe | looking... | 17:13 |
| edleafe | ok, just a few small changes. | 17:15 |
| cdent | yeah, good, thanks edleafe | 17:16 |
| cdent | away | 17:17 |
| cdent | as in, thar she blows | 17:17 |
| cdent | dtantsur|afk: you should see your +2 (if I’ve done it right) | 17:17 |
| *** gkadam_ has quit IRC | 17:22 | |
| openstackgerrit | Merged openstack-infra/shade master: Avoid tox_install.sh for constraints support https://review.openstack.org/508061 | 17:23 |
| *** d0ugal has quit IRC | 17:24 | |
| openstackgerrit | Merged openstack/python-openstacksdk master: Remove use of tox-siblings role https://review.openstack.org/523993 | 17:30 |
| *** d0ugal has joined #openstack-sdks | 17:35 | |
| *** jpich has quit IRC | 17:44 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 17:45 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 17:47 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 17:48 |
| *** david-lyle has joined #openstack-sdks | 17:51 | |
| *** nikhil has quit IRC | 17:51 | |
| thingee | with the merging of shade and python-openstacksdk, are extensions still going to be supported https://review.openstack.org/#/c/523834/ - shade normalized things is why I'm asking | 18:05 |
| *** ralonsoh has quit IRC | 18:18 | |
| *** jkilpatr has quit IRC | 18:23 | |
| mordred | thingee: we support some openstack extensions in shade already, so I thnk so - the main thing froma shade pov is in whether or not the extension in question can be exposed/presented in a way that makes sense or degrades in a reasonable way | 18:29 |
| mordred | thingee: like, if an extension adds a field to a resource, then we can always add that field to our returned data model and just fill in None if the cloud doens't have the extension ... or if the same info is given by different extensions slightly differently and we can rationalize it - then we can try to do that | 18:30 |
| mordred | thingee: but some extensions are things that don't make much sense in a general context - like something that exposed vendor-specific attributes - and those are ones we'd never add to the data model | 18:31 |
| mordred | thingee: long-winded way of saying "it depends/case-by-case" for the shade layer ... the sdk object layer is somewhat similar to shade's data model, but it's also closer to exposing the details of the cloud in question - so I could see an argument for some variation to be exposed, along with discovery flags, at thatlayer | 18:32 |
| *** ioggstream has quit IRC | 18:32 | |
| mordred | thingee: in the case of https://review.openstack.org/#/c/523834/ - that particular case seems reasonable for the responses - tags are a neutron feature that a deployer may or may not have enabled, so defaulting them to [] seems maybe reasonable? | 18:34 |
| thingee | yeah got it, thanks | 18:34 |
| thingee | and yes I agree it's reasonable | 18:34 |
| mordred | otoh - I'm not sure what the experience is going to be for a user that tries to add a tag to a resource on a cloud that doesn't support it - and I'm not sure that patch addresses that | 18:34 |
| mordred | (it's also possible that's already handled in the existing code, I need to go look) | 18:35 |
| *** jkilpatr has joined #openstack-sdks | 18:39 | |
| *** fzdarsky is now known as fzdarsky|afk | 18:39 | |
| *** openstackgerrit has quit IRC | 18:48 | |
| thingee | mordred: pretty sure that's just going give a 404 http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/network/v2/tag.py#n27 | 18:54 |
| thingee | just being a keystone session request. | 18:55 |
| *** gkadam has joined #openstack-sdks | 18:58 | |
| thingee | mordred: I'll check around for other instances, but this should probably have the _has_neutron_extension('tags') check and raise like the other place shade does it | 18:59 |
| thingee | yuk | 18:59 |
| *** thrash|biab is now known as thrash | 19:04 | |
| *** test234 has joined #openstack-sdks | 19:26 | |
| *** test234 has left #openstack-sdks | 19:26 | |
| *** cdent has quit IRC | 19:31 | |
| *** jkilpatr has quit IRC | 19:39 | |
| *** baffle has joined #openstack-sdks | 19:52 | |
| *** kmalloc has joined #openstack-sdks | 20:07 | |
| thingee | installing the latest python-openstacksdk, if I call openstack.OpenStackCloud() I get http://paste.openstack.org/show/627928/ | 20:38 |
| *** e0ne has joined #openstack-sdks | 20:39 | |
| thingee | ah I guess get_one_cloud() doesn't just get one without something passed in kwargs | 20:39 |
| mordred | thingee: yah - you need to give it at leat a cloud= | 20:43 |
| mordred | dtroyer: I've got 3 different keystoneauth patches I need to rebase/fix and get landed - would you like me to take a stab at taking the timing wrapper you have in osc and putting it directly on ksa.session? | 20:44 |
| mordred | dtroyer: since I'll be in there anyway? | 20:44 |
| *** openstackgerrit has joined #openstack-sdks | 20:45 | |
| openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: void tox_install.sh for constraints support https://review.openstack.org/524344 | 20:45 |
| *** e0ne has quit IRC | 20:58 | |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119 | 21:00 |
| *** gkadam has quit IRC | 21:01 | |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120 | 21:02 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121 | 21:02 |
| dtroyer | mordred: sure, I don't mind. I think we'll want to land a compat check into osc-lib for that, especially if you don't put it into Session | 21:04 |
| dtroyer | the version in osc itself appears to be unused and I never got around to removing it | 21:04 |
| thingee | mordred: so there are two interfaces to python-openstacksdk? I can openstack.OpenStackCloud.update_qos_policy() or use openstack.network.v2.qos_policy | 21:05 |
| thingee | mordred: Is there anywhere written of what the plan is there? | 21:06 |
| openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Avoid tox_install.sh for constraints support https://review.openstack.org/524344 | 21:06 |
| *** jkilpatr has joined #openstack-sdks | 21:09 | |
| openstackgerrit | Monty Taylor proposed openstack/os-api-ref master: void tox_install.sh for constraints support https://review.openstack.org/524344 | 21:09 |
| openstackgerrit | Monty Taylor proposed openstack/os-api-ref master: Add os-api-ref-nova-src job https://review.openstack.org/524353 | 21:09 |
| mordred | thingee: yes! there are two interfaces, and there is a plan written down ... (and also there is a plan to make the two interfaces be less completely different) | 21:10 |
| mordred | thingee: oh - I lied - there is a document, but it does not contain words about this - lemme fix | 21:11 |
| briancurtin | thingee: hello | 21:12 |
| openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Add os-api-ref-nova-src job https://review.openstack.org/524353 | 21:15 |
| thingee | briancurtin: hi | 21:16 |
| openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Add os-api-ref-nova-src job https://review.openstack.org/524353 | 21:18 |
| openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Avoid tox_install.sh for constraints support https://review.openstack.org/524344 | 21:18 |
| briancurtin | thingee: i ran by Sizzle Pie the other day. just thought you should know | 21:18 |
| thingee | briancurtin: and you didn't go in? | 21:19 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update the shade-merge document https://review.openstack.org/524391 | 21:19 |
| mordred | thingee: ^^ ok - I updated that doc a bit- it should probably be fundamentally rewritten and moved somewhere more discoverable | 21:20 |
| briancurtin | thingee: unfortunately not. a mid-run pizza didn’t sound great at the time, but actually sounds great now | 21:20 |
| mordred | briancurtin: the concept of a mid-run pizza might almost be enough to convince me to run | 21:22 |
| thingee | mordred: thanks | 21:22 |
| mordred | although I'd probably just end up skipping the first part of the run, going straight for pizza, then skipping the second part because I'm full | 21:22 |
| * dtroyer catches up… 'mid-run' is literal here, not 'stopped in' :) | 21:22 | |
| mordred | thingee: oh - it occurs to me there's another thing to put in there ... | 21:23 |
| adriant | mordred: I honestly don't mind working on bugs in the SDK, so potentially I'll get distracted and submit some patches for the Swift issues I've found. | 21:26 |
| mordred | adriant: woot! | 21:26 |
| adriant | mordred: and yes a function of flag for "a connection with just these args thanks" would be amazing. | 21:27 |
| adriant | or* flag | 21:27 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Remove version arg from updated ironic calls https://review.openstack.org/524393 | 21:28 |
| adriant | mordred: I submitted one patch already for swift related nonsense: https://review.openstack.org/#/c/524066/ | 21:28 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update the shade-merge document https://review.openstack.org/524391 | 21:30 |
| openstackgerrit | Krzysztof Klimonda proposed openstack-infra/shade master: Prioritize external networks for assigning FIP to instance https://review.openstack.org/524396 | 21:35 |
| *** dave-mccowan has quit IRC | 21:35 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Remove a bunch of things we promised to remove in 2H 2017 https://review.openstack.org/524399 | 21:40 |
| openstackgerrit | Monty Taylor proposed openstack/os-api-ref master: Add os-api-ref-nova-src job https://review.openstack.org/524353 | 21:45 |
| openstackgerrit | Monty Taylor proposed openstack/os-api-ref master: Avoid tox_install.sh for constraints support https://review.openstack.org/524344 | 21:45 |
| TheJulia | mordred: so hit a minor issue. python-ironicclient has retry logic baked in for handling nodes locked by the conductor. Any thoughts? I'm thinking the least invasive would be just to double check for the locks before operations that could be blocked by them and wait for the lock to clear, which would clear until ironic likely needs to go re-check the power state. | 22:00 |
| mordred | TheJulia: ah- interesting ... and yes, that seems like a good plan ... altough I suppose there's a race condition there and that something could still lock between the check and the operation - so we still probably want to handle geting a node-locked error | 22:04 |
| TheJulia | Yeah... | 22:05 |
| adriant | mordred: omfg. my notfound 404 issue. it's simple. no one stuck `exceptions.raise_from_response(resp)` in the _head function | 22:09 |
| adriant | although I'm curious if that breaks anything else | 22:09 |
| adriant | will submit a patch and we can test/review | 22:09 |
| mordred | adriant: woot! | 22:10 |
| TheJulia | mordred: I guess it only really matters on actions that are actually trying to make a change that requires a lock in the first place | 22:12 |
| * TheJulia wonders why port deletion even requires a lock.... | 22:12 | |
| dtroyer | that sounds like it involved neutron…. where I think the only atomic operation is "halt" | 22:16 |
| *** marst has quit IRC | 22:19 | |
| TheJulia | I guess there could be code someplace that requires the task object to be re-read, which could make things... fun if someone deleted ports while a task was running in ironic | 22:19 |
| *** edmondsw has quit IRC | 22:24 | |
| *** edmondsw has joined #openstack-sdks | 22:24 | |
| *** edmondsw has quit IRC | 22:25 | |
| *** edmondsw_ has joined #openstack-sdks | 22:27 | |
| *** edmondsw_ has quit IRC | 22:31 | |
| openstackgerrit | Merged openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119 | 22:33 |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120 | 22:33 |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121 | 22:33 |
| openstackgerrit | Lance Bragstad proposed openstack/python-openstackclient master: Add system role functionality https://review.openstack.org/524416 | 22:45 |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly https://review.openstack.org/524418 | 22:51 |
| adriant | mordred: this fixes my current swift issues with 404: https://review.openstack.org/#/c/524418/ | 22:52 |
| adriant | potentially may break other APIs that use HEAD differently, but I doubt it | 22:52 |
| adriant | I think someone just forgot to add that function all to resource.head | 22:52 |
| adriant | function call* | 22:52 |
| openstackgerrit | Merged openstack-infra/shade master: Remove version arg from updated ironic calls https://review.openstack.org/524393 | 22:57 |
| openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Display IPv6 addresses better https://review.openstack.org/524420 | 22:57 |
| *** bobh has quit IRC | 23:01 | |
| *** thrash is now known as thrash|g0ne | 23:26 | |
| thingee | adriant: curious why the key would be in the values? | 23:36 |
| thingee | for https://review.openstack.org/#/c/524066/1 | 23:36 |
| thingee | adriant: nevermind, found the storyboard ticket with the info | 23:37 |
| mordred | Shrews, briancurtin, dtroyer, SamYaple, thingee, TheJulia: I have discovered that pypy unittests are basically unpossible in the gate atm (the version of cryptography in global-requirements is not compatible with the version of pypy in xenial) | 23:38 |
| TheJulia | ugh | 23:39 |
| mordred | there is a ppa that has newer pypy, but that'll take a decent amount of consideration to deal with broadly - so I leaning toward just disabling the pypy tox env for now | 23:39 |
| mordred | until we can get a job set up that will *actually* test it | 23:39 |
| mordred | (it's been green so far because of a different bug that has caused it to not actually use pypy - fixing that causes the job to start (correctly) failing) | 23:40 |
| *** salv-orlando has quit IRC | 23:41 | |
| dtroyer | mordred: ++ on disabling for now… I'd say make it experimental but we know it's never going to actually work without tasks elsewhere first | 23:42 |
| mordred | yah, that was my thinking | 23:42 |
| dtroyer | mordred: fwiw, I'm testing a fix for OSC's -tips job… I finally duplicated it here | 23:42 |
| mordred | HOWEVER - I just had another idea I'm going to try out REAL quick ... we could tell it to run on fedora26 nodes instead of xenial nodes ... | 23:42 |
| mordred | dtroyer: woot! | 23:43 |
| dtroyer | I'd support the fedora alternative | 23:43 |
| * mordred double-checking versions real quick - if it's new enough I'll push up a patch to try it out | 23:43 | |
| mordred | yah. fedora-26 has new enough ... | 23:44 |
| dtroyer | you're going to love what the problem was :) if it succeeds here I'll push it up and see if you have a thought on what the correct fix actually is | 23:45 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | 23:46 |
| mordred | dtroyer: I can't wait :) | 23:46 |
| mordred | dtroyer: that *should* work for pypy ^^ - and is also a nice example of using a pre-existing job but overriding the node type :) | 23:47 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 23:47 |
| dtroyer | I think it's past the failure point ^^^^^ | 23:47 |
| mordred | dtroyer: oh. yah. whoops. | 23:47 |
| dtroyer | the question is what _is_ the correct thing to check for in that case, since it's SDK/Shade not o-c-c | 23:48 |
| dtroyer | this works though | 23:48 |
| dtroyer | damn… sopke too soon | 23:48 |
| mordred | yup. I think that's the right thing to check for for now | 23:48 |
| mordred | but I'm clearly not correct | 23:49 |
| dtroyer | it still took the 'old' branch, I think I'm not setting this up right…oh wait, … nevermind, let's let it run in the gate, it should work in spite of my local pilot error | 23:49 |
| dtroyer | dinnertime, will check up later | 23:50 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | 23:56 |
| mordred | dtroyer: I have the log stream open in a tab and am waiting to see it be happy | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!