Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default port is not changing after setting SUPERSET_WEBSERVER_PORT #23825

Open
3 tasks done
abhi1045 opened this issue Apr 26, 2023 · 6 comments
Open
3 tasks done

Default port is not changing after setting SUPERSET_WEBSERVER_PORT #23825

abhi1045 opened this issue Apr 26, 2023 · 6 comments
Labels
#bug Bug report

Comments

@abhi1045
Copy link

abhi1045 commented Apr 26, 2023

I'm trying to build a docker image of superset on a non default port.
I follow the steps mentioned in at this link and also this link.
But the issue is I'm changing the default port to 6011 but it is not changing.

How to reproduce the bug

  1. Go to 'superset_config.py'
    Add below code snippet
SUPERSET_WEBSERVER_PROTOCOL = "http"
SUPERSET_WEBSERVER_ADDRESS = "0.0.0.0"
SUPERSET_WEBSERVER_PORT = 6011

Screenshot 2023-04-26 at 1 51 09 PM

Expected results

When I'm building the docker using following commands

docker build -t superset .

docker run -p 8088:6011 superset

The port "6011" must have listening.

Actual results

Screenshot 2023-04-26 at 1 39 34 PM

It is still running on 8088 port, but should run on 6011 port.

Environment

  • browser type and version:
  • superset version: 2.1.0
  • python version: 3.9.13
  • node.js version: 16.17.0

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • I have checked the issue tracker for the same issue and I haven't found one similar.
@abhi1045 abhi1045 added the #bug Bug report label Apr 26, 2023
@sfirke
Copy link
Member

sfirke commented Apr 27, 2023

Hello, I believe that variable is no longer used - see this pending PR to update the docs: #22849 You might want WEBDRIVER_BASEURL instead.

@abhi1045
Copy link
Author

abhi1045 commented Apr 28, 2023

I have tried set WEBDRIVER_BASEURL in 'superset_config.py'

WEBDRIVER_BASEURL = "http://0.0.0.0:6011"

After that run those steps

docker build -t superset .

docker run -p 5000:6011 superset

Expected results

When, I hit "http://0.0.0.0:5000", the superset application runs.

So, the default port must be changed to port 6011 and I'm redirecting to port 5000, but this is not working.

Actual results

I'm not getting the application running at port 5000.

This is still Listening at: http://0.0.0.0:8088 and I'm not able to access it on either port

@sebastianliebscher
Copy link
Contributor

sebastianliebscher commented May 2, 2023

I can confirm this issue: Changing WEBDRIVER_BASEURL or SUPERSET_WEBSERVER_PORT does not change the port the superset webserver is listening at. Tested for Superset 2.1.0 and latest using docker-compose-non-dev.yml.

The only way I know to change the port is via environment variable SUPERSET_PORT.
docker-compose:

    environment:
      SUPERSET_PORT: 6011

docker run:

-e SUPERSET_PORT=6011

@rusackas
Copy link
Member

Is this still an issue in 3.x?

@sfirke
Copy link
Member

sfirke commented Mar 1, 2024

It seems like SUPERSET_PORT is the fix here. I think we're close to closing this. The only thing is how we want to document that. Normally I think of config.py as the default documentation of every Superset config variable. However this variable is not in config.py. Would it get picked up from config.py if we added SUPERSET_PORT=8088 which is present in /docker/.env-non-dev ? Then it might be more apparent to users how to change this. What do you think @sebastianliebscher ?

Someone mentioned this variable today in Slack as being key to their k8s deployment.

@rusackas
Copy link
Member

rusackas commented Sep 6, 2024

Anyone know the current state of affairs here, or what it'd take to close this?

Maybe @dosu-bot knows?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report
Projects
None yet
Development

No branches or pull requests

4 participants