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

[WIP] App-agnostic / Laravel support #1063

Closed
wants to merge 1 commit into from
Closed

[WIP] App-agnostic / Laravel support #1063

wants to merge 1 commit into from

Conversation

retlehs
Copy link
Member

@retlehs retlehs commented Jan 20, 2019

follow this branch/PR as we work on making trellis more app-agnostic and add laravel support out of the box

todo

  • docs updates
  • wordpress separation/tagging
  • laravel support

ref #914, #951

@vesper8
Copy link

vesper8 commented Mar 27, 2019

any progress on this? would love to have it!

@wassim
Copy link

wassim commented Jun 25, 2019

@retlehs is this still on the roadmap?

@retlehs
Copy link
Member Author

retlehs commented Jun 26, 2019

yup! but this comment also applies to this scenario:

#1085 (comment)

Right now there is not anyone that has expressed interest or has the free time to keep up with this.

Want to help? That would be great and appreciated 🌻 Want to financially support our OSS projects so we can help get people paid to fix things like this? We have a Patreon 🌱

@vesper8
Copy link

vesper8 commented Aug 2, 2019

is there a fork somewhere that uses a recent build of Trellis with added support for Laravel? Would love to test it out!

@pepijn-vanvlaanderen
Copy link

is there a fork somewhere that uses a recent build of Trellis with added support for Laravel? Would love to test it out!

Actually done this a few times, I created a fork of the current Trellis repo and edited it for a new project. You can find it here: https://github.com/pepijn-vanvlaanderen/trellis-laravel

@vesper8
Copy link

vesper8 commented Aug 10, 2019

wow thanks a lot @pepijn-vanvlaanderen ! that's perfect !

Does it also work as far as setting up supervisor tasks for horizon/websockets automatically?

@vesper8
Copy link

vesper8 commented Oct 11, 2019

@pepijn-vanvlaanderen

I've begun using your fork and I'm running into an error while provisioning a server.

TASK [redis : create /etc/redis] *********************************************************************************************
changed: [POC-001]

TASK [redis : check if redis user exists (ignore errors)] ********************************************************************
System info:
  Ansible 2.5.3; Darwin
  Trellis Head
---------------------------------------------------
non-zero return code
id: 'redis': no such user
fatal: [POC-001]: FAILED! => {"changed": false, "cmd": ["id", "redis"], "delta": "0:00:00.004985", "end": "2019-10-11 11:12:13.259672", "rc": 1, "start": "2019-10-11 11:12:13.254687", "stderr_lines": ["id: 'redis': no such user"], "stdout": "", "stdout_lines": []}
...ignoring

TASK [redis : add redis group] ***********************************************************************************************
changed: [POC-001]

TASK [redis : add redis user] ************************************************************************************************
changed: [POC-001]

Perhaps it's just a warning.. it's checking if the user exists before it's actually had a chance to create the user.. Have you experienced this as well?

@knowler
Copy link
Member

knowler commented Oct 11, 2019

@vesper8 Please make an issue on the fork not here.

@vesper8
Copy link

vesper8 commented Oct 12, 2019

@knowler I would have of course, but the fork does not allow issues to be created

@retlehs retlehs marked this pull request as draft November 12, 2020 20:02
@swalkinshaw
Copy link
Member

Closing this and #1123 because they are both quite old and out of date now. I'm open to someone contributing a newer PR to accomplish this but please post in either #914 or #951 first to discuss the solution.

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.

6 participants