-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[extension/observer/docker] add ListAndWatch to observer #5851
Conversation
5cc421d
to
2d3d9fc
Compare
556302f
to
4b6eb34
Compare
Not introduced here, but I remembered that the receiver creator uses endpoint target scheme of |
4b6eb34
to
c7db2e9
Compare
@rmfitzpatrick good catch, thanks, I just updated the target in this PR. |
431fe22
to
6175b4e
Compare
Please rebase |
6175b4e
to
82db734
Compare
82db734
to
064ab9a
Compare
Description: Adds the ListAndWatch capability to the Docker observer, and integration tests to help support it. There will be another PR after this one to add watching the docker container events stream to ListAndWatch to help modify endpoints in more real time, out of of the initial load and cache refresh interval.
Link to tracking Issue: #4446
Testing: Added new integration tests to test for some observer functionality. More will be added in the next PR when the container event stream is being watched and used to add/update/remove endpoints.