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

Test: Verify field immutability for all CRDs #1755

Merged
merged 9 commits into from
Nov 18, 2024

Conversation

Baarsgaard
Copy link
Contributor

@Baarsgaard Baarsgaard commented Nov 12, 2024

Decided to implement tests for immutable fields.
This also allows me to disable the slow Chainsaw Grafana Dashboard immutability test, which has been kept as snippets may be useful for other Chainsaw tests in the future.

I noticed that starting the envtest binaries takes a few seconds(~5), but running the Creates/Patches are almost free and take a few milliseconds.

@Baarsgaard Baarsgaard marked this pull request as ready for review November 12, 2024 19:57
@theSuess theSuess force-pushed the create_immutability_env_tests branch from 05e9987 to 1c74663 Compare November 18, 2024 09:18
@theSuess theSuess added this pull request to the merge queue Nov 18, 2024
Merged via the queue into grafana:master with commit 1ac8d8d Nov 18, 2024
14 checks passed
@Baarsgaard Baarsgaard deleted the create_immutability_env_tests branch November 18, 2024 11:00
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