Skip to content

Always send periodic progress requsts when collecting events.#21546

Open
serathius wants to merge 1 commit intoetcd-io:mainfrom
serathius:robustness-background-progress-request
Open

Always send periodic progress requsts when collecting events.#21546
serathius wants to merge 1 commit intoetcd-io:mainfrom
serathius:robustness-background-progress-request

Conversation

@serathius
Copy link
Copy Markdown
Member

Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius serathius force-pushed the robustness-background-progress-request branch from 20ae650 to 6376ee0 Compare March 29, 2026 15:50
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.40%. Comparing base (5aebee8) to head (6376ee0).

Additional details and impacted files

see 21 files with indirect coverage changes

@@            Coverage Diff             @@
##             main   #21546      +/-   ##
==========================================
- Coverage   68.45%   68.40%   -0.05%     
==========================================
  Files         428      428              
  Lines       35383    35383              
==========================================
- Hits        24221    24204      -17     
- Misses       9761     9772      +11     
- Partials     1401     1407       +6     

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5aebee8...6376ee0. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@k8s-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: henrybear327, nwnt, serathius

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cancel()
}
}
case <-time.After(100 * time.Millisecond):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: may consider making 100 as a const.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants