tagarr commented on code in PR #749:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/749#discussion_r1442824067
##########
flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/controller/FlinkDeploymentController.java:
##########
@@ -227,4 +235,28 @@ private boolean
validateDeployment(FlinkResourceContext<FlinkDeployment> ctx) {
}
return true;
}
+
+ private void setCRStatus(FlinkDeployment flinkApp) {
+ final List<Condition> conditions = new ArrayList<>();
+ FlinkDeploymentStatus deploymentStatus = flinkApp.getStatus();
+ switch (deploymentStatus.getJobManagerDeploymentStatus()) {
Review Comment:
I think we need a bit more logic here. We need to check if the status
doesn't already have a condition in it's list that matches the same type, we
need to then check the reason and message of this condition and only if they
are different replace the existing condition with the new one.
We shouldn't just replace the existing list of conditions with a new one,
there may be additional conditions of different types added later that would
get removed. This may be one of the reasons why the tests have had to be
modified to bump up the number of status changes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
