Skip to content

ci: upgrade to Sauce Connect 5 to fix SC4 deprecation - #34554

Merged
alan-agius4 merged 1 commit into
angular:20.3.xfrom
alan-agius4:ci/saucelabs-sc5-20.3.x
Oct 28, 2026
Merged

alan-agius4 merged 1 commit into
angular:20.3.xfrom
alan-agius4:ci/saucelabs-sc5-20.3.x

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

Starting July 31, 2026, Sauce Labs servers will no longer support Sauce Connect 4 traffic. This migrates the CI workflow on 20.3.x to use saucelabs/sauce-connect-action@v3 (Sauce Connect 5) and updates the Protractor Sauce Labs capabilities configuration (tunnelName and sauceRegion: 'us').

Starting July 31, 2026, Sauce Labs servers will no longer support Sauce
Connect 4 traffic. This commit migrates the CI workflow to use
saucelabs/sauce-connect-action@v3 (Sauce Connect 5) and refactors the
Protractor Sauce Labs capabilities configuration.
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Oct 28, 2026
@alan-agius4 alan-agius4 added target: lts This PR is targeting a version currently in long-term support action: merge The PR is ready for merge by the caretaker labels Oct 28, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Sauce Labs configuration by refactoring the capabilities definition and adding a region setting. A critical issue was identified regarding the use of 'tunnelName' in legacy JSON Wire Protocol capabilities; the reviewer correctly pointed out that 'tunnelIdentifier' must be used instead to ensure the tunnel is properly recognized by Sauce Labs, and provided a code suggestion to fix this.

@alan-agius4
alan-agius4 merged commit 5ec4fe3 into angular:20.3.x Oct 28, 2026
33 of 34 checks passed
@alan-agius4
alan-agius4 deleted the ci/saucelabs-sc5-20.3.x branch October 28, 2026 09:13
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: lts This PR is targeting a version currently in long-term support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants