Skip to content

Updated Dockerfile to match official image #9

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

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

rnsc
Copy link

@rnsc rnsc commented Aug 21, 2023

Hello,

Not sure if you're interested, but I wanted to replace the container image I had with 4.4 to a 5.x, but the Dockerfile from your repository was diverging a bit too far from the one of the official Mongo image.
Thus I couldn't do a drop-in replacement.

If this is not desired, I'm ok to close this, I'll keep it in my fork.

It also updates the workflow to build and push images to Docker hub, you need to set two secrets in your repository:

DOCKER_HUB_USER
DOCKER_HUB_TOKEN

This Docker hub push could be made optional if not desirable.

@rnsc rnsc force-pushed the updated-docker-build branch from 8cd65ec to a1f1a59 Compare August 21, 2023 08:34
@GermanAizek
Copy link
Owner

@rnsc, oh thanks for contribution.

@GermanAizek
Copy link
Owner

@rnsc, I created token, created secrets for repo and he can't login in action process.

image

image

@rnsc
Copy link
Author

rnsc commented Aug 21, 2023

@GermanAizek that's weird, login works properly on my repo with the same permissions scope...

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.

2 participants