Skip to content

CI: Github Actions parallel integration testing #3

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 61 commits into
base: master
Choose a base branch
from
Open

CI: Github Actions parallel integration testing #3

wants to merge 61 commits into from

Conversation

onetechnical
Copy link
Owner

Summary

Test Plan

onetechnical
onetechnical commented May 5, 2025
View reviewed changes
Copy link
Owner Author

@onetechnical onetechnical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/run amd64_test_nightly

onetechnical added 26 commits May 5, 2025 14:17
@onetechnical
Restore concurrency and add a debug for comment
a2ebeee
@onetechnical
Re-commented out concurrency limiter.
fc0e74f
@onetechnical
Change trigger to only issue_comment: created
29b70e5
@onetechnical
Add a minimal new CI file and comment out other jobs
b612e38
@onetechnical
Comment out 'on' from Github Actions file
97abc6c
@onetechnical
Naive approach to parallelize; dubious about overlapping matrices
1f6d8a1
@onetechnical
Fix some cache keys and make build step prerequisite for test
91fe464
@onetechnical
add workflow dispatch triggers and hardcode matrix platforms
d9598df
@onetechnical
Bug fixes to properly set up test
692c17c
@onetechnical
Try to remove the cache restoration and see if it's in the workspace
305dd31
@onetechnical
Try to use tar with workspace archiving to preserve permissions
859d315
@onetechnical
Move tar.gz to /tmp/ and shift go caching around
ec8322d
@onetechnical
Remove duplicative go caching and fix path for restore archive
5b8bfe7
@onetechnical
Try adding go installation to test step
0fb46df
@onetechnical
Ramp up to 8 instances and upload coverage artifacts
bc12b4a
@onetechnical
Disable unit tests and enable integration tests
b9dc1e2
@onetechnical
Re-zero index - I think I'm wrong
a8c2714
@onetechnical
Try only using four instances
b7b3cf4
@onetechnical
Add CIRCLECI=true to skip some flaky tests
dd5e9f5
@onetechnical
Try using tilde to set path to rest results and bump up concurrency
42b6d23
@onetechnical
Disable integration and enable e2e_expect tests
3a23b1d
@onetechnical
Test e2e_subs and comment out e2e_expect
402ca15
@onetechnical
Try all the tests to see if they work, but only amd64
af32722
@onetechnical
Fan in the verify_nightly check
f85d367
@onetechnical
Add creation of combined_testresults file, and only enable test
ad7ef1b
@onetechnical
Try all again, but switch to arm platform
0645eb0
onetechnical added 30 commits May 9, 2025 14:48
@onetechnical
Forgot to update to build for amd64, not arm
d3775d7
@onetechnical
Try to override env to denote GITHUB_ACTIONS
80094f5
@onetechnical
Re-enable other tests and add token to codecov
80e9ccc
@onetechnical
Only upload on amd64
39fd5ea
@onetechnical
Enable all platforms
c1dd653
@onetechnical
Add slack on fail steps, and for testing only do ubuntu-24.04
bfd6131
@onetechnical
Only enable build and force trigger slack
f61d327
@onetechnical
Also disable verify_nightly job
431d3a7
@onetechnical
Add incomplete upload job, and fix slack conditional
4e7c837
@onetechnical
Naive upload test
cbb9f78
@onetechnical
Bugfix extract workspace archive
dcccdff
@onetechnical
Revert setting S3 bucket in test_release.sh
999b9ac
@onetechnical
Don't forget to pass S3_RELEASE_BUCKET
24e60d4
@onetechnical
Add AWS_REGION variable
5bb21a7
@onetechnical
Add S3_REGION to env
10a62df
@onetechnical
Add AWS env variables to test release step
a2235a1
@onetechnical
Add overriding of S3 bucket to test_release.sh script
469624b
@onetechnical
Remove ubuntu 20.04 from test_release.sh script
e7fd069
@onetechnical
Try to escape ampersands
bf3cfe4
@onetechnical
Add AWS_REGION to test_release.sh
1576a4b
@onetechnical
Add CI for PR, and fix up nightly. Remove support for test release.
753dacd
@onetechnical
Bump up parallelism for PR tests
f0b8485
@onetechnical
Revert test release changes, as I'm removing it from the pipeline anyway
f8d3de6
@onetechnical
Revert other github actions
37f55a1
@onetechnical
Lower PR parallels and add job concurrency limit
27f8e60
@onetechnical
Test nightly with just build and upload on ubuntu-24.04, and try to a…
97dd06c 
…ssume role
@onetechnical
Bumup up integration parallelism to 8
6caf18a
@onetechnical
Add aws-region to configure aws credentials task
fa9c4d7
@onetechnical
Add id-token: write permissions
302abd9
@onetechnical
Bump integration tests back up to 16
55f9780
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
New Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@onetechnical