-
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
Annotate StrimziPodSets before rolling during CA renewal #10711
Conversation
Closes strimzi#10710 Signed-off-by: Katherine Stanley <[email protected]>
cluster-operator/src/main/java/io/strimzi/operator/cluster/model/WorkloadUtils.java
Outdated
Show resolved
Hide resolved
cluster-operator/src/main/java/io/strimzi/operator/cluster/model/WorkloadUtils.java
Outdated
Show resolved
Hide resolved
cluster-operator/src/main/java/io/strimzi/operator/cluster/operator/assembly/CaReconciler.java
Outdated
Show resolved
Hide resolved
cluster-operator/src/main/java/io/strimzi/operator/cluster/operator/assembly/CaReconciler.java
Outdated
Show resolved
Hide resolved
...r-operator/src/test/java/io/strimzi/operator/cluster/operator/assembly/CaReconcilerTest.java
Outdated
Show resolved
Hide resolved
/azp run regression |
Azure Pipelines successfully started running 1 pipeline(s). |
cluster-operator/src/main/java/io/strimzi/operator/cluster/operator/assembly/CaReconciler.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Katherine Stanley <[email protected]>
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 nit, LGTM otherwise.
.flatMap(podSet -> ReconcilerUtils.nodesFromPodSet(podSet).stream()) | ||
.collect(Collectors.toSet())); | ||
} else { | ||
return Future.succeededFuture(new LinkedHashSet<>()); |
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.
This could be just Set.of()
, or? Or is something added to it later? I don't think so.
/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.
LGTM
Signed-off-by: Katherine Stanley <[email protected]>
@katheris there seems to be checkstyle errors to be fixed. |
Signed-off-by: Katherine Stanley <[email protected]>
/azp run zookeeper-regression |
Azure Pipelines successfully started running 1 pipeline(s). |
Thanks for the PR. |
)" This reverts commit dcaa5e4.
Type of change
Select the type of your PR
Description
Update the CaReconciler to annotate the StrimziPodSets with the new CA cert generations before it rolls the Kafka nodes.
Checklist
Please go through this checklist and make sure all applicable tasks have been done