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

add docker-compose #105

Merged
merged 10 commits into from
Mar 20, 2021
Merged

Conversation

jbutler992
Copy link
Contributor

@jbutler992 jbutler992 commented Mar 18, 2021

This adds docker-compose capabilities

Includes nginx proxy for TLS termination

Includes postgresql per the getting started suggestions

@mattermod
Copy link
Contributor

Hello @jbutler992,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

@jbutler992
Copy link
Contributor Author

/check-cla

@jbutler992
Copy link
Contributor Author

Addresses #96 . Will leave it up to repository owner to determine if it closes the issue

docker-compose.yml Outdated Show resolved Hide resolved
@jbutler992
Copy link
Contributor Author

This check in test failure is also occurring on main. Once that's fixed I'll update with main again so the tests re-run

@jespino
Copy link
Contributor

jespino commented Mar 19, 2021

@jbutler992 it should be fixed now :)

@jbutler992
Copy link
Contributor Author

@jespino thanks, updated and passing now

"useSSL": false,
"webpath": "./pack",
"filespath": "./files",
"telemetry": true,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: a bit of indentation problem here.

Copy link
Contributor

@jespino jespino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks!

@jespino jespino merged commit 8251f8c into mattermost-community:main Mar 20, 2021
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.

5 participants