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

Remove X server requirement on CI and Deepnote #191

Open
RussTedrake opened this issue Sep 9, 2022 · 4 comments
Open

Remove X server requirement on CI and Deepnote #191

RussTedrake opened this issue Sep 9, 2022 · 4 comments
Assignees

Comments

@RussTedrake
Copy link
Owner

RussTedrake commented Sep 9, 2022

so that it doesn't need to get installed on the first call to StartMeshcat.

@RussTedrake
Copy link
Owner Author

done. I now call RUN /opt/drake/share/drake/setup/deepnote/install_nginx in the Dockerfile, but of course the service is not started each time the dockerfile is provisioned on Deepnote. The result is that the first call to StartMeshcat still says "installing nginx", but it at least returns very fast.

I tried adding RUN systemctl enable nginx to the dockerfile, but learned the docker does use sysctl / system.d. Adding CMD service nginx start didn't seem to get the job done, either.

cc @jwnimmer-tri, in case you have any better ideas. But I'm at least happy that it's faster.

@jwnimmer-tri
Copy link
Contributor

This all sounds fine.

FYI what we do for Drake tutorials is shown here:

https://drake.mit.edu/release_playbook.html#post-release-tutorials-updates

The only slow part is the apt, which we bake into the Docker image like so:

RUN apt-get -q update && apt-get -q install -y --no-install-recommends nginx-light xvfb && apt-get -q clean

We use an init.ipynb to do the install_xvfb. I realize now that it could probably do the install_nginx as well, which would maybe remove the printout from the actual notebooks for you?

@RussTedrake
Copy link
Owner Author

Thanks. I thought about init.ipynb for this. I'll go ahead with that.

@RussTedrake RussTedrake reopened this Sep 14, 2022
@RussTedrake RussTedrake changed the title install nginx in deepnote docker file Remove X server requirement on CI and Deepnote Dec 28, 2024
@RussTedrake
Copy link
Owner Author

I will finish this once RobotLocomotion/drake#22085 happens.

@RussTedrake RussTedrake self-assigned this Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants