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

Use alpine images #137

Merged
merged 3 commits into from
Nov 11, 2018
Merged

Use alpine images #137

merged 3 commits into from
Nov 11, 2018

Conversation

McSneaky
Copy link
Contributor

No description provided.

@mattrobenolt
Copy link
Contributor

I'm ok with the memcached change, but I'd rather not recommend Postgres on alpine. So if you wanna back that change out, I'll merge memcached.

@McSneaky
Copy link
Contributor Author

Reverted it back to regular image

I have been using Postgres Alpine image for quite some time in production and have not noticed any errors / anomalies.
But since I have not run any tests I can't be 100% sure it works the same as non Alpine version

@mattrobenolt mattrobenolt merged commit cd13427 into getsentry:master Nov 11, 2018
@mattrobenolt
Copy link
Contributor

The primary problem with alpine vs Debian images in Postgres is interapability with the data on disk. If you change from a Debian based image to Alpine or vice versa, you will corrupt your indices resulting in very confusing behaviors and unexpected results.

If you use Alpine for Postgres, you need to make sure you literally never switch to another distro or change the version of musl, and I’m not sure how often this happens in alpine world.

astagi added a commit to lotrekagency/onpremise that referenced this pull request Sep 12, 2019
* Use alpine images (getsentry#137)

* Use alpine images

* Updated memcahced version from 1.4. to 1.5

* Reverted Postgres back to regular image

* Update Dockerfile to get latest image (getsentry#162)

* Add Minimum Hardware Requirements (getsentry#165)

* Add Nginx

* Fix conf.d path

* Fix nginx with auto certificate creation
astagi added a commit to lotrekagency/onpremise that referenced this pull request Jan 8, 2020
* Use alpine images (getsentry#137)

* Use alpine images

* Updated memcahced version from 1.4. to 1.5

* Reverted Postgres back to regular image

* Update Dockerfile to get latest image (getsentry#162)

* Add Minimum Hardware Requirements (getsentry#165)

* Add Nginx

* Fix conf.d path

* Fix nginx

* Fix nginx with cerbot

* Add restart script

* Demonize up

Co-authored-by: McSneaky <kevin@synter.ee>
Co-authored-by: Pieter <pietering1@gmail.com>
Co-authored-by: Omid Raha <omidraha@users.noreply.github.com>
MaicolBen pushed a commit to hinthealth/onpremise that referenced this pull request Jul 20, 2020
* Use alpine images

* Updated memcahced version from 1.4. to 1.5

* Reverted Postgres back to regular image
@github-actions github-actions bot locked and limited conversation to collaborators Dec 14, 2020
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