Skip to content

Conversation

@tobybellwood
Copy link
Collaborator

We've moved the lagoon-cli image into the uselagoon namespace, this PR makes sure to pull that version (it's also M1 compatible)

@netlify
Copy link

netlify bot commented Dec 3, 2021

✔️ Deploy Preview for lando-lagoon ready!

🔨 Explore the source changes: e8524cb

🔍 Inspect the deploy log: https://app.netlify.com/sites/lando-lagoon/deploys/61b2f26c15d7a200087a5752

😎 Browse the preview: https://deploy-preview-16--lando-lagoon.netlify.app/404

@reynoldsalec
Copy link
Member

@tobybellwood do you have any idea what's causing these permission errors that seem to be borking the simple example test (https://github.com/lando/lagoon/runs/4461400707?check_suite_focus=true)? I haven't been able to replicate them on my local machine.

@tobybellwood
Copy link
Collaborator Author

I've had errors in actions too with the Lagoon images in Lando - will dig out where I got to.

@tobybellwood
Copy link
Collaborator Author

There's a Lando logs section here with a full set of startup logs
https://github.com/tobybellwood/drupal9-mariadb/runs/4147983910?check_suite_focus=true

I fiddled with the LANDO_WEBROOT_USER and LANDO_WEBROOT_GROUP to try to minimise the shuffling, but it still doesn't work

It looks like the HOST_GID isn't getting through to the perm_sweep command
perm_sweep $LANDO_WEBROOT_USER $(getent group "$LANDO_HOST_GID" | cut -d: -f1) $LANDO_RESET_DIR

cli_1      | userperms 04:36:39.70 INFO  ==> Doing the permission sweep.
cli_1      | chown: invalid group: ‘runner:/home’

@reynoldsalec
Copy link
Member

Weird, I'm mostly worried since we started having these failures in the last couple weeks, are these why you disabled Lando tests on the https://github.com/amazeeio/drupal-example-simple repo @tobybellwood?

@labboy0276
Copy link

labboy0276 commented Apr 21, 2022

Closing in favor of #31 to try and address the testing failures

@labboy0276 labboy0276 closed this Apr 21, 2022
This was referenced Apr 21, 2022
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.

3 participants