-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
New component observer/cfgardenobserver #33727
base: main
Are you sure you want to change the base?
New component observer/cfgardenobserver #33727
Conversation
Co-authored-by: Jose Riguera <[email protected]> Co-authored-by: Sam Clulow <[email protected]> Co-authored-by: Tomas Mota <[email protected]>
Co-authored-by: Jose Riguera <[email protected]> Co-authored-by: Sam Clulow <[email protected]> Co-authored-by: Tomas Mota <[email protected]>
This PR should also have a changelog entry, and the |
118247c
to
9da35ce
Compare
Thanks for the review @crobert-1 , we have made the requested changes. |
8b36717
to
54d1e9a
Compare
There are quite a few CI/CD action failures, and they appear to be related to the changes in this PR. I'll hold off on reviewing again until they're resolved. |
Thanks @crobert-1 and @MovieStoreGuy, I have made the changes and the ci is now passing. Thank you for your reviews |
Co-authored-by: Curtis Robert <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
Description:
We would like to implement a new observer for Cloudfoundry containers/applications. The idea is not make use of the main API but the local one, which is available as unix socket on each node and manages the containers lifecycle. The main API would remain as optional and only to get Application info (which only involves one GET http request once the app id is known).
Link to tracking Issue: 33618
Testing: First component PR
Documentation: Added Readme