[GitHub] varaga opened a new issue #5383: SuperSet-Druid Integration fails with LB Endpoint

[GitBox <gi...@apache.org>]

varaga opened a new issue #5383: SuperSet-Druid Integration fails with LB Endpoint
URL: https://github.com/apache/incubator-superset/issues/5383
 
 
   Make sure these boxes are checked before submitting your issue - thank you!
   
   - [x] I have checked the superset logs for python stacktraces and included it here as text if any
   - [x] I have reproduced the issue with at least the latest released version of superset
   - [x] I have checked the issue tracker for the same issue and I haven't found one similar
   
   I have deployed SuperSet in AWS. 
   Basically the LB is created without a port mapping.  Therefore the hostnames are proxied to the druid cluster nodes.
   Providing this hostname in the `host` and having the `port` null for both the coordinator & the broker fails.
   I'd expect SuperSet to have handled this instead of raising an exception as below?
   
   ### Superset version
   0.24.0
   
   ### Expected results
   SuperSet connects with Druid (Load Balancer) end points successfully
   
   ### Actual results
   fails with an InvalidURL. 
   
   `2018-07-12 16:15:59,790:ERROR:root:
   Failed to parse: coordinator-load-balancer:None 
   Traceback (most recent call last): 
   File "/usr/local/lib/python2.7/dist-packages/superset/connectors/druid/views.py", line 313, in refresh_datasources cluster.refresh_datasources(refreshAll=refreshAll)
    File "/usr/local/lib/python2.7/dist-packages/superset/connectors/druid/models.py", line 146, in refresh_datasources self.druid_version = self.get_druid_version() 
   File "/usr/local/lib/python2.7/dist-packages/superset/connectors/druid/models.py", line 136, in get_druid_version return json.loads(requests.get(endpoint).text)['version'] 
   File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 72, in get return request('get', url, params=params, **kwargs) 
   File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 58, in request return session.request(method=method, url=url, **kwargs) 
   File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 494, in request prep = self.prepare_request(req) File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 437, in prepare_request hooks=merge_hooks(request.hooks, self.hooks), 
   File "/usr/local/lib/python2.7/dist-packages/requests/models.py", line 305, in prepare self.prepare_url(url, params) 
   File "/usr/local/lib/python2.7/dist-packages/requests/models.py", line 373, in prepare_url raise InvalidURL(*e.args) 
   InvalidURL: Failed to parse: `coordinator-load-balancer:None``
   
   ### Steps to reproduce
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org