Skip to content

refactor(dockerfile): builder multistage pattern & random uid user support #35

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
Jul 31, 2018

Conversation

dlouzan
Copy link
Collaborator

@dlouzan dlouzan commented Jul 28, 2018

  • build image based on current upstream verdaccio refactor, only
    building the plugin code and then symlinking the plugin code
  • randomized uid support (e.g. openshift)
  • plugin install directory is read-only

@dlouzan dlouzan mentioned this pull request Jul 28, 2018
@dlouzan
Copy link
Collaborator Author

dlouzan commented Jul 28, 2018

Please note: this build depends on the work being done in verdaccio under verdaccio/verdaccio#864, in order to build this, it's needed to create a manual build of that verdaccio's PR and manually tag it:

# In verdaccio PR #864
cd ..
yarn build:docker
docker tag verdaccio:latest verdaccio/verdaccio:3.3

# Now build this image
cd ...
yarn build:docker
# available as verdaccio-gitlab:latest

@dlouzan dlouzan force-pushed the refactor/docker-multistage-build branch 2 times, most recently from 19a02fc to efea6da Compare July 30, 2018 08:54
@dlouzan
Copy link
Collaborator Author

dlouzan commented Jul 30, 2018

I'm changing this to make it depend on verdaccio/verdaccio:4.x-next. But until the changes in the pending PR in verdaccio for refactored dockerfile are applied, this won't work unless we build manually locally (see comment above)

@juanpicado
Copy link
Collaborator

It's out

docker pull verdaccio/verdaccio:4.x-next

@bufferoverflow
Copy link
Owner

This is great!

Please seperate the fixes for gitlab.js and package.json into a dedicated PR as they are not related to the docker build.

- build image based on current upstream verdaccio refactor, only
  building the plugin code and then symlinking the plugin code
- randomized uid support (e.g. openshift)
- plugin install directory is read-only
@dlouzan dlouzan force-pushed the refactor/docker-multistage-build branch from efea6da to 6edd701 Compare July 31, 2018 15:05
@bufferoverflow bufferoverflow merged commit 2b45a0b into master Jul 31, 2018
@dlouzan dlouzan deleted the refactor/docker-multistage-build branch August 2, 2018 16:13
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