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

Refactoring helm charts validation #6

Merged
merged 16 commits into from
May 18, 2021

Conversation

tei-k
Copy link
Contributor

@tei-k tei-k commented May 11, 2021

Description

https://scalar-labs.atlassian.net/browse/DLT-8981

Done

  • Fix k8s version to v1.19.7 for ci test. (Delete old v1.15.11 and v1.16.9)
  • Upgrade action version for helm/kind-action and helm/chart-testing-action.
  • Refactoring some.

Remark

kubeval just support v1.18.1

@tei-k tei-k force-pushed the refactoring-helm-charts-validation branch from 66affb3 to fc358bd Compare May 11, 2021 06:40
@tei-k tei-k force-pushed the refactoring-helm-charts-validation branch from fc358bd to cbde47f Compare May 11, 2021 06:45
@tei-k tei-k force-pushed the refactoring-helm-charts-validation branch from 6ca2b2f to c6cff3f Compare May 11, 2021 08:03
@tei-k tei-k marked this pull request as ready for review May 12, 2021 02:21
@tei-k tei-k self-assigned this May 12, 2021
ymorimo
ymorimo previously approved these changes May 12, 2021
Copy link
Contributor

@ymorimo ymorimo left a comment

Choose a reason for hiding this comment

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

LGTM!

@tei-k tei-k force-pushed the refactoring-helm-charts-validation branch from d04538f to f62d17e Compare May 14, 2021 07:23
superbrothers
superbrothers previously approved these changes May 16, 2021
feeblefakie
feeblefakie previously approved these changes May 17, 2021
Copy link
Contributor

@feeblefakie feeblefakie left a 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
Copy link
Contributor

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.

Copy link
Contributor Author

@tei-k tei-k May 17, 2021

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

@tei-k tei-k force-pushed the refactoring-helm-charts-validation branch from 00d868d to d9385c0 Compare May 17, 2021 09:45
Copy link
Contributor

@feeblefakie feeblefakie left a comment

Choose a reason for hiding this comment

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

LGMT! Thanks.

@feeblefakie
Copy link
Contributor

@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.

@feeblefakie feeblefakie merged commit 0efdab6 into main May 18, 2021
@feeblefakie feeblefakie deleted the refactoring-helm-charts-validation branch May 18, 2021 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants