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

feat(robustness): add liveness check and readiness check #536

Merged
merged 28 commits into from
Aug 7, 2024
Merged

feat(robustness): add liveness check and readiness check #536

merged 28 commits into from
Aug 7, 2024

Conversation

JasonHe-WQ
Copy link
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

For better robustness

Which issue(s) this PR fixes:

Issue #462

The connectivity of etcd and APIServer seems to be a build-in feature

Copy link

github-actions bot commented Jul 15, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@JasonHe-WQ
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

pkg/infra/persistence/elasticsearch/client.go Outdated Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
pkg/syncer/syncer.go Outdated Show resolved Hide resolved
pkg/core/health/check.go Show resolved Hide resolved
pkg/infra/persistence/elasticsearch/client.go Outdated Show resolved Hide resolved
pkg/core/route/route.go Outdated Show resolved Hide resolved
pkg/server/server.go Outdated Show resolved Hide resolved
pkg/server/config.go Outdated Show resolved Hide resolved
pkg/kubernetes/registry/types.go Outdated Show resolved Hide resolved
pkg/core/route/route.go Outdated Show resolved Hide resolved
pkg/core/health/server_check.go Outdated Show resolved Hide resolved
go.mod Show resolved Hide resolved
pkg/infra/search/storage/types.go Outdated Show resolved Hide resolved
pkg/core/health/check.go Outdated Show resolved Hide resolved
@elliotxx elliotxx added the area/server Server related. label Jul 29, 2024
@elliotxx elliotxx added this to the v0.5.0 milestone Jul 29, 2024
Copy link
Collaborator

@elliotxx elliotxx left a comment

Choose a reason for hiding this comment

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

lgtm

@elliotxx elliotxx merged commit a49de0a into KusionStack:main Aug 7, 2024
7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/server Server related.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants