Open mariomac opened 10 months ago
This is a great idea!
@mariomac I am not sure if this would work, particularly in Daemonset mode. There is a chance that, based on the discovery setting, there is not at least one valid executable on every single node the Daemonset is deployed on, meaning the probes will fail on the nodes without a valid executable.
Therefore, I think 'there is at least 1 instrumented executable' is not the best condition for the probes to succeed.
Maybe a better way would be to check the status of the HTTP server serving the metrics?
@marevers yeah, that's a good point. Probably now that Beyla is able to instrument multiple executables, we need to change this approach.
Some K8s deployments might require mandatory liveness and readiness probes. We could, only if a user activates it, provide and endpoint that would tell when beyla is live and is ready (e.g. when there is an instrumented executable).