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

chore: add non-dev docker-compose workflow #13143

Merged
merged 1 commit into from
Feb 18, 2021

Conversation

nytai
Copy link
Member

@nytai nytai commented Feb 16, 2021

SUMMARY

After some back and forth on #9880, I think it makes sense to provide a docker-compose workflow for non-dev use cases. Waiting for assets to build is a common pain point for users who just want to try out superset and I image we're losing quite a few adopters along the way.

With this new workflow, I was able to get superset up an running in < 1 minute.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TEST PLAN

  • run docker-compose -f docker-compose-non-dev.yml up in the base of repo
  • after a few seconds, superset is up and running at localhost:8088

ADDITIONAL INFORMATION

@nytai nytai requested review from craig-rueda and srinify February 16, 2021 00:27
@nytai nytai marked this pull request as ready for review February 16, 2021 00:32
Copy link
Member

@dpgaspar dpgaspar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I wonder if actually replacing the docker-compose.yaml by this, would better meet the user atraction/adoption goal here?

I can imagine new user's always starting the project by docker-compose up

Also beat on the new features almost always triggers tasks that need a webdriver and browser on the worker. So what's the real user gain here?

db:
env_file: docker/.env
image: postgres:10
container_name: superset_db
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you give these a different name from the dev compose? there's some automation around tests that use these names

docker/.env-non-dev Show resolved Hide resolved
docker/.env-non-dev Show resolved Hide resolved
SUPERSET_ENV=production
SUPERSET_LOAD_EXAMPLES=yes
CYPRESS_CONFIG=false
SUPERSET_PORT=8088
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better change it to something different from the dev env?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think there is a use case for running both dev and non-dev at the same time.

Copy link
Member

@dpgaspar dpgaspar Feb 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no use case, but I can imagine being used on the wild. It would also make it possible to run both by just a simple change.
But totally optional, feel free to ignore

@nytai
Copy link
Member Author

nytai commented Feb 16, 2021

@dpgaspar I would be in favor of making non-dev the default workflow, however I'm hesitant about breaking existing users workflows and causing some surprises. Also, I suspect that users would want to migrate to the dev workflow sooner or later (in order to update configs, and test out code changes). The main idea behind this is to provide a way for users to quickly test out superset without having to wait ~30mins for the assets to compile.

@nytai nytai merged commit aa50161 into apache:master Feb 18, 2021
@nytai nytai deleted the tai/docker-compose-non-dev branch February 18, 2021 17:55
@garden-of-delete
Copy link
Contributor

Awesome! Thanks Tai!

@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels preset-io size/L 🚢 1.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installing Superset via Docker-compose Up
4 participants