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

securityonion-elastic: Docker daemon.json conflict #1674

Closed
dougburks opened this issue Dec 10, 2019 · 4 comments
Closed

securityonion-elastic: Docker daemon.json conflict #1674

dougburks opened this issue Dec 10, 2019 · 4 comments
Assignees
Projects

Comments

@dougburks

This comment has been minimized.

Copy link
Contributor Author

@dougburks dougburks commented Dec 12, 2019

https://github.com/Security-Onion-Solutions/securityonion-elastic/blob/master/usr/sbin/so-elastic-configure-network

This script creates a /etc/daemon/docker.json with default configuration as follows:

{
  "bip": "172.17.0.1/24"
}

On some boxes, this file is causing docker to fail. Removing the file allows docker to start.

One option would be to update /usr/sbin/so-elastic-configure-network such that it checks to see if /etc/daemon/docker.json exists and has an MD5 of 0bcf8587dea814c9b712064c28da3d82. If so, remove the file. If not, then the user has modified the file and it should be retained. Additionally, /usr/sbin/so-elastic-configure-network can continue to include the code that updates /etc/daemon/docker.json with the DOCKER_BRIDGE from /etc/nsm/securityonion.conf (but only if it doesn't match the MD5 above).

@dougburks dougburks added this to To do in 16.04.6.4 via automation Dec 23, 2019
@dougburks dougburks moved this from To do to In progress in 16.04.6.4 Dec 23, 2019
@dougburks dougburks changed the title Docker daemon.json conflict securityonion-elastic: Docker daemon.json conflict Dec 23, 2019
@dougburks dougburks self-assigned this Dec 23, 2019
@dougburks

This comment has been minimized.

Copy link
Contributor Author

@dougburks dougburks commented Dec 24, 2019

@dougburks dougburks moved this from In progress to In Testing in 16.04.6.4 Jan 4, 2020
@weslambert

This comment has been minimized.

Copy link
Collaborator

@weslambert weslambert commented Jan 17, 2020

Looks good 👍

@weslambert weslambert moved this from In Testing to Tested in 16.04.6.4 Jan 17, 2020
@dougburks

This comment has been minimized.

@dougburks dougburks closed this Feb 5, 2020
16.04.6.4 automation moved this from Tested to Done Feb 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
16.04.6.4
  
Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.