| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-22 | |||
| 14:14:37 | SamYaple | ran into someone who thought the shade/openstacksdk merger was going into shade instead of openstacksdk being the place for future dev | |
| 14:16:42 | mordred | amotoki, dtroyer: oh, yes - migration to storyboard basically involves just deciding to migrate - then infra can arrange actually doing the migration | |
| 14:17:18 | mordred | amotoki, dtroyer: so if we want to migrate python-openstackclient, we can just add it to the list of repos to migrate when we migrate openstacksdk | |
| 14:17:25 | amotoki | mordred: sounds nice. | |
| 14:17:40 | mordred | SamYaple: it might not be a bad idea | |
| 14:17:53 | amotoki | mordred: as osc team, I can follow up minor things on migration of OSC | |
| 14:18:28 | mordred | amotoki: sweet! | |
| 14:20:07 | mordred | dhellmann: requestsexceptions doesn't actually have any tests (it's a very small librar that just squelches 4 useless warnings) | |
| 14:20:25 | dhellmann | mordred : ok, so we don't need a lower-constraints job for it? | |
| 14:20:26 | mordred | dhellmann: that said - looking at that patch to see what's up | |
| 14:20:36 | dhellmann | it has requirements, so... | |
| 14:20:54 | mordred | dhellmann: it only has test-requirements - and that only includes hacking | |
| 14:21:02 | dhellmann | ok. I'll abandon that patch | |
| 14:21:23 | mordred | dhellmann: cool. I love it when problems are solved by abandoning patches | |
| 14:21:40 | mordred | dhellmann: I should remove the py35,py27 envs from tox.ini ... | |
| 14:22:05 | dhellmann | my script isn't fancy enough to figure out whether tests are needed; it just makes the change to the repo | |
| 14:22:34 | mordred | dhellmann: well - I think that's the right choice for your script - things should in general have tests | |
| 14:22:45 | dhellmann | hmm, pbr is failing the py27 tests | |
| 14:22:49 | mordred | \o/ | |
| 14:22:49 | dhellmann | http://logs.openstack.org/97/555297/1/check/openstack-tox-py27/d977be1/ara/ | |
| 14:23:37 | dhellmann | and shade is failing the new lower-constraints job, so that's fun https://review.openstack.org/#/c/555299/ | |
| 14:23:52 | mordred | dhellmann: you're giving me all sorts of fun and joy today | |
| 14:24:06 | dhellmann | whee | |
| 14:24:32 | mordred | dhellmann: those are fun failures | |
| 14:24:34 | dhellmann | NoMockAddress exceptions | |
| 14:25:05 | dhellmann | does shade use a special test library for some of that? something that is maybe set too low? | |
| 14:25:15 | dhellmann | oh, those run under python 3, fwiw | |
| 14:25:24 | mordred | dhellmann: it uses requests_mock - and python3 should be fine | |
| 14:25:38 | mordred | dhellmann: I'll pull locally and see if I can figure out which thing is breaking it | |
| 14:25:43 | dhellmann | thank you | |
| 14:25:49 | mordred | dhellmann: the lower-constraints.txt is intended to be managed locally, yeah? | |
| 14:25:52 | dhellmann | yes | |
| 14:26:15 | dhellmann | there is not currently a test to verify that the values there are not higher than what is listed in requirements txt | |
| 14:26:57 | mordred | dhellmann: btw - your how-many-summits-have-I-been-to plugin is awesome | |
| 14:27:06 | dhellmann | haha | |
| 14:27:15 | dhellmann | I'm waiting for someone to notice the off-by-one error in the math | |
| 14:27:51 | mordred | dhellmann: I didn't look deeply enough to know if it allows expressing missed summits somewhere | |
| 14:28:06 | dhellmann | feature requests will be answered with commit privileges | |
| 14:28:11 | mordred | dhellmann: :) | |
| 14:28:28 | dhellmann | in fact I need to give cmurphy commit rights, since she sent a patch | |
| 14:28:47 | cmurphy | heh | |
| 14:31:04 | dhellmann | I'm trying this new thing where I instigate something and let other people finish it | |
| 14:32:10 | mordred | dhellmann: that strategy has worked pretty well for me over the years | |
| 14:32:54 | mordred | dhellmann: in fact, this last cycle I accomplished it by _saying_ I was going to do something and then doing no work on it so that cmurphy 'fixed' my buggy code - where the bug was "there is no code" | |
| 14:33:22 | toker_ | Hi guys, Im trying to debug a problem with octavia-dashboard, which makes a call to openstacksdk to find the public endpoint of octavia. I get an exception from the sdk like this 'SDKException: public endpoint for load_balancer service in regionOne region not found'... And I need to figure out why the sdk is giving me that answer, | |
| 14:33:28 | toker_ | totally new at this so .. | |
| 14:33:31 | mordred | so while I used to consider it the clarkb strategy (implement something poorly so that it annoys clark and he fixes it) - I think it's the cmurphy strategy now :) | |
| 14:33:40 | dhellmann | mordred : that's pretty darn zen | |
| 14:34:24 | mordred | toker_: I have a hunch it's going to be a _ vs - thing - I was just looking at an issue related to that this morning | |
| 14:34:52 | toker_ | Well, I tried changing the endpoint to _ | |
| 14:35:03 | toker_ | bit then I got another error instead :/ | |
| 14:35:31 | toker_ | service_type) File "/usr/lib/python2.7/site-packages/openstack/session.py", line 277, in _get_version_match for link in version["links"]: KeyError: 'links' | |
| 14:35:55 | mordred | toker_: are these errors in a test job or just locally for you? | |
| 14:36:15 | toker_ | no these are errrors in our POC of our openstack setup | |
| 14:36:59 | toker_ | so, openstack loadbalancer list works. But when octavia-dashboard tries to get the endpoint through the sdk, it fails. | |
| 14:37:10 | cmurphy | mordred: it was a fairly severe bug | |
| 14:37:23 | toker_ | And Ive been stuck here for hours trying to figure out why :( | |
| 14:38:12 | mordred | toker_: kk. I'm making a quick test script to hand you ... I bet we can figure it out | |
| 14:38:37 | toker_ | Oh, thanks! That's really nice | |
| 14:41:29 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/os-service-types master: Fix docs link in the README https://review.openstack.org/555324 | |
| 14:42:59 | mordred | toker_: http://paste.openstack.org/show/708969/ <-- if you run that and paste the output into paste.openstack.org we can see what's going wrong. | |
| 14:43:27 | mordred | toker_: (it assumes you're using env vars - if you have a named cloud in a clouds.yaml file, you'll want to change cloud='envvars') | |
| 14:43:58 | johnsom | toker_ note that you are on an older version of OpenStack, not queens | |
| 14:44:29 | mordred | for reference - this is me running that script against the vexxhost public cloud: http://paste.openstack.org/show/708970 | |
| 14:48:29 | toker_ | AttributeError: 'module' object has no attribute 'enable_logging' <-- | |
| 14:48:54 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | |
| 14:49:12 | mordred | toker_: ok. that's an older version of sdk then -I'm guessing from the 0.9 range | |
| 14:49:41 | toker_ | we are using OSP 12 (Pike) | |
| 14:50:23 | mordred | nod. and you're installing the octavia-dashboard from OSP 12 as well? | |
| 14:52:44 | toker_ | There is no official octavia-dashboard for OSP12. This is me trying to get the "master" working for OSP 12. | |
| 14:53:11 | toker_ | https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/lbaasv2.py#L38 | |
| 14:53:20 | toker_ | There is the connection to the SDK | |
| 14:59:10 | mordred | GOTCHA. so - this is a fun | |
| 14:59:55 | toker_ | Haha yes, very much so. 12 hours ago it was fun. Now its more like "god damn I just want this shit working" | |
| 14:59:57 | toker_ | :D | |
| 15:00:16 | mordred | toker_: :) | |
| 15:00:27 | mordred | toker_: how did you install the octavia-dashboard code? pip? globally or into a virtualenv? | |
| 15:01:20 | mordred | asking because I think running master octavia-dashboard with pike openstacksdk is going to be hard - but also wanting to keep troubleshooting paths to a minimum | |
| 15:01:35 | mordred | dhellmann: http://review.openstack.org/555299 should fix the shade l-c job | |
| 15:01:57 | mordred | dhellmann: it was requests-mock - 1.1.0 is not sufficient. I'm guessing I should go send in a patch to bump the min on it | |
| 15:02:49 | toker_ | mordred: well, I *think* it should work, if just could get the integration to SDK working. | |
| 15:03:15 | toker_ | its installed by cloning the repo "python setup.py install" inside the container running horizon.. | |
| 15:03:55 | toker_ | So basically, /usr/lib/python2.7/site-packages/octavia_dashboard/api/rest/lbaasv2.py <- making changes to this file and restarting the container everytime I try a change. | |
| 15:05:33 | mordred | toker_: gotcha. SO - could you try using the python in that container to run that script I pasted? | |
| 15:06:27 | toker_ | Yes, same error as I posted before. Hm, the guys in #openstack-lbaas channel actually says I need the 0.9.19 SDK | |
| 15:06:37 | toker_ | they got it working with that version... | |
| 15:06:38 | toker_ | hm | |
| 15:07:55 | mordred | for master of octavia-dashboard? that seems weird to me and like something we should track down - but if they can get you unbroken go for it :) | |
| 15:08:30 | toker_ | Well, I don't know.. Thats what one of the guys trying to help me said. | |
| 15:09:15 | toker_ | I would rather try to get it working with the current version of the SDK (then I dont need to try to upgrade the SDK that horizon may be depending on inside the container) | |
| 15:10:40 | toker_ | Is there something obvious with this code https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/lbaasv2.py#L38 that makes it not return the endpoint for the loadbalancer ( I mean the error is pretty verbose, right ? "SDKException: public endpoint for load_balancer service in regionOne region not found") | |
| 15:11:15 | toker_ | I'm not sure updating the SDK would help with that error ? | |
| 15:13:23 | toker_ | I want to understand why the SDK cant find the public endpoint - however getting there seems hard | |
| 15:18:03 | toker_ | I also tried to revert one specific commit, https://review.openstack.org/#/c/524011/2/octavia_dashboard/api/rest/lbaasv2.py , which supposedly could make it work with "an older sdk". But I still get the same error from the sdk that it cant find the public endpoint.. | |
| 15:20:32 | mordred | toker_: right - so - there's nothing in that code that would cause an error - the heavy-lifting of discovery is done in keystoneauth | |
| 15:21:33 | mordred | toker_: which is why I was curious about the output of that script - basically hoping something jumps out at me in the rest log | |
| 15:21:59 | toker_ | Can one do a similar script but for the older sDK ? | |
| 15:22:10 | mordred | so I think the things to verify are - which version of openstacksdk is installed in that container with octavia-dashboard | |
| 15:23:02 | toker_ | Ok, that is 0.9.17 | |
| 15:24:06 | mordred | cool. so - now we get to figure out how to debug the old sdk ... one sec | |