Skip to content

fix: validate-values script #560

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

Merged
merged 4 commits into from
Jul 1, 2025
Merged

fix: validate-values script #560

merged 4 commits into from
Jul 1, 2025

Conversation

mikhail-klimko
Copy link
Contributor

@mikhail-klimko mikhail-klimko commented Jun 30, 2025

What

Fix validate-values script to store kubectl get svc output in a file.

Why

kubectl.kubernetes.io/last-applied-configuration might contain unescaped new lines causing jq to fail when service_info saved to an env var
image

echo $service_info | jq -r '.items'
jq: parse error: Invalid string: control characters from U+0000 through U+001F must be escaped at line 10, column 1

Notes

@mikhail-klimko
Copy link
Contributor Author

/e2e

1 similar comment
@mikhail-klimko
Copy link
Contributor Author

/e2e

@mikhail-klimko mikhail-klimko marked this pull request as ready for review June 30, 2025 16:49
@mikhail-klimko mikhail-klimko requested a review from a team as a code owner June 30, 2025 16:49
@mikhail-klimko mikhail-klimko requested a review from ATGardner June 30, 2025 16:50
@mikhail-klimko mikhail-klimko merged commit 03e085c into main Jul 1, 2025
3 checks passed
@mikhail-klimko mikhail-klimko deleted the fix/validate-values branch July 1, 2025 07:22
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