Skip to content

Fix the Docker setup #2622

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

Merged
merged 1 commit into from
Nov 4, 2022
Merged

Fix the Docker setup #2622

merged 1 commit into from
Nov 4, 2022

Conversation

julienrf
Copy link
Contributor

@julienrf julienrf commented Nov 3, 2022

I found that our previous Docker setup was not good enough and was failing on some machines.

I propose the following changes, based on what has been done on the scala.epfl.ch website. The main “fix” is that we indicate a specific version of Ruby, rouge, and bundler. We also propose a different way to invoke docker-compose up that writes the generated files as a regular user (instead of root).

I found that our previous Docker setup was not good enough and was failing on some machines.

I propose the following changes, based on what has been done on the scala.epfl.ch website. The main “fix” is that we indicate a specific version of Ruby, rouge, and bundler. We also propose a different way to invoke docker-compose up that writes the generated files as a regular user (instead of root).
@julienrf julienrf requested review from SethTisue and fsalvi November 3, 2022 14:41
Copy link
Member

@SethTisue SethTisue left a comment

Choose a reason for hiding this comment

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

do you see "Docker guy" on my LinkedIn profile?!?!?!?! :-P

before this PR, on my Mac I got:

#0 103.4 ERROR:  Error installing jekyll:
#0 103.4 	The last version of rouge (>= 3.0, < 5.0) to support your Ruby & RubyGems was 3.30.0. Try installing it with `gem install rouge -v 3.30.0` and then running the current command again
#0 103.4 	rouge requires Ruby version >= 2.7. The current ruby version is 2.6.10.210.

and after this PR it worked

so I'd say that's an "Approve", even though I have no idea what the changes are about

as I also said at #1286,

how do I view the site? (visiting http://0.0.0.0:4000/ doesn't work

I guess because the site is running inside the container and my web browser is outside of the container? So this is "working" in the sense that site seems to build. But without understanding how to actually view the site, the Docker stuff still isn't actually helping me.

@julienrf
Copy link
Contributor Author

julienrf commented Nov 4, 2022

I guess you don't see the website because of VirtualBox.

@SethTisue
Copy link
Member

SethTisue commented Nov 4, 2022

(As an aside, I did switch from using VirtualBox to using the Docker desktop app.)

It appears from e.g. https://www.mend.io/free-developer-tools/blog/docker-expose-port/ that we should be able to add something to the Dockerfile that exposes port 4000. I might mess with that if I get a chance. (I joke about not wanting to mess with Docker, but it’s also been frustrating messing with rbenv and bundler and such over the years, so I actually am interested in improving the Docker option.)

@fsalvi
Copy link
Contributor

fsalvi commented Nov 4, 2022

Port 4000 is already exposed, it works fine under linux. I guess it's a bit different on mac...

Copy link
Contributor

@fsalvi fsalvi left a comment

Choose a reason for hiding this comment

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

Works fine under linux

@fsalvi fsalvi merged commit fd26923 into scala:main Nov 4, 2022
@julienrf julienrf deleted the fix-docker-setup branch May 15, 2023 07:09
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

Successfully merging this pull request may close these issues.

3 participants