minio / operator

Simple Kubernetes Operator for MinIO clusters :computer:
https://min.io/docs/minio/kubernetes/upstream/index.html
GNU Affero General Public License v3.0
1.19k stars 450 forks source link

Broken error log message: "Waiting for MinIO to be ready: %!s(<nil>)" #2308

Open tmohme opened 4 weeks ago

tmohme commented 4 weeks ago

The - in principle correct - log message suffers from incorrect argument replacement.

Expected Behavior

The error log message is (correctly & expectedly) produced when the MinIO server processes are not (yet) ready. Instead of "%!s()" I expect to find some kind of description of the underlying error.

Current Behavior

The log message hides the underlying error.

Possible Solution

In pkg/controller/main-controller.go, L#1366 (v6.0.3, 45110b9d), change the message preparation so that a description of the underlying error/cause is printed in the message.

Steps to Reproduce (for bugs)

  1. Have a MinIO tenant up and running.
  2. Make some changes to the tenant that induce temporary unavailability, like adding a new pool.

Context

We are going to manage multiple clusters and strive for complete automation. Therefore we need direct, succinct information in the log messages and can't effort to manually dig up the source-code, look what it does and what might have cause the error message in question.

Regression

No

Your Environment