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

fix(tests): delete flaky test #1565

Merged
merged 4 commits into from
Jul 24, 2024
Merged

fix(tests): delete flaky test #1565

merged 4 commits into from
Jul 24, 2024

Conversation

punya
Copy link
Contributor

@punya punya commented Jul 23, 2024

This test is already disabled for current and future versions of NodeJS,
and I don't think it's worth putting in the effort to fix flakiness when
the whole test will soon be a no-op.

Also fix other issues that were (deterministically) breaking the build:

  • Disable codecov: we were running out of API quota, and most other GCP repos had previously disabled it for unrelated reasons
  • Migrate eslint ignores from the "node" plugin to the "n" plugin

Fixes #1562, fixes #1525, fixes #1524, fixes #1523, fixes #1522, fixes #1500, fixes #1499, fixes #1498, fixes #1497, fixes #1496

This test is already disabled for current and future versions of NodeJS,
and I don't think it's worth putting in the effort to fix flakiness when
the whole test will soon be a no-op.
@punya punya requested review from a team as code owners July 23, 2024 18:35
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: cloudtrace Issues related to the googleapis/cloud-trace-nodejs API. labels Jul 23, 2024
aabmass

This comment was marked as resolved.

This comment was marked as outdated.

@punya punya force-pushed the delete-flaky-test branch from 2cc6689 to 5294588 Compare July 23, 2024 21:32
punya added 2 commits July 24, 2024 14:37
Codecov uploads are being throttled (429) because
we are exceeding monthly quota
@punya punya force-pushed the delete-flaky-test branch from c97a84d to 0e01a7d Compare July 24, 2024 14:59
@punya punya merged commit 78fde86 into main Jul 24, 2024
18 of 19 checks passed
@punya punya deleted the delete-flaky-test branch July 24, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment