Skip to content
This repository has been archived by the owner on Nov 23, 2023. It is now read-only.

feat(controller): add connector state monitoring with controller #72

Merged
merged 13 commits into from
Apr 7, 2023

Conversation

heiruwu
Copy link
Member

@heiruwu heiruwu commented Mar 29, 2023

Because

  • support connector state monitoring with controller

This commit

  • add public /watch endpoint for querying connector state from etcd through controller
  • add private /check endpoint for controller to probe and trigger longrunning checkConnector workflow

@linear
Copy link

linear bot commented Mar 29, 2023

@heiruwu heiruwu changed the title Heiru/ins 168 implement controller query in connector feat: add support for state monitoring with controller component Mar 29, 2023
@codecov
Copy link

codecov bot commented Mar 29, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (2b7d917) 9.41% compared to head (a4547fd) 9.41%.

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #72   +/-   ##
=====================================
  Coverage   9.41%   9.41%           
=====================================
  Files          6       6           
  Lines        393     393           
=====================================
  Hits          37      37           
  Misses       356     356           
Flag Coverage Δ
unittests 9.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@heiruwu heiruwu force-pushed the heiru/ins-168-implement-controller-query-in-connector branch from 6228386 to 24a3a17 Compare March 30, 2023 15:35
@heiruwu heiruwu force-pushed the heiru/ins-168-implement-controller-query-in-connector branch 2 times, most recently from 2b344d9 to 8a5d6a8 Compare April 3, 2023 15:06
@heiruwu heiruwu force-pushed the heiru/ins-168-implement-controller-query-in-connector branch from 8a5d6a8 to afbc890 Compare April 5, 2023 06:36
@heiruwu heiruwu changed the title feat: add support for state monitoring with controller component feat(controller): add connector state monitoring with controller Apr 5, 2023
@heiruwu heiruwu force-pushed the heiru/ins-168-implement-controller-query-in-connector branch from bc2e1a9 to 08664c5 Compare April 6, 2023 23:15
@heiruwu heiruwu marked this pull request as ready for review April 7, 2023 07:05
Copy link
Member

@pinglin pinglin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pinglin pinglin merged commit d4092e6 into main Apr 7, 2023
11 checks passed
@pinglin pinglin deleted the heiru/ins-168-implement-controller-query-in-connector branch April 7, 2023 13:08
pinglin pushed a commit that referenced this pull request Apr 7, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.9.0-alpha](v0.8.0-alpha...v0.9.0-alpha)
(2023-04-07)


### Features

* **controller:** add connector state monitoring with controller
([#72](#72))
([d4092e6](d4092e6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
pinglin pushed a commit that referenced this pull request Apr 9, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.9.0-alpha](v0.8.0-alpha...v0.9.0-alpha)
(2023-04-09)


### Features

* **controller:** add connector state monitoring with controller
([#72](#72))
([d4092e6](d4092e6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
2 participants