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

Clean up Dev hub's dashboard image and install mystmd,voila and voici packages #5851

Merged
merged 3 commits into from
Jul 15, 2024

Conversation

balajialg
Copy link
Contributor

@balajialg balajialg commented Jul 10, 2024

Ref:
https://anaconda.org/conda-forge/voila
https://anaconda.org/conda-forge/voila
https://anaconda.org/conda-forge/mystmd

Image was building for a long time locally (tried multiple iterations locally - it kept building. So, wanted to check whether it built via CI)

Okay, debugging locally

Commented out unnecessary packages for the dashboard image and added above packages which are needed to build dashboard. When I tested this image locally, it built successfully.

Screenshot 2024-07-10 at 4 54 16 PM

@balajialg balajialg changed the title Add voila and voici package to the Dev hub's dashboard image Clean up Dev hub's dashboard image and install mystmd ,voila and voici packages Jul 10, 2024
@balajialg balajialg changed the title Clean up Dev hub's dashboard image and install mystmd ,voila and voici packages Clean up Dev hub's dashboard image and install mystmd,voila and voici packages Jul 10, 2024
Copy link
Collaborator

@ryanlovett ryanlovett left a comment

Choose a reason for hiding this comment

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

Thanks for adding the dashboard packages! I've left some notes. The most important one is the jupyterhub version.

deployments/dev/images/secondary/environment.yml Outdated Show resolved Hide resolved
deployments/dev/images/secondary/environment.yml Outdated Show resolved Hide resolved
deployments/dev/images/secondary/environment.yml Outdated Show resolved Hide resolved
@balajialg
Copy link
Contributor Author

@ryanlovett Thanks for the detailed feedback. I just created a PR addressing all your comments. The image built successfully in my local docker setup.

@balajialg
Copy link
Contributor Author

I am merging the changes to staging inorder to test it

@balajialg balajialg merged commit 2aac624 into berkeley-dsep-infra:staging Jul 15, 2024
22 checks passed
@balajialg balajialg deleted the voila branch July 15, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants