Skip to content

ci: Integration test releases#1878

Open
collinjackson wants to merge 34 commits intomainfrom
collinjackson/integration-testing-2
Open

ci: Integration test releases#1878
collinjackson wants to merge 34 commits intomainfrom
collinjackson/integration-testing-2

Conversation

@collinjackson
Copy link
Collaborator

@collinjackson collinjackson commented Aug 4, 2025 •
edited
Loading

  • Now that presubmit integration testing on Linux is working, integration test builds on all platforms before creating the release. Exception: GitHub doesn't have available runners for Ubuntu arm64 so we cross compile and skip the integration test.
  • There is both a positive and negative test to ensure that the test harness is working.
  • Update docs.
  • Some refactoring of the CI workflow to avoid duplication.

@github-actions
Copy link

github-actions bot commented Aug 4, 2025 •
edited
Loading

Visit the preview URL for this PR (updated for commit c5a8b82):

https://nexus-cli--pr1878-collinjackson-integr-zq1es2jz.web.app

(expires Thu, 28 Aug 2025 20:31:24 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 815ec4c632754f56eccfacfc0919559f5a85a0f1

@collinjackson collinjackson requested a review from duc-nx August 4, 2025 20:43
@collinjackson
Merge remote-tracking branch 'origin/main' into collinjackson/integra…
fc264c1 
…tion-testing-2
@collinjackson
Remove comment
b88a7a4
@collinjackson
Merge remote-tracking branch 'origin/main' into collinjackson/integra…
64ae3a5 
…tion-testing-2
@collinjackson
ci(release): add manual checkbox overrides for docker build/push; gat…
95b9c37 
…e docker push to main; fix integration_test conflict and pattern gating
@collinjackson
ci: gate docker push to main with manual override; enforce bash for i…
7d861f9 
…ntegration tests; make script portable; use macos-13 for x86_64 to avoid Rosetta; do not change CLI success strings
@collinjackson
Remove comments
cd504db
@collinjackson
tests(integration): handle 429 early and try next node id; make ulimi…
8a54bb8 
…t non-fatal; maintain CLI success string; run mac x86_64 on macos-13
@collinjackson
tests(integration): on failure, rotate node IDs and retry a second pa…
fbf445c 
…ss; shuffle per attempt
@collinjackson
Merge remote-tracking branch 'origin/main' into collinjackson/integra…
e929858 
…tion-testing-2
@collinjackson
Fix test
cbd6bcf
@collinjackson
Fix success pattern
ce9d717
@collinjackson
Fix flaky windows test
2107703
@collinjackson
Increase pause
9a0b4ab
@collinjackson
Flakiness
cf04e1d
@collinjackson
Update docs
78024c5
@collinjackson
Consolidate
455b523
@collinjackson
matrix release
d1de149
@collinjackson
matrix
471f8ec
@collinjackson
Fix branching
173f909
@collinjackson
Remove matrix workflow
79e74f5
@collinjackson
Add inputs
90d08d4
@collinjackson
Syntax fix
0eecd3d
@collinjackson
make executable
7d29134
@collinjackson
Clean up scripts
4e46d88
@collinjackson collinjackson requested a review from duc-nx August 15, 2025 01:12
duc-nx
duc-nx approved these changes Aug 15, 2025
View reviewed changes
Copy link
Contributor

@duc-nx duc-nx left a comment

Choose a reason for hiding this comment

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

Just minor nits

@collinjackson collinjackson force-pushed the collinjackson/integration-testing-2 branch from 698e3e1 to ff49aa7 Compare August 15, 2025 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@duc-nx duc-nx duc-nx approved these changes

@TiGeRRGrowL TiGeRRGrowL TiGeRRGrowL approved these changes

@Yanuarsa14 Yanuarsa14 Yanuarsa14 approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@collinjackson @duc-nx @TiGeRRGrowL @Yanuarsa14