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

docker network connect network id on a running container does not return an error #8051

Open
hickeng opened this issue Jun 14, 2018 · 0 comments
Labels
component/portlayer/execution component/portlayer/network impact/doc/user Requires changes to official user documentation kind/enhancement Behavior that was intended, but we want to make better

Comments

@hickeng
Copy link
Member

hickeng commented Jun 14, 2018

This issue is for returning an appropriate error message if attempting to connect a running container to a network.

It may be that the appropriate implementation of this is is a general one, returning an error from Commit if any guestinfo keys that are not tagged with @non-persistent are present.

Example of current behaviour:

$ docker run -dit --net=bridge alpine /bin/ash
c7abb78201ba357b7edd9c653099601a5e280239670c651b15ce8f7872b6d662
$ docker network connect other-bridge c7
$ echo $?
0
$

Notes:
For support of docker network connect against a container that is:

  • running
  • connected to a bridge network
  • connecting to another bridge network

We must have either of:

  • Switch from datastore files to namespace usage #3286 and at least migrating the use of extraconfig so that we can have non-volatile keys set when VM is powered on
  • allowing disparate behaviour between vSphere versions with the volatile key bug fixed vs those without

#743 is for implementing this scenario.

For this to be viable in any of the other scenarios we need to enable hotadd for vNIC in the containerVMs.

@hickeng hickeng added kind/enhancement Behavior that was intended, but we want to make better component/portlayer/network impact/doc/user Requires changes to official user documentation labels Jun 14, 2018
@hickeng hickeng changed the title docker network connect bridge-net id on a running container that is already on a bridge network does not set IP in container docker network connect bridge-net id on a running container does not return an error Jun 14, 2018
@hickeng hickeng changed the title docker network connect bridge-net id on a running container does not return an error docker network connect network id on a running container does not return an error Jun 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/portlayer/execution component/portlayer/network impact/doc/user Requires changes to official user documentation kind/enhancement Behavior that was intended, but we want to make better
Projects
None yet
Development

No branches or pull requests

1 participant