Ping @crosbymichael @tibor @LKD4D @unclejack

Note: legacy make test-integration seems to be broken, but I can't find the root cause at the moment. Presumably I upset an arcane hidden dependency somewhere... Any help would be appreciated.

Unit tests and CLI integration tests seem to pass well.

Pretty much fails :) I'll take a look.

I indeed, don't see why the MTU and network init configs should not be done inside NewDaemon.

Thanks @tiborvass, that was it.

For now I'm leaving a FIXME to cleanup how daemonconfig is managed. Now that there isn't
a initserver job to convert an env from, I don't think we even need a daemonconfig package
at all... But let's worry about that in a separate patch.

Ready for review.

This depends no #7496.

LGTM, but don't merge before #7496

Ping

LGTM

This has a commit (with a different commitid) from the PR #7496 that has already been merged. Not sure if it's a problem.

@tiborvass it should be removed I think

@shykes do you have time to rebase this PR?

Yes will do this afternoon

On Monday, August 11, 2014, Michael Crosby [email protected] wrote:

@shykes https://github.com/shykes do you have time to rebase this PR?

—
Reply to this email directly or view it on GitHub
#7497 (comment).

Rebased

@shykes gofmt failed

#@^%#*@&^@^$@ fixed

LGTM

ping @crosbymichael @vieux @unclejack @tiborvass

LGTM