search

nextcloud / all-in-one

📦 The official Nextcloud installation method. Provides easy deployment and maintenance with most features included in this one Nextcloud instance.
https://hub.docker.com/r/nextcloud/all-in-one
GNU Affero General Public License v3.0
5.07k stars 587 forks source link

update the ipv6 documentation #4976

Closed szaimen closed 1 month ago

szaimen commented 1 month ago

Close https://github.com/nextcloud/all-in-one/issues/4887

I've also already updated the docs in https://github.com/nextcloud/all-in-one/discussions/2045

Zoey2936 commented 1 month ago

network mode bridge should be added to the compose.yaml, because otherwise docker compose will create a new network just for the mastercontainer, if we add network mode bridge the mastercontainer would be added to the bridge network, like it alsp happens when the mastercontainer is created using the docker run command. Also the issue in the mentioned discussion was closed: https://github.com/moby/moby/issues/41651 https://github.com/nextcloud/all-in-one/discussions/2557

Zoey2936 commented 1 month ago

And I think instead of #2045 the discussion #2557 should be mentioned (or we directly add a guide on how to recreate the network)

szaimen commented 1 month ago

Can you maybe make some suggestions to the code or commit your ideas directly?

Zoey2936 commented 1 month ago

not possible because of deleted lines or unchanged lines

szaimen commented 1 month ago

Committing to the branch is also not possible?

Zoey2936 commented 1 month ago

done and #2557 should be rewritten since the issue was closed with docker v26

szaimen commented 1 month ago

done

Thanks :)

and #2557 should be rewritten since the issue was closed with docker v26

What exactly do I need to change there?

Zoey2936 commented 1 month ago

and #2557 should be rewritten since the issue was closed with docker v26

What exactly do I need to change there?

the mentioned issue was closed

szaimen commented 1 month ago

and #2557 should be rewritten since the issue was closed with docker v26

What exactly do I need to change there?

the mentioned issue was closed

Yes it was closed but it doesnt look like the issue is fixed based on the conments?

Zoey2936 commented 1 month ago

but they seem to talk about ipv6 only containers?

szaimen commented 1 month ago

and #2557 should be rewritten since the issue was closed with docker v26

What exactly do I need to change there?

the mentioned issue was closed

Yes it was closed but it doesnt look like the issue is fixed based on the conments?

I closed the discussion since after testing this a bit I actually cannot reproduce the problem anymore :)

Zoey2936 commented 1 month ago

and #2557 should be rewritten since the issue was closed with docker v26

What exactly do I need to change there?

the mentioned issue was closed

Yes it was closed but it doesnt look like the issue is fixed based on the conments?

I closed the discussion since after testing this a bit I actually cannot reproduce the problem anymore :)

Can you then create a new discussion/wiki with a guide on how to recreate the nextcloud-aio network?

szaimen commented 1 month ago

and #2557 should be rewritten since the issue was closed with docker v26

What exactly do I need to change there?

the mentioned issue was closed

Yes it was closed but it doesnt look like the issue is fixed based on the conments?

I closed the discussion since after testing this a bit I actually cannot reproduce the problem anymore :)

Can you then create a new discussion/wiki with a guide on how to recreate the nextcloud-aio network?

done

szaimen commented 1 month ago

Thanks Zoey! :)