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

docker macvlan documentation #510

Merged
merged 5 commits into from
Jun 29, 2020
Merged

docker macvlan documentation #510

merged 5 commits into from
Jun 29, 2020

Conversation

stephdl
Copy link
Contributor

@stephdl stephdl commented Jun 14, 2020

documentation for the macvlan creation

@stephdl stephdl requested a review from mrmarkuz June 14, 2020 18:47
administrator-manual/en/docker.rst Show resolved Hide resolved
administrator-manual/en/docker.rst Outdated Show resolved Hide resolved
Copy link
Member

@mrmarkuz mrmarkuz left a comment

Choose a reason for hiding this comment

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

I found no errors but I noticed that the enableRepository db prop is missing (maybe to avoid autoupdate issues?)

@stephdl
Copy link
Contributor Author

stephdl commented Jun 16, 2020

thank @mrmarkuz, I missed it, the prop enableRepository is back

Comment on lines 139 to 141

The aeria network is not standard on docker, the developer can be contacted at https://github.com/devplayer0/docker-net-dhcp

Copy link
Member

Choose a reason for hiding this comment

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

I'm not still convinced that this should go on the manual. It's an implementation detail, let's move to the README of the package.

Suggested change
The aeria network is not standard on docker, the developer can be contacted at https://github.com/devplayer0/docker-net-dhcp

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed of the official documentation and added to the readme : NethServer/nethserver-docker#19

@stephdl stephdl requested a review from gsanchietti June 20, 2020 07:03
@gsanchietti gsanchietti merged commit a60d1c2 into NethServer:master Jun 29, 2020
gsanchietti pushed a commit that referenced this pull request Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants