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

sauce-connect: improve docker installation docs and document api-address changing in docker #2858

Merged
merged 3 commits into from
Jul 24, 2024

Conversation

mmatczuk
Copy link
Contributor

No description provided.

@mmatczuk mmatczuk force-pushed the mmt/docker_api_addr branch from 30f4e76 to 1921a9b Compare July 19, 2024 09:58
Copy link

Deploy preview ready for 2858!
https://docs.dev.saucelabs.net
built from 23b9a4f

Copy link

Deploy preview ready for 2858!
https://docs.dev.saucelabs.net
built from 7404aef

@Choraden Choraden force-pushed the mmt/docker_api_addr branch from 1921a9b to 3295aee Compare July 23, 2024 15:09
Copy link

Deploy preview ready for 2858!
https://docs.dev.saucelabs.net
built from 105e6c4

@Choraden
Copy link
Contributor

Should we link secure-connections/sauce-connect-5/operation/api-server/#docker somewhere in the docker installation page? Maybe in additional resources. Those things are connected, yet there is no direct way to access one from the other.

@mmatczuk
Copy link
Contributor Author

@Choraden added note

image

I'd be fixing it so the sauce-connect(api-server) commit could be reverted later.

@mmatczuk mmatczuk force-pushed the mmt/docker_api_addr branch from 151a93e to 0b78b80 Compare July 24, 2024 09:03
Copy link

Deploy preview ready for 2858!
https://docs.dev.saucelabs.net
built from 98c8f84

Copy link

Deploy preview ready for 2858!
https://docs.dev.saucelabs.net
built from 82ec49a

@mmatczuk mmatczuk merged commit a405c7a into main Jul 24, 2024
7 checks passed
@mmatczuk mmatczuk deleted the mmt/docker_api_addr branch July 24, 2024 09:22
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.

3 participants