Skip to content

Fix metrics errors - #3444

Merged
cjreynol merged 4 commits into
google:mainfrom
cjreynol:fix_metrics_errors
Oct 28, 2026
Merged

cjreynol merged 4 commits into
google:mainfrom
cjreynol:fix_metrics_errors

Conversation

@cjreynol

Copy link
Copy Markdown
Collaborator

Bug: 576310224

There are cases where another logic path fails and the directory has not
yet been created.  This logic defensively ensures that will not happen.

Bug: 576310224
Trying to be strict about requiring the value continues to craete
metrics failures in new environments due to the non-local nature of the
unique `user_id` file.

That is causing no metrics to be transmitted at all, which is strictly a
worse case.  We can look for blocks of data where the field is not
populated and try to resolve those issues after deployment, but we want
to get most of the data.

Bug: 576310224
See previous commit for reasoning.  Too many possible failures to be so
strict.

Bug: 576310224
@cjreynol
cjreynol requested a review from ser-io October 27, 2026 22:59
@cjreynol cjreynol added kokoro:force-run Trigger a presubmit build unconditionally. kokoro:run Run e2e tests. labels Oct 27, 2026
@cjreynol cjreynol self-assigned this Oct 27, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels Oct 27, 2026
No need to expose the lookup mechanism or environment variable strings.
@cjreynol cjreynol added kokoro:force-run Trigger a presubmit build unconditionally. kokoro:run Run e2e tests. labels Oct 27, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed kokoro:run Run e2e tests. kokoro:force-run Trigger a presubmit build unconditionally. labels Oct 27, 2026
@cjreynol
cjreynol added this pull request to the merge queue Oct 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 28, 2026
@cjreynol
cjreynol added this pull request to the merge queue Oct 28, 2026
Merged via the queue into google:main with commit 322ec19 Oct 28, 2026
56 checks passed
@cjreynol
cjreynol deleted the fix_metrics_errors branch October 28, 2026 20:19
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.

3 participants