-
Notifications
You must be signed in to change notification settings - Fork 12
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
use a tag in the bats image #27
Conversation
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.
thanks for the PR
can you update the image tag to use the digest as suggested?
also need to bump the chart version and sign the DCO
@@ -12,7 +12,7 @@ metadata: | |||
spec: | |||
containers: | |||
- name: {{ .Release.Name }}-test | |||
image: "bats/bats" | |||
image: "bats/bats:v1.10.0" |
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.
image: "bats/bats:v1.10.0" | |
image: "bats/bats:v1.10.0@sha256:33d5909905442e39afdb29af693516f213ace6b8100045d408475d5bd15196cb" |
f614d81
to
50d2200
Compare
@cpanato Fixed :) |
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.
thx
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
can you plese do the sign-off and squash the commits? then we can merge
5b8cc3d
to
f34e7a5
Compare
@cpanato done :) |
you missed to sign-off the commit, (DCO Check) |
Signed-off-by: Amit Sagtani <[email protected]>
f34e7a5
to
29daa87
Compare
ahhh.. I missed that.. Fixed now 👍 |
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.
cool
thanks!
When trying to bundle and deploy inbucket in a complete offline env, it returns because of not using a specific tag in the bats image.
Using a latest stable release tag, instead of latest by default.