| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-22 | |||
| 13:47:19 | frickler | dtroyer: mnaser: yes and the revert is failing due to some stackviz issue, see current discussion in -qa | |
| 13:47:30 | dtroyer | mnaser: revert: https://review.openstack.org/#/c/555016/ | |
| 13:49:02 | mnaser | okay, awesome, thanks frickler and dtroyer :) | |
| 13:49:59 | amotoki | slaweq and me need to understand how we can prevent this kind of breakage outside of neutron. I haven't figured out the whole picture | |
| 13:50:29 | amotoki | slaweq: sorry for pinging you :( | |
| 13:50:42 | slaweq | amotoki: no problem :) | |
| 13:50:52 | amotoki | hehe | |
| 13:52:01 | mnaser | amotoki: have neutron run that devstack osc job in gate? | |
| 13:52:11 | mnaser | assuming that job is fairly stable | |
| 13:54:47 | amotoki | mnaser: no, it is not part of neutron gate jobs. | |
| 13:55:13 | amotoki | mnaser: I am checking the relationship of osc-functional-devstack and neutron repo | |
| 13:55:54 | amotoki | basically they are not related directly | |
| 13:56:45 | mnaser | this type of thing can help catch things .. aka "hi im neutron and is my change breaking osc" seems like a reasonable thing | |
| 13:57:57 | dtroyer | for a Neutron change like that to break OSC means it is an API breakage, Using osc's functional tests as API tests is one way to catch that, it could also be done with stronger API testing in Neutron | |
| 13:58:50 | dtroyer | osc's functional tests do not exercise the APIs as thoroughly as I would hope dedicated API tests would do. | |
| 14:06:25 | amotoki | as my neutron hat, I believe neutron and tempest teams expect this should be caught in tempest/neutron-tempest-plugin should catch it. | |
| 14:06:41 | amotoki | i will investigate more tomorrow | |
| 14:14:03 | SamYaple | mordred: would a "think about using openstacksdk" warning be a good thing for shade? | |
| 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 | dhellmann | http://logs.openstack.org/97/555297/1/check/openstack-tox-py27/d977be1/ara/ | |
| 14:22:49 | mordred | \o/ | |
| 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.. | |