You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently KubernetesApiServiceDiscovery.readConfigVarFromFilesystem method returns None in case of an error reading from a file. This causes complications down the line when trying to identify what is going on. For example, I had an issue where reading the file threw permissions errors. Discovery process proceeds and simply fails to make k8s API requests because it no longer provides Bearer tokens. Error is still logged, but it gets quickly obscured by all the error logs about permissions for anonymous user account, so the root cause gets effectively hidden, making this a gotcha that each developer has to learn. I think this kind of exception should be treated more severely, maybe even by terminating the application, because this is basically a fatal error. Would implementing that make sense?
Versions used
Akka version: 2.6.13
Description
Currently
KubernetesApiServiceDiscovery.readConfigVarFromFilesystem
method returnsNone
in case of an error reading from a file. This causes complications down the line when trying to identify what is going on. For example, I had an issue where reading the file threw permissions errors. Discovery process proceeds and simply fails to make k8s API requests because it no longer provides Bearer tokens. Error is still logged, but it gets quickly obscured by all the error logs about permissions for anonymous user account, so the root cause gets effectively hidden, making this a gotcha that each developer has to learn. I think this kind of exception should be treated more severely, maybe even by terminating the application, because this is basically a fatal error. Would implementing that make sense?akka-management/discovery-kubernetes-api/src/main/scala/akka/discovery/kubernetes/KubernetesApiServiceDiscovery.scala
Lines 211 to 225 in 178497c
The text was updated successfully, but these errors were encountered: