Skip to content

Gracefully handle metadata look up for update_task check for bots in off GCE#5362

Open
dylanjew wants to merge 3 commits into
masterfrom
dylanj/metadata-check
Open

Gracefully handle metadata look up for update_task check for bots in off GCE#5362
dylanjew wants to merge 3 commits into
masterfrom
dylanj/metadata-check

Conversation

@dylanjew

@dylanjew dylanjew commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

We are making a bunch of invalid requests to the metadata server when the bot is off GCE. See crbug.com/534369736 for investigation and the corresponding error group

This PR checks that the bot can access the metadata server before making a GET request. If it can't it logs the case rather than an error.

Testing
Metrics in dev

@dylanjew
dylanjew requested a review from a team as a code owner July 14, 2026 20:45
@dylanjew
dylanjew force-pushed the dylanj/metadata-check branch from 9a49ec6 to 975354e Compare July 15, 2026 17:30
@dylanjew
dylanjew force-pushed the dylanj/bot-metrics branch from 4dd5c13 to 538dcc4 Compare July 15, 2026 17:30
Base automatically changed from dylanj/bot-metrics to master July 16, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant