Skip to content

Allow to pass HostConfig when creating a container#105

Merged
marcuslinke merged 2 commits intodocker-java:masterfrom
vjuranek:hostconfig
Dec 2, 2014
Merged

Allow to pass HostConfig when creating a container#105
marcuslinke merged 2 commits intodocker-java:masterfrom
vjuranek:hostconfig

Conversation

@vjuranek
Copy link
Copy Markdown
Contributor

@vjuranek vjuranek commented Dec 1, 2014

HostConfig should be passed when creating the container since API v1.15.

  • Move HostConfig into separate class
  • Use Links for container links instead of String
  • Add API for passing HostConfig when creating a container
  • Fix parsing of Links and Link

* Move HostConfig into separate class
* Use Links for container links instead of String
* Add API for passing HostConfig when creating a container
@gesellix
Copy link
Copy Markdown
Contributor

gesellix commented Dec 1, 2014

@gesellix
Copy link
Copy Markdown
Contributor

gesellix commented Dec 1, 2014

there's a note about HostConfig being deprecated when starting a container, directly above the v.1.15 section:

New! Passing the container's HostConfig on start is now deprecated. You should set this when creating the container.

@albers
Copy link
Copy Markdown
Contributor

albers commented Dec 1, 2014

@gesellix I was referring to /name1:/name2/alias as a new syntax for defining container links.

@gesellix
Copy link
Copy Markdown
Contributor

gesellix commented Dec 1, 2014

@albers sorry, didn't see that here. So, just ignore my comments :)

@albers
Copy link
Copy Markdown
Contributor

albers commented Dec 1, 2014

@gesellix Thanks anyway!

@albers
Copy link
Copy Markdown
Contributor

albers commented Dec 2, 2014

LGTM

marcuslinke added a commit that referenced this pull request Dec 2, 2014
Allow to pass HostConfig when creating a container
@marcuslinke marcuslinke merged commit 6527f4c into docker-java:master Dec 2, 2014
@marcuslinke
Copy link
Copy Markdown
Contributor

@vjuranek Thanks for the contribution!

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.

4 participants