[FLINK-35278] Occasional NPE on k8s operator status replacement

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: kubernetes-operator-1.9.0
• Component/s: Kubernetes Operator
• Labels: pull-request-available
• Assignee: Ferenc Csaky
• Reporter: Márton Balassi
• Created: 30/Apr/24 14:29
• Updated: 04/May/24 09:15
• Resolved: 04/May/24 09:15

Description

Infrequently we get a null pointer exception on status replacement:

logger: io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher
message: Error during error status handling.   

throwable: { [-]
     class: java.lang.NullPointerException
     msg: null
     stack: [ [-]
       org.apache.flink.kubernetes.operator.utils.StatusRecorder.replaceStatus(StatusRecorder.java:136)
       org.apache.flink.kubernetes.operator.utils.StatusRecorder.patchAndCacheStatus(StatusRecorder.java:97)
       org.apache.flink.kubernetes.operator.reconciler.ReconciliationUtils.toErrorStatusUpdateControl(ReconciliationUtils.java:438)
       org.apache.flink.kubernetes.operator.controller.FlinkDeploymentController.updateErrorStatus(FlinkDeploymentController.java:213)
       org.apache.flink.kubernetes.operator.controller.FlinkDeploymentController.updateErrorStatus(FlinkDeploymentController.java:60)
       io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleErrorStatusHandler(ReconciliationDispatcher.java:194)
       io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleReconcile(ReconciliationDispatcher.java:123)
       io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleDispatch(ReconciliationDispatcher.java:91)
       io.javaoperatorsdk.operator.processing.event.ReconciliationDispatcher.handleExecution(ReconciliationDispatcher.java:64)
       io.javaoperatorsdk.operator.processing.event.EventProcessor$ReconcilerExecutor.run(EventProcessor.java:452)
       java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
       java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
       java.lang.Thread.run(Thread.java:829)
     ]
   }

I suspect it probably is thrown by getResourceVersion() here:

https://github.com/apache/flink-kubernetes-operator/blob/main/flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/utils/StatusRecorder.java#L136

Issue Links

• links to: GitHub Pull Request #823