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

Support testing when Docker is not running locally #123

Merged
merged 2 commits into from
Feb 8, 2022
Merged

Support testing when Docker is not running locally #123

merged 2 commits into from
Feb 8, 2022

Conversation

dmikusa
Copy link

@dmikusa dmikusa commented Feb 2, 2022

Summary

This PR makes adjustments so that you can configure a remote host. It will look at and parse DOCKER_HOST. If not set, then it defaults to localhost which preserves the current behavior. If set, it will parse the URL and return the hostname so long as it's not a unix:// socket.

Use Cases

With the recent changes to Docker licensing, a user may not be running Docker on localhost. The occam code assumes that a test application will be accessible on localhost though. If you have a set up where Docker is running in a custom VM or even remotely. The occam matchers will always fail because nothing is available on localhost.

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

With the recent changes to Docker licensing, a user may not be running Docker on localhost. The occam code assumes that a test application will be accessible on `localhost` though. If you have a set up where Docker is running in a custom VM or even remotely. The occam matchers will always fail because nothing is available on localhost.

This PR makes adjustments so that you can configure a remote host. It will look at and parse `DOCKER_HOST`. If not set, then it defaults to `localhost` which preserves the current behavior. If set, it will parse the URL and return the hostname so long as it's not a `unix://` socket.

Signed-off-by: Daniel Mikusa <[email protected]>
@dmikusa dmikusa added the enhancement A new feature or request label Feb 2, 2022
@dmikusa dmikusa requested a review from a team as a code owner February 2, 2022 18:41
@sophiewigmore sophiewigmore merged commit 23547af into paketo-buildpacks:main Feb 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A new feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants