Skip to content
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

Feature/add custom k8s metadata collector #580

Merged

Conversation

NeJan2020
Copy link
Collaborator

  • Added an extra application for collecting k8s metadata named metadata-provider,with an API that list/watchk8s metadata

Description

Old:

- k8s-apiserver <--(watch)-- agent1,agent2,...,agentx

With new application

- k8s-apiserver  <-- (watch) -- metadata-provider 
- metadata-provider <--(watch)-- agent1,agent2,...,agentx

Motivation and Context

In large-scale cluster, many agents start at the same time, which directly stresses the k8s-apiserver. The capabilities and data provided by metadata-providers are much smaller than that provided by k8s-apiserver , which reduce flow pressure and avoid impact on the k8s-apiserver.

How Has This Been Tested?

TestCase will be added soon

Is there a better way?

Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
…o map with RWLock

Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
@NeJan2020 NeJan2020 force-pushed the feature/add-custom-k8s-metadata-collector branch from b4041a9 to dab5dfa Compare October 9, 2023 02:58
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
Signed-off-by: niejiangang <niejiangang@harmonycloud.cn>
@dxsup dxsup merged commit f55d64d into KindlingProject:main Nov 3, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants