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

chore: increase gce integration test timeout #411

Merged
merged 1 commit into from
Sep 29, 2022
Merged

Conversation

aabmass
Copy link
Contributor

@aabmass aabmass commented Sep 29, 2022

It is timing out waiting for the container to start https://github.com/GoogleCloudPlatform/opentelemetry-operations-js/pull/371/checks?check_run_id=8621886819

If this gets much longer, we may need to investigate. I had to increase it for operations-python repo as well, but operations-go doesn't seem to have a similar problem. It could be the docker image is too large.

@aabmass aabmass changed the title Increase gce integration test timeout chore: Increase gce integration test timeout Sep 29, 2022
@conventional-commit-lint-gcf
Copy link

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

It is timing out waiting for the container to start https://github.com/GoogleCloudPlatform/opentelemetry-operations-js/pull/371/checks?check_run_id=8621886819

If this gets much longer, we may need to investigate. I had to increase it for operations-python repo as well, but operations-go doesn't seem to have a similar problem. It could be the docker image is too large.
@aabmass aabmass changed the title chore: Increase gce integration test timeout chore: increase gce integration test timeout Sep 29, 2022
@aabmass aabmass enabled auto-merge (squash) September 29, 2022 15:52
@aabmass aabmass requested a review from a team September 29, 2022 15:53
@dashpole
Copy link
Contributor

operations-go has had the same problem. I've hit it a few times

@codecov
Copy link

codecov bot commented Sep 29, 2022

Codecov Report

Merging #411 (13125b5) into main (c64613e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #411   +/-   ##
=======================================
  Coverage   95.60%   95.60%           
=======================================
  Files          12       12           
  Lines         432      432           
  Branches       83       83           
=======================================
  Hits          413      413           
  Misses         19       19           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@aabmass aabmass merged commit 781b57b into main Sep 29, 2022
@punya punya deleted the increase-gce-timeout branch September 29, 2022 20:08
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.

2 participants