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

[RULE] Container App Health Probe that support HTTP #3111

Open
BernieWhite opened this issue Oct 13, 2024 · 0 comments
Open

[RULE] Container App Health Probe that support HTTP #3111

BernieWhite opened this issue Oct 13, 2024 · 0 comments
Labels
pillar: reliability Aligned to the Reliability pillar. rule: container-apps Rules for Container Apps

Comments

@BernieWhite
Copy link
Collaborator

BernieWhite commented Oct 13, 2024

Existing rule

No response

Suggested rule

If the container app uses HTTP based ingress, then health probes should be properly configured to support HTTP.

  • If properties.configuration.ingress.transport = http or http2 OR
  • If properties.configuration.ingress.targetPort = 80 or 8080 or 443

Health probes should be configured to support HTTP which has a granular status instead of open or closed that a TCP port has, to determine if a replica is available and ready to receive the ingress traffic.

Key points:

  • The default health probes use TCP, so no health probes configured is a fail.
  • Start up probe is often configured as TCP for several reasons to start the container so ignore that probe.
  • Check that Liveness and Readiness have httpGet configured.

Pillar

Reliability

Additional context

Similar to: https://azure.github.io/PSRule.Rules.Azure/en/rules/Azure.AppService.WebProbe/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pillar: reliability Aligned to the Reliability pillar. rule: container-apps Rules for Container Apps
Projects
None yet
Development

No branches or pull requests

1 participant