Hello,

I noticed recently that there was a PR merged
<https://github.com/apache/flink-kubernetes-operator/pull/969> that enables
Flink BlueGreen Deployments (Phase 1
<https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=337677648>).
While testing out this change, I noticed what I think is a bug. I realize
this change has not been officially released yet, but I wanted to still
bring it up in case no-one else has found it. When translating the
FlinkBlueGreenDeployment spec into a FlinkDeployment spec, I noticed that
it replaces all instances
<https://github.com/apache/flink-kubernetes-operator/blob/main/flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/utils/bluegreen/BlueGreenUtils.java#L72>
of the app name to include a -blue or -green suffix. This means it also
gets changed in some places where it might not be appropriate. In my
testing, it's so far caused issues with configmap names, the
initialSavepointPath field, metrics, and logging. Wondering if this is the
expected behavior?

Best,
Maheep
-- 

*Maheep Myneni*

Software Engineer

Reply via email to