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

ci: upgrade sauce-connect-action to v3 #534

Open
GraemeWatt opened this issue Sep 16, 2022 · 0 comments
Open

ci: upgrade sauce-connect-action to v3 #534

GraemeWatt opened this issue Sep 16, 2022 · 0 comments
Labels

Comments

@GraemeWatt
Copy link
Member

Following on from PR #533, we were using sauce-connect-action@main which broke after saucelabs/sauce-connect-action#53 was merged into the main branch, since the restUrl parameter was not being passed when starting the Docker container for Sauce Connect. The problem was fixed by instead pinning sauce-connect-action@v2. The Sauce Connect Proxy option --rest-url is deprecated in favour of --region, but specifying region: eu-central instead of restUrl: https://api.eu-central-1.saucelabs.com/rest/v1 in the ci.yml file did not work with either the main or v2 branch of sauce-connect-action. We should switch to v3 of sauce-connect-action after it is tagged and investigate if region works instead of restUrl. If not, the bug should be reported to the maintainers of sauce-connect-action. It could be checked whether the region option works when specified inside a configFile or as a SAUCE_REGION environment variable. Also, we should switch from tunnelIdentifier to tunnelName.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: To do
Development

No branches or pull requests

1 participant