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

KafkaRebalance resource should be checked for deprecated and unsupported fields #3637

Closed
scholzj opened this issue Sep 13, 2020 · 0 comments · Fixed by #4291
Closed

KafkaRebalance resource should be checked for deprecated and unsupported fields #3637

scholzj opened this issue Sep 13, 2020 · 0 comments · Fixed by #4291

Comments

@scholzj
Copy link
Member

scholzj commented Sep 13, 2020

Most of our resources which follow the main reconciliation cycle (Kafka, KafkaConnect, ...) are using the ValidationVisitor to check the resource and report any fields which are deprecated or which are on wrong / unsupported paths as warning into the log (and after #3623 also in resource status). However, this does not seem to apply to the KafkaRebalance which has its own cycle. The KafkaRebalanceAssemblyOperator should be updated to do this as well. While it currently doesn't have any deprecated fields, it can still have users have unsupported fields in the CR etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants