Skip to content

Docker deamon doesn't work with --fixed-cidr on windows#1916

Merged
fcrisciani merged 1 commit intomoby:masterfrom
msabansal:fixcidr
Dec 8, 2017
Merged

Docker deamon doesn't work with --fixed-cidr on windows#1916
fcrisciani merged 1 commit intomoby:masterfrom
msabansal:fixcidr

Conversation

@msabansal
Copy link
Copy Markdown
Contributor

Docker deamon fails to start when launched with --fixed-cidr parameter on windows.

This is because we are sending an invalid gateway address. Sending a gateway address was a temporary workaround which was added to prevent the daemon from crashing and is not needed any more so removing that.

@GordonTheTurtle
Copy link
Copy Markdown

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "fixcidr" git@github.com:msabansal/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Signed-off-by: Sandeep Bansal <sabansal@microsoft.com>
@fcrisciani fcrisciani self-requested a review August 18, 2017 21:22
Copy link
Copy Markdown

@fcrisciani fcrisciani left a comment

Choose a reason for hiding this comment

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

LGTM

@msabansal
Copy link
Copy Markdown
Contributor Author

msabansal commented Dec 8, 2017

@fcrisciani fcrisciani merged commit 822e5b5 into moby:master Dec 8, 2017
fcrisciani pushed a commit to fcrisciani/docker that referenced this pull request Dec 14, 2017
Windows fixes:
moby/libnetwork#1916
moby/libnetwork#2014
moby/libnetwork#2021

Fix for veth leak on daemon crash
Cleanup networkDB logic for node management

Signed-off-by: Flavio Crisciani <flavio.crisciani@docker.com>
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.

3 participants