Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-06-13
16:18:23 kmalloc wfm then
16:18:24 mordred and yes - I think a test for that case in ksa is a good idea
16:18:31 mordred but also we cross-gate :)
16:18:51 kmalloc cross-gate makes me happy. i wont ever say no to more quality behavior tests ;)
16:19:33 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add tests to verify behavior on '' in self link https://review.openstack.org/575158
16:19:49 mordred cdent: ^^ added a second test just to verify microversion info is found correctly
16:22:38 cdent mordred: Imma have to take your word for it, that last test its hard to tell what it is actually doing
16:22:48 cdent but overall sanity seems to be in place
16:25:45 mordred cdent: the last test is basically just showing that the metainfo about the service's endpoint (placement.get_endpoint_data()) contains the microversion info that's in the discovery document ... namely that keystoneauth properly discovered the microversion ranges
16:25:53 mordred cdent: but - taking my word for it is also fine :)
16:27:04 cdent yeah, i figured that out once I read the json files, but the assert_calls remains dark
16:40:39 mnaser i like that test, ill keep an eye on it
16:46:48 mordred cdent: yah - for that you have to know a bit more about all of the requests_mock fixtures in that test suite
16:47:30 mordred cdent: the point of "assert_calls()" is to make sure that at the end of the test the http calls you told it to mock were all called and nothing was called that you didn't tell it to mock
16:47:52 cdent ✔
16:48:04 mordred keeping in mind that the base class already mocks out the "get me a token" interaction from keystone
19:36:32 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Implement support for registered limits https://review.openstack.org/574475
19:36:32 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Implement support for project limits https://review.openstack.org/575211
22:22:35 openstackgerrit Adam Harwell proposed openstack/python-openstackclient master: Add ability to filter image list by tag https://review.openstack.org/575256
22:48:45 openstackgerrit Adam Harwell proposed openstack/python-openstackclient master: Add ability to filter image list by tag https://review.openstack.org/575256
23:14:18 rm_work running into some issues with openstacksdk between 0.9.19 and 0.10.0 with networking commands, maybe something with version discovery on old clouds?
23:14:47 rm_work I was gonna go through the commits and see if I could figure out what might have caused it, but noticed that between 0.9.19 and 0.10.0 the repo more than doubled in commit count T_T
23:14:48 rm_work anyone have any immediate thoughts about what might be going on?
23:18:50 rm_work http://paste.openstack.org/show/723420/
#openstack-sdks - 2018-06-14
01:02:10 openstackgerrit Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for security group https://review.openstack.org/547733
02:53:52 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Implement support for registered limits https://review.openstack.org/574475
03:06:02 openstackgerrit yanpuqing proposed openstack/python-openstackclient master: Do not require port argument when updating floating IP https://review.openstack.org/575057
08:29:03 frickler rm_work: your trace looks weird, can you show the same trace from a working environment (using 0.9.19 I assume)?
08:37:08 openstackgerrit Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359
08:40:34 openstackgerrit yanpuqing proposed openstack/osc-lib master: Delete the output in "find_bulk" api https://review.openstack.org/575372
10:22:01 openstackgerrit tianhui proposed openstack/python-openstackclient master: WIP: Compute: Add description support for flavor https://review.openstack.org/575390
11:10:45 frickler mordred: fyi, this is the detach_fip failure I mentioned earlier, together with another set of these bloo^H^Hinking unstable volume tests http://logs.openstack.org/59/557359/9/check/osc-functional-devstack-tips/afca297/testr_results.html.gz
11:14:03 frickler mordred: last failure in https://review.openstack.org/557935 also only has volume issues now, so with another 15 rechecks you might get two in a row to pass. probably would be good to get some cinder folk involved instead
11:55:51 openstackgerrit tianhui proposed openstack/python-openstackclient master: WIP: Compute: Add description support for flavor https://review.openstack.org/575390
12:20:40 mordred rm_work: I'd skip straight to 0.13 if you can - 0.10 was a little awkward. from the looksof that traceback it looks like an issue with version discovery and neutron
12:21:01 mordred I believe post-0.10 we added a hack back in to skip discovery for neutron since there are some broken things aout it
12:42:49 mordred frickler: ugh
12:58:27 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Turn OSC tips jobs non-voting https://review.openstack.org/575425
12:58:31 mordred frickler: ^^ I give up
13:03:46 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstackclient master: Fix subnet host_routes error https://review.openstack.org/540568
13:10:19 frickler mordred: that's sad, I was hoping we could isolate failing tests and get tempest to skip them, but I don't have the energy to go through that myself, either. so probably we'll just have to do the same for osc
13:10:22 frickler dtroyer: ^^
13:13:52 mordred frickler: yeha - I mean, I want to fix them ... but I have a hunch it'll take a while for us to have the collective energy/time to do so
13:14:52 mordred frickler: feel like giving the sdk patch a +2?
13:15:03 mordred Shrews: if you're around ... https://review.openstack.org/575425
13:15:12 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient master: Add ability to filter image list by tag https://review.openstack.org/575256
13:19:07 frickler mordred: done
13:20:59 Shrews mordred: do we have any idea what's going on there?
13:21:31 Shrews i.e., why so much flap now?
13:37:26 mordred Shrews: no clue
13:51:59 dtantsur edleafe: hey! FYI I have a meetup tonight, so won't show up at the meeting
13:52:21 dtantsur 6pm is quite inconvenient for a regular meeting on Thursday :)
13:59:26 dtroyer sigh… functional volume tests...
14:00:50 edleafe dtantsur: I have a meetup tonight too, but fortunately the API-SIG meeting is 11am for me :)
14:03:25 dtroyer so the volume_type_tests are just plain racy, not sure why this hasn't been more of an issue
14:05:59 dtantsur edleafe: that's the difference :)
14:39:24 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix volume type functional tests https://review.openstack.org/575448
14:43:29 mordred dtroyer: ++
14:57:35 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Implement support for registered limits https://review.openstack.org/574475
15:34:51 openstackgerrit Merged openstack/openstacksdk master: Turn OSC tips jobs non-voting https://review.openstack.org/575425
15:45:36 mriedem could use some help in pointers from the ksa/sdk/osc people here https://bugs.launchpad.net/python-novaclient/+bug/1776928
15:45:37 openstack Launchpad bug 1776928 in python-novaclient "Nova endpoint for v2.1 not parsed correctly, tries http instead of https" [Undecided,New]
15:45:52 mriedem i'm fairly certain this is code in novaclient that can be replaced with something in ksa
15:46:02 mriedem https://github.com/openstack/python-novaclient/blob/10.3.0/novaclient/v2/versions.py#L91
15:46:08 mriedem parsing the endpoint url properly
15:53:01 mordred mriedem: yes, for sure- although before we do there's two little patches for sdk I've been meaning to write
15:53:17 mordred but lemme get you a pointer to something
16:02:18 mriedem thanks
16:02:53 mordred mriedem: sigh. we added a method that is almost what you need :)
16:03:08 mordred but is decidedly too much - patches coming
16:08:09 mriedem the sort of good news is that osc handles this properly so we can say, "you really should just move to osc"
16:08:13 mriedem which is what we want long-term anyway
16:14:54 mordred yup
16:15:00 mordred I'll get you some patches anyway though
16:15:23 mordred there's a TODO earlier in that file from sdague about caching - which is also handled in ksa
16:25:22 edleafe cdent: https://etherpad.openstack.org/p/api-sig-newsletter is awaiting your critical pen
16:26:36 cdent my pen is weak, but will look
16:35:08 cdent edleafe: sorry for delay, wfm
16:36:33 edleafe cdent: roger that
16:43:52 rm_work mordred: sure, will get a stack from 0.13
16:48:32 rm_work mordred: http://paste.openstack.org/show/723484/
16:48:37 rm_work that's a FULL paste with 0.13.0
16:52:23 rm_work i have tested every version from 0.10.0 up
16:52:28 rm_work all seem the same
16:53:00 mordred rm_work: awesome. thnkas
16:54:44 mnaser https://review.openstack.org/#/c/574745
16:54:50 mnaser pretty straight forward thing
16:54:53 mnaser on the stack
16:56:01 mordred rm_work: oh - your cloud is returning 401 for / on neutron
16:56:20 mordred rm_work: do you have anything to do with the running of that cloud? or it's just one you're using?
16:56:20 rm_work I guess that's what Liberty did?
16:56:25 rm_work yeah we run it
16:56:36 rm_work I don't think we patched anything about that though...
16:56:39 mordred nod
16:58:34 rm_work that is correct tho, definitely 401 on /
17:02:30 mordred rm_work: do you mind running this? http://paste.openstack.org/show/723487
17:02:41 rm_work sure, one sec
17:02:57 mordred rm_work: I'm curious as to whether just updating things to send the token for discovery would work or not
17:06:37 rm_work err
17:06:39 rm_work hold on

Earlier   Later