Skip to content

Commit 215234e

Browse files
committed
Ask user login in make_images script. Remove testgres installation from container
1 parent 09fcbdc commit 215234e

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

Dockerfile.tmpl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ RUN if [ "${CHECK_CODE}" = "cppcheck" ] ; then \
1414
RUN if [ "${CHECK_CODE}" = "false" ] ; then \
1515
echo 'http://dl-3.alpinelinux.org/alpine/edge/main' > /etc/apk/repositories; \
1616
apk --no-cache add curl python3 gcc make musl-dev cmocka-dev;\
17-
pip3 install testgres; \
1817
fi
1918

2019
RUN mkdir -p /pg/data && \

make_images.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#!/usr/bin/env python
22

33
import subprocess
4+
import getpass
45

56
DOCKER_ID = 'pathman'
67
pg_versions = ['9.5','9.6','10']
@@ -17,11 +18,12 @@
1718
}
1819
}
1920

20-
password = input("Enter password for `docker login` for user `%s`: " % DOCKER_ID)
21+
user = input("Enter username for `docker login`: ")
22+
password = getpass.getpass()
2123
subprocess.check_output([
2224
'docker',
2325
'login',
24-
'-u', DOCKER_ID,
26+
'-u', user,
2527
'-p', password])
2628

2729
travis_conf_line = '- DOCKER_IMAGE=%s'

run_tests.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@ elif [ "$CHECK_CODE" = "cppcheck" ]; then
3535
exit $status
3636
fi
3737

38+
# we need testgres for pathman tests
39+
pip3 install testgres
40+
pip3 freeze | grep testgres
41+
3842
# don't forget to "make clean"
3943
make USE_PGXS=1 clean
4044

0 commit comments

Comments
 (0)