Skip to content
This repository was archived by the owner on Aug 26, 2022. It is now read-only.

Conversation

@aschmois
Copy link
Contributor

Story details: https://app.clubhouse.io/itprotv/story/69522

Changes:

  • No longer creating haskell user at this level since we only need non-root complexity for developing purposes (tmp-postgres)
  • Use ubuntu base image
    • This is a change we wanted to do
    • We decided on doing it now because amazon's base completely lost postgresql in the current repo and trying to solve that puzzle was not worth the time

Andres Schmois and others added 2 commits April 14, 2021 15:54
co-authored-by: Cody <cody@itpro.tv>
co-authored-by: Cody <cody@itpro.tv>
@aschmois aschmois requested a review from a team April 14, 2021 19:56
ncurses-devel \
netcat-openbsd \
perl \
postgresql-12 \
Copy link
Contributor Author

@aschmois aschmois Apr 14, 2021

Choose a reason for hiding this comment

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

It's possible we can tighten this dependency down to just whatever tmp-postgres requires, but it's very difficult to figure out the exact package and we've sunk too much time to make it worth it. The only issue installing all of psql is that we'll have a server running for no reason in the dev toolbox. (and a bigger file size)

FROM amazonlinux:2.0.20200722.0
FROM ubuntu:focal-20210401

ARG DEBIAN_FRONTEND=noninteractive
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because we're installing postgresql we need to tell it that we don't care what the setup values are. We can remove this when we no longer install all of postgres.

@aschmois aschmois merged commit ee39faa into master Apr 15, 2021
@aschmois aschmois deleted the ch69522-use-ubuntu branch April 15, 2021 13:38
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.

3 participants