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

Use docker containers as hosts for testing netplugin #37

Merged
merged 11 commits into from
Apr 7, 2015

Conversation

sachja
Copy link
Contributor

@sachja sachja commented Apr 5, 2015

No description provided.

4. Add $GOPATH/dockerhost to PATH
5. Set CONTIV_NODES to required number of nodes
6. Run start-dockerhosts

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to specify, either here or in prerequisites, that nsenter must be installed for this to work.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another pre-requisite is having docker installed on host as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added

@@ -14,7 +14,7 @@ echo "Checking gofmt..."
fmtRes=$(gofmt -l ./)
if [ -n "${fmtRes}" ]; then
echo -e "gofmt checking failed:\n${fmtRes}"
exit 1
# exit 1

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why exit with '0' code when go-fmt fails? Any reason for this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry that got in by mistake. Reverted.

Btw if i do a make build two times without clean i get gofmt errors.

@sachja
Copy link
Contributor Author

sachja commented Apr 6, 2015

i have pushed the changes handling all the comments . Pls take a relook .

@@ -112,6 +112,9 @@ In the examples directory [two_hosts_multiple_tenants.json](examples/two_hosts_m
[two_hosts_multiple_tenants_mix_vlan_vxlan.json](examples/two_hosts_multiple_tenants_mix_vlan_vxlan.json) shows the creation of a multi-tenant
(disjoint, overlapping) networks within a cluster.

####Trying the multi-host tests on a single machine using docker as hosts
If you cannot launch VM on your host one can also test the multi-host network by simulating hosts using docker containers. Please see [docs/Dockerhost.md](docs/Dockerhost.md) for instructions.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sachja overall LGTM
Suggest replacing If you cannot... with If you cannot launch a VM on your host, especially when the host is a VM itself, you can also...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

-------------
The following needs to be installed on the host machine
1. Docker
2. nsenter

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leave a pointer to nsenter repo so someone can get going)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@jainvipin
Copy link

@sachja - LGTM

sachja added a commit that referenced this pull request Apr 7, 2015
Use docker containers as hosts for testing netplugin
@sachja sachja merged commit d90fa5c into contiv:master Apr 7, 2015
dseevr pushed a commit to dseevr-dev/netplugin that referenced this pull request Nov 28, 2017
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.

None yet

3 participants