| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-13 | |||
| 18:18:20 | openstackgerrit | zhouxinyong proposed openstack/osc-lib master: Update the HTTP links to HTTPS in bug-1558690-1528b637f2c0a449.yaml. https://review.openstack.org/617768 | |
| 19:38:30 | openstackgerrit | zhouxinyong proposed openstack/js-openstack-lib master: Applying the HTTPS protocal in es2015.rst.To keep the website in this file be more robust,we'd better update the links to HTTPS type https://review.openstack.org/617805 | |
| 19:48:21 | openstackgerrit | zhouxinyong proposed openstack/keystoneauth master: Replacing the HTTP protocal with HTTPS in ksa_betamax_test_cassette.yaml. https://review.openstack.org/617811 | |
| #openstack-sdks - 2018-11-14 | |||
| 00:48:48 | openstackgerrit | Viktor Ekmark proposed openstack/openstacksdk master: new auth_url for ELASTX https://review.openstack.org/617851 | |
| 03:22:10 | openstackgerrit | liuyamin proposed openstack/python-openstackclient master: Replace assertEqual(True/False, expr) with assertTrue/assertFalse https://review.openstack.org/617862 | |
| 05:03:43 | openstackgerrit | Merged openstack/python-openstackclient master: Fix i18n issue https://review.openstack.org/617527 | |
| 06:51:07 | openstackgerrit | liuyamin proposed openstack/python-openstackclient master: Replace assertEqual(True/False, expr) with assertTrue/assertFalse https://review.openstack.org/617862 | |
| 12:15:13 | adriant | what's the progress on the openstackclient switching to the SDK? | |
| 12:38:58 | dtroyer | it is becoming clear that I need to let go of that… | |
| 13:11:39 | smcginnis | dtroyer: Maybe something to try to solicit additional help for on the ML? | |
| 13:53:52 | adriant | I'd be willing to lend a hand with it, at least with some things like switching the identity parts to use the SDK | |
| 19:20:29 | openstackgerrit | Ed Leafe proposed openstack/api-sig master: Add idempotent DELETE guidance https://review.openstack.org/616610 | |
| #openstack-sdks - 2018-11-15 | |||
| 06:35:31 | masuberu | good afternoon, this is the right place to ask about openstack python client? | |
| 06:35:54 | masuberu | more precisely python-neutronclient | |
| 10:53:14 | openstackgerrit | Merged openstack-infra/shade master: Use openSUSE 15.0 for testing https://review.openstack.org/617276 | |
| 11:56:49 | openstackgerrit | Rabi Mishra proposed openstack/osc-lib master: Fix issubclass() in python 3.7 https://review.openstack.org/618137 | |
| 13:29:10 | openstack | Launchpad bug 1743731 in python-openstackclient "object create cannot handle unicode file names" [Undecided,New] | |
| 13:29:10 | jmccarthy | Is this the correct irc channel to ask about the openstack client ? I was wondering about this unicode bug https://bugs.launchpad.net/python-openstackclient/+bug/1743731 | |
| 13:29:58 | jmccarthy | frickler: I mean https://storyboard.openstack.org/#!/story/1743731 :) | |
| 13:57:37 | smcginnis | jmccarthy: This is the correct channel for that. | |
| 13:59:02 | jmccarthy | smcginnis: Ok cool, I hope to try the suggestion in the bug and see how it goes, maybe someone has some additional insight ? | |
| 14:03:25 | smcginnis | jmccarthy: Most folks are tied up right now with Summit/Forum sessions, but would be great if you could try it out and share your experience. | |
| 14:03:47 | smcginnis | And maybe anyone else that has looked at it or has any thoughts about it swing by and can comment in the meantime. | |
| 14:26:15 | jmccarthy | smcginnis: Sure, it'll take me a while tho | |
| 15:21:33 | jmccarthy | Is there anything special needed with tox regarding locale ? I believe my env is ok, and altered an existing funcitonal test like so http://paste.openstack.org/show/734871/ however tox -e functional tells me http://paste.openstack.org/show/734873/ | |
| 15:36:16 | openstackgerrit | Tobias Urdin proposed openstack/openstacksdk master: Fix some spelling in documentation https://review.openstack.org/618198 | |
| 15:55:10 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Added image properties to Image class https://review.openstack.org/544544 | |
| 16:00:06 | edleafe | The API-SIG office hour has now officially begun | |
| 18:24:24 | openstackgerrit | zhouxinyong proposed openstack/oaktree master: Replacing the link in HACKING.rst https://review.openstack.org/618266 | |
| #openstack-sdks - 2018-11-16 | |||
| 02:41:08 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 03:41:49 | openstackgerrit | zhouxinyong proposed openstack/oaktree master: Optimizing the safety of the http link site in HACKING.rst https://review.openstack.org/618424 | |
| 07:20:00 | amito | hey, anyone here this time of day? :) | |
| 09:19:38 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 09:21:10 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: implement block-storage backup resource https://review.openstack.org/591445 | |
| 11:31:59 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 12:44:01 | openstackgerrit | Lars Kellogg-Stedman proposed openstack/openstacksdk master: _make_proxy attempts to referenced undefined variable api_version https://review.openstack.org/618519 | |
| 12:45:58 | gtema | dtansur, are you here? | |
| 12:49:55 | openstackgerrit | Lars Kellogg-Stedman proposed openstack/openstacksdk master: _make_proxy attempts to referenced undefined variable api_version https://review.openstack.org/618519 | |
| 13:04:46 | openstackgerrit | Lars Kellogg-Stedman proposed openstack/openstacksdk master: several errors in _make_proxy https://review.openstack.org/618519 | |
| 13:48:27 | openstackgerrit | Lars Kellogg-Stedman proposed openstack/openstacksdk master: several errors in _make_proxy https://review.openstack.org/618519 | |
| 15:02:46 | larsks | Hey folks. I'm running into a variety of problems with ansible's os_loadbalancer module, and it looks like it's tripping over problems in openstack/service_description.py. There appears to be a ton of untested code there (e.g., it's trying to raise exceptions.UnsupportedVersionWarning all over the place, but that exception doesn't seem to exist). | |
| 15:03:24 | larsks | ...but it's doing that because it doesn't seem to be able to determine the api version for the octavia service, and I'm wondering if someone here has thoughts on what might be going on. The openstack cli works fine. | |
| 15:05:37 | frickler | larsks: I fear lots of folks are still recovering from the summit, I'm off for the weekend now, too. probably best create a story at https://storyboard.openstack.org/#!/project/openstack/openstacksdk , include some error logs and mention the story here | |
| 15:06:01 | larsks | Curses! But sure, I'll do that. | |
| 15:06:19 | frickler | usually mordred might want to take a look, but he also seems to be busy recovering ;) | |
| 15:33:03 | gtema | larsks: what's your endpoint url for octavia? Do you also set/change octavia api version in your clouds.yaml? | |
| 15:36:20 | gtema | UnsupportedVersionWarning should be either created in exceptions.py or replaced with UnsupportedServiceVersion. Not sure what mordred would prefer | |
| 15:36:46 | larsks | gtema: The endpoint url does not hav a version (https://my.public.hostname:13876). I did try setting load_balancer_api_version: 2 in the clouds.yaml file but that didn't appear to resolve the issue. | |
| 15:38:04 | gtema | under "cli" you mean octaviaclient, right? | |
| 15:39:06 | larsks | gtema: I mean "openstack loadbalancer ...", which uses octaviaclient, yes. | |
| 15:39:47 | gtema | ok | |
| 15:41:00 | johnsom | larsks What version of Octavia do you have deployed? | |
| 15:42:10 | larsks | johnsom: 'octavia-api --version' tells me 2.0.2.dev42 | |
| 15:44:33 | johnsom | Queens. Ok, you should be good with the API versioning. It's v2. If you do an "openstack versions show" you should see lines like this: | |
| 15:44:45 | johnsom | https://www.irccloud.com/pastebin/uQXXdGAU/ | |
| 15:46:10 | larsks | johnsom: ah, there's a problem. The same command here shows: http://termbin.com/8rlw | |
| 15:46:45 | larsks | If I curl the endpointa address I get: http://termbin.com/om4q | |
| 15:48:32 | johnsom | Hmm, yeah, I have more versions because I'm running Rocky, but I'm not sure why they aren't being listed with the queens version that has one v2 | |
| 15:51:49 | larsks | johnsom: apparently 'openstack versions' doesn't like the output from octavia? http://termbin.com/bc0i | |
| 15:52:11 | johnsom | Ah, yeah, ok. Queens doesn't have the "links" stuff, so that openstack versions show is correct. It still shows the v2 API though, so you should be good to go. As you saw with the OpenStack client | |
| 15:53:40 | larsks | johnsom: here is the specific behavior I see: http://termbin.com/mlid | |
| 15:57:08 | amito | Hey all, I'm working on the integration for manila (shared file systems) and openstack SDK. It's in progress and I guess a patch will be uploaded sometime in the next few weeks. | |
| 15:57:50 | gtema | amito: cool | |
| 15:59:08 | gtema | larsks: isn't it working further? the proxy should be fine, it seems to just throw a warning | |
| 15:59:44 | gtema | can you add `conn.load_balancer.load_balancers()` | |
| 15:59:47 | larsks | gtema: the os_loadbalancer attempts to call cloud.load_balancer.find_load_balancer, which fails, since that's not a REST pass-through. | |
| 16:00:26 | gtema | larsks: how it fails? | |
| 16:00:35 | larsks | gtema: re: your question, AttributeError: 'Proxy' object has no attribute 'load_balancers' | |
| 16:00:45 | larsks | And the attempt to use find_load_balancer fails the same way. | |
| 16:00:47 | gtema | ok, so empty proxy | |
| 16:01:54 | johnsom | larsks I know a bunch of changes were made to API version discovery around that time in openstacksdk. I would try adding an explicit version "v2.0" to your cloud definition for the load-balancer service. | |
| 16:02:24 | larsks | johnsom: like this? load_balancer_api_version: "v2.0" | |
| 16:02:45 | johnsom | It might be v2 though. load-balancer_api_version I think | |
| 16:03:03 | johnsom | The service type has a hyphen in it | |
| 16:03:05 | larsks | I'm pretty sure that all "-" get convereted to "_" for use as keys, but I can put both in. | |
| 16:06:12 | larsks | johnsom: I've tried this: http://termbin.com/byhkv I've set the api version to "2", "2.0", "v2", and "v2.0", all seem to result in the same error. | |
| 16:10:36 | gtema | larsks: would you try adding `openstack.enable_logging(debug=True, http_debug=True)` into your test to see which requests are being sent? I'm extending SDK with own services, which also do not properly support discovery but I have not faced such behaviour so far | |
| 16:11:04 | larsks | gtema: sure, just a second. | |
| 16:13:50 | larsks | gtema: there are a bunch of "Skipping invalid version data. Missing links" in the debug output: http://termbin.com/tgff | |
| 16:20:23 | gtema | larsks: hmm. The version discovery response is certanly bad. In each version item there should be "links" attribute | |
| 16:21:46 | gtema | larsks: i guess if no response would have been sent - it would be better than that. To tell the truth - I have no clue so far. Parsing of this document is not done in SDK, but keystoneauth1 (I guess) | |
| 16:22:33 | gtema | but I would rather ask - why is this response wrong? | |
| 16:22:52 | gtema | https://wiki.openstack.org/wiki/VersionDiscovery#links | |
| 16:22:58 | larsks | As far as I know, that's just what this version of octavia spits out. | |
| 16:23:21 | larsks | johnsom: ^^^^ is that correct? | |
| 16:23:32 | gtema | sure, but I mean why is it spitting this out | |
| 16:24:28 | gtema | there might be bug in octavia (this version) or configuration. Since I do not have Octavia - I do not know for sure | |
| 16:24:45 | larsks | That I don't know. I just wish there were some way to skip the version discovery and *tell* openstacksdk what to use. | |
| 16:25:51 | johnsom | Well, the version discovery in openstacksdk changed, this is why octavia changed and added the links in a later version. Still, openstacksdk can use Octavia, it's how the CLI and horizon dashboard work. We just have some configuration issue in openstacksdk that is causing it to not be happy for some reason. | |
| 16:26:47 | johnsom | Yeah, there definitely is a way to tell the SDK to use a specific version, but you tried the way I know. Sadly, I haven't spent much time with openstacksdk in over a year. | |
| 16:26:56 | gtema | there is a way, but just temporarily - `openstacksdk:openstack.load_balancer.v2._proxy.Proxy.skip_discovery = True` | |
| 16:27:08 | gtema | but you will not get it landed in SDK | |
| 16:28:33 | gtema | johnsom: the problem is that octavia in this case does not return a valid version discovery doc, so I might try to process this correctly. But the root should be fixed | |
| 16:29:05 | johnsom | gtema It returned a valid discovery doc for the Queens timeframe | |
| 16:29:20 | johnsom | gtema He is running Queens | |
| 16:29:55 | gtema | are you sure links was not mandatory field? I am using Mitaka and services return this attribute | |
| 16:30:09 | johnsom | Since the CLI is working, the SDK works against the queens Octavia API | |
| 16:30:26 | johnsom | Yep, it was not required until later | |
| 16:30:44 | gtema | CLI is working because octaviaclient is not using SDK (I guess) | |
| 16:31:41 | gtema | yesm octaviaclient is not using SDK as of now | |