Skip to content

Conversation

@labboy0276
Copy link

@labboy0276 labboy0276 commented Apr 21, 2022

Recreated from PR #16

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 Apr 21, 2022

Deploy Preview for lando-lagoon ready!

Name Link
🔨 Latest commit 0f6ce91
🔍 Latest deploy log https://app.netlify.com/sites/lando-lagoon/deploys/626160580659be00096d9669
😎 Deploy Preview https://deploy-preview-31--lando-lagoon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@labboy0276
Copy link
Author

@tobybellwood I moved your code so I could adjust some stuffs so our tests will pass. However, I am still having an issue as you can see here: https://github.com/lando/lagoon/runs/6113182627?check_suite_focus=true

Any thoughts on to why it can't find the .lagoon.yml in the repo with lagoon cli now?

@tobybellwood
Copy link
Collaborator

Okies, will have a look today! It shouldn't have changed...

@tobybellwood
Copy link
Collaborator

ooh - there was a change in the way that lagoon-cli worked. We, in the dockerfile, touch an empty /root/.lagoon.yml (https://github.com/uselagoon/lagoon-cli/blob/main/Dockerfile#L22) to make sure it boots, but Lando overwrites the entire /root/ directory, so you'll need to make sure there is a .lagoon.yml file the root of the directory that's mounted in)
https://uselagoon.github.io/lagoon-cli/#running-as-a-docker-image

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