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

feat(api): improve crd conditions #487

Merged
merged 6 commits into from
Nov 5, 2024
Merged

Conversation

yaroslavborbat
Copy link
Member

@yaroslavborbat yaroslavborbat commented Oct 30, 2024

Description

Make all crds have the same conditions

Why do we need it, and what problem does it solve?

We have different specifications of conditions.
There is no validation of conditions in crds.

What is the expected result?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@yaroslavborbat yaroslavborbat removed the validation/skip/doc_changes Skip doc changes validation label Oct 31, 2024
@yaroslavborbat yaroslavborbat force-pushed the feat/api/crds-conditions branch from 344c9ef to 101a4b0 Compare October 31, 2024 07:01
@yaroslavborbat yaroslavborbat added the validation/skip/doc_changes Skip doc changes validation label Oct 31, 2024
Isteb4k
Isteb4k previously approved these changes Nov 1, 2024
Isteb4k
Isteb4k previously approved these changes Nov 2, 2024
Signed-off-by: yaroslavborbat <[email protected]>
Signed-off-by: yaroslavborbat <[email protected]>
Signed-off-by: yaroslavborbat <[email protected]>
Signed-off-by: yaroslavborbat <[email protected]>
Signed-off-by: yaroslavborbat <[email protected]>
Signed-off-by: yaroslavborbat <[email protected]>
@yaroslavborbat yaroslavborbat merged commit 44e2679 into main Nov 5, 2024
12 checks passed
@yaroslavborbat yaroslavborbat deleted the feat/api/crds-conditions branch November 5, 2024 09:02
danilrwx pushed a commit that referenced this pull request Nov 5, 2024
Make all crds have the same conditions
---------
Signed-off-by: yaroslavborbat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation/skip/doc_changes Skip doc changes validation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants