Skip to content
This repository has been archived by the owner on May 20, 2022. It is now read-only.

DOPS-215: Add MM_INSTALL_TYPE=docker environment variable. #497

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

metanerd
Copy link
Contributor

@metanerd metanerd commented Oct 2, 2020

Summary

Sets an environment variable to get an idea of the preferred installation type.

Ticket Link

https://mattermost.atlassian.net/browse/DOPS-215

@metanerd metanerd requested a review from cpanato October 2, 2020 15:00
@cpanato
Copy link
Contributor

cpanato commented Oct 6, 2020

after this gets merged can you cherry pick to branches 5.26/ 5.27 ? thanks!

@cpanato
Copy link
Contributor

cpanato commented Oct 6, 2020

did you validate when you start the docker container if this env is set there? @metanerd

@metanerd
Copy link
Contributor Author

metanerd commented Oct 6, 2020

@cpanato

I did not, but it looks good:

$ docker exec -it mattermost-docker_app_1 sh
~ $ printenv
MM_PASSWORD=mmuser_password
MM_VERSION=5.27.0
HOSTNAME=e126a4458346
SHLVL=1
HOME=/mattermost
MM_INSTALL_TYPE=docker
...

Copy link
Contributor

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

thanks!

@cpanato cpanato merged commit 1f5c1b2 into master Oct 7, 2020
@cpanato cpanato deleted the DOPS-215 branch October 7, 2020 12:06
metanerd added a commit that referenced this pull request Oct 8, 2020
braunsonm added a commit to trecnoc/mattermost-docker that referenced this pull request Feb 8, 2021
* bump MM to 5.25.0

* Remove sudo: required as it is deprecated. (mattermost#485)

Co-authored-by: francois-d <dfrancis2.primus.ca>

* update MM to 5.25.1

* bump mm to 5.25.2

* add TLS 1.3 support to web container (mattermost#488)

* bump MM to 5.26.0

* bump MM to 5.26.1

* update MM to 5.27.0

* Quote arguments (mattermost#492)

* DOPS-215: Add MM_INSTALL_TYPE environment variable. (mattermost#497)

* bump based release

* Update SSL ciphers and TLS versions in nginx config file (mattermost#501)

* update MM to 5.29.0

* Consistency of app/* (mattermost#502)

* be consistent with the use of tabs

* tabs; shellcheck; consistency

* use tabs
* use shellcheck to fix non POSIX stuff
* be consistent with the use of quotes and if-then-else / for-do-done

* bump to use 5.29.1 release

* Enable easy configuration of encrypted PostgreSQL connections with new optional DB_SSLMODE (defaults to current value of "disable") following values allowed by PostgreSQL (mattermost#506)

* change to support environment variable configuration of SSL mode for PostgreSQL connections

* Update Dockerfile (mattermost#508)

* Updating shell scripts to adhear better to Bash style guides (mattermost#510)

* Updated Shell Script Format

Signed-off-by: Spencer <wf6DJd8a3xSSCZbn@protonmail.com>

* Update Dockerfile (mattermost#511)

Co-authored-by: Carlos Panato <ctadeu@gmail.com>
Co-authored-by: Francois-D <46973576+Francois-D@users.noreply.github.com>
Co-authored-by: nvjacobo <nvjacobo@users.noreply.github.com>
Co-authored-by: Emanuele Panzeri <thePanz@users.noreply.github.com>
Co-authored-by: Elisabeth Kulzer <elikul@elikul.de>
Co-authored-by: Simon Staszkiewicz <sstaszkiewicz@copperleaf.com>
Co-authored-by: Yannic Haupenthal <tohn@users.noreply.github.com>
Co-authored-by: Dave Thompson <64275573+3leapsdet@users.noreply.github.com>
Co-authored-by: Amy Blais <amy_blais@hotmail.com>
Co-authored-by: wf6DJd8a3xSSCZbn <50090275+wf6DJd8a3xSSCZbn@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants