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

First timer tick for a pod status update takes too long #5693

Closed
smarterclayton opened this issue Mar 20, 2015 · 3 comments
Closed

First timer tick for a pod status update takes too long #5693

smarterclayton opened this issue Mar 20, 2015 · 3 comments
Assignees
Labels
area/usability priority/backlog Higher priority than priority/awaiting-more-evidence. sig/node Categorizes an issue or PR as relevant to SIG Node.

Comments

@smarterclayton
Copy link
Contributor

It appears that the first pod status update tick is fired on average 1/2 pod status frequency duration. This is unnecessarily long - I don't know if we have an existing issue to cover this, but it makes many small, simple pods take up to a minute to report ready (on their first time). @vmarmol added a note to convert this to a better mechanism, but I wanted to make it obvious.

Some thoughts:

  • The first transition between Pending and Running is the one that should fire with minimal latency - we should look to send these ASAP, in preference over all other status updates
  • Subsequent updates can come at lower priority
  • We should probably poll/check/update pod status at different intervals (inversely proportional to their age up to a cap), since most failures will happen in the first seconds of launch
@vmarmol
Copy link
Contributor

vmarmol commented Mar 20, 2015

Nice catch! @fgrzadkowski is working on #5624 which should alleviate this issue and is probably the right behavior.

@fgrzadkowski fgrzadkowski self-assigned this Mar 20, 2015
@fgrzadkowski
Copy link
Contributor

I hope to fix this by #5624.

@mbforbes mbforbes added sig/node Categorizes an issue or PR as relevant to SIG Node. priority/backlog Higher priority than priority/awaiting-more-evidence. labels Mar 20, 2015
@fgrzadkowski
Copy link
Contributor

Fixed by #5624

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/usability priority/backlog Higher priority than priority/awaiting-more-evidence. sig/node Categorizes an issue or PR as relevant to SIG Node.
Projects
None yet
Development

No branches or pull requests

4 participants