Repository navigation
fix: start the stack from a fresh clone - #23
Closed
YvesLepidus wants to merge 1 commit into
Closed
YvesLepidus wants to merge 1 commit into
YvesLepidus wants to merge 1 commit into
Conversation
Signed-off-by: YvesCesar <yvesamorim73@gmail.com>
Member
|
Superseded by #24. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
On a fresh clone of
main,make upfails for the site and Nextcloud stacks, and the WordPress → Nextcloud integration cannot reach Nextcloud.Changes
SITE_COMPOSEnow usesenv UID=... docker compose .... In bash, which is/bin/shon Fedora,UIDis readonly, soUID=... docker composefailed withUID: readonly variable.mysqlwas renamed todatabasein_refresh-nextcloud-imagesand_start-nextcloud. Thenextcloud-developmentbump in 7f556a7 renamed the service, which made the Makefile fail withno such service: mysql.IP_BIND=0.0.0.0was added toNEXTCLOUD_COMPOSE. The same bump made ports bind to127.0.0.1by default, so WordPress could no longer reach Nextcloud throughhost.docker.internal:8082. This restores the previous behavior._set-trusted-domainsstep addshost.docker.internalto Nextcloud'strusted_domains. Without it, requests from WordPress gotTrusted domain error.Validation
Tested with
make upon a fresh clone (Fedora 44, bash 5.3)::80), the site (:8081) and Nextcloud (:8082) all respond 200.wordpress_login_backend,admin_group_managerandgroupquotaare enabled, and theadmlibrecodegroup is created.curl http://host.docker.internal:8082/status.phpfrom the WordPress container returns 200.Notes
IP_BIND=0.0.0.0, the Nextcloud database port (3307) is also published on all interfaces.woocommerce-nextcloud-admin-group-managerrequires WordPress 7.0, butwordpress-docker:latestships 6.9.4, so the plugin is not activated.make upignores this error.