Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ZheSun88
Copy link
Contributor

@ZheSun88 ZheSun88 commented Oct 1, 2025

  • chore: Add GitHub Actions workflow for PR validation

Replaces TeamCity PR validation build with GitHub Actions workflow. Includes Maven build, test execution with Sauce Labs integration, and artifact collection for test failures.

πŸ€– Generated with Claude Code

  • Update validation.yml

  • fix: Update Sauce Connect to v5.x and configure PRO_KEY environment variable

  • Upgrade Sauce Connect from 4.9.2 to 5.2.0
  • Update command-line arguments to use new v5 syntax (--username, --access-key, --tunnel-name)
  • Configure TB_LICENSE secret as PRO_KEY environment variable
  • Remove inline license parameter from Maven command

πŸ€– Generated with Claude Code

  • fix: Correct Sauce Connect 5.x download URL format
  • Fix URL to include version in path: /sauce-connect/{VERSION}/sauce-connect-{VERSION}_linux.x86_64.tar.gz
  • Update to latest version 5.2.3
  • Fix extracted directory path (./sauce-connect/sc instead of ./sauce-connect-{VERSION}/sc)

πŸ€– Generated with Claude Code

  • refactor: Merge format-check workflow into validation workflow
  • Add format-check job to validation workflow to run in parallel with build-and-test
  • Remove separate format-check.yml workflow file
  • Format validation now runs as part of the main PR validation pipeline

πŸ€– Generated with Claude Code

  • feat: Add Maven cache to format-check job
  • Add Maven dependency caching to format-check job
  • Uses same cache key as build-and-test job for artifact sharing
  • Prevents redundant downloads between parallel jobs

πŸ€– Generated with Claude Code

  • fix: Run format-check and build-and-test jobs sequentially
  • Add 'needs: format-check' to build-and-test job
  • Ensures proper cache sharing between jobs
  • Prevents cache conflicts and race conditions
  • Format check must pass before build and tests run

πŸ€– Generated with Claude Code

  • chore: set TB License

  • fix: Correct Sauce Connect binary path for version 5.x

The extracted directory structure for Sauce Connect 5.x is sc-${VERSION}-linux/bin/sc, not sauce-connect/sc

πŸ€– Generated with Claude Code

  • fix: Improve Sauce Connect setup and monitoring
  • Use hardcoded version in path (5.2.3) instead of variable expansion
  • Wait for specific "Sauce Connect is up" message instead of sleeping
  • Fail fast if SC process dies or doesn't start within 2 minutes
  • Save PID for proper cleanup
  • Include SC logs in failure artifacts for debugging

πŸ€– Generated with Claude Code

  • Fix version

  • refactor: Use official sauce-connect-action v3 instead of manual setup

  • Replace manual Sauce Connect download and setup with saucelabs/sauce-connect-action@v3
  • Remove manual cleanup step as the action handles it automatically
  • Simplify tunnel ID management
  • Remove sc.log from artifacts as it's not generated with the action

πŸ€– Generated with Claude Code

Split test artifacts into two separate uploads:

  • error-screenshots: Contains only screenshot files from test failures
  • test-reports: Contains surefire and failsafe reports

This makes it easier to find and download just the screenshots when debugging failures.

πŸ€– Generated with Claude Code

  • fix: Enable localhost proxying for Sauce Connect tunnel

Add --proxy-localhost flag to allow Sauce Labs to connect to localhost addresses, which is required for the test setup.

πŸ€– Generated with Claude Code

  • fix: Use correct proxyLocalhost parameter for Sauce Connect

Replace scArgs with the proper proxyLocalhost: allow parameter to enable localhost proxying in Sauce Connect tunnel.

πŸ€– Generated with Claude Code

  • no phantomjs

  • fix: Exclude failsafe-summary.xml from test result parsing

Change file patterns to only include TEST-*.xml files, excluding failsafe-summary.xml which is not in JUnit XML format and causes the test result publisher to fail.

πŸ€– Generated with Claude Code

  • upgrade java version to 21 and remove master branch

Artur- and others added 3 commits October 1, 2025 13:57
* chore: Add GitHub Actions workflow for PR validation

