kyma-project / application-connector-manager

Apache License 2.0
0 stars 12 forks source link

Error hidden when accessing Gardener configmap `shoot-info` #302

Open tobiscr opened 4 months ago

tobiscr commented 4 months ago

Description

The ACM is not reporting the error when it tries to detect the domain of the Gardener cluster.

This make debugging unnecessary difficult (e.g. in case of permission errors). We should, beside the error message of not being able to read the domain-name, also report the error message we received from K8s.

See https://github.com/kyma-project/application-connector-manager/blob/b910f97b1084195d13e465a3090b938c64d57597/pkg/reconciler/detect_domain.go#L30

AC:

Expected result

ACM is showing an error message which includes the root-casue if it's failing to access the shoot-info Configmap.

Actual result

Just the error unable to detect domain is reported, but not the root-cause why it failed to read the Gardener config map.

Steps to reproduce

Simulate to use a non-Gardener cluster by renaming the shoot-info CM or use a K3d cluster. Install ACM manually afterwards and see how it complains about the missing domain.

Troubleshooting