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.

Sign up for GitHub

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

Jump to bottom

feat: Customize the readiness probe of APISIX pod #545

Closed
joyyir opened this issue May 23, 2023 · 2 comments
Closed

feat: Customize the readiness probe of APISIX pod #545

joyyir opened this issue May 23, 2023 · 2 comments

Comments

@joyyir
Copy link
Contributor

joyyir commented May 23, 2023 •
edited

The current version of APISIX helm chart cannot change the readiness probe of APISIX pod. (Source Code)
It would be nice to support customizing the readiness probe of APISIX pod. It is probably needed to change the readiness probe, especially when using external plugins. Currently, the 10 second delay is not sufficient to prepare external plugins.
@joyyir joyyir changed the title feat: customize the readiness probe of APISIX pod feat: Customize the readiness probe of APISIX pod May 23, 2023
@Gallardot
Copy link
Member

Agree. Welcome to submit a PR to make it possible!

@joyyir joyyir mentioned this issue May 31, 2023
Merged
Gallardot pushed a commit that referenced this issue Jun 18, 2023
@joyyir
feat: Customize the readiness probe of APISIX pod (#545) (#548)
0c7d61b 
* feat: support readinessProbe for apisix pod (#545)

* docs: update README.md (#545)

---------

Co-authored-by: 장준영(Junyeong Jang)/Core플랫폼개발팀/11ST <junyeong.jang@sk.com>
@lingsamuel
Copy link
Member

Closing since the PR merged

joyyir pushed a commit to joyyir/apisix-helm-chart that referenced this issue Jan 3, 2024
feat: support readinessProbe for apisix pod (apache#545)
5bfe732
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Gallardot @joyyir @lingsamuel