-
Notifications
You must be signed in to change notification settings - Fork 86
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
docs: update documentation to support new version of dockerfile #183
Conversation
This commit changes the version of both argocd and ksops to newer versions and changes the paths to reflect what is configured in the new slim docker image introduced in viaduct-ai#180 ArgoCD v2.6.7 is the latest version available on the argoproj docker registry at the time of this commit.
|
||
# Copy the plugin to kustomize plugin path | ||
COPY --from=ksops-builder /go/src/github.com/viaduct-ai/kustomize-sops/* $KUSTOMIZE_PLUGIN_PATH/viaduct.ai/v1/${PKG_NAME}/ | ||
COPY --from=ksops-builder /usr/local/bin/ksops $KUSTOMIZE_PLUGIN_PATH/viaduct.ai/v1/${PKG_NAME}/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously I just had install -D ksops /custom-tools/kustomize-plugins/viaduct.ai/v1/ksops/ksops;
Should we change the working directory of the container to keep that working?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As in change the work dir of the KSOPS image to /usr/local/bin/
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@james-callahan Let me know if this PR addresses: https://github.com/viaduct-ai/kustomize-sops/pull/185/files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, all good thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM -- Thank you for fixing @therealdwright 🚀
|
||
# Copy the plugin to kustomize plugin path | ||
COPY --from=ksops-builder /go/src/github.com/viaduct-ai/kustomize-sops/* $KUSTOMIZE_PLUGIN_PATH/viaduct.ai/v1/${PKG_NAME}/ | ||
COPY --from=ksops-builder /usr/local/bin/ksops $KUSTOMIZE_PLUGIN_PATH/viaduct.ai/v1/${PKG_NAME}/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As in change the work dir of the KSOPS image to /usr/local/bin/
?
This commit changes the version of both argocd and ksops to newer
versions and changes the paths to reflect what is configured in the
new slim docker image introduced in #180
ArgoCD v2.6.7 is the latest version available on the argoproj docker
registry at the time of this commit.