-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Set status Git/BucketStateStores #247
Comments
Update the issue to include publishing events, and only for State store. Similar issue for Destination #330 |
Closing this one because:
So we believe setting status in Destination will provide enough feedback to our users. Therefore picking up #330 and closing this issue. |
In #330 we introduced Destination status. I think we should still implement State Store status for the following scenario:
Ideally that should trigger a reconciliation of the state store, and its status should now be "not ready". That, in turn, is already set to trigger a reconciliation of the destination -- which would update the destination status accordingly. i suggest we reopen and implement this @ChunyiLyu @catmo-syntasso wdyt? |
Context
Users would like to have feedback after they set up their State Store.
Update Git/BucketStateStores status, logs, and publish events to indicate wether it can successfully authenticate, and give reasonable errors back.
The text was updated successfully, but these errors were encountered: