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

Attempts to make tests work better using the buildkite plugin #190

Closed
wants to merge 17 commits into from

Conversation

jagoosw
Copy link
Collaborator

@jagoosw jagoosw commented Jul 26, 2024

Not making this a draft or the CI won't run

@jagoosw jagoosw changed the title Another attempt to make tests work Attempts to make tests work better using the buildkite plugin Jul 26, 2024
@jagoosw jagoosw added documentation Improvements or additions to documentation tests labels Jul 26, 2024
@jagoosw
Copy link
Collaborator Author

jagoosw commented Jul 26, 2024

So this may be working now, so I'm going to make a branch from this branch and PR it into main, then make them both run at the same time to see if that works as a stress test

@jagoosw
Copy link
Collaborator Author

jagoosw commented Jul 26, 2024

This is way way slower for some reason? In #186 (old version of pipeline) the tests finish in <20min but this is still running after 45min.

@jagoosw
Copy link
Collaborator Author

jagoosw commented Jul 26, 2024

Maybe I should experiment with what we're caching

@jagoosw
Copy link
Collaborator Author

jagoosw commented Jul 26, 2024

Ah it realises that these branches are duplicates I think

@jagoosw
Copy link
Collaborator Author

jagoosw commented Aug 2, 2024

This does seem a lot more robust now but also a lot slower, I will investigate adding more caching

@jagoosw
Copy link
Collaborator Author

jagoosw commented Oct 4, 2024

I don't think we should do it this way since its so slow

@jagoosw jagoosw closed this Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant