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

Add liveness and readiness probes to api gateway #11730

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

skhalash
Copy link
Contributor

@skhalash skhalash commented Jul 22, 2021

Description

A new way of monitoring Kyma runtime availability requires readiness probes to be configured for all workloads (deployments, stateful sets, daemon sets). api-gateway does not expose a health check endpoint, so as a temporary workaround, the metrics endpoint will be used for this purpose.

Changes proposed in this pull request:

  • Use existing metrics endpoint as a liveness/readiness probe

Related issue(s)
See also #11726

@skhalash skhalash added kind/feature Categorizes issue or PR as related to a new feature. area/api-gateway Issues or PRs related to api-gateway labels Jul 22, 2021
@kyma-bot kyma-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 22, 2021
@skhalash skhalash linked an issue Jul 22, 2021 that may be closed by this pull request
@skhalash skhalash changed the title Add liveness and readiness probes to the api gateway Add liveness and readiness probes to api gateway Jul 22, 2021
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jul 23, 2021
Copy link
Contributor

@strekm strekm left a comment

Choose a reason for hiding this comment

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

/meow

@kyma-bot
Copy link
Contributor

@strekm: cat image

In response to this:

/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@kyma-bot kyma-bot merged commit 88696e6 into kyma-project:main Jul 27, 2021
@kyma-bot
Copy link
Contributor

@skhalash: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Rerun command
pre-main-kyma-gke-integration-cosigned e27346c link /test pre-main-kyma-gke-integration-cosigned

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@skhalash skhalash deleted the api-gateway-readiness-probe branch July 27, 2021 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api-gateway Issues or PRs related to api-gateway kind/feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me! size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a health check endpoint to the api-gateway-controller
5 participants