Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-11-30
03:33:01 openstackgerrit Hengqing Hu proposed openstack/python-openstacksdk master: Add pools attribute to load balancer heath monitor https://review.openstack.org/522962
04:57:14 SamYaple TheJulia: you can put whiskey in anything if you try hard enough
04:57:47 TheJulia SamYaple: Indeed, although I think most people will prefer a glass of some sort.
05:22:00 openstackgerrit Adrian Turjak proposed openstack/python-openstacksdk master: Stop osSDK mangling Swift metadata keys https://review.openstack.org/524066
05:25:53 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:26:49 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:34:32 adriant hmmmm... anyone here who has played with swift support in the openstackSDK?
05:34:54 adriant I'm trying to do HEAD on a container to see if it exists before I try creating it.
05:35:28 adriant but well... `x = conn.object_store.get_container_metadata('not_a_real_container')` returns a valid resource object
05:35:40 adriant it's not a 'real' container
05:35:58 adriant but it has some attrs
05:36:19 adriant I can't for the life of me find a value to say "this is not a real container"
05:36:30 adriant which is broken
05:36:33 adriant like, pretty broken
05:36:55 adriant swiftclient sucks, but at least it throws a 404 type error
05:38:42 adriant `x = conn.compute.get_server("not a real server")` does throw an error
05:40:33 adriant `x = conn.object_store.get_object_metadata("not a real container", "fake object")` also returns a not none value :/
05:41:32 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:40 adriant wow, object_store in the SDK is broken
05:42:35 adriant I think this is a good time to call it quits for today...
08:35:01 ldl can I through the server id to get the server image snapshots?
10:43:46 openstackgerrit Édouard Thuleau proposed openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported https://review.openstack.org/523834
14:31:07 mordred adriant: morning! so ...
14:31:35 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:37:03 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:38:00 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:56 mordred adriant: hopefully we'll at least have a $something you can use related to the new-world-order soonish
14:42:08 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:44:41 cmurphy mordred: i've heard of that, i think it's this https://docs.python.org/2/library/doctest.html
14:45:55 mordred cmurphy: ah! yes - that's what I was thinking of, thanks!
14:46:20 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:47:13 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:48:27 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:49:06 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:53:24 mordred rcarrillocruz, Shrews, jlk: https://github.com/ansible/ansible/pull/20974 has been updated (the rebase on that SUCKED)
15:02:07 Shrews mordred: seems to have failed tests still
15:24:26 mordred yah - I'll fix that next
15:24:38 mordred Shrews: sake of argument, what do you think about this: https://gist.github.com/ce8b1c90cf972bfc104e3eb38ae74762 as an alternative approach?
15:34:59 Shrews mordred: hmm, i sort of like that
15:35:51 mordred Shrews: cool, me too ... jlk, rcarrillocruz, SamYaple: ^^ what about y'all ?
15:36:16 rcarrillocruz think latter is cleaner
15:52:45 thingee mordred: where are these roles defined again? https://review.openstack.org/#/c/523993
15:53:23 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938
15:54:10 cdent api-sig meeting in #openstack-meeting-3 in a few minutes
15:59:12 mordred thingee: in that case, tox-siblings is/was defined in openstack-infra/zuul-jobs
15:59:21 thingee was?
15:59:46 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:56 thingee got it
16:00:00 thingee right
16:00:12 mordred thingee: but we needed to remove use of it first - that patch chain is a fun DAG of depends-on changes :)
16:00:29 mordred I sohuld have left needed-by comments on everything to make it easier to follow :)
16:00:42 mordred thingee: here's teh final patch: https://review.openstack.org/#/c/523996
16:01:15 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:02:15 SamYaple mordred: yea seems ok. im not too fussy as long as its consistent
16:03:38 thingee mordred: I can't even find ensure-tox role being defined
16:06:05 mordred thingee: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/ensure-tox
16:06:28 mordred thingee: we have a todo-list item to have zuul generate a global documentation of all the jobs and roles
16:06:41 thingee mordred: ah ha, that's why I didn't find it.
16:07:31 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:08:18 thingee mordred: are you just looking for an auto generate approach?
16:08:37 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:09:19 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:10:18 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:11:11 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:12:56 thingee mordred: ok, I went through your series for removal in other project's pre.yaml files.
16:13:00 thingee only +1
16:13:02 thingee s :)
16:16:17 mordred thingee: woot! thanks
16:31:49 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:32:05 rcarrillocruz my understanding is that we already get webhooks from ansible/ansible
16:32:12 rcarrillocruz so we could potentially just start ?
16:32:34 mordred rcarrillocruz: yes- the plan is to start running the shade-ansible-functional-devstack jobs on them
16:33:07 rcarrillocruz is that on hold till zuul v3, or just pending
16:33:09 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:18 rcarrillocruz if pending, i'm happy to get the ball running
16:33:21 rcarrillocruz AH
16:33:25 rcarrillocruz the rate limiting i assume?
16:33:52 mordred rcarrillocruz: it's unclear - we need to dig in a little and see
16:34:03 rcarrillocruz k, when that's sorted i'm happy to help on that, just ping me
16:34:17 mordred rcarrillocruz: we should *probably* get this finished nad merged too: https://review.openstack.org/#/c/504267/
16:34:20 rcarrillocruz also, OSA folks seem interested on testing against ansible devel
16:35:03 rcarrillocruz and they bundle a few modules due to lack of os_* functionality
16:35:11 rcarrillocruz which i'm hoping to tackle soon
16:35:23 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:25 rcarrillocruz like, they have os_magnum_container
16:35:34 mordred ah - fun
16:35:45 rcarrillocruz and some keystone stuff, cos for some reason upstream lacks some stuff
16:35:54 mordred rcarrillocruz: jeez - we're going to have to figure out a good name for that- os_magnum_container isn't it ...
16:36:01 rcarrillocruz lol
16:36:15 rcarrillocruz yeah, iirc 'container' was taken by swift :S
16:37:03 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:06 mordred rcarrillocruz: yup
16:37:50 mordred rcarrillocruz: so we need an answer for that for shade and for ansible
16:37:59 mordred rcarrillocruz: the only magnum function we have in shade currently is cluster_template
16:38:26 rcarrillocruz yeah, i think that's my fault
16:38:33 rcarrillocruz i started putting magnum stuff on shade
16:38:43 rcarrillocruz then ELOSTJOB
16:38:49 rcarrillocruz and you renamed it to cluster_template
16:38:50 rcarrillocruz meaning
16:39:03 rcarrillocruz is a halfway work, i never completed putting magnum on it

Earlier   Later