-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add topicId to KafkaTopic status #9708
Conversation
This should close strimzi#9651. Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks.
/azp run regression |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One comment, but otherwise this LGTM. Thanks @fvaleri.
@@ -37,4 +38,13 @@ public String getTopicName() { | |||
public void setTopicName(String topicName) { | |||
this.topicName = topicName; | |||
} | |||
|
|||
@Description("Topic id") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're exposing this to client applications. We should try to define what it means so such applications can use it correctly. This is a bit tricky because we have the possibility of conflicting KafkaTopics. I think this is OK:
@Description("Topic id") | |
@Description("The topic's id. For a KafkaTopic with the ready condition, this will change only if the topic gets deleted and recreated with the same name.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
/azp run regression |
Azure Pipelines successfully started running 1 pipeline(s). |
This should close #9651.