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

Enhance HTTPD pod liveness/readiness probes #218

Merged
merged 1 commit into from
Sep 13, 2017
Merged

Enhance HTTPD pod liveness/readiness probes #218

merged 1 commit into from
Sep 13, 2017

Conversation

fbladilo
Copy link
Contributor

@fbladilo fbladilo commented Sep 13, 2017 •
edited
Loading

  • The httpd pod is considered alive if a httpd process is spawned
  • The httpd pod is considered ready if a socket can be open on port 80

@fbladilo
Enhance HTTPD pod liveness/readiness probes
f780b86 
- The httpd pod is considered alive if an httpd process is spawned
- The httpd pod is considered ready if a socket can be open on port 80
@fbladilo
Copy link
Contributor Author

@bdunne @carbonin Please have a look, seems like a more reasonable strategy for probes. Related to #211

@fbladilo fbladilo requested review from bdunne and carbonin September 13, 2017 15:19
@carbonin carbonin self-assigned this Sep 13, 2017
@carbonin carbonin added this to the Sprint 69 Ending Sep 18, 2017 milestone Sep 13, 2017
@carbonin carbonin merged commit 3a4e140 into ManageIQ:master Sep 13, 2017
@fbladilo fbladilo mentioned this pull request Sep 13, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carbonin carbonin carbonin approved these changes

@bdunne bdunne Awaiting requested review from bdunne bdunne is a code owner

Assignees

@carbonin carbonin

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fbladilo @carbonin