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

validate eks files #3984

Merged
merged 2 commits into from
Oct 29, 2024
Merged

validate eks files #3984

merged 2 commits into from
Oct 29, 2024

Conversation

kedarv
Copy link
Member

@kedarv kedarv commented Oct 29, 2024

We have automated tooling that modifies eks files, but the validator workflow is setup will cause those automated changes to error if there are modified files that are not able to be validated at all (https://github.com/Yelp/paasta/blob/kedar-validate-eks/paasta_tools/config_utils.py#L253-L266)

I think this should just work given the eks schema exists here: https://github.com/Yelp/paasta/blob/334e743605aeb0d7fac01c34dd38d74013bb156d/paasta_tools/cli/schemas/eks_schema.json

@kedarv kedarv changed the title Kedar validate eks validate eks files Oct 29, 2024
@kedarv kedarv marked this pull request as ready for review October 29, 2024 17:14
@kedarv kedarv requested a review from a team as a code owner October 29, 2024 17:14
@kedarv kedarv requested a review from nemacysts October 29, 2024 17:19
@nemacysts nemacysts merged commit 5d78470 into master Oct 29, 2024
10 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.

2 participants