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

Ensure folders exist to allow installation #2

Merged
merged 11 commits into from
Jul 10, 2019
Merged

Ensure folders exist to allow installation #2

merged 11 commits into from
Jul 10, 2019

Conversation

fooman
Copy link
Contributor

@fooman fooman commented Jul 9, 2019

bin/magento setup:install checks for writeability of a few folders. Syncing just the .htaccess file allows them to be created.

@davidalger davidalger changed the base branch from master to develop July 10, 2019 15:54
@davidalger davidalger merged commit ee0d385 into wardenenv:develop Jul 10, 2019
@davidalger
Copy link
Collaborator

This has been included in release 0.1.3. Thanks for the PR!

@davidalger
Copy link
Collaborator

davidalger commented Jul 10, 2019

Documenting the testing process I used to validate this indeed fixes the issue for posterity's sake:

  1. Install codebase on host machine

     composer create-project --ignore-platform-reqs --repository-url=https://repo.magento.com/ \
         magento/project-community-edition m2test 2.3.2
    
  2. Start the project environment

     cd m2test
     
     warden env-init m2test
     warden sign-certificate m2test.test
     warden env up -d
     warden sync start
    
  3. Launch a shell session and install Magento

     warden shell
    
     ## Run Magento install routine
     bin/magento setup:install \
         --backend-frontname=backend \
         --amqp-host=rabbitmq \
         --amqp-port=5672 \
         --amqp-user=guest \
         --amqp-password=guest \
         --db-host=db \
         --db-name=magento \
         --db-user=magento \
         --db-password=magento \
         --http-cache-hosts=varnish:80 \
         --session-save=redis \
         --session-save-redis-host=redis \
         --session-save-redis-port=6379 \
         --session-save-redis-db=2 \
         --session-save-redis-max-concurrency=20 \
         --cache-backend=redis \
         --cache-backend-redis-server=redis \
         --cache-backend-redis-db=0 \
         --cache-backend-redis-port=6379 \
         --page-cache=redis \
         --page-cache-redis-server=redis \
         --page-cache-redis-db=1 \
         --page-cache-redis-port=6379
    
     ## Generate an admin user
     ADMIN_PASS="$(cat /dev/urandom | base64 | head -n1 | sed 's/[^a-zA-Z0-9]//g' | colrm 17)"
     ADMIN_USER=localadmin
     bin/magento admin:user:create \
         --admin-password="${ADMIN_PASS}" \
         --admin-user="${ADMIN_USER}" \
         --admin-firstname="Local" \
         --admin-lastname="Admin" \
         --admin-email="${ADMIN_USER}@example.com"
     printf "u: %s\np: %s\n" "${ADMIN_USER}" "${ADMIN_PASS}"
    
  4. Open site in browser at following URLs:

davidalger pushed a commit that referenced this pull request Apr 16, 2021
Merge current development branch
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.

2 participants