Skip to content

MWI: Add AllServicesReported method to readyz.Registry#60059

Merged
boxofrad merged 1 commit intomasterfrom
boxofrad/tbot-readyz-block
Oct 20, 2025
Merged

MWI: Add AllServicesReported method to readyz.Registry#60059
boxofrad merged 1 commit intomasterfrom
boxofrad/tbot-readyz-block

Conversation

@boxofrad
Copy link
Copy Markdown
Contributor

@boxofrad boxofrad commented Oct 8, 2025

This method will be used by the heartbeat service to wait until services have reported their initial status (with a timeout) before submitting the initial heartbeat. Otherwise, we'd potentially report all services as "initializing" for 30 minutes in daemon mode, or forever in long-shot mode.

@boxofrad boxofrad added machine-id no-changelog Indicates that a PR does not require a changelog entry labels Oct 8, 2025
@github-actions github-actions bot requested a review from doggydogworld October 8, 2025 17:55
@boxofrad boxofrad changed the title MWI: Add AllServicesReported method to readyz.Register MWI: Add AllServicesReported method to readyz.Registry Oct 8, 2025
@boxofrad boxofrad force-pushed the boxofrad/tbot-readyz-block branch from 2a25403 to 48b0203 Compare October 9, 2025 15:34
@boxofrad boxofrad added this pull request to the merge queue Oct 13, 2025
@boxofrad boxofrad removed this pull request from the merge queue due to a manual request Oct 13, 2025
@boxofrad boxofrad force-pushed the boxofrad/tbot-readyz-block branch from 48b0203 to ec9ad91 Compare October 13, 2025 10:42
@boxofrad boxofrad changed the base branch from master to boxofrad/tbot-oneshot-status October 13, 2025 10:43
@boxofrad boxofrad force-pushed the boxofrad/tbot-oneshot-status branch from d907f78 to 191382e Compare October 20, 2025 11:41
@boxofrad boxofrad force-pushed the boxofrad/tbot-readyz-block branch from ec9ad91 to 2afeaa2 Compare October 20, 2025 11:41
Base automatically changed from boxofrad/tbot-oneshot-status to master October 20, 2025 14:29
This method will be used by the heartbeat service to wait until services have
reported their initial status (with a timeout) before submitting the initial
heartbeat. Otherwise, we'd potentially report all services as "initializing"
for 30 minutes in daemon mode, or forever in long-shot mode.
@boxofrad boxofrad force-pushed the boxofrad/tbot-readyz-block branch from 2afeaa2 to f763e57 Compare October 20, 2025 14:39
@boxofrad boxofrad added this pull request to the merge queue Oct 20, 2025
Merged via the queue into master with commit 64e7eaa Oct 20, 2025
41 checks passed
@boxofrad boxofrad deleted the boxofrad/tbot-readyz-block branch October 20, 2025 15:31
boxofrad added a commit that referenced this pull request Oct 22, 2025
boxofrad added a commit that referenced this pull request Oct 22, 2025
boxofrad added a commit that referenced this pull request Oct 29, 2025
boxofrad added a commit that referenced this pull request Oct 29, 2025
github-merge-queue bot pushed a commit that referenced this pull request Oct 29, 2025
* [v17] MWI: Automatically report service statuses in oneshot mode

Backport #60148 to branch/v17

* [v17] MWI: Add `AllServicesReported` method to `readyz.Register`

Backport #60059 to branch/v17

* [v17] MWI: Wait for service health before sending first heartbeat

Backport #60087 to branch/v17

* [v17] MWI: Add service health to bot heartbeats

Backport #60093 to branch/v17

* [v17] MWI: Simpler auto-generated `tbot` service names

Backport #60052 to branch/v17

* Fix `testing/synctest` on CI

* Fix linting of synctest files on CI

* [v17] MWI: Fix flaky test in SPIFFE Workload APIs

Backport #60668 to branch/v17
github-merge-queue bot pushed a commit that referenced this pull request Oct 29, 2025
* [v18] MWI: Automatically report service statuses in oneshot mode

Backport #60148 to branch/v18

* [v18] MWI: Add `AllServicesReported` method to `readyz.Register`

Backport #60059 to branch/v18

* [v18] MWI: Wait for service health before sending first heartbeat

Backport #60087 to branch/v18

* [v18] MWI: Add service health to bot heartbeats

Backport #60093 to branch/v18

* [v18] MWI: Simpler auto-generated `tbot` service names

Backport #60052 to branch/v18

* Fix linting of synctest files on CI

* [v18] MWI: Fix flaky test in SPIFFE Workload APIs

Backport #60668 to branch/v18
mmcallister pushed a commit that referenced this pull request Nov 6, 2025
This method will be used by the heartbeat service to wait until services have
reported their initial status (with a timeout) before submitting the initial
heartbeat. Otherwise, we'd potentially report all services as "initializing"
for 30 minutes in daemon mode, or forever in long-shot mode.
mmcallister pushed a commit that referenced this pull request Nov 19, 2025
This method will be used by the heartbeat service to wait until services have
reported their initial status (with a timeout) before submitting the initial
heartbeat. Otherwise, we'd potentially report all services as "initializing"
for 30 minutes in daemon mode, or forever in long-shot mode.
mmcallister pushed a commit that referenced this pull request Nov 20, 2025
This method will be used by the heartbeat service to wait until services have
reported their initial status (with a timeout) before submitting the initial
heartbeat. Otherwise, we'd potentially report all services as "initializing"
for 30 minutes in daemon mode, or forever in long-shot mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

machine-id no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants