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: e2e test timeout 3 #1379

Draft
wants to merge 1 commit into
base: hmatulis-e2e-timeout-test-merge-target
Choose a base branch
from

Conversation

hsmatulisgoogle
Copy link
Collaborator

DO NOT MERGE.

This PR is meant to test the whether e2e test timeouts affecting the 0.15 release branch affect the commit of the last PR merged in

Signed-off-by: Henrique Spanoudis Matulis <[email protected]>
@bwplotka
Copy link
Collaborator

Tip: You can convert it to a draft to show it's an experiment.

@bwplotka
Copy link
Collaborator

Looks like it works?

@bernot-dev
Copy link
Collaborator

Looks like it works?

Yeah, looks like the longest e2e test took 11 minutes. Maybe just an incremental performance regression in the e2e tests. I'm curious if this performance difference is detectable in Prombench. It could actually be related to the newer version of Go, but that might be difficult to isolate with confidence.

@hsmatulisgoogle hsmatulisgoogle marked this pull request as draft February 13, 2025 21:02
@hsmatulisgoogle
Copy link
Collaborator Author

Tip: You can convert it to a draft to show it's an experiment.

Done!

Looks like it works?

To clarify for posterity: this PR targets a release/0.15~3 to bisect the issue. The issue ended up being related to the metadata changes here 953d736 being skewed from our released tags, which should now be fixed

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