Skip to content

Comments

Add CRDHook annotations#1914

Merged
Slach merged 1 commit intoAltinity:0.26.0from
eyyu:crdhook-annotations
Feb 2, 2026
Merged

Add CRDHook annotations#1914
Slach merged 1 commit intoAltinity:0.26.0from
eyyu:crdhook-annotations

Conversation

@eyyu
Copy link
Contributor

@eyyu eyyu commented Jan 30, 2026

Add custom annotations to CRD Hook Resources

Thanks for taking the time to contribute to clickhouse-operator!

Please, read carefully instructions on how to make a Pull Request.

This will help a lot for maintainers to adopt your Pull Request.

Important items to consider before making a Pull Request

Please check items PR complies to:

  • [ ✔️ ] All commits in the PR are squashed. More info
  • [ ✔️ ] The PR is made into dedicated next-release branch, not into master branch1. More info
  • [ ✔️ ] The PR is signed. More info

Add custom annotations to CRD Hook Resources

Signed-off-by: Eva Yu <eva.yw.yu@gmail.com>
Copy link
Collaborator

@Slach Slach left a comment

Choose a reason for hiding this comment

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

looks good enough for me

@Slach
Copy link
Collaborator

Slach commented Jan 30, 2026

@eyyu did you test it locally? if yes, let's merge

@eyyu eyyu marked this pull request as ready for review January 30, 2026 22:39
@eyyu
Copy link
Contributor Author

eyyu commented Jan 30, 2026

@Slach Yep, tested locally and confirmed no issues found deploying with crdHook.enabled. Additional annotations are added for resources

@Slach Slach merged commit 420b97c into Altinity:0.26.0 Feb 2, 2026
2 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