Replaces TeamCity PR validation build with GitHub Actions workflow.
Includes Maven build, test execution with Sauce Labs integration,
and artifact collection for test failures.

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* Update validation.yml

* fix: Update Sauce Connect to v5.x and configure PRO_KEY environment variable

- Upgrade Sauce Connect from 4.9.2 to 5.2.0
- Update command-line arguments to use new v5 syntax (--username, --access-key, --tunnel-name)
- Configure TB_LICENSE secret as PRO_KEY environment variable
- Remove inline license parameter from Maven command

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* fix: Correct Sauce Connect 5.x download URL format

- Fix URL to include version in path: /sauce-connect/{VERSION}/sauce-connect-{VERSION}_linux.x86_64.tar.gz
- Update to latest version 5.2.3
- Fix extracted directory path (./sauce-connect/sc instead of ./sauce-connect-{VERSION}/sc)

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* refactor: Merge format-check workflow into validation workflow

- Add format-check job to validation workflow to run in parallel with build-and-test
- Remove separate format-check.yml workflow file
- Format validation now runs as part of the main PR validation pipeline

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* feat: Add Maven cache to format-check job

- Add Maven dependency caching to format-check job
- Uses same cache key as build-and-test job for artifact sharing
- Prevents redundant downloads between parallel jobs

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* fix: Run format-check and build-and-test jobs sequentially

- Add 'needs: format-check' to build-and-test job
- Ensures proper cache sharing between jobs
- Prevents cache conflicts and race conditions
- Format check must pass before build and tests run

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* chore: set TB License

* fix: Correct Sauce Connect binary path for version 5.x

The extracted directory structure for Sauce Connect 5.x is
sc-${VERSION}-linux/bin/sc, not sauce-connect/sc

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* fix: Improve Sauce Connect setup and monitoring

- Use hardcoded version in path (5.2.3) instead of variable expansion
- Wait for specific "Sauce Connect is up" message instead of sleeping
- Fail fast if SC process dies or doesn't start within 2 minutes
- Save PID for proper cleanup
- Include SC logs in failure artifacts for debugging

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* Fix version

* refactor: Use official sauce-connect-action v3 instead of manual setup

- Replace manual Sauce Connect download and setup with saucelabs/sauce-connect-action@v3
- Remove manual cleanup step as the action handles it automatically
- Simplify tunnel ID management
- Remove sc.log from artifacts as it's not generated with the action

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* use existing version, see saucelabs/sauce-connect-action#101

* Secrets

* .

* region

* fix: Separate error screenshots into their own artifact

Split test artifacts into two separate uploads:
- error-screenshots: Contains only screenshot files from test failures
- test-reports: Contains surefire and failsafe reports

This makes it easier to find and download just the screenshots when debugging failures.

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* fix: Enable localhost proxying for Sauce Connect tunnel

Add --proxy-localhost flag to allow Sauce Labs to connect to localhost
addresses, which is required for the test setup.

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* fix: Use correct proxyLocalhost parameter for Sauce Connect

Replace scArgs with the proper proxyLocalhost: allow parameter
to enable localhost proxying in Sauce Connect tunnel.

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* no phantomjs

* fix: Exclude failsafe-summary.xml from test result parsing

Change file patterns to only include TEST-*.xml files, excluding
failsafe-summary.xml which is not in JUnit XML format and causes
the test result publisher to fail.

πŸ€– Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* upgrade java version to 21 and remove master branch

---------

Co-authored-by: Claude <[email protected]>
Co-authored-by: Zhe Sun <[email protected]>
Co-authored-by: Marco Collovati <[email protected]>
Co-authored-by: Mikhail Shabarov <[email protected]>
@ZheSun88 ZheSun88 changed the title chore: Add GitHub Actions workflow for PR validation (#2007) chore: Add GitHub Actions workflow for PR validation (#2007) (9.5) Oct 1, 2025
@ZheSun88 ZheSun88 enabled auto-merge (squash) October 1, 2025 12:28
@ZheSun88 ZheSun88 merged commit d5681af into 9.5 Oct 3, 2025
5 checks passed
@ZheSun88 ZheSun88 deleted the pick-gha-9.5 branch October 3, 2025 07:17
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.

4 participants