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

fix: docker-compose up ModuleNotFoundError: {...} superset #31379

Closed
wants to merge 3 commits into from

Conversation

mistercrunch
Copy link
Member

As reported here our Dockerfile is currently broken. Somehow the copy command was copying the superset/superset folder instead of superset/ in some setups.

@dosubot dosubot bot added the install:docker Installation - docker container label Dec 10, 2024
@mistercrunch mistercrunch marked this pull request as draft December 10, 2024 17:31
@rusackas
Copy link
Member

I have all the clueless questions about how it broke, why it worked before, and why more people aren't running into this, etc.

This looks like it'll copy in a lot more stuff, will that lead to any bloat/slowdown?

Otherwise this looks very stampable to me when it's ready for review.

@mistercrunch
Copy link
Member Author

from my troubleshoot

  • uv has some quirks around switching users
  • docker cache works in mysterious ways, somehow my setup was working with what I assume was older docker cache that should have been validated (?)

Rolling back using uv in docker, tested it works from clean cache

@mistercrunch
Copy link
Member Author

closing in favor of #31385

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
install:docker Installation - docker container review:draft size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants