Skip to content

Add libnetwork agent mode support#1149

Merged
sanimej merged 2 commits intomoby:masterfrom
mrjana:agent
May 5, 2016
Merged

Add libnetwork agent mode support#1149
sanimej merged 2 commits intomoby:masterfrom
mrjana:agent

Conversation

@mrjana
Copy link
Copy Markdown
Contributor

@mrjana mrjana commented May 2, 2016

libnetwork agent mode is a mode where libnetwork can act as a local
agent for network and discovery plumbing alone while the state
management is done elsewhere. This completes the support for making
libnetwork and its associated drivers to be completely independent of a
k/v store(if needed) and work purely based on the state information
passed along by some some external controller or manager. This does not
mean that libnetwork support for decentralized state management via a
k/v store is removed.

Signed-off-by: Jana Radhakrishnan mrjana@docker.com

@mavenugo
Copy link
Copy Markdown
Contributor

mavenugo commented May 2, 2016

@mrjana before starting the review. I just wanted to express my 👏

libnetwork agent mode is a mode where libnetwork can act as a local
agent for network and discovery plumbing alone while the state
management is done elsewhere. This completes the support for making
libnetwork and its associated drivers to be completely independent of a
k/v store(if needed) and work purely based on the state information
passed along by some some external controller or manager. This does not
mean that libnetwork support for decentralized state management via a
k/v store is removed.

Signed-off-by: Jana Radhakrishnan <mrjana@docker.com>
@mrjana mrjana force-pushed the agent branch 2 times, most recently from ae19464 to ef2dda7 Compare May 3, 2016 02:48
@aboch
Copy link
Copy Markdown
Contributor

aboch commented May 5, 2016

LGTM

Comment thread api/api.go Outdated
setFctList = append(setFctList, libnetwork.CreateOptionAlias(name, alias))
}

log.Printf("Entering api join ...")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I am guessing these logs were added for debugging. If so you may want to remove them.

Signed-off-by: Jana Radhakrishnan <mrjana@docker.com>
@mrjana
Copy link
Copy Markdown
Contributor Author

mrjana commented May 5, 2016

@aboch Removed the unnecessary debug logs.

@sanimej
Copy link
Copy Markdown

sanimej commented May 5, 2016

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants