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

Dashboard doesn't send client update when a pod terminates #1539

Closed
bpoland opened this issue Sep 24, 2021 · 1 comment · Fixed by #1642
Closed

Dashboard doesn't send client update when a pod terminates #1539

bpoland opened this issue Sep 24, 2021 · 1 comment · Fixed by #1642
Assignees
Labels
bug Something isn't working dashboard ui
Milestone

Comments

@bpoland
Copy link
Contributor

bpoland commented Sep 24, 2021

Summary

When watching the rollouts dashboard during a canary rollout, the dashboard usually updates automatically as the rollout progresses through the steps. When a pod starts terminating it updates to show a blue circle box, but when the pod finishes terminating there is no update sent to the client UI.

This is generally not an issue while the rollout is progressing since another update usually comes soon after a pod terminates (e.g. when another pod gets started). But the last pod to terminate in the old ReplicaSet stays with the blue circle indefinitely, and there's no update when the rollout fully completes. Refreshing the page manually shows the correct status, but that shouldn't be necessary -- the dashboard should send an update to the client when the pod terminates.

Diagnostics

Controller + dashboard v1.1.0-rc1


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

@bpoland bpoland added the bug Something isn't working label Sep 24, 2021
@harikrongali harikrongali added this to the v1.2 milestone Sep 24, 2021
@perenesenko
Copy link
Member

perenesenko commented Sep 29, 2021

I will look at this one.
@harikrongali

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dashboard ui
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants