yangwwei edited a comment on issue #72: Add ability to support 3rd party K8s operator integration URL: https://github.com/apache/incubator-yunikorn-k8shim/pull/72#issuecomment-583608990 Hi @wilfred-s I am done with my changes, can you help to review? The PR is pretty big, that is because I've done a big refractory of the current code base in the shim. The purpose is to decouple the app-lifecycle-management out. So if you look at pkg/appmgmt/interfaces, this package defines the common interfaces used for app management, that can be used for different types of operators, including current logic (where I put them in general pkg). Each operator framework needs to watch on certain CRD resources, and waiting for the signals from K8s informers. That's why I introduced `AppManager`, and a compositive service `AppManagementService`. So all registered app managers can run independently, they just discover apps/tasks and push those info to the cache. Then the common logic will start to submit reuqests/handle responses to/from yunikorn-core. For detail of the design, please see design doc: https://github.com/apache/incubator-yunikorn-core/blob/design/docs/design/pluggable-app-management.md.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
