Show errors in status conditions
Closed this issue · 3 comments
If the operator encounters an error when calling the Aiven API (for instance if there are invalid values in the resource being updated), those errors are logged, but nothing is reflected on the actual resource.
Since the resource is the interface for the developer, it would be expected that any errors are shown there in some way.
It could for instance be by updating the Conditions fields with proper details for the current status and last known state/message.
+1 for this feature. A common issue for us, is that devs reduces the partition count on a topic.
When the operator tries to reconcile, the request is rejected by the aiven api.
This error is only shown as an event on the KafkaTopic, and is easy to miss:
unable to create or update aiven instance: cannot update Kafka Topic: 400:
{"errors":[{"error_code":"kafka_topic_invalid_config","message":"Reducing partition count is not supported","status":400}],"message":"Reducing partition count is not supported"}
Relevant part of the guide on API conventions from SIG architecture:
https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md#typical-status-properties
It looks like this was solved in v0.16.0, so I'm closing this.