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

Container order change clashes with readiness probes #3304

Closed
yaron2 opened this issue Jun 16, 2021 · 1 comment
Closed

Container order change clashes with readiness probes #3304

yaron2 opened this issue Jun 16, 2021 · 1 comment
Assignees
@yaron2
Labels
kind/bug Something isn't working P0

Comments

@yaron2
Copy link
Member

yaron2 commented Jun 16, 2021

The PR to make Dapr the first container in the pod causes a regression in cases where apps have readiness probes and has Dapr waiting for the app to respond on a given port.

This causes Dapr to detect the app is running only when the failure threshold of the user app is met.
@yaron2 yaron2 added kind/bug Something isn't working P0 labels Jun 16, 2021
@yaron2 yaron2 self-assigned this Jun 16, 2021
@yaron2
Copy link
Member Author

yaron2 commented Jun 16, 2021

Closed via #3300.

@yaron2 yaron2 closed this as completed Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yaron2 yaron2

Labels
kind/bug Something isn't working P0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yaron2