| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-22 | |||
| 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 | |
| 15:24:35 | toker_ | ok ok | |
| 15:26:59 | 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:27:20 | toker_ | oh :/ | |
| 15:27:27 | toker_ | Hm, thats interesting | |
| 15:31:22 | mordred | toker_: yah. well - all of the code to handle this was basically completely replaced in the 0.10 release | |
| 15:31:48 | 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:55 | toker_ | Oh, for the better then you mean ? It wasnt working properly before ? | |
| 15:31:59 | toker_ | Ok thats very nice of you | |
| 15:39:41 | 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:40:01 | mordred | toker_: it's obviously not a GENERAL fix, but the general fix involves upgrading more things | |
| 15:40:53 | 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:42:35 | toker_ | laugh ? How could I laugh, you just fixed a problem I've been struggling with for over 10 hours !! | |
| 15:42:57 | toker_ | This is a very hacky setup at the moment, so adding more hacks to it doesn't really matter. | |
| 15:43:09 | toker_ | We are going to reinstall everything when OSP 13 is out anyway. | |
| 15:43:38 | toker_ | Omg, so simple fix! | |
| 15:43:41 | toker_ | Thank you | |
| 15:43:57 | toker_ | took you ten minutes to find, I've never would have found it :) | |
| 15:45:58 | toker_ | AttributeError: 'Proxy' object has no attribute 'listeners' <- stumbled on this now when trying to list listeners ... | |
| 15:46:09 | toker_ | But thats fine, dont really matter. | |
| 15:48:28 | mordred | woot! | |
| 15:49:01 | toker_ | https://github.com/openstack/python-openstacksdk/commit/d1b242ee850f286f1fae56021f2edf13cde1eb9c <- maybe I need this | |
| 15:49:16 | toker_ | one guy said that in the lbaas-channel | |
| 15:51:47 | toker_ | hm dont think thats related -. | |
| 16:01:45 | 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:15:00 | openstackgerrit | Doug Hellmann proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | |
| 16:25:12 | dhellmann | mordred : I updated your update to also change the test-requirements.txt file ^^ | |
| 17:04:11 | elmiko | edleafe dtantsur , ready for review https://etherpad.openstack.org/p/api-sig-newsletter | |
| 17:07:08 | dtantsur | elmiko: one comment there, everything else LGTM | |
| 17:07:13 | elmiko | cool | |
| 17:07:15 | dtantsur | maybe I just misunderstood the phrase though | |
| 17:07:31 | elmiko | no no, good spot | |
| 17:07:55 | elmiko | i missed a word lol | |
| 17:12:21 | edleafe | elmiko: LGTM | |
| 17:13:40 | elmiko | thanks! | |
| 17:18:11 | mordred | dhellmann: sweet | |
| 18:21:52 | openstackgerrit | Doug Hellmann proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | |
| 18:25:36 | openstackgerrit | Doug Hellmann proposed openstack/cliff master: add lower-constraints job https://review.openstack.org/555405 | |
| 19:13:48 | 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:14:01 | jroll | or experience running a cloud like that, and could point to something I'm missing | |
| 19:14:25 | 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:19:52 | cdent | jroll: is the service catalog returning reasonable entries? | |
| 19:20:18 | cdent | also you might be interested in https://review.openstack.org/#/c/459869/ | |
| 19:22:11 | mordred | jroll: shade should totally work with suburls.. | |
| 19:22:24 | mordred | jroll: as long as the service-catalog and version discovery documents are correct | |
| 19:22:59 | mordred | jroll: I'm not sure how to configure the services to return the correct thing in their version discovery docs... | |
| 19:23:18 | 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:31 | jroll | mordred: right, that last part is the catch | |
| 19:23:38 | jroll | oh, devstack does that now? I can look at that | |
| 19:23:41 | mordred | yah | |
| 19:23:49 | mordred | iirc it's the default deployment now | |
| 19:24:42 | 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:25:07 | jroll | mordred: yeah, I was curious if we were doing something wrong with the service or with shade | |
| 19:25:41 | 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:26:55 | 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:27:12 | jroll | mordred: indeed :) | |
| 19:27:52 | 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:29:24 | jroll | mordred: ha, I'll pass for now. I know I'm missing something obvious, I just don't know what | |
| 19:41:40 | openstackgerrit | Bailey Johnson proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537 | |
| 19:44:31 | 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:39 | jroll | so probably an issue with our apache config | |
| 19:55:08 | openstackgerrit | Bailey Johnson proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537 | |
| #openstack-sdks - 2018-03-23 | |||
| 00:06:50 | openstackgerrit | melissaml proposed openstack/cliff master: fix typos in documentation https://review.openstack.org/555507 | |
| 00:47:51 | openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/555551 | |
| 01:02:43 | openstackgerrit | OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/555558 | |
| 01:37:30 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/555579 | |
| 01:38:50 | openstackgerrit | OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/551601 | |
| 03:19:11 | openstackgerrit | Merged openstack/cliff master: fix typos in documentation https://review.openstack.org/555507 | |
| 04:03:49 | openstackgerrit | Doug Hellmann proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625 | |
| 06:06:30 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix api request miss microversions https://review.openstack.org/555231 | |
| 07:13:14 | openstackgerrit | chenyb4 proposed openstack/python-openstackclient master: fix doc title format error https://review.openstack.org/555666 | |
| 07:45:30 | openstackgerrit | OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902 | |
| 07:51:32 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/553257 | |
| 09:22:55 | openstackgerrit | takahara.kengo proposed openstack/python-openstacksdk master: Add instance_ha service https://review.openstack.org/555710 | |
| 09:24:10 | openstackgerrit | takahara.kengo proposed openstack/python-openstacksdk master: Add instance_ha service https://review.openstack.org/555710 | |
| 09:36:45 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Fix api request miss microversions https://review.openstack.org/555231 | |
| 12:33:33 | openstackgerrit | Doug Hellmann proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625 | |
| 12:42:19 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 13:07:47 | mnaser | do we need a new release of python-openstacksdk with https://review.openstack.org/#/c/554615/ to fix the osc-functional-devstack job? | |
| 13:08:00 | mnaser | -tips job seems to be fixed | |
| 13:09:33 | mnaser | mordred: ^ | |
| 13:16:36 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename python-openstacksdk to openstacksdk in zuul.yaml https://review.openstack.org/555765 | |
| 13:16:37 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update python-openstacksdk references to openstacksdk https://review.openstack.org/555766 | |
| 13:42:06 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: fix doc title format error https://review.openstack.org/555778 | |
| 13:44:21 | chenyb4 | Hi, mordred | |
| 13:45:13 | chenyb4 | mordred, Can you help me review this ptach. https://review.openstack.org/#/c/555231/ | |
| 13:45:38 | chenyb4 | thanks | |
| 13:46:23 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Rename python-openstacksdk to openstacksdk https://review.openstack.org/555779 | |
| 13:57:51 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Rename python-openstascksdk to openstacksdk https://review.openstack.org/555792 | |