-
Notifications
You must be signed in to change notification settings - Fork 1
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
Refactoring helm charts validation #6
Conversation
66affb3
to
fc358bd
Compare
fc358bd
to
cbde47f
Compare
6ca2b2f
to
c6cff3f
Compare
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!
d04538f
to
f62d17e
Compare
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! Thanks.
I would like to add more versions to the test.
strategy: | ||
matrix: | ||
k8s: | ||
- v1.15.7 | ||
- v1.16.4 | ||
- v1.19.7 |
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.
I think it should be tested with multiple supported versions. But we haven't decided what versions to support. So, let's discuss and update.
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.
Updated k8s version in d9385c0, but can't find latest image.
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.
The latest image (v1.21.x) was provided 2 days ago.
00d868d
to
d9385c0
Compare
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.
LGMT! Thanks.
@ymorimo @superbrothers The approvals are dismissed due to a minor change in the versions to test, but it is mostly unchanged. So, I will merge it. Please take a look quickly when you get another chance. |
Description
https://scalar-labs.atlassian.net/browse/DLT-8981
Done
v1.19.7
for ci test. (Delete oldv1.15.11
andv1.16.9
)helm/kind-action
andhelm/chart-testing-action
.Remark
kubeval
just support v1.18.1