Skip to content

Update test images #40

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.

Sign up for GitHub

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

Merged
SaitTalhaNisanci merged 1 commit into from
Aug 26, 2020
Merged

Update test images #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions circleci/images/citusupgradetester/files/sbin/install-extdeps
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,12 @@ echo "deb http://apt.postgresql.org/pub/repos/apt/ ${codename}-pgdg main" > \
echo "Installing server-dev packages..." >&2
apt-get update && apt-get install -y --no-install-recommends postgresql-server-dev-$PG_MAJOR

# we need autoconf and build-essential for running the tests
# we need other libraries for configure to pass, even though we could use
# --without-libcurl while configuring, we prefer this for now.
apt-get install -y autoconf build-essential libcurl4-openssl-dev libicu-dev \
libreadline-dev libselinux1-dev libxslt-dev libssl-dev

# build list of postgres
echo "Installing postgres packages..." >&2
apt-get install -y --no-install-recommends postgresql-$PG_MAJOR
Expand Down
6 changes: 6 additions & 0 deletions circleci/images/exttester/files/sbin/install-testdeps
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,12 @@ EOF
echo "Installing tools for testing PostgreSQL extensions..." >&2
apt-get update && apt-get install -y --no-install-recommends make postgresql-common perl libcurl3 "postgresql-${PG_MAJOR}" "postgresql-server-dev-${PG_MAJOR}"

# we need autoconf and build-essential for running the tests
# we need other libraries for configure to pass, even though we could use
# --without-libcurl while configuring, we prefer this for now.
apt-get install -y autoconf build-essential libcurl4-openssl-dev \
libicu-dev libreadline-dev libselinux1-dev libxslt-dev libssl-dev

Comment on lines +61 to +66
Copy link
Copy Markdown
Contributor

@thanodnl thanodnl Aug 26, 2020

Choose a reason for hiding this comment

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

Was scared this could add a lot of data. Did an experiment by commenting out this line and check the layer size difference in docker image history. This showed these packages add only 2mb.

# Give permission for usr so that circleci can write citus artifacts
chmod o+w -R /usr/

Expand Down
7 changes: 7 additions & 0 deletions circleci/images/failtester/files/sbin/install-testdeps
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,13 @@ EOF
echo "Installing tools for testing PostgreSQL extensions..." >&2
apt-get update && apt-get install -y --no-install-recommends make postgresql-common \
perl libcurl3 "postgresql-${PG_MAJOR}" "postgresql-server-dev-${PG_MAJOR}"

# we need autoconf and build-essential for running the tests
# we need other libraries for configure to pass, even though we could use
# --without-libcurl while configuring, we prefer this for now.
apt-get install -y autoconf build-essential libcurl4-openssl-dev \
libicu-dev libreadline-dev libselinux1-dev libxslt-dev libssl-dev


echo "Installing tools for failure testing..." >&2
pip3 install -Ir /tmp/etc/requirements.txt
Expand Down
6 changes: 6 additions & 0 deletions circleci/images/pgupgradetester/files/sbin/install-testdeps
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,12 @@ sd_pkgs=( "${pg_majors[@]/#/postgresql-server-dev-}" )
pg_pkgs=( "${pg_majors[@]/#/postgresql-}" )
apt-get update && apt-get install -y --no-install-recommends make postgresql-common perl libcurl3 "${sd_pkgs[@]}" "${pg_pkgs[@]}"

# we need autoconf and build-essential for running the tests
# we need other libraries for configure to pass, even though we could use
# --without-libcurl while configuring, we prefer this for now.
apt-get install -y autoconf build-essential libcurl4-openssl-dev \
libicu-dev libreadline-dev libselinux1-dev libxslt-dev libssl-dev

echo "Installing tools for failure testing..." >&2
pip install -Ir /tmp/etc/requirements.txt

Expand Down