| *** salv-orlando has joined #openstack-sdks | 00:27 | |
| *** salv-orlando has quit IRC | 00:32 | |
| *** chenyb4 has joined #openstack-sdks | 01:04 | |
| adriant | mordred: I don't know if we can make it nicer, but having to pass in the service itself when calling actions on the resources is a pain: | 01:07 |
|---|---|---|
| adriant | test_project.assign_role_to_user(conn.identity, test_user, member_role) | 01:07 |
| adriant | not to mention, having those kind of calls directly on the proxy might be nicer, since I don't always want to have to query for the project first if I know what the ids are of all the resources I want to act on are. | 01:08 |
| adriant | *also on the proxy | 01:08 |
| *** salv-orlando has joined #openstack-sdks | 01:27 | |
| *** harlowja has quit IRC | 01:31 | |
| *** salv-orlando has quit IRC | 01:32 | |
| *** dayou has quit IRC | 02:18 | |
| *** salv-orlando has joined #openstack-sdks | 02:28 | |
| *** ldl has joined #openstack-sdks | 02:31 | |
| *** salv-orlando has quit IRC | 02:34 | |
| *** edmondsw has joined #openstack-sdks | 02:42 | |
| adriant | mordred: an error this time. Trying to see what response the sdk gives me when trying to create a keystone user that already exists: HttpException: HttpException: Unknown error | 02:42 |
| adriant | mordred: looks like an easy one. We aren't catching 409s in the raise from exception function | 02:44 |
| adriant | I'll throw up a patch for that | 02:45 |
| *** edmondsw has quit IRC | 02:49 | |
| *** shu-mutow has joined #openstack-sdks | 03:05 | |
| * adriant is very very confused | 03:05 | |
| adriant | mordred: https://github.com/openstack/python-openstacksdk/blob/master/openstack/exceptions.py#L63 is evaluating as false... expect I KNOW there is a response with status 409... | 03:06 |
| adriant | except* | 03:06 |
| adriant | if I print response.text in that very place I get: "{"error": {"message": "Conflict occurred attempting to store user - Duplicate Entry", "code": 409, "title": "Conflict"}}" | 03:07 |
| adriant | so I have no clue what magic is making a response of type 409 eval to false... | 03:08 |
| *** dave-mccowan has quit IRC | 03:11 | |
| *** annp has joined #openstack-sdks | 03:14 | |
| adriant | wait... wtf response always evals to false | 03:16 |
| adriant | OH | 03:16 |
| adriant | it is false when it isn't a safe status code... I'm willing to bet that's it | 03:16 |
| * adriant goes to read requests code | 03:16 | |
| adriant | mordred: we're using it wrong: https://github.com/requests/requests/blob/master/requests/models.py#L663-L671 | 03:17 |
| * adriant makes another patch | 03:17 | |
| *** salv-orlando has joined #openstack-sdks | 03:30 | |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Add 409 ConflictException https://review.openstack.org/555142 | 03:31 |
| *** harlowja has joined #openstack-sdks | 03:32 | |
| *** salv-orlando has quit IRC | 03:35 | |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Fix response always being False https://review.openstack.org/555143 | 03:37 |
| *** d0ugal_ has joined #openstack-sdks | 03:39 | |
| *** d0ugal has quit IRC | 03:39 | |
| * adriant finds all the best bugs | 03:53 | |
| *** harlowja has quit IRC | 03:54 | |
| *** edmondsw has joined #openstack-sdks | 04:05 | |
| *** edmondsw has quit IRC | 04:10 | |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Fix response always being False https://review.openstack.org/555143 | 04:14 |
| *** salv-orlando has joined #openstack-sdks | 04:31 | |
| *** salv-orlando has quit IRC | 04:36 | |
| *** gildub has quit IRC | 04:51 | |
| *** gildub has joined #openstack-sdks | 05:05 | |
| *** salv-orlando has joined #openstack-sdks | 05:29 | |
| *** ldl has quit IRC | 05:33 | |
| *** edmondsw has joined #openstack-sdks | 05:54 | |
| *** edmondsw has quit IRC | 05:59 | |
| *** e0ne has joined #openstack-sdks | 06:15 | |
| *** e0ne has quit IRC | 06:33 | |
| *** e0ne has joined #openstack-sdks | 06:46 | |
| *** e0ne has quit IRC | 06:52 | |
| *** d0ugal_ has quit IRC | 07:14 | |
| *** d0ugal has joined #openstack-sdks | 07:14 | |
| *** d0ugal has quit IRC | 07:14 | |
| *** d0ugal has joined #openstack-sdks | 07:14 | |
| *** zacksh has quit IRC | 07:25 | |
| *** Hazelesque has quit IRC | 07:25 | |
| *** slaweq has quit IRC | 07:25 | |
| *** Anticimex has quit IRC | 07:25 | |
| *** tobixen has quit IRC | 07:25 | |
| *** RuiChen has quit IRC | 07:25 | |
| *** Kvisle has quit IRC | 07:25 | |
| *** timss has quit IRC | 07:25 | |
| *** slaweq has joined #openstack-sdks | 07:26 | |
| *** Hazelesque has joined #openstack-sdks | 07:26 | |
| *** Kvisle has joined #openstack-sdks | 07:26 | |
| *** Hazelesque has quit IRC | 07:26 | |
| *** Hazelesque has joined #openstack-sdks | 07:26 | |
| *** RuiChen has joined #openstack-sdks | 07:27 | |
| *** zacksh has joined #openstack-sdks | 07:27 | |
| *** timss has joined #openstack-sdks | 07:27 | |
| *** tobixen has joined #openstack-sdks | 07:28 | |
| *** salv-orlando has quit IRC | 07:33 | |
| *** markvoelker has quit IRC | 07:35 | |
| *** ralonsoh has joined #openstack-sdks | 07:39 | |
| *** edmondsw has joined #openstack-sdks | 07:42 | |
| *** edmondsw has quit IRC | 07:46 | |
| *** GreenBlood has joined #openstack-sdks | 08:23 | |
| openstackgerrit | Andreas Jaeger proposed openstack/keystoneauth master: Remove tox_install.sh and align with constraints consumption https://review.openstack.org/550837 | 08:23 |
| *** GreenBlood has quit IRC | 08:31 | |
| *** salv-orlando has joined #openstack-sdks | 08:34 | |
| *** markvoelker has joined #openstack-sdks | 08:34 | |
| *** yolanda_ is now known as yolanda | 08:35 | |
| *** salv-orlando has quit IRC | 08:40 | |
| *** GreenBlood has joined #openstack-sdks | 08:47 | |
| *** jpena|off is now known as jpena | 08:48 | |
| *** jpich has joined #openstack-sdks | 08:54 | |
| *** e0ne has joined #openstack-sdks | 08:56 | |
| *** e0ne has quit IRC | 09:07 | |
| *** e0ne has joined #openstack-sdks | 09:11 | |
| *** e0ne has quit IRC | 09:15 | |
| *** gkadam has joined #openstack-sdks | 09:19 | |
| *** gildub has quit IRC | 09:29 | |
| *** edmondsw has joined #openstack-sdks | 09:30 | |
| *** shu-mutow is now known as shu-mutow-AWAY | 09:32 | |
| *** edmondsw has quit IRC | 09:35 | |
| *** salv-orlando has joined #openstack-sdks | 09:35 | |
| *** salv-orlando has quit IRC | 09:38 | |
| *** salv-orlando has joined #openstack-sdks | 09:38 | |
| *** dayou has joined #openstack-sdks | 09:38 | |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Add 409 ConflictException https://review.openstack.org/555142 | 09:42 |
| *** dayou has quit IRC | 09:44 | |
| *** dayou has joined #openstack-sdks | 09:53 | |
| *** ioggstream has joined #openstack-sdks | 10:01 | |
| openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix api request in header miss microversions https://review.openstack.org/555231 | 10:03 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/553257 | 10:07 |
| openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix api request in header miss microversions https://review.openstack.org/555231 | 10:08 |
| *** e0ne has joined #openstack-sdks | 10:10 | |
| *** chenyb4 has quit IRC | 10:14 | |
| *** dtantsur|afk is now known as dtantsur | 10:36 | |
| mordred | adriant: you do find all the best bugs! | 10:47 |
| *** kmy has quit IRC | 10:51 | |
| *** kmy has joined #openstack-sdks | 10:52 | |
| openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Remove tox_install.sh and align with constraints consumption https://review.openstack.org/550837 | 10:55 |
| openstackgerrit | Merged openstack-infra/shade master: Updated from global requirements https://review.openstack.org/552305 | 10:59 |
| *** chenyb4 has joined #openstack-sdks | 11:44 | |
| openstackgerrit | Merged openstack/python-openstacksdk master: Fix TypeError in case of FloatingIP add and remove https://review.openstack.org/552112 | 11:57 |
| *** chenyb4 has quit IRC | 12:03 | |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix response always being False https://review.openstack.org/555143 | 12:07 |
| mordred | rods, slaweq: if you have a sec, https://review.openstack.org/#/c/555142 and https://review.openstack.org/#/c/555143 are both nice bugfixes | 12:08 |
| slaweq | mordred: both +W | 12:09 |
| mordred | does anyone know if chenyb4 / Yuanbin.Chen is on IRC? | 12:10 |
| adriant | mordred: that requests response is not none fix is gonna fail zuul after that rebase | 12:11 |
| adriant | since... the second is not none... doesn't catch the empty dict | 12:12 |
| mordred | adriant: oh yeah? | 12:12 |
| mordred | oh - fun | 12:12 |
| adriant | and somewhere someone is passing in a empty dict | 12:12 |
| adriant | so we may have to be silly and do if type(response) is Response: | 12:12 |
| mordred | adriant: like, somewhere something is passing an empty dict instead of an actual Response object? | 12:12 |
| adriant | mordred: looks like | 12:13 |
| adriant | that's what the test failure looked like | 12:13 |
| adriant | I'm just about to run it locally and check again | 12:13 |
| mordred | adriant: have I mentioned recently that I really miss strong typing? | 12:13 |
| adriant | yeah... normally I don't hit these issues, but every now and again you wish we had it | 12:14 |
| mordred | adriant: cool - I'd love to find/fix the calling location if we can - but yeah, I can see maybe needing to do isinstance(type, Response) ... but that just seems so sad | 12:14 |
| *** edmondsw has joined #openstack-sdks | 12:15 | |
| adriant | isinstance! of course, silly me I use that one but always forget it | 12:15 |
| mordred | :) | 12:15 |
| mordred | I have to remind myself of it all the time | 12:15 |
| adriant | is this and instance to use it? ;) | 12:15 |
| adriant | an* | 12:15 |
| * adriant fails at his own terrible pun | 12:15 | |
| adriant | mordred: self.request_id = response.headers.get('x-openstack-request-id')" | 12:18 |
| adriant | b"AttributeError: 'dict' object has no attribute 'headers'" | 12:18 |
| adriant | yeah, somewhere it's a dict | 12:18 |
| openstackgerrit | Merged openstack/os-client-config master: pypy is not checked at gate https://review.openstack.org/554955 | 12:23 |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Fix response always being False https://review.openstack.org/555143 | 12:23 |
| openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Fix response always being False https://review.openstack.org/555143 | 12:25 |
| *** chenyb4 has joined #openstack-sdks | 12:27 | |
| adriant | mordred: a better fix was found so (to try that pun again), this isn't an instance where we type check. :P | 12:28 |
| *** e0ne has quit IRC | 12:31 | |
| mordred | adriant: \o/ | 12:33 |
| *** cdent has joined #openstack-sdks | 12:34 | |
| *** markvoelker has quit IRC | 12:34 | |
| *** markvoelker has joined #openstack-sdks | 12:34 | |
| *** chenyb4 has quit IRC | 12:34 | |
| *** thrash|g0ne is now known as thrash | 12:43 | |
| *** e0ne has joined #openstack-sdks | 12:45 | |
| *** bobh has joined #openstack-sdks | 13:06 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/requestsexceptions master: add lower-constraints job https://review.openstack.org/555298 | 13:07 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | 13:07 |
| openstackgerrit | Merged openstack/python-openstacksdk master: Add 409 ConflictException https://review.openstack.org/555142 | 13:18 |
| *** dtantsur is now known as dtantsur|brb | 13:21 | |
| amotoki | dtroyer: regarding storyboard migration along with SDK in the dev list, do you have any specific concern? In general, I can take care of the migration if it happens. | 13:26 |
| dhellmann | mordred : what's up with requestsexceptions? I don't see any of the unit test jobs running against https://review.openstack.org/#/c/555298/ | 13:27 |
| dtroyer | amotoki: the only thing I thought we should do would be a pass through the bugs and clean up obsolete ones, but IIRc all get migrated anyway so I'm not sure that is important | 13:28 |
| dtroyer | amotoki: if you want to handle the migration, that would be awesome, thanks! | 13:28 |
| amotoki | dtroyer: honestly I don't know what kind of tools we actually have. On obsolete bugs, I think we can look thru them even after the migration. | 13:32 |
| amotoki | dtroyer: tags in storyboard looks like global (from my limited experience), but it needs to be clarified. | 13:33 |
| amotoki | I only have experiences on storyboard around octavia migration. | 13:34 |
| dtroyer | amotoki: sure, that is more than I have with Storyboard :) | 13:34 |
| amotoki | dtroyer: from my impressions so far, it seems we have no big concern on osc migration to storyboard. | 13:40 |
| amotoki | dtroyer: I can care things during migrations and discuss broader topics with mordred and/or infra team. | 13:40 |
| amotoki | dtroyer: i will send a follow-up mail to the list | 13:40 |
| dtroyer | amotoki: sounds good, thanks | 13:42 |
| mnaser | i think the openstackclient gate is broken? | 13:42 |
| mnaser | osc-functional-devstack seems to always fail | 13:42 |
| amotoki | mordred: in my understanding so far, we have a tool to support migration from launchpad to storyboard, right? | 13:43 |
| mnaser | it looks like creating a network segment now does not have a description of 'None' but an empty string | 13:44 |
| amotoki | mordred: based on chats with dtroyer here, there is no big blocking issue to block OSC storyboard migration. the remaining thing is how we can automate it. | 13:45 |
| dtroyer | mnaser: frickler chased that down yesterday to a neutron commit that is being reverted | 13:45 |
| frickler | dtroyer: mnaser: yes and the revert is failing due to some stackviz issue, see current discussion in -qa | 13:47 |
| dtroyer | mnaser: revert: https://review.openstack.org/#/c/555016/ | 13:47 |
| mnaser | okay, awesome, thanks frickler and dtroyer :) | 13:49 |
| 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:49 |
| amotoki | slaweq: sorry for pinging you :( | 13:50 |
| slaweq | amotoki: no problem :) | 13:50 |
| amotoki | hehe | 13:50 |
| mnaser | amotoki: have neutron run that devstack osc job in gate? | 13:52 |
| mnaser | assuming that job is fairly stable | 13:52 |
| amotoki | mnaser: no, it is not part of neutron gate jobs. | 13:54 |
| amotoki | mnaser: I am checking the relationship of osc-functional-devstack and neutron repo | 13:55 |
| amotoki | basically they are not related directly | 13:55 |
| 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:56 |
| 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:57 |
| dtroyer | osc's functional tests do not exercise the APIs as thoroughly as I would hope dedicated API tests would do. | 13:58 |
| 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 |
| amotoki | i will investigate more tomorrow | 14:06 |
| *** gkadam has quit IRC | 14:09 | |
| SamYaple | mordred: would a "think about using openstacksdk" warning be a good thing for shade? | 14:14 |
| SamYaple | ran into someone who thought the shade/openstacksdk merger was going into shade instead of openstacksdk being the place for future dev | 14:14 |
| *** dtantsur|brb is now known as dtantsur | 14:15 | |
| mordred | amotoki, dtroyer: oh, yes - migration to storyboard basically involves just deciding to migrate - then infra can arrange actually doing the migration | 14:16 |
| 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 |
| amotoki | mordred: sounds nice. | 14:17 |
| mordred | SamYaple: it might not be a bad idea | 14:17 |
| amotoki | mordred: as osc team, I can follow up minor things on migration of OSC | 14:17 |
| mordred | amotoki: sweet! | 14:18 |
| mordred | dhellmann: requestsexceptions doesn't actually have any tests (it's a very small librar that just squelches 4 useless warnings) | 14:20 |
| dhellmann | mordred : ok, so we don't need a lower-constraints job for it? | 14:20 |
| mordred | dhellmann: that said - looking at that patch to see what's up | 14:20 |
| dhellmann | it has requirements, so... | 14:20 |
| mordred | dhellmann: it only has test-requirements - and that only includes hacking | 14:20 |
| dhellmann | ok. I'll abandon that patch | 14:21 |
| mordred | dhellmann: cool. I love it when problems are solved by abandoning patches | 14:21 |
| mordred | dhellmann: I should remove the py35,py27 envs from tox.ini ... | 14:21 |
| dhellmann | my script isn't fancy enough to figure out whether tests are needed; it just makes the change to the repo | 14:22 |
| mordred | dhellmann: well - I think that's the right choice for your script - things should in general have tests | 14:22 |
| dhellmann | hmm, pbr is failing the py27 tests | 14:22 |
| dhellmann | http://logs.openstack.org/97/555297/1/check/openstack-tox-py27/d977be1/ara/ | 14:22 |
| mordred | \o/ | 14:22 |
| dhellmann | and shade is failing the new lower-constraints job, so that's fun https://review.openstack.org/#/c/555299/ | 14:23 |
| mordred | dhellmann: you're giving me all sorts of fun and joy today | 14:23 |
| dhellmann | whee | 14:24 |
| mordred | dhellmann: those are fun failures | 14:24 |
| dhellmann | NoMockAddress exceptions | 14:24 |
| dhellmann | does shade use a special test library for some of that? something that is maybe set too low? | 14:25 |
| dhellmann | oh, those run under python 3, fwiw | 14:25 |
| mordred | dhellmann: it uses requests_mock - and python3 should be fine | 14:25 |
| mordred | dhellmann: I'll pull locally and see if I can figure out which thing is breaking it | 14:25 |
| dhellmann | thank you | 14:25 |
| mordred | dhellmann: the lower-constraints.txt is intended to be managed locally, yeah? | 14:25 |
| dhellmann | yes | 14:25 |
| 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 |
| mordred | dhellmann: btw - your how-many-summits-have-I-been-to plugin is awesome | 14:26 |
| dhellmann | haha | 14:27 |
| dhellmann | I'm waiting for someone to notice the off-by-one error in the math | 14:27 |
| mordred | dhellmann: I didn't look deeply enough to know if it allows expressing missed summits somewhere | 14:27 |
| dhellmann | feature requests will be answered with commit privileges | 14:28 |
| mordred | dhellmann: :) | 14:28 |
| dhellmann | in fact I need to give cmurphy commit rights, since she sent a patch | 14:28 |
| cmurphy | heh | 14:28 |
| dhellmann | I'm trying this new thing where I instigate something and let other people finish it | 14:31 |
| *** toker_ has joined #openstack-sdks | 14:31 | |
| mordred | dhellmann: that strategy has worked pretty well for me over the years | 14:32 |
| 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:32 |
| 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 |
| toker_ | totally new at this so .. | 14:33 |
| 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 |
| dhellmann | mordred : that's pretty darn zen | 14:33 |
| 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 |
| toker_ | Well, I tried changing the endpoint to _ | 14:34 |
| toker_ | bit then I got another error instead :/ | 14:35 |
| 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 |
| mordred | toker_: are these errors in a test job or just locally for you? | 14:35 |
| toker_ | no these are errrors in our POC of our openstack setup | 14:36 |
| toker_ | so, openstack loadbalancer list works. But when octavia-dashboard tries to get the endpoint through the sdk, it fails. | 14:36 |
| cmurphy | mordred: it was a fairly severe bug | 14:37 |
| toker_ | And Ive been stuck here for hours trying to figure out why :( | 14:37 |
| mordred | toker_: kk. I'm making a quick test script to hand you ... I bet we can figure it out | 14:38 |
| toker_ | Oh, thanks! That's really nice | 14:38 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/os-service-types master: Fix docs link in the README https://review.openstack.org/555324 | 14:41 |
| 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:42 |
| 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 |
| johnsom | toker_ note that you are on an older version of OpenStack, not queens | 14:43 |
| mordred | for reference - this is me running that script against the vexxhost public cloud: http://paste.openstack.org/show/708970 | 14:44 |
| toker_ | AttributeError: 'module' object has no attribute 'enable_logging' <-- | 14:48 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | 14:48 |
| mordred | toker_: ok. that's an older version of sdk then -I'm guessing from the 0.9 range | 14:49 |
| toker_ | we are using OSP 12 (Pike) | 14:49 |
| mordred | nod. and you're installing the octavia-dashboard from OSP 12 as well? | 14:50 |
| toker_ | There is no official octavia-dashboard for OSP12. This is me trying to get the "master" working for OSP 12. | 14:52 |
| toker_ | https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/lbaasv2.py#L38 | 14:53 |
| toker_ | There is the connection to the SDK | 14:53 |
| *** bobh has quit IRC | 14:57 | |
| mordred | GOTCHA. so - this is a fun | 14:59 |
| 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 |
| toker_ | :D | 14:59 |
| mordred | toker_: :) | 15:00 |
| mordred | toker_: how did you install the octavia-dashboard code? pip? globally or into a virtualenv? | 15:00 |
| 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 |
| mordred | dhellmann: http://review.openstack.org/555299 should fix the shade l-c job | 15:01 |
| 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:01 |
| toker_ | mordred: well, I *think* it should work, if just could get the integration to SDK working. | 15:02 |
| toker_ | its installed by cloning the repo "python setup.py install" inside the container running horizon.. | 15:03 |
| 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:03 |
| mordred | toker_: gotcha. SO - could you try using the python in that container to run that script I pasted? | 15:05 |
| 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 |
| toker_ | they got it working with that version... | 15:06 |
| toker_ | hm | 15:06 |
| 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:07 |
| toker_ | Well, I don't know.. Thats what one of the guys trying to help me said. | 15:08 |
| 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:09 |
| 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:10 |
| toker_ | I'm not sure updating the SDK would help with that error ? | 15:11 |
| toker_ | I want to understand why the SDK cant find the public endpoint - however getting there seems hard | 15:13 |
| *** dave-mccowan has joined #openstack-sdks | 15:15 | |
| 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:18 |
| 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:20 |
| 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 |
| toker_ | Can one do a similar script but for the older sDK ? | 15:21 |
| mordred | so I think the things to verify are - which version of openstacksdk is installed in that container with octavia-dashboard | 15:22 |
| toker_ | Ok, that is 0.9.17 | 15:23 |
| mordred | cool. so - now we get to figure out how to debug the old sdk ... one sec | 15:24 |
| toker_ | ok ok | 15:24 |
| mordred | toker_: ok. well - if it makes you feel any better, I can confirm that sdk 0.9.19 does not find the load balancer service in the vexxhost public cloud either - with the same error message | 15:26 |
| toker_ | oh :/ | 15:27 |
| toker_ | Hm, thats interesting | 15:27 |
| *** bobh has joined #openstack-sdks | 15:28 | |
| *** bobh has quit IRC | 15:30 | |
| *** bobh has joined #openstack-sdks | 15:30 | |
| mordred | toker_: yah. well - all of the code to handle this was basically completely replaced in the 0.10 release | 15:31 |
| mordred | toker_: but - I'm working on a patch for you ... since I can reproduce your issue here, it's a little easier | 15:31 |
| toker_ | Oh, for the better then you mean ? It wasnt working properly before ? | 15:31 |
| toker_ | Ok thats very nice of you | 15:31 |
| mordred | toker_: ok. don't laugh ... http://paste.openstack.org/show/708978 <-- if you apply that to the files in your container - it should fix you | 15:39 |
| mordred | toker_: it's obviously not a GENERAL fix, but the general fix involves upgrading more things | 15:40 |
| mordred | toker_: part of the issue is that pike octavia doesn't have a working version discovery document ... I believe johnsom got that fixed this last cycle | 15:40 |
| toker_ | laugh ? How could I laugh, you just fixed a problem I've been struggling with for over 10 hours !! | 15:42 |
| toker_ | This is a very hacky setup at the moment, so adding more hacks to it doesn't really matter. | 15:42 |
| toker_ | We are going to reinstall everything when OSP 13 is out anyway. | 15:43 |
| toker_ | Omg, so simple fix! | 15:43 |
| toker_ | Thank you | 15:43 |
| toker_ | took you ten minutes to find, I've never would have found it :) | 15:43 |
| toker_ | AttributeError: 'Proxy' object has no attribute 'listeners' <- stumbled on this now when trying to list listeners ... | 15:45 |
| toker_ | But thats fine, dont really matter. | 15:46 |
| mordred | woot! | 15:48 |
| toker_ | https://github.com/openstack/python-openstacksdk/commit/d1b242ee850f286f1fae56021f2edf13cde1eb9c <- maybe I need this | 15:49 |
| toker_ | one guy said that in the lbaas-channel | 15:49 |
| toker_ | hm dont think thats related -. | 15:51 |
| *** fbaca has quit IRC | 15:53 | |
| mordred | no - that shouldn't be related to the listeners issue - although in general that patch should be fine and safe to backport if you do need to get the list of pools on a health_monitor | 16:01 |
| *** harlowja has joined #openstack-sdks | 16:04 | |
| *** harlowja has quit IRC | 16:09 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | 16:15 |
| *** thrash is now known as thrash|biab | 16:15 | |
| dhellmann | mordred : I updated your update to also change the test-requirements.txt file ^^ | 16:25 |
| *** bobh has quit IRC | 16:29 | |
| *** cdent has left #openstack-sdks | 16:46 | |
| *** thrash|biab is now known as thrash | 16:49 | |
| elmiko | edleafe dtantsur , ready for review https://etherpad.openstack.org/p/api-sig-newsletter | 17:04 |
| * elmiko notices that cdent left before review, makes note on tally sheet | 17:04 | |
| dtantsur | elmiko: one comment there, everything else LGTM | 17:07 |
| elmiko | cool | 17:07 |
| dtantsur | maybe I just misunderstood the phrase though | 17:07 |
| elmiko | no no, good spot | 17:07 |
| elmiko | i missed a word lol | 17:07 |
| edleafe | elmiko: LGTM | 17:12 |
| elmiko | thanks! | 17:13 |
| mordred | dhellmann: sweet | 17:18 |
| *** dtantsur is now known as dtantsur|afk | 17:23 | |
| *** ioggstream has quit IRC | 17:31 | |
| *** jpich has quit IRC | 17:31 | |
| *** jroll has joined #openstack-sdks | 17:39 | |
| *** e0ne has quit IRC | 17:39 | |
| *** Anticimex has joined #openstack-sdks | 17:50 | |
| *** jpena is now known as jpena|off | 18:00 | |
| *** boris_42_ has joined #openstack-sdks | 18:13 | |
| *** harlowja has joined #openstack-sdks | 18:15 | |
| *** harlowja_ has joined #openstack-sdks | 18:17 | |
| *** harlowja has quit IRC | 18:19 | |
| *** dave-mccowan has quit IRC | 18:20 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | 18:21 |
| openstackgerrit | Doug Hellmann proposed openstack/cliff master: add lower-constraints job https://review.openstack.org/555405 | 18:25 |
| *** ralonsoh has quit IRC | 18:31 | |
| *** cdent has joined #openstack-sdks | 18:53 | |
| jroll | does anyone have experience running nodepool or shade against clouds that put services on subpaths, e.g. https://mycloud/nova/v2? we're playing around with that and shade seems to follow the links returned from / and not seeing a path prefix of sorts in most project's config (nova has one, but it's host and all, not just the path) | 19:13 |
| jroll | or experience running a cloud like that, and could point to something I'm missing | 19:14 |
| jroll | considering starting a thread on the mailing list about making a consistent config between projects for that, if there isn't something obvious I'm missing here | 19:14 |
| *** zer0c00l has joined #openstack-sdks | 19:18 | |
| cdent | jroll: is the service catalog returning reasonable entries? | 19:19 |
| cdent | also you might be interested in https://review.openstack.org/#/c/459869/ | 19:20 |
| mordred | jroll: shade should totally work with suburls.. | 19:22 |
| mordred | jroll: as long as the service-catalog and version discovery documents are correct | 19:22 |
| mordred | jroll: I'm not sure how to configure the services to return the correct thing in their version discovery docs... | 19:22 |
| mordred | jroll: but devstack is running with suburl layout these days and shade's functional tests work against it -so it's gotta be possible _somehow_ | 19:23 |
| jroll | mordred: right, that last part is the catch | 19:23 |
| jroll | oh, devstack does that now? I can look at that | 19:23 |
| mordred | yah | 19:23 |
| mordred | iirc it's the default deployment now | 19:23 |
| mordred | jroll: but the tl;dr is that shade uses version discovery (except for neutron and swift) - and treats the values in the version discovery documents as being *mostly* the source of truth | 19:24 |
| jroll | mordred: yeah, I was curious if we were doing something wrong with the service or with shade | 19:25 |
| mordred | jroll: we do discard whatever host is returned in the version docs - because that is FREQUENTLY wrong in many configs (services return their rfc1918 ip in the url they make - so we replace the version discovery host, scheme and port with the host, scheme and port from the catalog) | 19:25 |
| mordred | jroll: it's always possible we've got bugs, but shade is a pretty good litmus test for "is this service configured correctly" :) | 19:26 |
| jroll | mordred: indeed :) | 19:27 |
| mordred | jroll: (if you ever want WAY more information on this topic I'm happy to assault you - but I'd like consent first :) ) | 19:27 |
| jroll | mordred: ha, I'll pass for now. I know I'm missing something obvious, I just don't know what | 19:29 |
| openstackgerrit | Bailey Johnson proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537 | 19:41 |
| jroll | to follow up, I dug in the code harder and turns out it comes from req.application_url here (at least in nova): https://docs.pylonsproject.org/projects/webob/en/stable/#urls | 19:44 |
| jroll | so probably an issue with our apache config | 19:44 |
| openstackgerrit | Bailey Johnson proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537 | 19:55 |
| *** boris_42_ has quit IRC | 20:41 | |
| *** cdent has quit IRC | 20:45 | |
| *** edmondsw has quit IRC | 20:56 | |
| *** edmondsw has joined #openstack-sdks | 20:56 | |
| *** edmondsw has quit IRC | 21:01 | |
| *** salv-orlando has quit IRC | 21:19 | |
| *** salv-orlando has joined #openstack-sdks | 21:19 | |
| *** salv-orlando has quit IRC | 21:24 | |
| *** salv-orlando has joined #openstack-sdks | 21:26 | |
| *** dayou has quit IRC | 22:42 | |
| *** dayou has joined #openstack-sdks | 22:43 | |
| *** gildub has joined #openstack-sdks | 22:43 | |
| *** edmondsw has joined #openstack-sdks | 22:46 | |
| *** dayou has quit IRC | 22:52 | |
| *** dayou has joined #openstack-sdks | 22:55 | |
| *** ccha has quit IRC | 23:21 | |
| *** e0ne has joined #openstack-sdks | 23:22 | |
| *** halbbalda has quit IRC | 23:44 | |
| *** halbbalda has joined #openstack-sdks | 23:45 | |
| *** calebb has quit IRC | 23:50 | |
| *** halbbalda has quit IRC | 23:51 | |
| *** calebb has joined #openstack-sdks | 23:52 | |
| *** e0ne_ has joined #openstack-sdks | 23:54 | |
| *** e0ne has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!