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.

Sign up for GitHub

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

Jump to bottom

Docker sock option #1956

Merged

rssk merged 3 commits into master from docker-sock-option

Aug 28, 2024

Contributor

rssk commented Aug 28, 2024

Please check if the PR fulfills these requirements

The commit message follows our guidelines
If your PR contains DB doc changes have you made an API migration and a localDB migration if necessary?
Have you checked to ensure there aren't other open Pull Requests for the same update/change?
Tests for the changes have been added (for bug fixes / features)
Passes e2e testing
Passes lint
Docs have been added / updated if necessary (for bug fixes / features)

What kind of change does this PR introduce?

Programmatically determines the docker context

What is the current behavior?
Currently if a non standard context, or multiple engines are installed, odd behavior can occur
Does this PR introduce a breaking change?
No

rssk added 3 commits

August 22, 2024 19:05


          add prod compose

f35ce4f


          add method to determine current docker context in use

1a5740a


          update ci config to new image

81334cd

rssk merged commit 1c5dadd into master

1 check passed

rssk deleted the docker-sock-option branch

August 28, 2024 17:26

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

Labels

None yet