-
Notifications
You must be signed in to change notification settings - Fork 42
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
Introducing the Container provider #151
Conversation
class VMBroker: | ||
class Broker: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change++
Looks like I need to account for differences in unconfigured configs for containers to get the checks to pass on GH |
One test failing because AT provider is actually trying to connect, so I need to look into that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK with some typos and one question.
@swadeley @Griffin-Sullivan thanks for the reviews! The latest update should include and/or address everything so far |
2a46d69
to
f0aa330
Compare
``` | ||
pip install broker[podman] | ||
or | ||
pip install broker[docker] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason these commands fail when using zsh
, I think its worth mentioning you may need to switch to bash
for them to install correctly
This pr introduces the container provider. The first supported bind is Docker, though a Podman bind is in progress. A number of other supporting concepts have also been introduced, most notably are "binds". These are small abstraction layers around an external API that ease the integration into Broker.
This pr introduces the container provider.
The first supported bind is Docker, though a Podman bind is in progress.
A number of other supporting concepts have also been introduced, most
notably are "binds". These are small abstraction layers around an
external API that ease the integration into Broker.