Error code not added

[ialidzhikov]

Error code not added

[ialidzhikov]

I think we should make changes to make possible from the ConfigValidator to return provider related error such as ERR_INFRA_UNAUTHENTICATED and ERR_INFRA_UNAUTHORIZED. The reason to introduce the ConfigValidator interface was to perform prerequisite checks before the Infrastructure reconciliation - for example make sure the VPC exists, and has the needed fields enabled; make sure the Network exists, etc. Usually the ConfigValidator create a client, get the cloud provider resource and then check it. So, this get operation is the first operation that is executed with the Shoot's cloud provider credentials and that can fail due to invalid credentials. To improve the ops productivity we should rather think about how to make possible the error categorization in that case.

[ialidzhikov]

/cc @acumino

[kon-angelo]

@ialidzhikov I think the error for this is coming from ConfigValidator so this is probably not easily fixed in the extension if I am not mistaken

[acumino]

/assign

[acumino]

The error from configValidator is supposed to be considered only for configuration error and that is already being checked here.
We can consider other error codes also for configValidator error by putting error code check here. But this is not being done in any provider extension. So I would propose to close this issues.

[acumino]

/close
Fixed by #745