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

Updated values.local.yaml to include default mongodb passwords #68

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

mhaswell-bcgov
Copy link

All pods now come up without error. App pods can connect to mongodb.
The app route is not accessible locally yet, will need another ticket.

Added EXPERIMENTAL_DOCKER_DESKTOP_FORCE_QEMU env variable to mongodb values which allows an image built for amd64 arch to work on arm64
Updated readme to document how to deploy to Kubernetes through Docker Desktop
Copy link

@robrien-bcgov robrien-bcgov left a comment

Choose a reason for hiding this comment

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

helm isn't familiar to me, but this looks pretty much like a docker compose or k8s yaml file.

The readme seems to be saying to use helm like Kustomize, so that makes sense.

I wonder if helm can do all the same things as Kustomize

replicaSetKey: 'replicasetpassword'
password: 'password'
extraEnvVars:
- name: EXPERIMENTAL_DOCKER_DESKTOP_FORCE_QEMU

Choose a reason for hiding this comment

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

is this what made the pods come up without errors?

Copy link
Author

Choose a reason for hiding this comment

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

That's right. The mongodb pods were failing to start up because mongodb doesn't release images for arm64 architecture. This env variable seems to make it so Docker Desktop uses the QEMU emulator instead of Rosetta which for whatever reason makes it possible to run the image built for amd64 on arm64. I got this information from here: docker/for-mac#7137

@mhaswell-bcgov mhaswell-bcgov merged commit ac72f08 into v5 Nov 14, 2024
3 checks passed
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