Skip to content

Merge master into release-1.1 #1044

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

Merged
merged 98 commits into from
Nov 1, 2017
Merged

Merge master into release-1.1 #1044

merged 98 commits into from
Nov 1, 2017

Conversation

dseevr
Copy link
Contributor

@dseevr dseevr commented Nov 1, 2017

No description provided.

Erik Hollensbe and others added 30 commits December 3, 2015 14:56
Signed-off-by: Erik Hollensbe <[email protected]>
Signed-off-by: Erik Hollensbe <[email protected]>
Reorganize objdb and add basic consul support
Fix a typo

Merging this small change in a hurry so that I can pick it up in netplugin
move mutex to global level
Get current lock holder by reading from statestore
Move away from using etcd modifiedIndex in compare-and-swap
etcdService.go: add error checks for host parsing in service watch
dseevr and others added 29 commits October 20, 2017 16:28
Adding debugs in netplugin/scripts/build.sh
Without this patch, when a node first time start netplugin, and if
there wasn't netmaster started ever on the node, the ovs logging
directory will not exist and no logging will be able to be found.
This patch makes sure the ovs logging directory created before
service starts.
Also added a wait step before ovsdb ready, and logging level.
changes the indentation to 4 spaces and make codes more readable

Signed-off-by: Wei Tie <[email protected]>
* Removed GOPATH check

Signed-off-by: Vikram Hosakote <[email protected]>

* Removed doc

Signed-off-by: Vikram Hosakote <[email protected]>
etcdClient.go: fixed a bug where nil was returned instead of err
'devbuild' moniker is not needed, that is a contiv/install default
convention

Simplify the tag name to just the SHA, which ties the image tag to the
actual contents, and not be variable based on a build time environment variable

* Added some comments
* compile-with-docker was not using the tag variable

Signed-off-by: Chris Plock <[email protected]>
git-subtree-dir: objdb
git-subtree-mainline: 83e1bda
git-subtree-split: 77da460
…spell` was complaining about it

Signed-off-by: Bill Robinson <[email protected]>
Subtree objdb repo into netplugin
objdb/modeldb/modeldb.go: fix import path
…db import paths

This also pulled in the changes from contiv-experimental/contivmodel#68 which had not previously been included.

Signed-off-by: Bill Robinson <[email protected]>
image tag is just sha, rm 'devbuild' default
Godeps,vendor: update contivmodel to the latest version + use new objdb import paths
There are no vendored changes with this commit.  This is necessary to resolve an issue with Godeps where it barfs on dependency detection because "client-go" had an empty Godeps.json

Signed-off-by: Bill Robinson <[email protected]>
Godeps/Godeps.json: update client-go version
objdb: remove now-unused Makefile and Vagrantfile
Remove test/systemtests/cfg.json so that it will be ignored
* Contiv network is not deleted when docker network is deleted

Steps to reproduce:
1. Create a docker network with docker network CLI
2. Ensure docker network and contiv network are created (docker network ls and netctl network ls)
3. Remove docker network.
4. Check docker network ls, docker network is deleted
5. Check netctl networ ls, contiv network is still test.

The root cause is due to the dockernetworkstate mapping is planned to delete after
contiv network is delete. However, the determination of contiv network deletion is
determined by the existance of the dockernetworkstate. Thefore, if the state is still
there, the contiv network will be unable to delete.

The fix is to delete the dockernetworkstate before proceed to contiv network deletion

Signed-off-by: Kahou Lei [email protected]

* Print out error

* Add unit test and also refactor some code to make the code more testable.

* Add more unit test and fix typos

* Add more testing

* Add key value pair in fake state

* Refactor base on comments

* Add more check in unit test.

Take out return statement when docker network is not there.
support ip-ip policy in EPG.
Signed-off-by: Vikram Hosakote <[email protected]>
Adding test/systemtests/cfg.json.example
@dseevr dseevr merged commit 4e216bf into release-1.1 Nov 1, 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.