From 82b9c3b753da2d5607ffd8ccf9da44c028199b3e Mon Sep 17 00:00:00 2001 From: Lukas Javorsky Date: Thu, 15 Jan 2026 12:21:52 +0000 Subject: [PATCH] The repository is available in https://github.com/sclorg organization. Images are pushed to quay.io/fedora organization from the upstream repo. --- .gitignore | 0 12 | 1 - Dockerfile | 83 -- Dockerfile.fedora | 1 - README.md | 1 - common/.travis.yml | 4 - common/LICENSE | 202 --- common/Makefile | 46 - common/README.md | 126 -- common/build.sh | 226 --- common/check_imagestreams.py | 73 - common/clean.sh | 21 - common/common.mk | 150 -- common/generate.sh | 184 --- common/run-shellcheck.sh | 44 - common/squash.py | 23 - common/tag.sh | 34 - common/test-lib-openshift.sh | 1278 ----------------- common/test-lib-remote-openshift.sh | 107 -- common/test-lib.sh | 901 ------------ common/test-remote-cluster.sh | 38 - common/test.sh | 67 - common/tests/.gitignore | 3 - common/tests/check_imagestreams.sh | 9 - common/tests/failures/check/Makefile | 4 - common/tests/failures/check/common | 1 - common/tests/failures/check/v0/Dockerfile | 2 - common/tests/failures/check/v0/README.md | 1 - common/tests/failures/check/v0/test/run | 3 - common/tests/imagestreams/testing_file.json | 53 - common/tests/remote-containers.sh | 103 -- common/tests/squash/.gitignore | 1 - common/tests/squash/squash.sh | 24 - common/tests/test-lib/image_availability | 17 - common/tests/test-lib/path_foreach | 24 - common/tests/test-lib/public_image_name | 17 - common/tests/test-lib/random_string | 17 - common/tests/test-lib/test_npm | 16 - common/update-generated.sh | 72 - dead.package | 1 + examples/custom-config/README | 2 - .../postgresql-cfg/10_shared_buffers.conf | 1 - examples/enable-ssl/postgresql-cfg/ssl.conf | 5 - .../postgresql-pre-start/enable_ssl.sh | 4 - examples/enable-ssl/server.crt | 77 - examples/enable-ssl/server.key | 28 - .../postgresql-cfg/s2i-extending.conf | 4 - .../postgresql-start/set_passwords.sh | 4 - examples/pgaudit/README | 10 - .../pgaudit/postgresql-cfg/10_pgaudit.conf | 1 - examples/postgresql-ephemeral-template.json | 253 ---- examples/postgresql-persistent-template.json | 277 ---- examples/replica/README.md | 240 ---- examples/replica/postgresql_replica.json | 370 ----- examples/s2i-dump-data/.s2i/bin/assemble | 36 - examples/s2i-dump-data/init/init.sql | 2 - .../postgresql-pre-start/10_boot.sh | 3 - help.md | 1 - root/help.1 | 404 ------ root/usr/bin/container-entrypoint | 3 - root/usr/bin/run-postgresql | 58 - root/usr/bin/run-postgresql-master | 5 - root/usr/bin/run-postgresql-slave | 39 - root/usr/bin/usage | 4 - root/usr/libexec/check-container | 27 - root/usr/libexec/fix-permissions | 39 - .../container-scripts/postgresql/README.md | 330 ----- .../container-scripts/postgresql/common.sh | 479 ------ ...ustom-postgresql-replication.conf.template | 7 - .../openshift-custom-postgresql.conf.template | 21 - .../openshift-custom-recovery.conf.template | 8 - .../container-scripts/postgresql/scl_enable | 3 - .../postgresql/start/set_passwords.sh | 23 - s2i/bin/assemble | 14 - s2i/bin/run | 1 - s2i/bin/usage | 1 - test/check_imagestreams.py | 1 - test/examples | 1 - test/pagila.sh | 42 - test/pg-test-lib.sh | 147 -- test/run | 1 - test/run-openshift | 1 - test/run-openshift-local-cluster | 456 ------ test/run-openshift-remote-cluster | 36 - test/run_migration_test | 56 - test/run_test | 929 ------------ test/run_test.orig | 847 ----------- test/run_upgrade_test | 97 -- test/test-app/postgresql-init/backup_user.sh | 12 - test/test-lib-openshift.sh | 1 - test/test-lib-postgresql.sh | 39 - test/test-lib.sh | 1 - 92 files changed, 1 insertion(+), 9428 deletions(-) delete mode 100644 .gitignore delete mode 120000 12 delete mode 100644 Dockerfile delete mode 120000 Dockerfile.fedora delete mode 120000 README.md delete mode 100644 common/.travis.yml delete mode 100644 common/LICENSE delete mode 100644 common/Makefile delete mode 100644 common/README.md delete mode 100755 common/build.sh delete mode 100755 common/check_imagestreams.py delete mode 100755 common/clean.sh delete mode 100644 common/common.mk delete mode 100755 common/generate.sh delete mode 100755 common/run-shellcheck.sh delete mode 100755 common/squash.py delete mode 100755 common/tag.sh delete mode 100644 common/test-lib-openshift.sh delete mode 100644 common/test-lib-remote-openshift.sh delete mode 100644 common/test-lib.sh delete mode 100755 common/test-remote-cluster.sh delete mode 100755 common/test.sh delete mode 100644 common/tests/.gitignore delete mode 100755 common/tests/check_imagestreams.sh delete mode 100644 common/tests/failures/check/Makefile delete mode 120000 common/tests/failures/check/common delete mode 100644 common/tests/failures/check/v0/Dockerfile delete mode 100644 common/tests/failures/check/v0/README.md delete mode 100755 common/tests/failures/check/v0/test/run delete mode 100644 common/tests/imagestreams/testing_file.json delete mode 100755 common/tests/remote-containers.sh delete mode 100644 common/tests/squash/.gitignore delete mode 100755 common/tests/squash/squash.sh delete mode 100755 common/tests/test-lib/image_availability delete mode 100755 common/tests/test-lib/path_foreach delete mode 100755 common/tests/test-lib/public_image_name delete mode 100755 common/tests/test-lib/random_string delete mode 100755 common/tests/test-lib/test_npm delete mode 100755 common/update-generated.sh create mode 100644 dead.package delete mode 100644 examples/custom-config/README delete mode 100644 examples/custom-config/postgresql-cfg/10_shared_buffers.conf delete mode 100644 examples/enable-ssl/postgresql-cfg/ssl.conf delete mode 100644 examples/enable-ssl/postgresql-pre-start/enable_ssl.sh delete mode 100644 examples/enable-ssl/server.crt delete mode 100644 examples/enable-ssl/server.key delete mode 100644 examples/extending-image/postgresql-cfg/s2i-extending.conf delete mode 100644 examples/extending-image/postgresql-start/set_passwords.sh delete mode 100644 examples/pgaudit/README delete mode 100644 examples/pgaudit/postgresql-cfg/10_pgaudit.conf delete mode 100644 examples/postgresql-ephemeral-template.json delete mode 100644 examples/postgresql-persistent-template.json delete mode 100644 examples/replica/README.md delete mode 100644 examples/replica/postgresql_replica.json delete mode 100755 examples/s2i-dump-data/.s2i/bin/assemble delete mode 100644 examples/s2i-dump-data/init/init.sql delete mode 100644 examples/s2i-dump-data/postgresql-pre-start/10_boot.sh delete mode 120000 help.md delete mode 100644 root/help.1 delete mode 100755 root/usr/bin/container-entrypoint delete mode 100755 root/usr/bin/run-postgresql delete mode 100755 root/usr/bin/run-postgresql-master delete mode 100755 root/usr/bin/run-postgresql-slave delete mode 100755 root/usr/bin/usage delete mode 100755 root/usr/libexec/check-container delete mode 100755 root/usr/libexec/fix-permissions delete mode 100644 root/usr/share/container-scripts/postgresql/README.md delete mode 100644 root/usr/share/container-scripts/postgresql/common.sh delete mode 100644 root/usr/share/container-scripts/postgresql/openshift-custom-postgresql-replication.conf.template delete mode 100644 root/usr/share/container-scripts/postgresql/openshift-custom-postgresql.conf.template delete mode 100644 root/usr/share/container-scripts/postgresql/openshift-custom-recovery.conf.template delete mode 100644 root/usr/share/container-scripts/postgresql/scl_enable delete mode 100644 root/usr/share/container-scripts/postgresql/start/set_passwords.sh delete mode 100755 s2i/bin/assemble delete mode 120000 s2i/bin/run delete mode 100755 s2i/bin/usage delete mode 120000 test/check_imagestreams.py delete mode 120000 test/examples delete mode 100755 test/pagila.sh delete mode 100644 test/pg-test-lib.sh delete mode 120000 test/run delete mode 120000 test/run-openshift delete mode 100755 test/run-openshift-local-cluster delete mode 100755 test/run-openshift-remote-cluster delete mode 100755 test/run_migration_test delete mode 100755 test/run_test delete mode 100755 test/run_test.orig delete mode 100755 test/run_upgrade_test delete mode 100644 test/test-app/postgresql-init/backup_user.sh delete mode 120000 test/test-lib-openshift.sh delete mode 100644 test/test-lib-postgresql.sh delete mode 120000 test/test-lib.sh diff --git a/.gitignore b/.gitignore deleted file mode 100644 index e69de29..0000000 diff --git a/12 b/12 deleted file mode 120000 index 945c9b4..0000000 --- a/12 +++ /dev/null @@ -1 +0,0 @@ -. \ No newline at end of file diff --git a/Dockerfile b/Dockerfile deleted file mode 100644 index 8b99d9a..0000000 --- a/Dockerfile +++ /dev/null @@ -1,83 +0,0 @@ -FROM registry.fedoraproject.org/f34/s2i-core:latest - -# PostgreSQL image for OpenShift. -# Volumes: -# * /var/lib/psql/data - Database cluster for PostgreSQL -# Environment: -# * $POSTGRESQL_USER - Database user name -# * $POSTGRESQL_PASSWORD - User's password -# * $POSTGRESQL_DATABASE - Name of the database to create -# * $POSTGRESQL_ADMIN_PASSWORD (Optional) - Password for the 'postgres' -# PostgreSQL administrative account - -ENV NAME=postgresql \ - VERSION=0 \ - ARCH=x86_64 \ - \ - POSTGRESQL_VERSION=12 \ - POSTGRESQL_PREV_VERSION=11 \ - HOME=/var/lib/pgsql \ - PGUSER=postgres \ - APP_DATA=/opt/app-root - -ENV SUMMARY="PostgreSQL is an advanced Object-Relational database management system" \ - DESCRIPTION="PostgreSQL is an advanced Object-Relational database management system (DBMS). \ -The image contains the client and server programs that you'll need to \ -create, run, maintain and access a PostgreSQL DBMS server." - -LABEL summary="$SUMMARY" \ - description="$DESCRIPTION" \ - io.k8s.description="$DESCRIPTION" \ - io.k8s.display-name="PostgreSQL 12" \ - io.openshift.expose-services="5432:postgresql" \ - io.openshift.tags="database,postgresql,postgresql12" \ - com.redhat.component="$NAME" \ - maintainer="SoftwareCollections.org " \ - name="$FGC/$NAME" \ - version="0" \ - usage="docker run -d --name postgresql_database -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -p 5432:5432 $FGC/$NAME" - -EXPOSE 5432 - -COPY root/usr/libexec/fix-permissions /usr/libexec/fix-permissions - -# This image must forever use UID 26 for postgres user so our volumes are -# safe in the future. This should *never* change, the last test is there -# to make sure of that. -RUN INSTALL_PKGS="rsync tar gettext bind-utils postgresql-server postgresql-contrib nss_wrapper " && \ - INSTALL_PKGS+="findutils xz" && \ - INSTALL_PKGS+=" pgaudit" && \ - dnf -y module enable postgresql:12 && \ - dnf -y --setopt=tsflags=nodocs install $INSTALL_PKGS && \ - rpm -V $INSTALL_PKGS && \ - dnf clean all && \ - test "$(id postgres)" = "uid=26(postgres) gid=26(postgres) groups=26(postgres)" && \ - mkdir -p /var/lib/pgsql/data && \ - /usr/libexec/fix-permissions /var/lib/pgsql /var/run/postgresql - -# Get prefix path and path to scripts rather than hard-code them in scripts -ENV CONTAINER_SCRIPTS_PATH=/usr/share/container-scripts/postgresql - -COPY root / -COPY ./s2i/bin/ $STI_SCRIPTS_PATH - -VOLUME ["/var/lib/pgsql/data"] - -# S2I permission fixes -# -------------------- -# 1. unless specified otherwise (or - equivalently - we are in OpenShift), s2i -# build process would be executed as 'uid=26(postgres) gid=26(postgres)'. -# Such process wouldn't be able to execute the default 'assemble' script -# correctly (it transitively executes 'fix-permissions' script). So let's -# add the 'postgres' user into 'root' group here -# -# 2. we call fix-permissions on $APP_DATA here directly (UID=0 during build -# anyways) to assure that s2i process is actually able to _read_ the -# user-specified scripting. -RUN usermod -a -G root postgres && \ - /usr/libexec/fix-permissions --read-only "$APP_DATA" - -USER 26 - -ENTRYPOINT ["container-entrypoint"] -CMD ["run-postgresql"] diff --git a/Dockerfile.fedora b/Dockerfile.fedora deleted file mode 120000 index 1d1fe94..0000000 --- a/Dockerfile.fedora +++ /dev/null @@ -1 +0,0 @@ -Dockerfile \ No newline at end of file diff --git a/README.md b/README.md deleted file mode 120000 index d359f03..0000000 --- a/README.md +++ /dev/null @@ -1 +0,0 @@ -root/usr/share/container-scripts/postgresql/README.md \ No newline at end of file diff --git a/common/.travis.yml b/common/.travis.yml deleted file mode 100644 index 8df16d3..0000000 --- a/common/.travis.yml +++ /dev/null @@ -1,4 +0,0 @@ -language: shell -os: linux -script: - - make shellcheck diff --git a/common/LICENSE b/common/LICENSE deleted file mode 100644 index 7a4a3ea..0000000 --- a/common/LICENSE +++ /dev/null @@ -1,202 +0,0 @@ - - Apache License - Version 2.0, January 2004 - http://www.apache.org/licenses/ - - TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION - - 1. Definitions. - - "License" shall mean the terms and conditions for use, reproduction, - and distribution as defined by Sections 1 through 9 of this document. - - "Licensor" shall mean the copyright owner or entity authorized by - the copyright owner that is granting the License. - - "Legal Entity" shall mean the union of the acting entity and all - other entities that control, are controlled by, or are under common - control with that entity. For the purposes of this definition, - "control" means (i) the power, direct or indirect, to cause the - direction or management of such entity, whether by contract or - otherwise, or (ii) ownership of fifty percent (50%) or more of the - outstanding shares, or (iii) beneficial ownership of such entity. - - "You" (or "Your") shall mean an individual or Legal Entity - exercising permissions granted by this License. - - "Source" form shall mean the preferred form for making modifications, - including but not limited to software source code, documentation - source, and configuration files. - - "Object" form shall mean any form resulting from mechanical - transformation or translation of a Source form, including but - not limited to compiled object code, generated documentation, - and conversions to other media types. - - "Work" shall mean the work of authorship, whether in Source or - Object form, made available under the License, as indicated by a - copyright notice that is included in or attached to the work - (an example is provided in the Appendix below). - - "Derivative Works" shall mean any work, whether in Source or Object - form, that is based on (or derived from) the Work and for which the - editorial revisions, annotations, elaborations, or other modifications - represent, as a whole, an original work of authorship. For the purposes - of this License, Derivative Works shall not include works that remain - separable from, or merely link (or bind by name) to the interfaces of, - the Work and Derivative Works thereof. - - "Contribution" shall mean any work of authorship, including - the original version of the Work and any modifications or additions - to that Work or Derivative Works thereof, that is intentionally - submitted to Licensor for inclusion in the Work by the copyright owner - or by an individual or Legal Entity authorized to submit on behalf of - the copyright owner. For the purposes of this definition, "submitted" - means any form of electronic, verbal, or written communication sent - to the Licensor or its representatives, including but not limited to - communication on electronic mailing lists, source code control systems, - and issue tracking systems that are managed by, or on behalf of, the - Licensor for the purpose of discussing and improving the Work, but - excluding communication that is conspicuously marked or otherwise - designated in writing by the copyright owner as "Not a Contribution." - - "Contributor" shall mean Licensor and any individual or Legal Entity - on behalf of whom a Contribution has been received by Licensor and - subsequently incorporated within the Work. - - 2. Grant of Copyright License. Subject to the terms and conditions of - this License, each Contributor hereby grants to You a perpetual, - worldwide, non-exclusive, no-charge, royalty-free, irrevocable - copyright license to reproduce, prepare Derivative Works of, - publicly display, publicly perform, sublicense, and distribute the - Work and such Derivative Works in Source or Object form. - - 3. Grant of Patent License. Subject to the terms and conditions of - this License, each Contributor hereby grants to You a perpetual, - worldwide, non-exclusive, no-charge, royalty-free, irrevocable - (except as stated in this section) patent license to make, have made, - use, offer to sell, sell, import, and otherwise transfer the Work, - where such license applies only to those patent claims licensable - by such Contributor that are necessarily infringed by their - Contribution(s) alone or by combination of their Contribution(s) - with the Work to which such Contribution(s) was submitted. If You - institute patent litigation against any entity (including a - cross-claim or counterclaim in a lawsuit) alleging that the Work - or a Contribution incorporated within the Work constitutes direct - or contributory patent infringement, then any patent licenses - granted to You under this License for that Work shall terminate - as of the date such litigation is filed. - - 4. Redistribution. You may reproduce and distribute copies of the - Work or Derivative Works thereof in any medium, with or without - modifications, and in Source or Object form, provided that You - meet the following conditions: - - (a) You must give any other recipients of the Work or - Derivative Works a copy of this License; and - - (b) You must cause any modified files to carry prominent notices - stating that You changed the files; and - - (c) You must retain, in the Source form of any Derivative Works - that You distribute, all copyright, patent, trademark, and - attribution notices from the Source form of the Work, - excluding those notices that do not pertain to any part of - the Derivative Works; and - - (d) If the Work includes a "NOTICE" text file as part of its - distribution, then any Derivative Works that You distribute must - include a readable copy of the attribution notices contained - within such NOTICE file, excluding those notices that do not - pertain to any part of the Derivative Works, in at least one - of the following places: within a NOTICE text file distributed - as part of the Derivative Works; within the Source form or - documentation, if provided along with the Derivative Works; or, - within a display generated by the Derivative Works, if and - wherever such third-party notices normally appear. The contents - of the NOTICE file are for informational purposes only and - do not modify the License. You may add Your own attribution - notices within Derivative Works that You distribute, alongside - or as an addendum to the NOTICE text from the Work, provided - that such additional attribution notices cannot be construed - as modifying the License. - - You may add Your own copyright statement to Your modifications and - may provide additional or different license terms and conditions - for use, reproduction, or distribution of Your modifications, or - for any such Derivative Works as a whole, provided Your use, - reproduction, and distribution of the Work otherwise complies with - the conditions stated in this License. - - 5. Submission of Contributions. Unless You explicitly state otherwise, - any Contribution intentionally submitted for inclusion in the Work - by You to the Licensor shall be under the terms and conditions of - this License, without any additional terms or conditions. - Notwithstanding the above, nothing herein shall supersede or modify - the terms of any separate license agreement you may have executed - with Licensor regarding such Contributions. - - 6. Trademarks. This License does not grant permission to use the trade - names, trademarks, service marks, or product names of the Licensor, - except as required for reasonable and customary use in describing the - origin of the Work and reproducing the content of the NOTICE file. - - 7. Disclaimer of Warranty. Unless required by applicable law or - agreed to in writing, Licensor provides the Work (and each - Contributor provides its Contributions) on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or - implied, including, without limitation, any warranties or conditions - of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A - PARTICULAR PURPOSE. You are solely responsible for determining the - appropriateness of using or redistributing the Work and assume any - risks associated with Your exercise of permissions under this License. - - 8. Limitation of Liability. In no event and under no legal theory, - whether in tort (including negligence), contract, or otherwise, - unless required by applicable law (such as deliberate and grossly - negligent acts) or agreed to in writing, shall any Contributor be - liable to You for damages, including any direct, indirect, special, - incidental, or consequential damages of any character arising as a - result of this License or out of the use or inability to use the - Work (including but not limited to damages for loss of goodwill, - work stoppage, computer failure or malfunction, or any and all - other commercial damages or losses), even if such Contributor - has been advised of the possibility of such damages. - - 9. Accepting Warranty or Additional Liability. While redistributing - the Work or Derivative Works thereof, You may choose to offer, - and charge a fee for, acceptance of support, warranty, indemnity, - or other liability obligations and/or rights consistent with this - License. However, in accepting such obligations, You may act only - on Your own behalf and on Your sole responsibility, not on behalf - of any other Contributor, and only if You agree to indemnify, - defend, and hold each Contributor harmless for any liability - incurred by, or claims asserted against, such Contributor by reason - of your accepting any such warranty or additional liability. - - END OF TERMS AND CONDITIONS - - APPENDIX: How to apply the Apache License to your work. - - To apply the Apache License to your work, attach the following - boilerplate notice, with the fields enclosed by brackets "[]" - replaced with your own identifying information. (Don't include - the brackets!) The text should be enclosed in the appropriate - comment syntax for the file format. We also recommend that a - file or class name and description of purpose be included on the - same "printed page" as the copyright notice for easier - identification within third-party archives. - - Copyright [yyyy] [name of copyright owner] - - Licensed under the Apache License, Version 2.0 (the "License"); - you may not use this file except in compliance with the License. - You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - - Unless required by applicable law or agreed to in writing, software - distributed under the License is distributed on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - See the License for the specific language governing permissions and - limitations under the License. \ No newline at end of file diff --git a/common/Makefile b/common/Makefile deleted file mode 100644 index 183f48f..0000000 --- a/common/Makefile +++ /dev/null @@ -1,46 +0,0 @@ -SHELL := /usr/bin/env bash - -all: - @echo >&2 "Only 'make check' allowed" - - -TESTED_IMAGES = \ - postgresql-container \ - s2i-python-container \ - s2i-nodejs-container - -.PHONY: check test all check-failures - - -TEST_LIB_TESTS = \ - path_foreach \ - random_string \ - test_npm \ - image_availability \ - public_image_name - -$(TEST_LIB_TESTS): - @echo " RUN TEST '$@'" ; \ - $(SHELL) tests/test-lib/$@ || $(SHELL) -x tests/lib/$@ - -test-lib-foreach: - -check-test-lib: $(TEST_LIB_TESTS) - -test: check - -shellcheck: - ./run-shellcheck.sh `git ls-files *.sh` - -check-failures: check-test-lib - cd tests/failures/check && make tag && ! make check && make clean - grep -q "Red Hat Enterprise Linux release 8" /etc/system-release || cd tests/failures/check && make tag SKIP_SQUASH=0 - -check-squash: - ./tests/squash/squash.sh - -check-latest-imagestream: - cd tests && ./check_imagestreams.sh - -check: check-failures check-squash check-latest-imagestream - TESTED_IMAGES="$(TESTED_IMAGES)" tests/remote-containers.sh diff --git a/common/README.md b/common/README.md deleted file mode 100644 index 10d134b..0000000 --- a/common/README.md +++ /dev/null @@ -1,126 +0,0 @@ -Common build helpers for sclorg containers -========================================== - -This repository is aimed to be added as git submodule into particular -containers' source repositories. By default, the path to submodule should be -named 'common'. - -Usage ------ - -This section explains the usage of the shared scripts in this repository when -it is used as a submodule in a container source repository. - -Once you have the repository set as a submodule include `common.mk` into your -root Makefile in order to access the default rules used to call shared scripts. - -**Default rules:** - -`make` or `make build` -This rule will build an image without tagging it with any tags after it is built. -After the image finishes building the scripts will squash the image using `docker-squash`. -`make build` will also expect a `README.md` so that it can transfrom it into -a man page that gets added to the image so make sure it is available and that -you have the `go-md2man` tool installed on your host. - - -`make tag` -Use this rule if you want to tag an image after it is built. It will be tagged with -two tags - name:latest and name:version. -Depends on `build` - -`make test` or `make check` -This rule will run the testsuite scripts contained in the container source repositories. -It expects the test to be available at `$gitroot/$version/test/run` -Depends on `tag` as some tests might need to have the images tagged (s2i). - -`make test-openshift` -Similar to `make test` but runs testsuite for Openshift 3, expected to be found at -`$gitroot/$version/test/run-openshift` - -`make test-openshift-4` -Similar to `make test` but runs testsuite for Openshift 4, expected to be found at -`$gitroot/$version/test/run-openshift-remote-cluster` - -`make test-with-conu` -The rule is similar to `make test`. It runs a test suite written using [conu -library](https://github.com/user-cont/conu). The path to the test script is -meant to be at `$gitroot/$version/test/run-conu`. By default the test suite is -being run in the current environment. You can also run the tests in a container -by defining variable `CONU_IMAGE`. Container images with conu are available in -[this docker hub repository](docker.io/usercont/conu:0.6.2), a good value for -the variable is `docker.io/usercont/conu:0.6.2`. - -`make clean` -Runs scripts that clean-up the working dir. Depends on the `clean-images` rule by default -and additional clean rules can be provided through the `clean-hook` variable. - -`make clean-images` -Best-effort to remove the last set of images that have been built using the scripts. - -`make shellcheck` -Check the shell syntax of the files specified by `$SHELLCHECK_FILES` variable. -See `SHELLCHECK_FILES` variable description below for more info (default is `.`). -The files matching this specification are then filtered, to not show results twice -for symlinks. Only files with a suffix `.sh` or shell shebang are scanned with -the `shellcheck` utility. See [run-shellcheck.sh](./run-shellcheck.sh) in this repo for more detailed info. -Once the shell syntax issues are fixed, CI that runs `make shellcheck` for each PR can be -turned on by putting [.travis.yml](.travis.yml) file into the root of the image's repository. -[.travis.yml](https://github.com/sclorg/container-common-scripts/blob/master/.travis.yml) -for its content. - -**There are additional variables that you can use that the default rules are prepared to -work with:** - -`VERSIONS` -Names of the directories in which the Dockerfiles are contained. Needs to be defined in your -Dockerfile for the scripts to know which versions to build. - -`OS` -OS version you want to build the images for. Currently the scripts are able to build for -centos (default), centos6, centos8, rhel7, rhel8 and fedora. - -`SKIP_SQUASH` -When set to 1 the build script will skip the squash phase of the build. - -`CUSTOM_REPO` -Set this variable to the path to your local .repo files you want to have available inside -the image while building. Useful for building rhel-based images on an unsubscribed box. -Be aware that you cannot write to any .repo files used this way inside the image as they -will be mounted into the image as read-only. - -`UPDATE_BASE` -Set to 1 if you want the build script to always pull the base image when available. - -`DOCKER_BUILD_CONTEXT` -Use this variable in case you want to have a different context for your builds. By default -the context of the build is the versioned directory the Dockerfiles are contained in. - -`SHELLCHECK_FILES` -One or more files or directories to be scanned by the shellcheck, default is `.`, which -means a whole repository directory. If a directory is provided then all of its content -is scanned as well. - -`clean-hook` -Append Makefile rules to this variable to make sure additional cleaning actions are run -when `make clean` is called. - -Regression tests ----------------- - -`make check` -Runs the tests of few images that use this set of scripts. If the tests of those -images pass, this repo is considered to be working. - -`make shellcheck` -Check the shell syntax of all `*.sh` files tracked by the git in this repository. - -Dependencies for testsuite: - -- /usr/bin/docker (either `docker` or `podman` + `podman-docker`) -- docker-squash (if `docker` is used, otherwise set `SKIP_SQUASH`, which is done automatically on non-EL-7) -- git -- go-md2man -- make -- source-to-image -- shellcheck diff --git a/common/build.sh b/common/build.sh deleted file mode 100755 index 4983e79..0000000 --- a/common/build.sh +++ /dev/null @@ -1,226 +0,0 @@ -#!/bin/bash - -# This script is used to build the OpenShift Docker images. -# -# OS - Specifies distribution - "rhel7", "rhel8", "centos7", "centos8" or "fedora" -# VERSION - Specifies the image version - (must match with subdirectory in repo) -# VERSIONS - Must be set to a list with possible versions (subdirectories) - -set -e - -script_name=$(readlink -f "$0") -script_dir=$(dirname "$script_name") - -OS=${1-$OS} -VERSION=${2-$VERSION} - -error() { echo "ERROR: $*" ; false ; } - - -# _parse_output_inner -# ------------------- -# Helper function for 'parse_output'. -# We need to avoid case statements in $() for older Bash versions (per issue -# postgresql-container#35, mac ships with 3.2). -# Example of problematic statement: echo $(case i in i) echo i;; esac) -_parse_output_inner () -{ - set -o pipefail - { - case $stream in - stdout|1|"") - eval "$command" | tee >(cat - >&"$stdout_fd") - ;; - stderr|2) - set +x # avoid stderr pollution - eval "$command" {free_fd}>&1 1>&"$stdout_fd" 2>&"$free_fd" | tee >(cat - >&"$stderr_fd") - ;; - esac - # Inherit correct exit status. - (exit "${PIPESTATUS[0]}") - } | eval "$filter" -} - - -# parse_output COMMAND FILTER_COMMAND OUTVAR [STREAM={stderr|stdout}] -# ------------------------------------------------------------------- -# Parse standard (error) output of COMMAND with FILTER_COMMAND and store the -# output into variable named OUTVAR. STREAM might be 'stdout' or 'stderr', -# defaults to 'stdout'. The filtered output stays (live) printed to terminal. -# This method doesn't create any explicit temporary files. -# Defines: -# ${$OUTVAR}: Set to FILTER_COMMAND output. -parse_output () -{ - local command=$1 filter=$2 var=$3 stream=$4 - local raw_output='' rc=0 - { - # shellcheck disable=SC2034 - raw_output=$(_parse_output_inner) - } {stdout_fd}>&1 {stderr_fd}>&2 - rc=$? - eval "$var=\$raw_output" - (exit $rc) -} - -# "best-effort" cleanup of previous image -function clean_image { - if test -f .image-id.raw; then - local previous_id - previous_id=$(cat .image-id.raw) - if test "$IMAGE_ID" != "$previous_id"; then - # Also remove squashed image since it will change anyway - docker rmi "$previous_id" "$(cat .image-id)" || : - rm -f ".image-id.raw" ".image-id" || : - fi - fi -} - -# Pull image based on FROM, before we build our own. -function pull_image { - local dockerfile="$1" - local loops=10 - local loop=0 - - # Get image_name from Dockerfile before pulling. - while read -r line; do - if ! grep -q "^FROM" <<< "$line"; then - continue - fi - - image_name=$(echo "$line" | cut -d ' ' -f2) - - # In case FROM scratch is defined, skip it - if [[ x"$image_name" == "xscratch" ]]; then - continue - fi - echo "-> Pulling image $image_name before building image from $dockerfile." - # Sometimes in Fedora case it fails with HTTP 50X - # Check if the image is available locally and try to pull it if it is not - if [[ "$(docker images -q "$image_name" 2>/dev/null)" != "" ]]; then - echo "The image $image_name is already pulled." - continue - fi - - # Try pulling the image to see if it is accessible - # WORKAROUND: Since Fedora registry sometimes fails randomly, let's try it more times - while ! docker pull "$image_name"; do - ((loop++)) || : - echo "Pulling image $image_name failed." - [ "$loop" -gt "$loops" ] && { echo "It happened $loops times. Giving up." ; return 1; } - echo "Let's wait $((loop*5)) seconds and try again." - sleep "$((loop*5))" - done - - done < "$dockerfile" -} - -# Perform docker build but append the LABEL with GIT commit id at the end -function docker_build_with_version { - local dockerfile="$1" - local exclude=.exclude-${OS} - if [ -e "$exclude" ]; then - echo "-> $exclude file exists for version $dir, skipping build." - clean_image - return - fi - if [ ! -e "$dockerfile" ]; then - echo "-> $dockerfile for version $dir does not exist, skipping build." - clean_image - return - fi - echo "-> Version ${dir}: building image from '${dockerfile}' ..." - - git_version=$(git rev-parse --short HEAD) - BUILD_OPTIONS+=" --label io.openshift.builder-version=\"${git_version}\"" - if [[ "${UPDATE_BASE}" == "1" ]]; then - BUILD_OPTIONS+=" --pull=true" - fi - if [ -n "$CUSTOM_REPO" ]; then - if [ -f "$CUSTOM_REPO" ]; then - BUILD_OPTIONS+=" -v $CUSTOM_REPO:/etc/yum.repos.d/sclorg_custom.repo:Z" - elif [ -d "$CUSTOM_REPO" ]; then - BUILD_OPTIONS+=" -v $CUSTOM_REPO:/etc/yum.repos.d/:Z" - else - echo "ERROR: file type not known: $CUSTOM_REPO" >&2 - fi - fi - - pull_image "$dockerfile" - - # shellcheck disable=SC2016 - parse_output 'docker build '"$BUILD_OPTIONS"' -f "$dockerfile" "${DOCKER_BUILD_CONTEXT}"' \ - "tail -n 1 | awk '/Successfully built|(^--> )?(Using cache )?[a-fA-F0-9]+$/{print \$NF}'" \ - IMAGE_ID - clean_image - echo "$IMAGE_ID" > .image-id.raw - - squash "${dockerfile}" - echo "$IMAGE_ID" > .image-id -} - -# squash DOCKERFILE -# ----------------- -# Use python library docker_squash[1] and squash the result image -# when necessary. -# [1] https://github.com/goldmann/docker-squash -# Reads: -# $IMAGE_ID -# Sets: -# $IMAGE_ID -squash () -{ - local base squashed_from squashed='' unsquashed=$IMAGE_ID - test "$SKIP_SQUASH" = 1 && return 0 - - if test -f .image-id.squashed; then - squashed=$(cat .image-id.squashed) - # We (maybe) already have squashed file. - if test -f .image-id.squashed_from; then - squashed_from=$(cat .image-id.squashed_from) - if test "$squashed_from" = "$IMAGE_ID"; then - # $squashed is up2date - IMAGE_ID=$squashed - echo "Image '$unsquashed' already squashed as '$squashed'" - return 0 - fi - fi - - # We are going to squash now, so if there's existing squashed image, try - # to do the best-effort 'rmi' to not waste memory unnecessarily. - docker rmi "$squashed" || : - fi - - base=$(awk '/^FROM/{print $2}' "$1") - - echo "Squashing the image '$unsquashed' from '$base' layer." - IMAGE_ID=$("${PYTHON-python3}" "$script_dir"/squash.py "$unsquashed" "$base") - - echo "Squashed as '$IMAGE_ID'." - - echo "$unsquashed" > .image-id.squashed_from - echo "$IMAGE_ID" > .image-id.squashed -} - -# Versions are stored in subdirectories. You can specify VERSION variable -# to build just one single version. By default we build all versions -dirs=${VERSION:-$VERSIONS} - -for dir in ${dirs}; do - pushd "${dir}" > /dev/null - if [ "$OS" == "rhel8" ] || [ "$OS" == "rhel8-candidate" ]; then - docker_build_with_version Dockerfile.rhel8 - elif [ "$OS" == "rhel7" ] || [ "$OS" == "rhel7-candidate" ]; then - docker_build_with_version Dockerfile.rhel7 - elif [ "$OS" == "fedora" ] || [ "$OS" == "fedora-candidate" ]; then - docker_build_with_version Dockerfile.fedora - elif [ "$OS" == "centos6" ] || [ "$OS" == "centos6-candidate" ]; then - docker_build_with_version Dockerfile.centos6 - elif [ "$OS" == "centos8" ] || [ "$OS" == "centos8-candidate" ]; then - docker_build_with_version Dockerfile.centos8 - else - docker_build_with_version Dockerfile - fi - - popd > /dev/null -done diff --git a/common/check_imagestreams.py b/common/check_imagestreams.py deleted file mode 100755 index 6da0f63..0000000 --- a/common/check_imagestreams.py +++ /dev/null @@ -1,73 +0,0 @@ -#!/bin/env python3 - -import sys -import json -import logging -import os - -from pathlib import Path -from typing import Dict - -IMAGESTREAMS_DIR: str = "imagestreams" - - -class ImageStreamChecker(object): - version: str = "" - results: Dict = {} - - def __init__(self, version: str): - self.version = version - - def load_json_file(self, filename: Path): - with open(str(filename)) as f: - return json.load(f) - - def check_version(self, json_dict: Dict): - res = [] - for tags in json_dict["spec"]["tags"]: - # The name can be"" or "-elX" or "-ubiX" - if (tags["name"] == self.version or - tags["name"].startswith(self.version + '-')): - res.append(tags) - return res - - def check_latest_tag(self, json_dict: Dict): - latest_tag_correct: bool = False - for tags in json_dict["spec"]["tags"]: - if tags["name"] != "latest": - continue - # The latest can link to either "" or "-elX" or "-ubiX" - if tags["from"]["name"] == self.version or tags["from"]["name"].startswith(self.version + '-'): - latest_tag_correct = True - return latest_tag_correct - - def check_imagestreams(self): - p = Path(".") - json_files = p.glob(f"{IMAGESTREAMS_DIR}/*.json") - if not json_files: - print(f"No json files present in {IMAGESTREAMS_DIR}.") - return 0 - for f in json_files: - if os.environ.get("TARGET") in ("rhel7", "centos7") and "aarch64" in str(f): - print("Imagestream aarch64 is not supported on rhel7") - continue - print(f"Checking file {str(f)}.") - json_dict = self.load_json_file(f) - if not (self.check_version(json_dict) and self.check_latest_tag(json_dict)): - print(f"The latest version is not present in {str(f)} or in latest tag.") - self.results[f] = False - if self.results: - return 1 - print("Imagestreams contains the latest version.") - return 0 - - -if __name__ == "__main__": - if len(sys.argv) != 2: - logging.fatal("%s: %s", sys.argv[0], "VERSION as an argument was not provided") - sys.exit(1) - - print(f"Version to check is {sys.argv[1]}.") - isc = ImageStreamChecker(version=sys.argv[1]) - sys.exit(isc.check_imagestreams()) - diff --git a/common/clean.sh b/common/clean.sh deleted file mode 100755 index 075d1bf..0000000 --- a/common/clean.sh +++ /dev/null @@ -1,21 +0,0 @@ -#! /bin/sh - -set -e -test -f auto_targets.mk && rm auto_targets.mk - -for version -do - remove_images= - for idfile in .image-id.raw .image-id.squashed; do - # shellcheck disable=SC2039 - test ! -f "$version/$idfile" || remove_images+=" $(cat "$version/$idfile")" - done - - for image in $remove_images; do - # shellcheck disable=SC2046 - docker rm -f $(docker ps -q -a -f "ancestor=$image") 2>/dev/null || : - docker rmi -f "$image" || : - done - - rm -rf "$version"/.image-id* -done diff --git a/common/common.mk b/common/common.mk deleted file mode 100644 index 83ab47b..0000000 --- a/common/common.mk +++ /dev/null @@ -1,150 +0,0 @@ -# SKIP_SQUASH = 0/1 -# ================= -# If set to '0', images are automatically squashed. '1' disables -# squashing. By default only RHEL containers are squashed. - -SHELL := /usr/bin/env bash - -ifndef common_dir - common_dir = common -endif - -build = $(SHELL) $(common_dir)/build.sh -test = $(SHELL) $(common_dir)/test.sh -testr = $(SHELL) $(common_dir)/test-remote-cluster.sh -shellcheck = $(SHELL) $(common_dir)/run-shellcheck.sh -tag = $(SHELL) $(common_dir)/tag.sh -clean = $(SHELL) $(common_dir)/clean.sh - -DG ?= /bin/dg - -generator = DG="$(DG)" $(SHELL) $(common_dir)/generate.sh - - -# pretty printers -# --------------- -__PROLOG = $(if $(VERBOSE),,@echo " $(1) " $@;) -V_LN = $(call __PROLOG,LN ) -V_DG = $(call __PROLOG,DG ) -V_DGM = $(call __PROLOG,DGM) -V_CP = $(call __PROLOG,CP ) - -CDIR = mkdir -p "$$(dirname "$@")" || exit 1 ; - -ifeq ($(TARGET),rhel8) - SKIP_SQUASH ?= 1 - OS := rhel8 - DOCKERFILE ?= Dockerfile.rhel8 -else ifeq ($(TARGET),rhel7) - SKIP_SQUASH ?= 0 - OS := rhel7 - DOCKERFILE ?= Dockerfile.rhel7 -else ifeq ($(TARGET),fedora) - OS := fedora - DOCKERFILE ?= Dockerfile.fedora -else ifeq ($(TARGET),centos6) - OS := centos6 - DOCKERFILE ?= Dockerfile.centos6 -else ifeq ($(TARGET),centos8) - OS := centos8 - DOCKERFILE ?= Dockerfile.centos8 -else - OS := centos7 - DOCKERFILE ?= Dockerfile -endif - -SKIP_SQUASH ?= 1 -DOCKER_BUILD_CONTEXT ?= . -SHELLCHECK_FILES ?= . - -script_env = \ - SKIP_SQUASH=$(SKIP_SQUASH) \ - UPDATE_BASE=$(UPDATE_BASE) \ - OS=$(OS) \ - CLEAN_AFTER=$(CLEAN_AFTER) \ - DOCKER_BUILD_CONTEXT=$(DOCKER_BUILD_CONTEXT) \ - OPENSHIFT_NAMESPACES="$(OPENSHIFT_NAMESPACES)" \ - CUSTOM_REPO="$(CUSTOM_REPO)" - -# TODO: switch to 'build: build-all' once parallel builds are relatively safe -.PHONY: build build-serial build-all -build: build-serial -build-serial: - @$(MAKE) -j1 build-all - -build-all: $(VERSIONS) - @for i in $(VERSIONS); do \ - test -f $$i/.image-id || continue ; \ - echo -n "$(BASE_IMAGE_NAME) $$i => " ; \ - cat $$i/.image-id ; \ - done - -.PHONY: $(VERSIONS) -$(VERSIONS): % : %/root/help.1 - VERSION="$@" $(script_env) $(build) - -.PHONY: test check -check: test - -test: script_env += TEST_MODE=true - -# The tests should ideally depend on $IMAGE_ID only, but see PR#19 for more info -# while we need to depend on 'tag' instead of 'build'. -test: tag - VERSIONS="$(VERSIONS)" $(script_env) $(test) - -.PHONY: test-with-conu -test-with-conu: script_env += TEST_CONU_MODE=true -test-with-conu: tag - VERSIONS="$(VERSIONS)" $(script_env) $(test) - -.PHONY: test-openshift-4 -test-openshift-4: script_env += TEST_OPENSHIFT_4=true -test-openshift-4: tag - VERSIONS="$(VERSIONS)" BASE_IMAGE_NAME="$(BASE_IMAGE_NAME)" $(script_env) $(test) - -.PHONY: test-openshift -test-openshift: script_env += TEST_OPENSHIFT_MODE=true -test-openshift: tag - VERSIONS="$(VERSIONS)" BASE_IMAGE_NAME="$(BASE_IMAGE_NAME)" $(script_env) $(test) - -.PHONY: shellcheck -shellcheck: - $(shellcheck) $(SHELLCHECK_FILES) - -.PHONY: tag -tag: build - VERSIONS="$(VERSIONS)" $(script_env) $(tag) - -.PHONY: clean clean-hook clean-images clean-versions -clean: clean-images - @$(MAKE) --no-print-directory clean-hook - -clean-images: - $(clean) $(VERSIONS) - -clean-versions: - rm -rf $(VERSIONS) - -%root/help.1: %README.md - mkdir -p $(@D) - go-md2man -in "$^" -out "$@" - chmod a+r "$@" - -generate-all: generate - -MANIFEST_FILE ?= manifest.sh - -auto_targets.mk: $(MANIFEST_FILE) - MANIFEST_FILE="$(MANIFEST_FILE)" \ - VERSIONS="$(VERSIONS)" \ - $(generator) - -# triggers build of auto_targets.mk automatically --include auto_targets.mk - -# We have to remove auto_targets.mk here, otherwise subsequent make calls -# with different VERSIONS=* option keeps the auto_targets.mk unchanged. -.PHONY: generate -generate: $(DISTGEN_TARGETS) $(DISTGEN_MULTI_TARGETS) $(COPY_TARGETS) $(SYMLINK_TARGETS) - rm auto_targets.mk diff --git a/common/generate.sh b/common/generate.sh deleted file mode 100755 index 7dfd931..0000000 --- a/common/generate.sh +++ /dev/null @@ -1,184 +0,0 @@ -#!/bin/bash - -# This script is used to create image directories using distgen, cp or ln -# It requires "$MANIFEST_FILE" (defaults to manifest.sh) file to be present in -# image repository. -# The manifest file should contain set of rules in form: -# =" -# src= -# dest= -# mode=; -# ...; -# " -# -# Supported type rules are now COPY_RULES, DISTGEN_RULES and SYMLINKS_RULES -# for real example see https://github.com/sclorg/postgresql-container/blob/master/manifest.sh - -# shellcheck disable=SC1090 -source "$MANIFEST_FILE" - -die () { echo "FATAL: $*" ; exit 1 ; } - -nl=' -' - -test -f auto_targets.mk && rm auto_targets.mk -DG="${DG-/bin/dg}" -[ ! -x "$DG" ] && echo " Error: distgen binary not found or not executable in $DG" && \ -echo " Make sure distgen is properly installed on your host in $DG, or provide a path to your distgen binary via \$DG" && exit 1 - -DISTGEN_COMBINATIONS=$("$DG" --multispec specs/multispec.yml --multispec-combinations) - - -clean_rule_variables(){ - src="" - dest="" - mode="" - link_target="" - link_name="" -} - -parse_rules() { - targets="" - OLD_IFS=$IFS - IFS=";" - for rule in $rules; do - if [ -z "$(echo "$rule"| tr -d '[:space:]')" ]; then - continue - fi - clean_rule_variables - eval "$rule" - - # shellcheck disable=SC2016 - cdir='$(CDIR)' - case "$creator" in - copy) - [[ -z "$src" ]] && echo "src has to be specified in copy rule" && exit 1 - [[ -z "$dest" ]] && echo "dest has to be specified in copy rule" && exit 1 - core_subst=$core - prolog="\$(V_CP)$cdir" - ;; - distgen) - [[ -z "$src" ]] && echo "src has to be specified in distgen rule" && exit 1 - [[ -z "$dest" ]] && echo "dest has to be specified in distgen rule" && exit 1 - core_subst=$core - prolog="\$(V_DG)$cdir" - ;; - distgen_multi) - [[ -z "$src" ]] && echo "src has to be specified in distgen rule" && exit 1 - [[ -z "$dest" ]] && echo "dest has to be specified in distgen rule" && exit 1 - - if [[ "$dest" == "Dockerfile.rhel7" ]]; then - if ! [[ "$DG_CONF" =~ rhel-7-x86_64.yaml ]]; then - continue - fi - elif [[ "$dest" == "Dockerfile.rhel8" ]]; then - if ! [[ "$DG_CONF" =~ rhel-8-x86_64.yaml ]]; then - continue - fi - elif [[ "$dest" == "Dockerfile.centos8" ]]; then - if ! [[ "$DG_CONF" =~ centos-8-x86_64.yaml ]]; then - continue - fi - elif [[ "$dest" == *"Dockerfile.fedora" ]]; then - if ! [[ "$DG_CONF" =~ fedora-[0-9]{,2}-x86_64.yaml ]]; then - continue - fi - elif [[ "$dest" == *"Dockerfile" ]]; then - if ! [[ "$DG_CONF" =~ centos-[0-9]{,2}-x86_64.yaml ]]; then - continue - fi - fi - prolog="\$(V_DGM)$cdir" - core_subst=$core - ;; - link) - [[ -z "$link_name" ]] && echo "link_name has to be specified in link rule" && exit 1 - [[ -z "$link_target" ]] && echo "link_target has to be specified in link rule" && exit 1 - dest="$link_name" - # shellcheck disable=SC2001 - core_subst=$(echo "$core" | sed -e "s~__link_target__~${link_target}~g") - prolog="\$(V_LN)$cdir" - ;; - esac - - case $version$dest$src in - *' '*) die "space not allowed in version, dest, src" ;; - esac - - target=$version/$dest - targets+="\\$nl $target" - - cat >> auto_targets.mk << EOF -$nl$target: $src \$(MANIFEST_FILE) - $prolog \\ - $core_subst${mode:+"; \\ - chmod $mode '\$@'"} -EOF - done - IFS=$OLD_IFS -} - -for version in ${VERSIONS}; do - # Get a working combination of distgen options for this version - while read -r combination; do - # line looks like: --distro rhel-7-x86_64.yaml --multispec-selector version=9.4 - echo "$combination" | grep "version=$version" &>/dev/null && break - done <<< "$DISTGEN_COMBINATIONS" - [ -z "$combination" ] && die "Could not find a working distgen options combination for version $version" - - # copy targets - rules="$COPY_RULES" - core="cp \$< \$@" - creator="copy" - parse_rules - COPY_TARGETS+="$targets" - - - # distgen targets - rules="$DISTGEN_RULES" - core="\$(DG) --multispec specs/multispec.yml \\ - --template \"\$<\" $combination --output \"\$@\"" - creator="distgen" - parse_rules - DISTGEN_TARGETS+="$targets" - - - rules=$SYMLINK_RULES - core="ln -nfs __link_target__ \$@" - creator="link" - parse_rules - SYMLINK_TARGETS+="$targets" -done - -while read -r combination; do - # line looks like: --distro rhel-7-x86_64.yaml --multispec-selector version=9.4 - eval 'set -- $combination' - case $4 in - version=*) version=${4##*=} ;; - *) die "version not found" - esac - case $2 in - *x86_64*) DG_CONF=$2 ;; - *) die "invalid --distro option" - esac - # distgen multi targets - rules="$DISTGEN_MULTI_RULES" - core="\$(DG) --multispec specs/multispec.yml \\ - --template \"\$<\" \\ - --output \"\$@\" \\ - $combination" - creator="distgen_multi" - parse_rules - DISTGEN_MULTI_TARGETS+="$targets" -done <<< "$DISTGEN_COMBINATIONS" - -cat -v >> auto_targets.mk < [ ... ]" -} - -verbose() { - if [ "${VERBOSE_OUTPUT}" -eq 1 ] ; then - echo "$@" >&2 - fi -} - -if [ $# -eq 0 ] ; then - echo "ERROR: No arguments given." - usage - exit 1 -fi - -case $1 in - -v|--verbose) VERBOSE_OUTPUT=1; shift ;; -esac - -filter_files() { - while read -r file ; do - if [ -L "$file" ] ; then - verbose "Ignoring symlink $file." - continue - fi - verbose "Will scan $file" - echo "$file" - done -} - -detect_shell_files() { - find -H "$@" -type f -not -path '*/\.git/*' -exec grep -l '^#!/bin/\(bash\|sh\)' {} + - find -H "$@" -name '*.sh' -not -path '*/\.git/*' -} - -# Run shellcheck on all files (we should also ignore symlinks) -detect_shell_files "$@" | filter_files | sort -u | xargs shellcheck - -# vim: set tabstop=2:shiftwidth=2:expandtab: diff --git a/common/squash.py b/common/squash.py deleted file mode 100755 index e92de5f..0000000 --- a/common/squash.py +++ /dev/null @@ -1,23 +0,0 @@ -#! /bin/python - -import sys -import logging -from platform import python_version - -try: - from docker_squash import squash -except ImportError: - logging.fatal("please install 'docker_squash' for Python {0}".format(python_version())) - sys.exit(1) - - -if __name__ == "__main__": - if len(sys.argv) != 3: - logging.fatal("%s: %s", sys.argv[0], msg) - sys.exit(1) - - print(squash.Squash( - log=logging.getLogger(), - image=sys.argv[1], - from_layer=sys.argv[2]).run() - ) diff --git a/common/tag.sh b/common/tag.sh deleted file mode 100755 index 0c67aa0..0000000 --- a/common/tag.sh +++ /dev/null @@ -1,34 +0,0 @@ -#!/bin/bash - -# This script is used to tag the OpenShift Docker images. -# -# Resulting image will be tagged: 'name:version' and 'name:latest'. Name and version -# are values of labels from resulted image -# -# VERSIONS - Must be set to a list with possible versions (subdirectories) - -set -e - -for dir in ${VERSIONS}; do - [ ! -e "${dir}/.image-id" ] && echo "-> Image for version $dir not built, skipping tag." && continue - pushd "${dir}" > /dev/null - IMAGE_ID=$(cat .image-id) - name=$(docker inspect -f "{{.Config.Labels.name}}" "$IMAGE_ID") - version=$(docker inspect -f "{{.Config.Labels.version}}" "$IMAGE_ID") - commit_date=$(git show -s HEAD --format=%cd --date=short | sed 's/-//g') - date_and_hash="${commit_date}-$(git rev-parse --short HEAD)" - - echo "-> Tagging image '$IMAGE_ID' as '$name:$version' and '$name:latest' and '$name:$date_and_hash'" - docker tag "$IMAGE_ID" "$name:$version" - docker tag "$IMAGE_ID" "$name:latest" - docker tag "$IMAGE_ID" "$name:$date_and_hash" - - for suffix in squashed raw; do - id_file=.image-id.$suffix - if test -f "$id_file"; then - docker tag "$(cat "$id_file")" "$name:$suffix" || rm .image-id."$suffix" - fi - done - - popd > /dev/null -done diff --git a/common/test-lib-openshift.sh b/common/test-lib-openshift.sh deleted file mode 100644 index c11a025..0000000 --- a/common/test-lib-openshift.sh +++ /dev/null @@ -1,1278 +0,0 @@ -# shellcheck shell=bash -# some functions are used from test-lib.sh, that is usually in the same dir -# shellcheck source=/dev/null -source "$(dirname "${BASH_SOURCE[0]}")"/test-lib.sh - -# Set of functions for testing docker images in OpenShift using 'oc' command - -# A variable containing the overall test result; must be changed to 0 in the end -# of the testing script: -# OS_TESTSUITE_RESULT=0 -# And the following trap must be set, in the beginning of the test script: -# trap ct_os_cleanup EXIT SIGINT -OS_TESTSUITE_RESULT=1 -OS_CLUSTER_STARTED_BY_TEST=0 - -function ct_os_cleanup() { - if [ $OS_TESTSUITE_RESULT -eq 0 ] ; then - # shellcheck disable=SC2153 - echo "OpenShift tests for ${IMAGE_NAME} succeeded." - else - # shellcheck disable=SC2153 - echo "OpenShift tests for ${IMAGE_NAME} failed." - fi -} - -# ct_os_check_compulsory_vars -# --------------------------- -# Check the compulsory variables: -# * IMAGE_NAME specifies a name of the candidate image used for testing. -# * VERSION specifies the major version of the MariaDB in format of X.Y -# * OS specifies RHEL version (e.g. OS=rhel7) -function ct_os_check_compulsory_vars() { - # shellcheck disable=SC2016 - test -n "${IMAGE_NAME-}" || ( echo 'make sure $IMAGE_NAME is defined' >&2 ; exit 1) - # shellcheck disable=SC2016 - test -n "${VERSION-}" || ( echo 'make sure $VERSION is defined' >&2 ; exit 1) - # shellcheck disable=SC2016 - test -n "${OS-}" || ( echo 'make sure $OS is defined' >&2 ; exit 1) -} - -# ct_os_get_status -# -------------------- -# Returns status of all objects to make debugging easier. -function ct_os_get_status() { - oc get all - oc status -} - -# ct_os_print_logs -# -------------------- -# Returns status of all objects and logs from all pods. -function ct_os_print_logs() { - ct_os_get_status - while read -r pod_name; do - echo "INFO: printing logs for pod ${pod_name}" - oc logs "${pod_name}" - done < <(oc get pods --no-headers=true -o custom-columns=NAME:.metadata.name) -} - -# ct_os_enable_print_logs -# -------------------- -# Enables automatic printing of pod logs on ERR. -function ct_os_enable_print_logs() { - set -E - trap ct_os_print_logs ERR -} - -# ct_get_public_ip -# -------------------- -# Returns best guess for the IP that the node is accessible from other computers. -# This is a bit funny heuristic, simply goes through all IPv4 addresses that -# hostname -I returns and de-prioritizes IP addresses commonly used for local -# addressing. The rest of addresses are taken as public with higher probability. -function ct_get_public_ip() { - local hostnames - local public_ip='' - local found_ip - hostnames=$(hostname -I) - for guess_exp in '127\.0\.0\.1' '192\.168\.[0-9\.]*' '172\.[0-9\.]*' \ - '10\.[0-9\.]*' '[0-9\.]*' ; do - found_ip=$(echo "${hostnames}" | grep -oe "${guess_exp}") - if [ -n "${found_ip}" ] ; then - # shellcheck disable=SC2001 - hostnames=$(echo "${hostnames}" | sed -e "s/${found_ip}//") - public_ip="${found_ip}" - fi - done - if [ -z "${public_ip}" ] ; then - echo "ERROR: public IP could not be guessed." >&2 - return 1 - fi - echo "${public_ip}" -} - -# ct_os_run_in_pod POD_NAME CMD -# -------------------- -# Runs [cmd] in the pod specified by prefix [pod_prefix]. -# Arguments: pod_name - full name of the pod -# Arguments: cmd - command to be run in the pod -function ct_os_run_in_pod() { - local pod_name="$1" ; shift - - oc exec "$pod_name" -- "$@" -} - -# ct_os_get_service_ip SERVICE_NAME -# -------------------- -# Returns IP of the service specified by [service_name]. -# Arguments: service_name - name of the service -function ct_os_get_service_ip() { - local service_name="${1}" ; shift - oc get "svc/${service_name}" -o yaml | grep clusterIP | \ - cut -d':' -f2 | grep -oe '172\.30\.[0-9\.]*' -} - - -# ct_os_get_all_pods_status -# -------------------- -# Returns status of all pods. -function ct_os_get_all_pods_status() { - oc get pods -o custom-columns=Ready:status.containerStatuses[0].ready,NAME:.metadata.name -} - -# ct_os_get_all_pods_name -# -------------------- -# Returns the full name of all pods. -function ct_os_get_all_pods_name() { - oc get pods --no-headers -o custom-columns=NAME:.metadata.name -} - -# ct_os_get_pod_status POD_PREFIX -# -------------------- -# Returns status of the pod specified by prefix [pod_prefix]. -# Note: Ignores -build and -deploy pods -# Arguments: pod_prefix - prefix or whole ID of the pod -function ct_os_get_pod_status() { - local pod_prefix="${1}" ; shift - ct_os_get_all_pods_status | grep -e "${pod_prefix}" | grep -Ev "(build|deploy)$" \ - | awk '{print $1}' | head -n 1 -} - -# ct_os_get_build_pod_status POD_PREFIX -# -------------------- -# Returns status of the build pod specified by prefix [pod_prefix]. -# Arguments: pod_prefix - prefix or whole ID of the pod -function ct_os_get_build_pod_status() { - local pod_prefix="${1}" ; shift - local query="custom-columns=NAME:.metadata.name,Ready:status.phase" - oc get pods -o "$query" | grep -e "${pod_prefix}" | grep -E "\-build\s" \ - | sort -u | awk '{print $2}' | tail -n 1 -} - -# ct_os_get_pod_name POD_PREFIX -# -------------------- -# Returns the full name of pods specified by prefix [pod_prefix]. -# Note: Ignores -build and -deploy pods -# Arguments: pod_prefix - prefix or whole ID of the pod -function ct_os_get_pod_name() { - local pod_prefix="${1}" ; shift - ct_os_get_all_pods_name | grep -e "^${pod_prefix}" | grep -Ev "(build|deploy)$" -} - -# ct_os_get_pod_ip POD_NAME -# -------------------- -# Returns the ip of the pod specified by [pod_name]. -# Arguments: pod_name - full name of the pod -function ct_os_get_pod_ip() { - local pod_name="${1}" - oc get pod "$pod_name" --no-headers -o custom-columns=IP:status.podIP -} - -# ct_os_check_pod_readiness POD_PREFIX STATUS -# -------------------- -# Checks whether the pod is ready. -# Arguments: pod_prefix - prefix or whole ID of the pod -# Arguments: status - expected status (true, false) -function ct_os_check_pod_readiness() { - local pod_prefix="${1}" ; shift - local status="${1}" ; shift - test "$(ct_os_get_pod_status "${pod_prefix}")" == "${status}" -} - -# ct_os_wait_pod_ready POD_PREFIX TIMEOUT -# -------------------- -# Wait maximum [timeout] for the pod becomming ready. -# Arguments: pod_prefix - prefix or whole ID of the pod -# Arguments: timeout - how many seconds to wait seconds -function ct_os_wait_pod_ready() { - local pod_prefix="${1}" ; shift - local timeout="${1}" ; shift - # If there is a build pod - wait for it to finish first - sleep 3 - if ct_os_get_all_pods_name | grep -E "${pod_prefix}.*-build"; then - SECONDS=0 - echo -n "Waiting for ${pod_prefix} build pod to finish ..." - while ! [ "$(ct_os_get_build_pod_status "${pod_prefix}")" == "Succeeded" ] ; do - echo -n "." - [ "${SECONDS}" -gt "${timeout}0" ] && echo " FAIL" && return 1 - sleep 3 - done - echo " DONE" - fi - SECONDS=0 - echo -n "Waiting for ${pod_prefix} pod becoming ready ..." - while ! ct_os_check_pod_readiness "${pod_prefix}" "true" ; do - echo -n "." - [ "${SECONDS}" -gt "${timeout}" ] && echo " FAIL" && return 1 - sleep 3 - done - echo " DONE" -} - -# ct_os_wait_rc_ready POD_PREFIX TIMEOUT -# -------------------- -# Wait maximum [timeout] for the rc having desired number of replicas ready. -# Arguments: pod_prefix - prefix of the replication controller -# Arguments: timeout - how many seconds to wait seconds -function ct_os_wait_rc_ready() { - local pod_prefix="${1}" ; shift - local timeout="${1}" ; shift - SECONDS=0 - echo -n "Waiting for ${pod_prefix} having desired numbers of replicas ..." - while ! test "$( (oc get --no-headers statefulsets; oc get --no-headers rc) 2>/dev/null \ - | grep "^${pod_prefix}" | awk '$2==$3 {print "ready"}')" == "ready" ; do - echo -n "." - [ "${SECONDS}" -gt "${timeout}" ] && echo " FAIL" && return 1 - sleep 3 - done - echo " DONE" -} - -# ct_os_deploy_pure_image IMAGE [ENV_PARAMS, ...] -# -------------------- -# Runs [image] in the openshift and optionally specifies env_params -# as environment variables to the image. -# Arguments: image - prefix or whole ID of the pod to run the cmd in -# Arguments: env_params - environment variables parameters for the images. -function ct_os_deploy_pure_image() { - local image="${1}" ; shift - # ignore error exit code, because oc new-app returns error when image exists - oc new-app "${image}" "$@" || : - # let openshift cluster to sync to avoid some race condition errors - sleep 3 -} - -# ct_os_deploy_s2i_image IMAGE APP [ENV_PARAMS, ... ] -# -------------------- -# Runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. -# Arguments: image - prefix or whole ID of the pod to run the cmd in -# Arguments: app - url or local path to git repo with the application sources. -# Arguments: env_params - environment variables parameters for the images. -function ct_os_deploy_s2i_image() { - local image="${1}" ; shift - local app="${1}" ; shift - # ignore error exit code, because oc new-app returns error when image exists - oc new-app "${image}~${app}" --strategy=source "$@" || : - - # let openshift cluster to sync to avoid some race condition errors - sleep 3 -} - -# ct_os_deploy_template_image TEMPLATE [ENV_PARAMS, ...] -# -------------------- -# Runs template in the openshift and optionally gives env_params to use -# specific values in the template. -# Arguments: template - prefix or whole ID of the pod to run the cmd in -# Arguments: env_params - environment variables parameters for the template. -# Example usage: ct_os_deploy_template_image mariadb-ephemeral-template.yaml \ -# DATABASE_SERVICE_NAME=mysql-57-centos7 \ -# DATABASE_IMAGE=mysql-57-centos7 \ -# MYSQL_USER=testu \ -# MYSQL_PASSWORD=testp \ -# MYSQL_DATABASE=testdb -function ct_os_deploy_template_image() { - local template="${1}" ; shift - oc process -f "${template}" "$@" | oc create -f - - # let openshift cluster to sync to avoid some race condition errors - sleep 3 -} - -# _ct_os_get_uniq_project_name -# -------------------- -# Returns a uniq name of the OpenShift project. -function _ct_os_get_uniq_project_name() { - local r - while true ; do - r=${RANDOM} - mkdir /var/tmp/sclorg-test-${r} &>/dev/null && echo sclorg-test-${r} && break - done -} - -# ct_os_new_project [PROJECT] -# -------------------- -# Creates a new project in the openshfit using 'os' command. -# Arguments: project - project name, uses a new random name if omitted -# Expects 'os' command that is properly logged in to the OpenShift cluster. -# Not using mktemp, because we cannot use uppercase characters. -# The OPENSHIFT_CLUSTER_PULLSECRET_PATH environment variable can be set -# to contain a path to a k8s secret definition which will be used -# to authenticate to image registries. -# shellcheck disable=SC2120 -function ct_os_new_project() { - if [ "${CT_SKIP_NEW_PROJECT:-false}" == 'true' ] ; then - echo "Creating project skipped." - return - fi - local project_name="${1:-$(_ct_os_get_uniq_project_name)}" ; shift || : - oc new-project "${project_name}" - # let openshift cluster to sync to avoid some race condition errors - sleep 3 - if test -n "${OPENSHIFT_CLUSTER_PULLSECRET_PATH:-}" -a -e "${OPENSHIFT_CLUSTER_PULLSECRET_PATH:-}"; then - oc create -f "$OPENSHIFT_CLUSTER_PULLSECRET_PATH" - # add registry pullsecret to the serviceaccount if provided - secret_name=$(grep '^\s*name:' "$OPENSHIFT_CLUSTER_PULLSECRET_PATH" | awk '{ print $2 }') - oc secrets link --for=pull default "$secret_name" - fi -} - -# ct_os_delete_project [PROJECT] -# -------------------- -# Deletes the specified project in the openshfit -# Arguments: project - project name, uses the current project if omitted -# shellcheck disable=SC2120 -function ct_os_delete_project() { - if [ "${CT_SKIP_NEW_PROJECT:-false}" == 'true' ] ; then - echo "Deleting project skipped, cleaning objects only." - # when not having enough privileges (remote cluster), it might fail and - # it is not a big problem, so ignore failure in this case - ct_delete_all_objects || : - return - fi - local project_name="${1:-$(oc project -q)}" ; shift || : - oc delete project "${project_name}" -} - -# ct_delete_all_objects -# ----------------- -# Deletes all objects within the project. -# Handy when we have one project and want to run more tests. -function ct_delete_all_objects() { - for x in bc builds dc is isimage istag po pv pvc rc routes secrets svc ; do - oc delete "$x" --all - done - # for some objects it takes longer to be really deleted, so a dummy sleep - # to avoid some races when other test can see not-yet-deleted objects and can fail - sleep 10 -} - -# ct_os_docker_login -# -------------------- -# Logs in into docker daemon -# Uses global REGISRTY_ADDRESS environment variable for arbitrary registry address. -# Does not do anything if REGISTRY_ADDRESS is set. -function ct_os_docker_login() { - [ -n "${REGISTRY_ADDRESS:-}" ] && "REGISTRY_ADDRESS set, not trying to docker login." && return 0 - # docker login fails with "404 page not found" error sometimes, just try it more times - # shellcheck disable=SC2034 - for i in $(seq 12) ; do - # shellcheck disable=SC2015 - docker login -u developer -p "$(oc whoami -t)" "${REGISRTY_ADDRESS:-172.30.1.1:5000}" && return 0 || : - sleep 5 - done - return 1 -} - -# ct_os_upload_image IMAGE [IMAGESTREAM] -# -------------------- -# Uploads image from local registry to the OpenShift internal registry. -# Arguments: image - image name to upload -# Arguments: imagestream - name and tag to use for the internal registry. -# In the format of name:tag ($image_name:latest by default) -# Uses global REGISRTY_ADDRESS environment variable for arbitrary registry address. -function ct_os_upload_image() { - local input_name="${1}" ; shift - local image_name=${input_name##*/} - local imagestream=${1:-$image_name:latest} - local output_name - - output_name="${REGISRTY_ADDRESS:-172.30.1.1:5000}/$(oc project -q)/$imagestream" - - ct_os_docker_login - docker tag "${input_name}" "${output_name}" - docker push "${output_name}" -} - -# ct_os_is_tag_exists IS_NAME TAG -# -------------------- -# Checks whether the specified tag exists for an image stream -# Arguments: is_name - name of the image stream -# Arguments: tag - name of the tag (usually version) -function ct_os_is_tag_exists() { - local is_name=$1 ; shift - local tag=$1 ; shift - oc get is "${is_name}" -n openshift -o=jsonpath='{.spec.tags[*].name}' | grep -qw "${tag}" -} - -# ct_os_template_exists T_NAME -# -------------------- -# Checks whether the specified template exists for an image stream -# Arguments: t_name - template name of the image stream -function ct_os_template_exists() { - local t_name=$1 ; shift - oc get templates -n openshift | grep -q "^${t_name}\s" -} - -# ct_os_install_in_centos -# -------------------- -# Installs os cluster in CentOS -function ct_os_install_in_centos() { - yum install -y centos-release-openshift-origin - yum install -y wget git net-tools bind-utils iptables-services bridge-utils\ - bash-completion origin-clients docker origin-clients -} - - -# ct_os_cluster_up [DIR, IS_PUBLIC, CLUSTER_VERSION] -# -------------------- -# Runs the local OpenShift cluster using 'oc cluster up' and logs in as developer. -# Arguments: dir - directory to keep configuration data in, random if omitted -# Arguments: is_public - sets either private or public hostname for web-UI, -# use "true" for allow remote access to the web-UI, -# "false" is default -# Arguments: cluster_version - version of the OpenShift cluster to use, empty -# means default version of `oc`; example value: 3.7; -# also can be specified outside by OC_CLUSTER_VERSION -function ct_os_cluster_up() { - ct_os_cluster_running && echo "Cluster already running. Nothing is done." && return 0 - ct_os_logged_in && echo "Already logged in to a cluster. Nothing is done." && return 0 - - mkdir -p /var/tmp/openshift - local dir="${1:-$(mktemp -d /var/tmp/openshift/os-data-XXXXXX)}" ; shift || : - local is_public="${1:-'false'}" ; shift || : - local default_cluster_version=${OC_CLUSTER_VERSION:-} - local cluster_version=${1:-${default_cluster_version}} ; shift || : - if ! grep -qe '--insecure-registry.*172\.30\.0\.0' /etc/sysconfig/docker ; then - sed -i "s|OPTIONS='|OPTIONS='--insecure-registry 172.30.0.0/16 |" /etc/sysconfig/docker - fi - - systemctl stop firewalld || : - setenforce 0 - iptables -F - - systemctl restart docker - local cluster_ip="127.0.0.1" - [ "${is_public}" == "true" ] && cluster_ip=$(ct_get_public_ip) - - if [ -n "${cluster_version}" ] ; then - # if $cluster_version is not set, we simply use oc that is available - ct_os_set_path_oc "${cluster_version}" - fi - - mkdir -p "${dir}"/{config,data,pv} - case $(oc version| head -n 1) in - "oc v3.1"?.*) - oc cluster up --base-dir="${dir}/data" --public-hostname="${cluster_ip}" - ;; - "oc v3."*) - oc cluster up --host-data-dir="${dir}/data" --host-config-dir="${dir}/config" \ - --host-pv-dir="${dir}/pv" --use-existing-config --public-hostname="${cluster_ip}" - ;; - *) - echo "ERROR: Unexpected oc version." >&2 - return 1 - ;; - esac - oc version - oc login -u system:admin - oc project default - ct_os_wait_rc_ready docker-registry 180 - ct_os_wait_rc_ready router 30 - oc login -u developer -p developer - OS_CLUSTER_STARTED_BY_TEST=1 - # let openshift cluster to sync to avoid some race condition errors - sleep 3 -} - -# ct_os_cluster_down -# -------------------- -# Shuts down the local OpenShift cluster using 'oc cluster down' -function ct_os_cluster_down() { - if [ ${OS_CLUSTER_STARTED_BY_TEST:-0} -eq 1 ] ; then - echo "Cluster started by the test, shutting down." - oc cluster down - else - echo "Cluster not started by the test, shutting down skipped." - fi -} - -# ct_os_cluster_running -# -------------------- -# Returns 0 if oc cluster is running -function ct_os_cluster_running() { - oc cluster status &>/dev/null -} - -# ct_os_logged_in -# --------------- -# Returns 0 if logged in to a cluster (remote or local) -function ct_os_logged_in() { - oc whoami >/dev/null -} - -# ct_os_set_path_oc OC_VERSION -# -------------------- -# This is a trick that helps using correct version of the `oc`: -# The input is version of the openshift in format v3.6.0 etc. -# If the currently available version of oc is not of this version, -# it first takes a look into /usr/local/oc-/bin directory, -# and if not found there it downloads the community release from github. -# In the end the PATH variable is changed, so the other tests can still use just 'oc'. -# Arguments: oc_version - X.Y part of the version of OSE (e.g. 3.9) -function ct_os_set_path_oc() { - local oc_version - local oc_path - - oc_version=$(ct_os_get_latest_ver "$1") - - if oc version | grep -q "oc ${oc_version%.*}." ; then - echo "Binary oc found already available in version ${oc_version}: $(command -v oc) Doing noting." - return 0 - fi - - # first check whether we already have oc available in /usr/local - local installed_oc_path="/usr/local/oc-${oc_version%.*}/bin" - - if [ -x "${installed_oc_path}/oc" ] ; then - oc_path="${installed_oc_path}" - echo "Binary oc found in ${installed_oc_path}" >&2 - else - # oc not available in /usr/local, try to download it from github (community release) - oc_path="/tmp/oc-${oc_version}-bin" - ct_os_download_upstream_oc "${oc_version}" "${oc_path}" - fi - if [ -z "${oc_path}" ] ; then - echo "ERROR: oc not found installed, nor downloaded" >&1 - return 1 - fi - export PATH="${oc_path}:${PATH}" - if ! oc version | grep -q "oc ${oc_version%.*}." ; then - echo "ERROR: something went wrong, oc located at ${oc_path}, but oc of version ${oc_version} not found in PATH ($PATH)" >&1 - return 1 - else - echo "PATH set correctly, binary oc found in version ${oc_version}: $(command -v oc)" - fi -} - -# ct_os_get_latest_ver VERSION_PART_X -# -------------------- -# Returns full version (vX.Y.Z) from part of the version (X.Y) -# Arguments: vxy - X.Y part of the version -# Returns vX.Y.Z variant of the version -function ct_os_get_latest_ver(){ - local vxy="v$1" - for vz in {3..0} ; do - curl -sif "https://github.com/openshift/origin/releases/tag/${vxy}.${vz}" >/dev/null && echo "${vxy}.${vz}" && return 0 - done - echo "ERROR: version ${vxy} not found in https://github.com/openshift/origin/tags" >&2 - return 1 -} - -# ct_os_download_upstream_oc OC_VERSION OUTPUT_DIR -# -------------------- -# Downloads a particular version of openshift-origin-client-tools from -# github into specified output directory -# Arguments: oc_version - version of OSE (e.g. v3.7.2) -# Arguments: output_dir - output directory -function ct_os_download_upstream_oc() { - local oc_version=$1 - local output_dir=$2 - - # check whether we already have the binary in place - [ -x "${output_dir}/oc" ] && return 0 - - mkdir -p "${output_dir}" - # using html output instead of https://api.github.com/repos/openshift/origin/releases/tags/${oc_version}, - # because API is limited for number of queries if not authenticated - tarball=$(curl -si "https://github.com/openshift/origin/releases/tag/${oc_version}" | grep -o -e "openshift-origin-client-tools-${oc_version}-[a-f0-9]*-linux-64bit.tar.gz" | head -n 1) - - # download, unpack the binaries and then put them into output directory - echo "Downloading https://github.com/openshift/origin/releases/download/${oc_version}/${tarball} into ${output_dir}/" >&2 - curl -sL https://github.com/openshift/origin/releases/download/"${oc_version}"/"${tarball}" | tar -C "${output_dir}" -xz - mv -f "${output_dir}"/"${tarball%.tar.gz}"/* "${output_dir}/" - - rmdir "${output_dir}"/"${tarball%.tar.gz}" -} - - -# ct_os_test_s2i_app_func IMAGE APP CONTEXT_DIR CHECK_CMD [OC_ARGS] -# -------------------- -# Runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. Then check the container by arbitrary -# function given as argument (such an argument may include string, -# that will be replaced with actual IP). -# Arguments: image - prefix or whole ID of the pod to run the cmd in (compulsory) -# Arguments: app - url or local path to git repo with the application sources (compulsory) -# Arguments: context_dir - sub-directory inside the repository with the application sources (compulsory) -# Arguments: check_command - CMD line that checks whether the container works (compulsory; '' will be replaced with actual IP) -# Arguments: oc_args - all other arguments are used as additional parameters for the `oc new-app` -# command, typically environment variables (optional) -function ct_os_test_s2i_app_func() { - local image_name=${1} - local app=${2} - local context_dir=${3} - local check_command=${4} - local oc_args=${5:-} - local image_name_no_namespace=${image_name##*/} - local service_name="${image_name_no_namespace%%:*}-testing" - local namespace - - if [ $# -lt 4 ] || [ -z "${1}" ] || [ -z "${2}" ] || [ -z "${3}" ] || [ -z "${4}" ]; then - echo "ERROR: ct_os_test_s2i_app_func() requires at least 4 arguments that cannot be emtpy." >&2 - return 1 - fi - - # shellcheck disable=SC2119 - ct_os_new_project - - namespace=${CT_NAMESPACE:-"$(oc project -q)"} - - if [ "${CT_EXTERNAL_REGISTRY:-false}" == 'true' ] ; then - ct_os_import_image_ocp4 "${image_name}" "${image_name_no_namespace}" - else - # Create a specific imagestream tag for the image so that oc cannot use anything else - if [ "${CT_SKIP_UPLOAD_IMAGE:-false}" == 'true' ] ; then - echo "Importing image ${image_name} as ${namespace}/${image_name_no_namespace}" - # Use --reference-policy=local to pull remote image content to the cluster - # Works around the issue of builder pods not having access to registry.redhat.io - oc tag --source=docker "${image_name}" "${namespace}/${image_name_no_namespace}" --insecure=true --reference-policy=local - ct_os_wait_stream_ready "${image_name_no_namespace}" "${namespace}" - else - echo "Uploading image ${image_name} as ${image_name_no_namespace}" - ct_os_upload_image "${image_name}" "${image_name_no_namespace}" - fi - fi - - local app_param="${app}" - if [ -d "${app}" ] ; then - # for local directory, we need to copy the content, otherwise too smart os command - # pulls the git remote repository instead - app_param=$(ct_obtain_input "${app}") - fi - - # shellcheck disable=SC2086 - ct_os_deploy_s2i_image "${image_name_no_namespace}" "${app_param}" \ - --context-dir="${context_dir}" \ - --name "${service_name}" \ - ${oc_args} - - if [ -d "${app}" ] ; then - # in order to avoid weird race seen sometimes, let's wait shortly - # before starting the build explicitly - sleep 5 - oc start-build "${service_name}" --from-dir="${app_param}" - fi - - ct_os_wait_pod_ready "${service_name}" 300 - - local ip - local check_command_exp - local image_id - - # get image ID from the deployment config - image_id=$(oc get "deploymentconfig.apps.openshift.io/${service_name}" -o custom-columns=IMAGE:.spec.template.spec.containers[*].image | tail -n 1) - - ip=$(ct_os_get_service_ip "${service_name}") - # shellcheck disable=SC2001 - check_command_exp=$(echo "$check_command" | sed -e "s//$ip/g" -e "s||${image_id}|g") - - echo " Checking APP using $check_command_exp ..." - local result=0 - eval "$check_command_exp" || result=1 - - ct_os_service_image_info "${service_name}" - - if [ $result -eq 0 ] ; then - echo " Check passed." - else - echo " Check failed." - fi - - # shellcheck disable=SC2119 - ct_os_delete_project - return $result -} - -# ct_os_test_s2i_app IMAGE APP CONTEXT_DIR EXPECTED_OUTPUT [PORT, PROTOCOL, RESPONSE_CODE, OC_ARGS, ... ] -# -------------------- -# Runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. Then check the http response. -# Arguments: image - prefix or whole ID of the pod to run the cmd in (compulsory) -# Arguments: app - url or local path to git repo with the application sources (compulsory) -# Arguments: context_dir - sub-directory inside the repository with the application sources (compulsory) -# Arguments: expected_output - PCRE regular expression that must match the response body (compulsory) -# Arguments: port - which port to use (optional; default: 8080) -# Arguments: protocol - which protocol to use (optional; default: http) -# Arguments: response_code - what http response code to expect (optional; default: 200) -# Arguments: oc_args - all other arguments are used as additional parameters for the `oc new-app` -# command, typically environment variables (optional) -function ct_os_test_s2i_app() { - local image_name=${1} - local app=${2} - local context_dir=${3} - local expected_output=${4} - local port=${5:-8080} - local protocol=${6:-http} - local response_code=${7:-200} - local oc_args=${8:-} - - if [ $# -lt 4 ] || [ -z "${1}" ] || [ -z "${2}" ] || [ -z "${3}" ] || [ -z "${4}" ]; then - echo "ERROR: ct_os_test_s2i_app() requires at least 4 arguments that cannot be emtpy." >&2 - return 1 - fi - - ct_os_test_s2i_app_func "${image_name}" \ - "${app}" \ - "${context_dir}" \ - "ct_os_test_response_internal '${protocol}://:${port}' '${response_code}' '${expected_output}'" \ - "${oc_args}" -} - -# ct_os_test_template_app_func IMAGE APP IMAGE_IN_TEMPLATE CHECK_CMD [OC_ARGS] -# -------------------- -# Runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. Then check the container by arbitrary -# function given as argument (such an argument may include string, -# that will be replaced with actual IP). -# Arguments: image_name - prefix or whole ID of the pod to run the cmd in (compulsory) -# Arguments: template - url or local path to a template to use (compulsory) -# Arguments: name_in_template - image name used in the template -# Arguments: check_command - CMD line that checks whether the container works (compulsory; '' will be replaced with actual IP) -# Arguments: oc_args - all other arguments are used as additional parameters for the `oc new-app` -# command, typically environment variables (optional) -# Arguments: other_images - some templates need other image to be pushed into the OpenShift registry, -# specify them in this parameter as "|", where "" is a full image name -# (including registry if needed) and "" is a tag under which the image should be available -# in the OpenShift registry. -function ct_os_test_template_app_func() { - local image_name=${1} - local template=${2} - local name_in_template=${3} - local check_command=${4} - local oc_args=${5:-} - local other_images=${6:-} - - if [ $# -lt 4 ] || [ -z "${1}" ] || [ -z "${2}" ] || [ -z "${3}" ] || [ -z "${4}" ]; then - echo "ERROR: ct_os_test_template_app_func() requires at least 4 arguments that cannot be emtpy." >&2 - return 1 - fi - - local service_name="${name_in_template}-testing" - local image_tagged="${name_in_template}:${VERSION}" - local namespace - - # shellcheck disable=SC2119 - ct_os_new_project - - namespace=${CT_NAMESPACE:-"$(oc project -q)"} - # Create a specific imagestream tag for the image so that oc cannot use anything else - if [ "${CT_EXTERNAL_REGISTRY:-false}" == 'true' ] ; then - ct_os_import_image_ocp4 "${image_name}" "${image_tagged}" - else - if [ "${CT_SKIP_UPLOAD_IMAGE:-false}" == 'true' ] ; then - echo "Importing image ${image_name} as ${image_tagged}" - # Use --reference-policy=local to pull remote image content to the cluster - # Works around the issue of builder pods not having access to registry.redhat.io - oc tag --source=docker "${image_name}" "${namespace}/${image_tagged}" --insecure=true --reference-policy=local - ct_os_wait_stream_ready "${image_tagged}" "${namespace}" - else - echo "Uploading image ${image_name} as ${image_tagged}" - ct_os_upload_image "${image_name}" "${image_tagged}" - fi - fi - if [ "${CT_SKIP_UPLOAD_IMAGE:-false}" == 'false' ] ; then - # upload also other images, that template might need (list of pairs in the format | - local image_tag_a - local i_t - for i_t in ${other_images} ; do - echo "${i_t}" - IFS='|' read -ra image_tag_a <<< "${i_t}" - docker pull "${image_tag_a[0]}" - if [ "${CT_EXTERNAL_REGISTRY:-false}" == 'true' ] ; then - ct_os_import_image_ocp4 "${image_tag_a[0]}" "${image_tag_a[1]}" - else - ct_os_upload_image "${image_tag_a[0]}" "${image_tag_a[1]}" - fi - done - fi - - # get the template file from remote or local location; if not found, it is - # considered an internal template name, like 'mysql', so use the name - # explicitly - local local_template - - local_template=$(ct_obtain_input "${template}" 2>/dev/null || echo "--template=${template}") - # shellcheck disable=SC2086 - oc new-app "${local_template}" \ - --name "${name_in_template}" \ - -p NAMESPACE="${namespace}" \ - ${oc_args} - - ct_os_wait_pod_ready "${service_name}" 300 - - local ip - local check_command_exp - local image_id - - # get image ID from the deployment config - image_id=$(oc get "deploymentconfig.apps.openshift.io/${service_name}" -o custom-columns=IMAGE:.spec.template.spec.containers[*].image | tail -n 1) - - ip=$(ct_os_get_service_ip "${service_name}") - # shellcheck disable=SC2001 - check_command_exp=$(echo "$check_command" | sed -e "s//$ip/g" -e "s||${image_id}|g") - - echo " Checking APP using $check_command_exp ..." - local result=0 - eval "$check_command_exp" || result=1 - - ct_os_service_image_info "${service_name}" - - if [ $result -eq 0 ] ; then - echo " Check passed." - else - echo " Check failed." - fi - - # shellcheck disable=SC2119 - ct_os_delete_project - return $result -} - -# params: -# ct_os_test_template_app IMAGE APP IMAGE_IN_TEMPLATE EXPECTED_OUTPUT [PORT, PROTOCOL, RESPONSE_CODE, OC_ARGS, ... ] -# -------------------- -# Runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. Then check the http response. -# Arguments: image_name - prefix or whole ID of the pod to run the cmd in (compulsory) -# Arguments: template - url or local path to a template to use (compulsory) -# Arguments: name_in_template - image name used in the template -# Arguments: expected_output - PCRE regular expression that must match the response body (compulsory) -# Arguments: port - which port to use (optional; default: 8080) -# Arguments: protocol - which protocol to use (optional; default: http) -# Arguments: response_code - what http response code to expect (optional; default: 200) -# Arguments: oc_args - all other arguments are used as additional parameters for the `oc new-app` -# command, typically environment variables (optional) -# Arguments: other_images - some templates need other image to be pushed into the OpenShift registry, -# specify them in this parameter as "|", where "" is a full image name -# (including registry if needed) and "" is a tag under which the image should be available -# in the OpenShift registry. -function ct_os_test_template_app() { - local image_name=${1} - local template=${2} - local name_in_template=${3} - local expected_output=${4} - local port=${5:-8080} - local protocol=${6:-http} - local response_code=${7:-200} - local oc_args=${8:-} - local other_images=${9:-} - - if [ $# -lt 4 ] || [ -z "${1}" ] || [ -z "${2}" ] || [ -z "${3}" ] || [ -z "${4}" ]; then - echo "ERROR: ct_os_test_template_app() requires at least 4 arguments that cannot be emtpy." >&2 - return 1 - fi - - ct_os_test_template_app_func "${image_name}" \ - "${template}" \ - "${name_in_template}" \ - "ct_os_test_response_internal '${protocol}://:${port}' '${response_code}' '${expected_output}'" \ - "${oc_args}" \ - "${other_images}" -} - -# ct_os_test_image_update IMAGE_NAME OLD_IMAGE ISTAG CHECK_FUNCTION OC_ARGS -# -------------------- -# Runs an image update test with [image] uploaded to [is] imagestream -# and checks the services using an arbitrary function provided in [check_function]. -# Arguments: image_name - prefix or whole ID of the pod to run the cmd in (compulsory) -# Arguments: old_image - valid name of the image from the registry -# Arguments: istag - imagestream to upload the images into (compulsory) -# Arguments: check_function - command to be run to check functionality of created services (compulsory) -# Arguments: oc_args - arguments to use during oc new-app (compulsory) -ct_os_test_image_update() { - local image_name=$1; shift - local old_image=$1; shift - local istag=$1; shift - local check_function=$1; shift - local service_name=${image_name##*/} - local ip="" check_command_exp="" - - echo "Running image update test for: $image_name" - # shellcheck disable=SC2119 - ct_os_new_project - - # Get current image from repository and create an imagestream - docker pull "$old_image:latest" 2>/dev/null - ct_os_upload_image "$old_image" "$istag" - - # Setup example application with curent image - oc new-app "$@" --name "$service_name" - ct_os_wait_pod_ready "$service_name" 60 - - # Check application output - ip=$(ct_os_get_service_ip "$service_name") - check_command_exp=${check_function///$ip} - ct_assert_cmd_success "$check_command_exp" - - # Tag built image into the imagestream and wait for rebuild - ct_os_upload_image "$image_name" "$istag" - ct_os_wait_pod_ready "${service_name}-2" 60 - - # Check application output - ip=$(ct_os_get_service_ip "$service_name") - check_command_exp=${check_function///$ip} - ct_assert_cmd_success "$check_command_exp" - - # shellcheck disable=SC2119 - ct_os_delete_project -} - -# ct_os_deploy_cmd_image IMAGE_NAME -# -------------------- -# Runs a special command pod, a pod that does nothing, but includes utilities for testing. -# A typical usage is a mysql pod that includes mysql commandline, that we need for testing. -# Running commands inside this command pod is done via ct_os_cmd_image_run function. -# The pod is not run again if already running. -# Arguments: image_name - image to be used as a command pod -function ct_os_deploy_cmd_image() { - local image_name=${1} - oc get pod command-app &>/dev/null && echo "command POD already running" && return 0 - echo "command POD not running yet, will start one called command-app" - oc create -f - <" - local sleep_time=3 - local attempt=1 - local result=1 - local status - local response_code - local response_file - local util_image_name='python:3.6' - - response_file=$(mktemp /tmp/ct_test_response_XXXXXX) - ct_os_deploy_cmd_image "${util_image_name}" - - while [ "${attempt}" -le "${max_attempts}" ]; do - ct_os_cmd_image_run "curl --connect-timeout 10 -s -w '%{http_code}' '${url}'" >"${response_file}" && status=0 || status=1 - if [ "${status}" -eq 0 ]; then - response_code=$(tail -c 3 "${response_file}") - if [ "${response_code}" -eq "${expected_code}" ]; then - result=0 - fi - grep -qP -e "${body_regexp}" "${response_file}" || result=1; - # Some services return 40x code until they are ready, so let's give them - # some chance and not end with failure right away - # Do not wait if we already have expected outcome though - if [ "${result}" -eq 0 ] || [ "${attempt}" -gt "${ignore_error_attempts}" ] || [ "${attempt}" -eq "${max_attempts}" ] ; then - break - fi - fi - attempt=$(( attempt + 1 )) - sleep "${sleep_time}" - done - rm -f "${response_file}" - return "${result}" -} - -# ct_os_get_image_from_pod -# ------------------------ -# Print image identifier from an existing pod to stdout -# Argument: pod_prefix - prefix or full name of the pod to get image from -ct_os_get_image_from_pod() { - local pod_prefix=$1 ; shift - local pod_name - pod_name=$(ct_os_get_pod_name "$pod_prefix") - oc get "po/${pod_name}" -o yaml | sed -ne 's/^\s*image:\s*\(.*\)\s*$/\1/ p' | head -1 -} - -# ct_os_check_cmd_internal -# ---------------- -# Runs a specified command, checks exit code and compares the output with expected regexp. -# That all is done inside an image in the cluster, so the function is used -# typically in clusters that are not accessible outside. -# The check is repeated until timeout. -# Argument: util_image_name - name of the image in the cluster that is used for running the cmd -# Argument: service_name - kubernetes' service name to work with (IP address is taken from this one) -# Argument: check_command - command that is run within the util_image_name container -# Argument: expected_content_match - regexp that must be in the output (use .* to ignore check) -# Argument: timeout - number of seconds to wait till the check succeeds -function ct_os_check_cmd_internal() { - local util_image_name=$1 ; shift - local service_name=$1 ; shift - local check_command=$1 ; shift - local expected_content_match=${1:-.*} ; shift - local timeout=${1:-60} ; shift || : - - : " Service ${service_name} check ..." - - local output - local ret - local ip - local check_command_exp - - ip=$(ct_os_get_service_ip "${service_name}") - # shellcheck disable=SC2001 - check_command_exp=$(echo "$check_command" | sed -e "s//$ip/g") - - ct_os_deploy_cmd_image "${util_image_name}" - SECONDS=0 - - echo -n "Waiting for ${service_name} service becoming ready ..." - while true ; do - output=$(ct_os_cmd_image_run "$check_command_exp") - ret=$? - echo "${output}" | grep -qe "${expected_content_match}" || ret=1 - if [ ${ret} -eq 0 ] ; then - echo " PASS" - return 0 - fi - echo -n "." - [ ${SECONDS} -gt "${timeout}" ] && break - sleep 3 - done - echo " FAIL" - return 1 -} - -# ct_os_test_image_stream_template IMAGE_STREAM_FILE TEMPLATE_FILE SERVICE NAME [TEMPLATE_PARAMS] -# ------------------------ -# Creates an image stream and deploys a specified template. Then checks that a pod runs. -# Argument: image_stream_file - local or remote file with the image stream definition -# Argument: template_file - local file name with a template -# Argument: service_name - how the pod will be named (prefix) -# Argument: template_params (optional) - parameters for the template, like image stream version -function ct_os_test_image_stream_template() { - local image_stream_file=${1} - local template_file=${2} - local service_name=${3} - local template_params=${4:-} - local local_image_stream_file - local local_template_file - - if [ $# -lt 3 ] || [ -z "${1}" ] || [ -z "${2}" ] || [ -z "${3}" ]; then - echo "ERROR: ct_os_test_image_stream() requires at least 3 arguments that cannot be empty." >&2 - return 1 - fi - - echo "Running image stream test for stream ${image_stream_file} and template ${template_file}" - # shellcheck disable=SC2119 - ct_os_new_project - - local_image_stream_file=$(ct_obtain_input "${image_stream_file}") - local_template_file=$(ct_obtain_input "${template_file}") - oc create -f "${local_image_stream_file}" - - # shellcheck disable=SC2086 - if ! ct_os_deploy_template_image "${local_template_file}" -p NAMESPACE="${CT_NAMESPACE:-$(oc project -q)}" ${template_params} ; then - echo "ERROR: ${template_file} could not be loaded" - return 1 - # Deliberately not runnig ct_os_delete_project here because user either - # might want to investigate or the cleanup is done with the cleanup trap. - # Most functions depend on the set -e anyway at this point. - fi - ct_os_wait_pod_ready "${service_name}" 120 - - # shellcheck disable=SC2119 - ct_os_delete_project -} - -# ct_os_wait_stream_ready IMAGE_STREAM_FILE NAMESPACE [ TIMEOUT ] -# ------------------------ -# Waits max timeout seconds till a [stream] is available in the [namespace]. -# Arguments: image_stream - stream name (usuallly :) -# Arguments: namespace - namespace name -# Arguments: timeout - how many seconds to wait -function ct_os_wait_stream_ready() { - local image_stream=${1} - local namespace=${2} - local timeout=${3:-60} - # It takes some time for the first time before the image is pulled in - SECONDS=0 - echo -n "Waiting for ${namespace}/${image_stream} to become available ..." - while ! oc get -n "${namespace}" istag "${image_stream}" &>/dev/null; do - if [ "$SECONDS" -gt "${timeout}" ] ; then - echo "FAIL: ${namespace}/${image_stream} not available after ${timeout}s:" - echo "oc get -n ${namespace} istag ${image_stream}" - oc get -n "${namespace}" istag "${image_stream}" - return 1 - fi - sleep 3 - echo -n . - done - echo " DONE" -} - -# ct_os_test_image_stream_s2i IMAGE_STREAM_FILE IMAGE_NAME APP CONTEXT_DIR EXPECTED_OUTPUT [PORT, PROTOCOL, RESPONSE_CODE, OC_ARGS, ... ] -# -------------------- -# Check the imagestream with an s2i app check. First it imports the given image stream, then -# it runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. Then check the http response. -# Argument: image_stream_file - local or remote file with the image stream definition -# Argument: image_name - container image we test (or name of the existing image stream in : format) -# Argument: app - url or local path to git repo with the application sources (compulsory) -# Argument: context_dir - sub-directory inside the repository with the application sources (compulsory) -# Argument: expected_output - PCRE regular expression that must match the response body (compulsory) -# Argument: port - which port to use (optional; default: 8080) -# Argument: protocol - which protocol to use (optional; default: http) -# Argument: response_code - what http response code to expect (optional; default: 200) -# Argument: oc_args - all other arguments are used as additional parameters for the `oc new-app` -# command, typically environment variables (optional) -function ct_os_test_image_stream_s2i() { - local image_stream_file=${1} - local image_name=${2} - local app=${3} - local context_dir=${4} - local expected_output=${5} - local port=${6:-8080} - local protocol=${7:-http} - local response_code=${8:-200} - local oc_args=${9:-} - local result - local local_image_stream_file - - echo "Running image stream test for stream ${image_stream_file} and application ${app} with context ${context_dir}" - - # shellcheck disable=SC2119 - ct_os_new_project - - local_image_stream_file=$(ct_obtain_input "${image_stream_file}") - oc create -f "${local_image_stream_file}" - - # ct_os_test_s2i_app creates a new project, but we already need - # it before for the image stream import, so tell it to skip this time - CT_SKIP_NEW_PROJECT=true \ - ct_os_test_s2i_app "${IMAGE_NAME}" "${app}" "${context_dir}" "${expected_output}" \ - "${port}" "${protocol}" "${response_code}" "${oc_args}" - result=$? - - # shellcheck disable=SC2119 - ct_os_delete_project - - return $result -} - -# ct_os_test_image_stream_quickstart IMAGE_STREAM_FILE TEMPLATE IMAGE_NAME NAME_IN_TEMPLATE EXPECTED_OUTPUT [PORT, PROTOCOL, RESPONSE_CODE, OC_ARGS, OTHER_IMAGES ] -# -------------------- -# Check the imagestream with an s2i app check. First it imports the given image stream, then -# it runs [image] and [app] in the openshift and optionally specifies env_params -# as environment variables to the image. Then check the http response. -# Argument: image_stream_file - local or remote file with the image stream definition -# Argument: template_file - local file name with a template -# Argument: image_name - container image we test (or name of the existing image stream in : format) -# Argument: name_in_template - image name used in the template -# Argument: expected_output - PCRE regular expression that must match the response body (compulsory) -# Argument: port - which port to use (optional; default: 8080) -# Argument: protocol - which protocol to use (optional; default: http) -# Argument: response_code - what http response code to expect (optional; default: 200) -# Argument: oc_args - all other arguments are used as additional parameters for the `oc new-app` -# command, typically environment variables (optional) -# Argument: other_images - some templates need other image to be pushed into the OpenShift registry, -# specify them in this parameter as "|", where "" is a full image name -# (including registry if needed) and "" is a tag under which the image should be available -# in the OpenShift registry. -function ct_os_test_image_stream_quickstart() { - local image_stream_file=${1} - local template_file=${2} - local image_name=${3} - local name_in_template=${4} - local expected_output=${5} - local port=${6:-8080} - local protocol=${7:-http} - local response_code=${8:-200} - local oc_args=${9:-} - local other_images=${10:-} - local result - local local_image_stream_file - local local_template_file - - echo "Running image stream test for stream ${image_stream_file} and quickstart template ${template_file}" - - # shellcheck disable=SC2119 - ct_os_new_project - - local_image_stream_file=$(ct_obtain_input "${image_stream_file}") - local_template_file=$(ct_obtain_input "${template_file}") - oc create -f "${local_image_stream_file}" - - # ct_os_test_template_app creates a new project, but we already need - # it before for the image stream import, so tell it to skip this time - CT_SKIP_NEW_PROJECT=true \ - ct_os_test_template_app "${image_name}" \ - "${local_template_file}" \ - "${name_in_template}" \ - "${expected_output}" \ - "${port}" "${protocol}" "${response_code}" "${oc_args}" "${other_images}" - - result=$? - - # shellcheck disable=SC2119 - ct_os_delete_project - - return $result -} - -# ct_os_service_image_info SERVICE_NAME -# -------------------- -# Shows information about the image used by a specified service. -# Argument: service_name - Service name (uesd for deployment config) -function ct_os_service_image_info() { - local service_name=$1 - local image_id - local namespace - - # get image ID from the deployment config - image_id=$(oc get "deploymentconfig.apps.openshift.io/${service_name}" -o custom-columns=IMAGE:.spec.template.spec.containers[*].image | tail -n 1) - namespace=${CT_NAMESPACE:-"$(oc project -q)"} - - echo " Information about the image we work with:" - oc get deploymentconfig.apps.openshift.io/"${service_name}" -o yaml | grep lastTriggeredImage - # for s2i builds, the resulting image is actually in the current namespace, - # so if the specified namespace does not succeed, try the current namespace - oc get isimage -n "${namespace}" "${image_id##*/}" -o yaml || oc get isimage "${image_id##*/}" -o yaml -} -# vim: set tabstop=2:shiftwidth=2:expandtab: diff --git a/common/test-lib-remote-openshift.sh b/common/test-lib-remote-openshift.sh deleted file mode 100644 index c73c071..0000000 --- a/common/test-lib-remote-openshift.sh +++ /dev/null @@ -1,107 +0,0 @@ -# shellcheck shell=bash -# some functions are used from test-lib.sh, that is usually in the same dir -# shellcheck source=/dev/null -source "$(dirname "${BASH_SOURCE[0]}")"/test-lib.sh - -# Set of functions for testing docker images in OpenShift using 'oc' command - -# A variable containing the overall test result; must be changed to 0 in the end -# of the testing script: -# OS_TESTSUITE_RESULT=0 -# And the following trap must be set, in the beginning of the test script: -# trap ct_os_cleanup EXIT SIGINT - -# ct_os_set_path_oc_4 OC_VERSION -# -------------------- -# This is a trick that helps using correct version 4 of the `oc`: -# The input is version of the openshift in format 4.4 etc. -# If the currently available version of oc is not of this version, -# it first takes a look into /usr/local/oc-/bin directory, - -# Arguments: oc_version - X.Y part of the version of OSE (e.g. 3.9) -function ct_os_set_path_oc_4() { - echo "Setting OCP4 client" - local oc_version=$1 - local installed_oc_path="/usr/local/oc-v${oc_version}/bin" - echo "PATH ${installed_oc_path}" - if [ -x "${installed_oc_path}/oc" ] ; then - oc_path="${installed_oc_path}" - echo "Binary oc found in ${installed_oc_path}" >&2 - else - echo "OCP4 not found" - return 1 - fi - export PATH="${oc_path}:${PATH}" - oc version - if ! oc version | grep -q "Client Version: ${oc_version}." ; then - echo "ERROR: something went wrong, oc located at ${oc_path}, but oc of version ${oc_version} not found in PATH ($PATH)" >&1 - return 1 - else - echo "PATH set correctly, binary oc found in version ${oc_version}: $(command -v oc)" - fi -} - -# ct_os_prepare_ocp4 -# ------------------ -# Prepares environment for testing images in OpenShift 4 environment -# -# -function ct_os_set_ocp4() { - local login - OS_OC_CLIENT_VERSION=${OS_OC_CLIENT_VERSION:-4.4} - ct_os_set_path_oc_4 "${OS_OC_CLIENT_VERSION}" - - oc version - - login=$(cat "$KUBEPASSWORD") - oc login -u kubeadmin -p "$login" - echo "Login to OpenShift ${OS_OC_CLIENT_VERSION} is DONE" - # let openshift cluster to sync to avoid some race condition errors - sleep 3 -} - -function ct_os_upload_image_external_registry() { - local input_name="${1}" ; shift - local image_name=${input_name##*/} - local imagestream=${1:-$image_name:latest} - local output_name - - ct_os_login_external_registry - - output_name="${INTERNAL_DOCKER_REGISTRY}/rhscl-ci-testing/$imagestream" - - docker images - docker tag "${input_name}" "${output_name}" - docker push "${output_name}" -} - - -function ct_os_login_external_registry() { - local docker_token - # docker login fails with "404 page not found" error sometimes, just try it more times - # shellcheck disable=SC2034 - echo "loging" - [ -z "${INTERNAL_DOCKER_REGISTRY:-}" ] && "INTERNAL_DOCKER_REGISTRY has to be set for working with Internal registry" && return 1 - # shellcheck disable=SC2034 - for i in $(seq 12) ; do - # shellcheck disable=SC2015 - docker_token=$(cat "$DOCKER_UPSHIFT_TOKEN") - # shellcheck disable=SC2015 - docker login -u rhscl-ci-testing -p "$docker_token" "${INTERNAL_DOCKER_REGISTRY}" && return 0 || : - sleep 5 - done - return 1 -} - -function ct_os_import_image_ocp4() { - local image_name="${1}"; shift - local imagestream=${1:-$image_name:latest} - local namespace - - namespace=${CT_NAMESPACE:-"$(oc project -q)"} - deploy_image_name="${INTERNAL_DOCKER_REGISTRY}/rhscl-ci-testing/${imagestream}" - echo "Uploading image ${image_name} as ${deploy_image_name} , ${imagestream} into external registry." - ct_os_upload_image_external_registry "${image_name}" "${imagestream}" - echo "Import image into OpenShift 4 environment " - oc import-image "${namespace}/${imagestream}" --from="${deploy_image_name}" --confirm --reference-policy=local -} diff --git a/common/test-lib.sh b/common/test-lib.sh deleted file mode 100644 index deefbd7..0000000 --- a/common/test-lib.sh +++ /dev/null @@ -1,901 +0,0 @@ -# shellcheck shell=bash -# -# Test a container image. -# -# Always use sourced from a specific container testfile -# -# reguires definition of CID_FILE_DIR -# CID_FILE_DIR=$(mktemp --suffix=_test_cidfiles -d) -# reguires definition of TEST_LIST -# TEST_LIST="\ -# ctest_container_creation -# ctest_doc_content" - -# Container CI tests -# abbreviated as "ct" - -# may be redefined in the specific container testfile -EXPECTED_EXIT_CODE=0 - -# ct_cleanup -# -------------------- -# Cleans up containers used during tests. Stops and removes all containers -# referenced by cid_files in CID_FILE_DIR. Dumps logs if a container exited -# unexpectedly. Removes the cid_files and CID_FILE_DIR as well. -# Uses: $CID_FILE_DIR - path to directory containing cid_files -# Uses: $EXPECTED_EXIT_CODE - expected container exit code -function ct_cleanup() { - ct_show_resources - for cid_file in "$CID_FILE_DIR"/* ; do - local container - container=$(cat "$cid_file") - - : "Stopping and removing container $container..." - docker stop "$container" - exit_status=$(docker inspect -f '{{.State.ExitCode}}' "$container") - if [ "$exit_status" != "$EXPECTED_EXIT_CODE" ]; then - : "Dumping logs for $container" - docker logs "$container" - fi - docker rm -v "$container" - rm "$cid_file" - done - rmdir "$CID_FILE_DIR" - : "Done." -} - -# ct_enable_cleanup -# -------------------- -# Enables automatic container cleanup after tests. -function ct_enable_cleanup() { - trap ct_cleanup EXIT SIGINT -} - -# ct_check_envs_set env_filter check_envs loop_envs [env_format] -# -------------------- -# Compares values from one list of environment variable definitions against such list, -# checking if the values are present and have a specific format. -# Argument: env_filter - optional string passed to grep used for -# choosing which variables to filter out in env var lists. -# Argument: check_envs - list of env var definitions to check values against -# Argument: loop_envs - list of env var definitions to check values for -# Argument: env_format (optional) - format string for bash substring deletion used -# for checking whether the value is contained in check_envs. -# Defaults to: "*VALUE*", VALUE string gets replaced by actual value from loop_envs -function ct_check_envs_set { - local env_filter check_envs env_format - env_filter=$1; shift - check_envs=$1; shift - loop_envs=$1; shift - env_format=${1:-"*VALUE*"} - while read -r variable; do - [ -z "$variable" ] && continue - var_name=$(echo "$variable" | awk -F= '{ print $1 }') - stripped=$(echo "$variable" | awk -F= '{ print $2 }') - filtered_envs=$(echo "$check_envs" | grep "^$var_name=") - [ -z "$filtered_envs" ] && { echo "$var_name not found during \` docker exec\`"; return 1; } - old_IFS=$IFS - # For each such variable compare its content with the `docker exec` result, use `:` as delimiter - IFS=: - for value in $stripped; do - # If the falue checked does not go through env_filter we do not care about it - echo "$value" | grep -q "$env_filter" || continue - if [ -n "${filtered_envs##${env_format//VALUE/$value}}" ]; then - echo " Value $value is missing from variable $var_name" - echo "$filtered_envs" - return 1 - fi - done - done <<< "$(echo "$loop_envs" | grep "$env_filter" | grep -v "^PWD=")" - IFS=$old_IFS -} - -# ct_get_cid [name] -# -------------------- -# Prints container id from cid_file based on the name of the file. -# Argument: name - name of cid_file where the container id will be stored -# Uses: $CID_FILE_DIR - path to directory containing cid_files -function ct_get_cid() { - local name="$1" ; shift || return 1 - cat "$CID_FILE_DIR/$name" -} - -# ct_get_cip [id] -# -------------------- -# Prints container ip address based on the container id. -# Argument: id - container id -function ct_get_cip() { - local id="$1" ; shift - docker inspect --format='{{.NetworkSettings.IPAddress}}' "$(ct_get_cid "$id")" -} - -# ct_wait_for_cid [cid_file] -# -------------------- -# Holds the execution until the cid_file is created. Usually run after container -# creation. -# Argument: cid_file - name of the cid_file that should be created -function ct_wait_for_cid() { - local cid_file=$1 - local max_attempts=10 - local sleep_time=1 - local attempt=1 - local result=1 - while [ $attempt -le $max_attempts ]; do - [ -f "$cid_file" ] && [ -s "$cid_file" ] && return 0 - : "Waiting for container start..." - attempt=$(( attempt + 1 )) - sleep $sleep_time - done - return 1 -} - -# ct_assert_container_creation_fails [container_args] -# -------------------- -# The invocation of docker run should fail based on invalid container_args -# passed to the function. Returns 0 when container fails to start properly. -# Argument: container_args - all arguments are passed directly to dokcer run -# Uses: $CID_FILE_DIR - path to directory containing cid_files -function ct_assert_container_creation_fails() { - local ret=0 - local max_attempts=10 - local attempt=1 - local cid_file=assert - set +e - local old_container_args="${CONTAINER_ARGS-}" - # we really work with CONTAINER_ARGS as with a string - # shellcheck disable=SC2124 - CONTAINER_ARGS="$@" - if ct_create_container "$cid_file" ; then - local cid - cid=$(ct_get_cid "$cid_file") - - while [ "$(docker inspect -f '{{.State.Running}}' "$cid")" == "true" ] ; do - sleep 2 - attempt=$(( attempt + 1 )) - if [ "$attempt" -gt "$max_attempts" ]; then - docker stop "$cid" - ret=1 - break - fi - done - exit_status=$(docker inspect -f '{{.State.ExitCode}}' "$cid") - if [ "$exit_status" == "0" ]; then - ret=1 - fi - docker rm -v "$cid" - rm "$CID_FILE_DIR/$cid_file" - fi - [ -n "$old_container_args" ] && CONTAINER_ARGS="$old_container_args" - set -e - return "$ret" -} - -# ct_create_container [name, command] -# -------------------- -# Creates a container using the IMAGE_NAME and CONTAINER_ARGS variables. Also -# stores the container id to a cid_file located in the CID_FILE_DIR, and waits -# for the creation of the file. -# Argument: name - name of cid_file where the container id will be stored -# Argument: command - optional command to be executed in the container -# Uses: $CID_FILE_DIR - path to directory containing cid_files -# Uses: $CONTAINER_ARGS - optional arguments passed directly to docker run -# Uses: $IMAGE_NAME - name of the image being tested -function ct_create_container() { - local cid_file="$CID_FILE_DIR/$1" ; shift - # create container with a cidfile in a directory for cleanup - # shellcheck disable=SC2086 - docker run --cidfile="$cid_file" -d ${CONTAINER_ARGS:-} "$IMAGE_NAME" "$@" - ct_wait_for_cid "$cid_file" || return 1 - : "Created container $(cat "$cid_file")" -} - -# ct_scl_usage_old [name, command, expected] -# -------------------- -# Tests three ways of running the SCL, by looking for an expected string -# in the output of the command -# Argument: name - name of cid_file where the container id will be stored -# Argument: command - executed inside the container -# Argument: expected - string that is expected to be in the command output -# Uses: $CID_FILE_DIR - path to directory containing cid_files -# Uses: $IMAGE_NAME - name of the image being tested -function ct_scl_usage_old() { - local name="$1" - local command="$2" - local expected="$3" - local out="" - : " Testing the image SCL enable" - out=$(docker run --rm "${IMAGE_NAME}" /bin/bash -c "${command}") - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[/bin/bash -c \"${command}\"] Expected '${expected}', got '${out}'" >&2 - return 1 - fi - out=$(docker exec "$(ct_get_cid "$name")" /bin/bash -c "${command}" 2>&1) - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[exec /bin/bash -c \"${command}\"] Expected '${expected}', got '${out}'" >&2 - return 1 - fi - out=$(docker exec "$(ct_get_cid "$name")" /bin/sh -ic "${command}" 2>&1) - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[exec /bin/sh -ic \"${command}\"] Expected '${expected}', got '${out}'" >&2 - return 1 - fi -} - -# ct_doc_content_old [strings] -# -------------------- -# Looks for occurence of stirngs in the documentation files and checks -# the format of the files. Files examined: help.1 -# Argument: strings - strings expected to appear in the documentation -# Uses: $IMAGE_NAME - name of the image being tested -function ct_doc_content_old() { - local tmpdir - tmpdir=$(mktemp -d) - local f - : " Testing documentation in the container image" - # Extract the help files from the container - # shellcheck disable=SC2043 - for f in help.1 ; do - docker run --rm "${IMAGE_NAME}" /bin/bash -c "cat /${f}" >"${tmpdir}/$(basename "${f}")" - # Check whether the files contain some important information - for term in "$@" ; do - if ! grep -F -q -e "${term}" "${tmpdir}/$(basename "${f}")" ; then - echo "ERROR: File /${f} does not include '${term}'." >&2 - return 1 - fi - done - # Check whether the files use the correct format - for term in TH PP SH ; do - if ! grep -q "^\.${term}" "${tmpdir}/help.1" ; then - echo "ERROR: /help.1 is probably not in troff or groff format, since '${term}' is missing." >&2 - return 1 - fi - done - done - : " Success!" -} - -# full_ca_file_path -# Return string for full path to CA file -function full_ca_file_path() -{ - echo "/etc/pki/ca-trust/source/anchors/RH-IT-Root-CA.crt" -} -# ct_mount_ca_file -# ------------------ -# Check if /etc/pki/certs/RH-IT-Root-CA.crt file exists -# return mount string for containers or empty string -function ct_mount_ca_file() -{ - # mount CA file only if NPM_REGISTRY variable is present. - local mount_parameter="" - if [ -n "$NPM_REGISTRY" ] && [ -f "$(full_ca_file_path)" ]; then - mount_parameter="-v $(full_ca_file_path):$(full_ca_file_path):Z" - fi - echo "$mount_parameter" -} - -# ct_build_s2i_npm_variables URL_TO_NPM_JS_SERVER -# ------------------------------------------ -# Function returns -e NPM_MIRROR and -v MOUNT_POINT_FOR_CAFILE -# or empty string -function ct_build_s2i_npm_variables() -{ - npm_variables="" - if [ -n "$NPM_REGISTRY" ] && [ -f "$(full_ca_file_path)" ]; then - npm_variables="-e NPM_MIRROR=$NPM_REGISTRY $(ct_mount_ca_file)" - fi - echo "$npm_variables" -} - -# ct_npm_works -# -------------------- -# Checks existance of the npm tool and runs it. -function ct_npm_works() { - local tmpdir - tmpdir=$(mktemp -d) - : " Testing npm in the container image" - local cid_file="${tmpdir}/cid" - if ! docker run --rm "${IMAGE_NAME}" /bin/bash -c "npm --version" >"${tmpdir}/version" ; then - echo "ERROR: 'npm --version' does not work inside the image ${IMAGE_NAME}." >&2 - return 1 - fi - - # shellcheck disable=SC2046 - docker run -d $(ct_mount_ca_file) --rm --cidfile="$cid_file" "${IMAGE_NAME}-testapp" - - # Wait for the container to write it's CID file - ct_wait_for_cid "$cid_file" || return 1 - - if ! docker exec "$(cat "$cid_file")" /bin/bash -c "npm --verbose install jquery && test -f node_modules/jquery/src/jquery.js" >"${tmpdir}/jquery" 2>&1 ; then - echo "ERROR: npm could not install jquery inside the image ${IMAGE_NAME}." >&2 - return 1 - fi - - if [ -n "$NPM_REGISTRY" ] && [ -f "$(full_ca_file_path)" ]; then - if ! grep -qo "$NPM_REGISTRY" "${tmpdir}/jquery"; then - echo "ERROR: Internal repository is NOT set. Even it is requested." - return 1 - fi - fi - - if [ -f "$cid_file" ]; then - docker stop "$(cat "$cid_file")" - rm "$cid_file" - fi - : " Success!" -} - -# ct_binary_found_from_df binary [path] -# -------------------- -# Checks if a binary can be found in PATH during Dockerfile build -# Argument: binary - name of the binary to test accessibility for -# Argument: path - optional path in which the binary should reside in -# /opt/rh by default -function ct_binary_found_from_df() { - local tmpdir - local binary=$1; shift - local binary_path=${1:-"^/opt/rh"} - tmpdir=$(mktemp -d) - : " Testing $binary in build from Dockerfile" - - # Create Dockerfile that looks for the binary - cat <"$tmpdir/Dockerfile" -FROM $IMAGE_NAME -RUN which $binary | grep "$binary_path" -EOF - # Build an image, looking for expected path in the output - if ! docker build -f "$tmpdir/Dockerfile" --no-cache "$tmpdir"; then - echo " ERROR: Failed to find $binary in Dockerfile!" >&2 - return 1 - fi - : " Success!" -} - -# ct_check_exec_env_vars [env_filter] -# -------------------- -# Checks if all relevant environment variables from `docker run` -# can be found in `docker exec` as well. -# Argument: env_filter - optional string passed to grep used for -# choosing which variables to check in the test case. -# Defaults to X_SCLS and variables containing /opt/app-root, /opt/rh -# Uses: $CID_FILE_DIR - path to directory containing cid_files -# Uses: $IMAGE_NAME - name of the image being tested -function ct_check_exec_env_vars() { - local tmpdir exec_envs cid old_IFS env_filter - local var_name stripped filtered_envs run_envs - env_filter=${1:-"^X_SCLS=\|/opt/rh\|/opt/app-root"} - tmpdir=$(mktemp -d) - CID_FILE_DIR=${CID_FILE_DIR:-$(mktemp -d)} - # Get environment variables from `docker run` - run_envs=$(docker run --rm "$IMAGE_NAME" /bin/bash -c "env") - # Get environment variables from `docker exec` - ct_create_container "test_exec_envs" bash -c "sleep 1000" >/dev/null - cid=$(ct_get_cid "test_exec_envs") - exec_envs=$(docker exec "$cid" env) - # Filter out variables we are not interested in - # Always check X_SCLS, ignore PWD - # Check variables from `docker run` that have alternative paths inside (/opt/rh, /opt/app-root) - ct_check_envs_set "$env_filter" "$exec_envs" "$run_envs" "*VALUE*" || return 1 - echo " All values present in \`docker exec\`" - return 0 -} - -# ct_check_scl_enable_vars [env_filter] -# -------------------- -# Checks if all relevant environment variables from `docker run` -# are set twice after a second call of `scl enable $SCLS`. -# Argument: env_filter - optional string passed to grep used for -# choosing which variables to check in the test case. -# Defaults to paths containing enabled SCLS in the image -# Uses: $IMAGE_NAME - name of the image being tested -function ct_check_scl_enable_vars() { - local tmpdir exec_envs cid old_IFS env_filter enabled_scls - local var_name stripped filtered_envs loop_envs - env_filter=$1 - tmpdir=$(mktemp -d) - enabled_scls=$(docker run --rm "$IMAGE_NAME" /bin/bash -c "echo \$X_SCLS") - if [ -z "$env_filter" ]; then - for scl in $enabled_scls; do - [ -z "$env_filter" ] && env_filter="/$scl" && continue - # env_filter not empty, append to the existing list - env_filter="$env_filter|/$scl" - done - fi - # Get environment variables from `docker run` - loop_envs=$(docker run --rm "$IMAGE_NAME" /bin/bash -c "env") - run_envs=$(docker run --rm "$IMAGE_NAME" /bin/bash -c "X_SCLS= scl enable $enabled_scls env") - # Check if the values are set twice in the second set of envs - ct_check_envs_set "$env_filter" "$run_envs" "$loop_envs" "*VALUE*VALUE*" || return 1 - echo " All scl_enable values present" - return 0 -} - -# ct_path_append PATH_VARNAME DIRECTORY -# ------------------------------------- -# Append DIRECTORY to VARIABLE of name PATH_VARNAME, the VARIABLE must consist -# of colon-separated list of directories. -ct_path_append () -{ - if eval "test -n \"\${$1-}\""; then - eval "$1=\$2:\$$1" - else - eval "$1=\$2" - fi -} - - -# ct_path_foreach PATH ACTION [ARGS ...] -# -------------------------------------- -# For each DIR in PATH execute ACTION (path is colon separated list of -# directories). The particular calls to ACTION will look like -# '$ ACTION directory [ARGS ...]' -ct_path_foreach () -{ - local dir dirlist action save_IFS - save_IFS=$IFS - IFS=: - dirlist=$1 - action=$2 - shift 2 - for dir in $dirlist; do "$action" "$dir" "$@" ; done - IFS=$save_IFS -} - - -# ct_run_test_list -# -------------------- -# Execute the tests specified by TEST_LIST -# Uses: $TEST_LIST - list of test names -function ct_run_test_list() { - for test_case in $TEST_LIST; do - : "Running test $test_case" - # shellcheck source=/dev/null - [ -f "test/$test_case" ] && source "test/$test_case" - # shellcheck source=/dev/null - [ -f "../test/$test_case" ] && source "../test/$test_case" - $test_case - done; -} - -# ct_gen_self_signed_cert_pem -# --------------------------- -# Generates a self-signed PEM certificate pair into specified directory. -# Argument: output_dir - output directory path -# Argument: base_name - base name of the certificate files -# Resulted files will be those: -# /-cert-selfsigned.pem -- public PEM cert -# /-key.pem -- PEM private key -ct_gen_self_signed_cert_pem() { - local output_dir=$1 ; shift - local base_name=$1 ; shift - mkdir -p "${output_dir}" - openssl req -newkey rsa:2048 -nodes -keyout "${output_dir}"/"${base_name}"-key.pem -subj '/C=GB/ST=Berkshire/L=Newbury/O=My Server Company' > "${base_name}"-req.pem - openssl req -new -x509 -nodes -key "${output_dir}"/"${base_name}"-key.pem -batch > "${output_dir}"/"${base_name}"-cert-selfsigned.pem -} - -# ct_obtain_input FILE|DIR|URL -# -------------------- -# Either copies a file or a directory to a tmp location for local copies, or -# downloads the file from remote location. -# Resulted file path is printed, so it can be later used by calling function. -# Arguments: input - local file, directory or remote URL -function ct_obtain_input() { - local input=$1 - local extension="${input##*.}" - - # Try to use same extension for the temporary file if possible - [[ "${extension}" =~ ^[a-z0-9]*$ ]] && extension=".${extension}" || extension="" - - local output - output=$(mktemp "/var/tmp/test-input-XXXXXX$extension") - if [ -f "${input}" ] ; then - cp -f "${input}" "${output}" - elif [ -d "${input}" ] ; then - rm -f "${output}" - cp -r -LH "${input}" "${output}" - elif echo "${input}" | grep -qe '^http\(s\)\?://' ; then - curl "${input}" > "${output}" - else - echo "ERROR: file type not known: ${input}" >&2 - return 1 - fi - echo "${output}" -} - -# ct_test_response -# ---------------- -# Perform GET request to the application container, checks output with -# a reg-exp and HTTP response code. -# Argument: url - request URL path -# Argument: expected_code - expected HTTP response code -# Argument: body_regexp - PCRE regular expression that must match the response body -# Argument: max_attempts - Optional number of attempts (default: 20), three seconds sleep between -# Argument: ignore_error_attempts - Optional number of attempts when we ignore error output (default: 10) -ct_test_response() { - local url="$1" - local expected_code="$2" - local body_regexp="$3" - local max_attempts=${4:-20} - local ignore_error_attempts=${5:-10} - - : " Testing the HTTP(S) response for <${url}>" - local sleep_time=3 - local attempt=1 - local result=1 - local status - local response_code - local response_file - response_file=$(mktemp /tmp/ct_test_response_XXXXXX) - while [ "${attempt}" -le "${max_attempts}" ]; do - curl --connect-timeout 10 -s -w '%{http_code}' "${url}" >"${response_file}" && status=0 || status=1 - if [ "${status}" -eq 0 ]; then - response_code=$(tail -c 3 "${response_file}") - if [ "${response_code}" -eq "${expected_code}" ]; then - result=0 - fi - grep -qP -e "${body_regexp}" "${response_file}" || result=1; - # Some services return 40x code until they are ready, so let's give them - # some chance and not end with failure right away - # Do not wait if we already have expected outcome though - if [ "${result}" -eq 0 ] || [ "${attempt}" -gt "${ignore_error_attempts}" ] || [ "${attempt}" -eq "${max_attempts}" ] ; then - break - fi - fi - attempt=$(( attempt + 1 )) - sleep "${sleep_time}" - done - rm -f "${response_file}" - return "${result}" -} - -# ct_registry_from_os OS -# ---------------- -# Transform operating system string [os] into registry url -# Argument: OS - string containing the os version -ct_registry_from_os() { - local registry="" - case $1 in - rhel*) - registry=registry.redhat.io - ;; - *) - registry=docker.io - ;; - esac - echo "$registry" -} - - # ct_get_public_image_name OS BASE_IMAGE_NAME VERSION -# ---------------- -# Transform the arguments into public image name -# Argument: OS - string containing the os version -# Argument: BASE_IMAGE_NAME - string containing the base name of the image as defined in the Makefile -# Argument: VERSION - string containing the version of the image as defined in the Makefile -ct_get_public_image_name() { - local os=$1; shift - local base_image_name=$1; shift - local version=$1; shift - - local public_image_name - local registry - - registry=$(ct_registry_from_os "$os") - if [ "x$os" == "xrhel7" ]; then - public_image_name=$registry/rhscl/$base_image_name-${version//./}-rhel7 - elif [ "x$os" == "xrhel8" ]; then - public_image_name=$registry/rhel8/$base_image_name-${version//./} - elif [ "x$os" == "xcentos7" ]; then - public_image_name=$registry/centos/$base_image_name-${version//./}-centos7 - elif [ "x$os" == "xcentos8" ]; then - public_image_name=$registry/centos/$base_image_name-${version//./}-centos8 - fi - - echo "$public_image_name" -} - -# ct_assert_cmd_success CMD -# ---------------- -# Evaluates [cmd] and fails if it does not succeed. -# Argument: CMD - Command to be run -function ct_assert_cmd_success() { - echo "Checking '$*' for success ..." - if ! eval "$@" &>/dev/null; then - echo " FAIL" - return 1 - fi - echo " PASS" - return 0 -} - -# ct_assert_cmd_failure CMD -# ---------------- -# Evaluates [cmd] and fails if it succeeds. -# Argument: CMD - Command to be run -function ct_assert_cmd_failure() { - echo "Checking '$*' for failure ..." - if eval "$@" &>/dev/null; then - echo " FAIL" - return 1 - fi - echo " PASS" - return 0 -} - - -# ct_random_string [LENGTH=10] -# ---------------------------- -# Generate pseudorandom alphanumeric string of LENGTH bytes, the -# default length is 10. The string is printed on stdout. -ct_random_string() -( - export LC_ALL=C - dd if=/dev/urandom count=1 bs=10k 2>/dev/null \ - | tr -dc 'a-z0-9' \ - | fold -w "${1-10}" \ - | head -n 1 -) - -# ct_s2i_usage IMG_NAME [S2I_ARGS] -# ---------------------------- -# Create a container and run the usage script inside -# Argument: IMG_NAME - name of the image to be used for the container run -# Argument: S2I_ARGS - Additional list of source-to-image arguments, currently unused. -ct_s2i_usage() -{ - local img_name=$1; shift - local s2i_args="$*"; - local usage_command="/usr/libexec/s2i/usage" - docker run --rm "$img_name" bash -c "$usage_command" -} - -# ct_s2i_build_as_df APP_PATH SRC_IMAGE DST_IMAGE [S2I_ARGS] -# ---------------------------- -# Create a new s2i app image from local sources in a similar way as source-to-image would have used. -# Argument: APP_PATH - local path to the app sources to be used in the test -# Argument: SRC_IMAGE - image to be used as a base for the s2i build -# Argument: DST_IMAGE - image name to be used during the tagging of the s2i build result -# Argument: S2I_ARGS - Additional list of source-to-image arguments. -# Only used to check for pull-policy=never and environment variable definitions. -ct_s2i_build_as_df() -{ - local app_path=$1; shift - local src_image=$1; shift - local dst_image=$1; shift - local s2i_args="$*"; - local local_app=upload/src/ - local local_scripts=upload/scripts/ - local user_id= - local df_name= - local tmpdir= - local incremental=false - local mount_options="" - - # Run the entire thing inside a subshell so that we do not leak shell options outside of the function - ( - # Error out if any part of the build fails - set -e - - # Use /tmp to not pollute cwd - tmpdir=$(mktemp -d) - df_name=$(mktemp -p "$tmpdir" Dockerfile.XXXX) - cd "$tmpdir" - # Check if the image is available locally and try to pull it if it is not - docker images "$src_image" &>/dev/null || echo "$s2i_args" | grep -q "pull-policy=never" || docker pull "$src_image" - user=$(docker inspect -f "{{.Config.User}}" "$src_image") - # Default to root if no user is set by the image - user=${user:-0} - # run the user through the image in case it is non-numeric or does not exist - # NOTE: The '-eq' test is used to check if $user is numeric as it will fail if $user is not an integer - if ! [ "$user" -eq "$user" ] 2>/dev/null && ! user_id=$(docker run --rm "$src_image" bash -c "id -u $user 2>/dev/null"); then - echo "ERROR: id of user $user not found inside image $src_image." - echo "Terminating s2i build." - return 1 - else - user_id=${user_id:-$user} - fi - echo "$s2i_args" | grep -q "\-\-incremental" && incremental=true - if $incremental; then - inc_tmp=$(mktemp -d --tmpdir incremental.XXXX) - setfacl -m "u:$user_id:rwx" "$inc_tmp" - # Check if the image exists, build should fail (for testing use case) if it does not - docker images "$dst_image" &>/dev/null || (echo "Image $dst_image not found."; false) - # Run the original image with a mounted in volume and get the artifacts out of it - cmd="if [ -s /usr/libexec/s2i/save-artifacts ]; then /usr/libexec/s2i/save-artifacts > \"$inc_tmp/artifacts.tar\"; else touch \"$inc_tmp/artifacts.tar\"; fi" - docker run --rm -v "$inc_tmp:$inc_tmp:Z" "$dst_image" bash -c "$cmd" - # Move the created content into the $tmpdir for the build to pick it up - mv "$inc_tmp/artifacts.tar" "$tmpdir/" - fi - # Strip file:// from APP_PATH and copy its contents into current context - mkdir -p "$local_app" - cp -r "${app_path/file:\/\//}/." "$local_app" - [ -d "$local_app/.s2i/bin/" ] && mv "$local_app/.s2i/bin" "$local_scripts" - # Create a Dockerfile named df_name and fill it with proper content - #FIXME: Some commands could be combined into a single layer but not sure if worth the trouble for testing purposes - cat <"$df_name" -FROM $src_image -LABEL "io.openshift.s2i.build.image"="$src_image" \\ - "io.openshift.s2i.build.source-location"="$app_path" -USER root -COPY $local_app /tmp/src -EOF - [ -d "$local_scripts" ] && echo "COPY $local_scripts /tmp/scripts" >> "$df_name" && - echo "RUN chown -R $user_id:0 /tmp/scripts" >>"$df_name" - echo "RUN chown -R $user_id:0 /tmp/src" >>"$df_name" - # Check for custom environment variables inside .s2i/ folder - if [ -e "$local_app/.s2i/environment" ]; then - # Remove any comments and add the contents as ENV commands to the Dockerfile - sed '/^\s*#.*$/d' "$local_app/.s2i/environment" | while read -r line; do - echo "ENV $line" >>"$df_name" - done - fi - # Filter out env var definitions from $s2i_args and create Dockerfile ENV commands out of them - echo "$s2i_args" | grep -o -e '\(-e\|--env\)[[:space:]=]\S*=\S*' | sed -e 's/-e /ENV /' -e 's/--env[ =]/ENV /' >>"$df_name" - # Check if CA autority is present on host and add it into Dockerfile - [ -f "$(full_ca_file_path)" ] && echo "RUN cd /etc/pki/ca-trust/source/anchors && update-ca-trust extract" >>"$df_name" - - # Add in artifacts if doing an incremental build - if $incremental; then - { echo "RUN mkdir /tmp/artifacts" - echo "ADD artifacts.tar /tmp/artifacts" - echo "RUN chown -R $user_id:0 /tmp/artifacts" ; } >>"$df_name" - fi - - echo "USER $user_id" >>"$df_name" - # If exists, run the custom assemble script, else default to /usr/libexec/s2i/assemble - if [ -x "$local_scripts/assemble" ]; then - echo "RUN /tmp/scripts/assemble" >>"$df_name" - else - echo "RUN /usr/libexec/s2i/assemble" >>"$df_name" - fi - # If exists, set the custom run script as CMD, else default to /usr/libexec/s2i/run - if [ -x "$local_scripts/run" ]; then - echo "CMD /tmp/scripts/run" >>"$df_name" - else - echo "CMD /usr/libexec/s2i/run" >>"$df_name" - fi - - # Check if -v parameter is present in s2i_args and add it into docker build command - mount_options=$(echo "$s2i_args" | grep -o -e '\(-v\)[[:space:]]\.*\S*' || true) - - # Run the build and tag the result - # shellcheck disable=SC2086 - docker build $mount_options -f "$df_name" --no-cache=true -t "$dst_image" . - ) -} - -# ct_check_image_availability PUBLIC_IMAGE_NAME -# ---------------------------- -# Pull an image from the public repositories to see if the image is already available. -# Argument: PUBLIC_IMAGE_NAME - string containing the public name of the image to pull -ct_check_image_availability() { - local public_image_name=$1; - - # Try pulling the image to see if it is accessible - if ! docker pull "$public_image_name" &>/dev/null; then - echo "$public_image_name could not be downloaded via 'docker'" - return 1 - fi -} - -# ct_check_latest_imagestreams -# ----------------------------- -# Check if the latest version present in Makefile in the variable VERSIONS -# is present in all imagestreams. -# Also the latest tag in the imagestreams has to contain the latest version -ct_check_latest_imagestreams() { - local latest_version= - local test_lib_dir= - - # We only maintain imagestreams for RHEL and CentOS (Community) - if [[ "$OS" =~ ^fedora.* ]] ; then - echo "Imagestreams for Fedora are not maintained, skipping ct_check_latest_imagestreams" - return 0 - fi - - # Check only lines which starts with VERSIONS - latest_version=$(grep '^VERSIONS' Makefile | rev | cut -d ' ' -f 1 | rev ) - # Fall back to previous version if the latest is excluded for this OS - [ -f "$latest_version/.exclude-$OS" ] && latest_version=$(grep '^VERSIONS' Makefile | rev | cut -d ' ' -f 2 | rev ) - # Only test the imagestream once, when the version matches - # ignore the SC warning, $VERSION is always available - # shellcheck disable=SC2153 - if [ "$latest_version" == "$VERSION" ]; then - test_lib_dir=$(dirname "$(readlink -f "$0")") - python3 "${test_lib_dir}/check_imagestreams.py" "$latest_version" - else - echo "Image version $VERSION is not latest, skipping ct_check_latest_imagestreams" - fi -} - -# ct_show_resources -# ---------------- -# Prints the available resources -ct_show_resources() -{ - echo "Resources info:" - echo "Memory:" - free -h - echo "Storage:" - df -h - echo "CPU" - lscpu -} - -# ct_test_app_dockerfile -# ----------------------------- -# Argument: dockerfile - path to a Dockerfile that will be used for building an image -# (must work with an application directory called 'app-src') -# Argument: app_url - git URI with a testing application -# Argument: body_regexp - PCRE regular expression that must match the response body -# Argument: app_dir - name of the application directory that is used in the Dockerfile -# Argument: port - Optional port number (default: 8080) -ct_test_app_dockerfile() { - local dockerfile=$1 - local app_url=$2 - local expected_text=$3 - local app_dir=$4 # this is a directory that must match with the name in the Dockerfile - local port=${5:-8080} - local app_image_name=myapp - local ret - local cname=app_dockerfile - - if [ -z "$app_dir" ] ; then - echo "ERROR: Option app_dir not set. Terminating the Dockerfile build." - return 1 - fi - - if ! [ -r "${dockerfile}" ] || ! [ -s "${dockerfile}" ] ; then - echo "ERROR: Dockerfile ${dockerfile} does not exist or is empty." - echo "Terminating the Dockerfile build." - return 1 - fi - - CID_FILE_DIR=${CID_FILE_DIR:-$(mktemp -d)} - local dockerfile_abs - dockerfile_abs=$(readlink -f "${dockerfile}") - tmpdir=$(mktemp -d) - pushd "$tmpdir" >/dev/null - cp "${dockerfile_abs}" Dockerfile - - # Rewrite the source image to what we test - sed -i -e "s|^FROM.*$|FROM $IMAGE_NAME|" Dockerfile - # a bit more verbose, but should help debugging failures - echo "Using this Dockerfile:" - cat Dockerfile - - if ! git clone "${app_url}" "${app_dir}" ; then - echo "ERROR: Git repository ${app_url} cannot be cloned into ${app_dir}." - echo "Terminating the Dockerfile build." - return 1 - fi - - echo "Building '${app_image_name}' image using docker build" - if ! docker build --no-cache=true -t "${app_image_name}" . ; then - echo "ERROR: The image cannot be built from ${dockerfile} and application ${app_url}." - echo "Terminating the Dockerfile build." - return 1 - fi - - if ! docker run -d --cidfile="${CID_FILE_DIR}/app_dockerfile" --rm "${app_image_name}" ; then - echo "ERROR: The image ${app_image_name} cannot be run for ${dockerfile} and application ${app_url}." - echo "Terminating the Dockerfile build." - return 1 - fi - echo "Waiting for ${app_image_name} to start" - ct_wait_for_cid "${CID_FILE_DIR}/app_dockerfile" - - ip="$(ct_get_cip "${cname}")" - ct_test_response "http://$ip:${port}" 200 "${expected_text}" - ret=$? - - # cleanup - docker kill "$(ct_get_cid "${cname}")" - sleep 2 - docker rmi "${app_image_name}" - popd >/dev/null - rm -rf "${tmpdir}" - rm -f "${CID_FILE_DIR}/${cname}" - return $ret -} - -# vim: set tabstop=2:shiftwidth=2:expandtab: diff --git a/common/test-remote-cluster.sh b/common/test-remote-cluster.sh deleted file mode 100755 index 6ccfb32..0000000 --- a/common/test-remote-cluster.sh +++ /dev/null @@ -1,38 +0,0 @@ -#!/bin/bash - -# This script is used to test container images integrated in the OpenShift. -# -# VERSIONS - Must be set to a list with possible versions (subdirectories) -# -# This script expects oc command to exist and logged in to a working cluster. - -set -e - -export OS=${OS:-rhel7} - -if [ "${OS}" == "rhel7" ] ; then - NAMESPACE=${NAMESPACE:-rhscl/} - REGISTRY=${REGISTRY:-registry.access.redhat.com/} -else - NAMESPACE=${NAMESPACE:-centos/} -fi - -export NAMESPACE -export REGISTRY - -for dir in ${VERSIONS}; do - [ ! -e "${dir}/.image-id" ] && echo "-> Image for version $dir not built, skipping OpenShift 4 tests." && continue - pushd "${dir}" > /dev/null - - export IMAGE_NAME="${NAMESPACE}${BASE_IMAGE_NAME}-${dir//./}-${OS}" - - if [[ -x test/run-openshift-remote-cluster ]]; then - VERSION="${dir}" test/run-openshift-remote-cluster - else - echo "-> Tests for OpenShift 4 are not present. Add run-openshift-remote-cluster script, skipping" - fi - - popd > /dev/null -done - -# vim: set tabstop=2:shiftwidth=2:expandtab: diff --git a/common/test.sh b/common/test.sh deleted file mode 100755 index f619bb7..0000000 --- a/common/test.sh +++ /dev/null @@ -1,67 +0,0 @@ -#!/bin/bash - -# This script is used to test the OpenShift Docker images. -# -# TEST_MODE - If set, run regular test suite -# TEST_OPENSHIFT_MODE - If set, run OpenShift tests (if present) -# VERSIONS - Must be set to a list with possible versions (subdirectories) - -set -e - -for dir in ${VERSIONS}; do - [ ! -e "${dir}/.image-id" ] && echo "-> Image for version $dir not built, skipping tests." && continue - pushd "${dir}" > /dev/null - IMAGE_ID=$(cat .image-id) - export IMAGE_ID - IMAGE_VERSION=$(docker inspect -f "{{.Config.Labels.version}}" "$IMAGE_ID") - # Kept also IMAGE_NAME as some tests might still use that. - IMAGE_NAME="$(docker inspect -f "{{.Config.Labels.name}}" "$IMAGE_ID"):$IMAGE_VERSION" - export IMAGE_NAME - - if [ -n "${TEST_MODE}" ]; then - VERSION=$dir test/run - fi - - if [ -n "${TEST_CONU_MODE}" ]; then - if [[ -x test/run-conu ]]; then - if [ -n "${CONU_IMAGE}" ]; then - echo "-> Running conu tests in a container" - docker run \ - --net=host \ - -e VERSION="${dir}" \ - -e IMAGE_NAME \ - --rm \ - --security-opt label=disable \ - -ti \ - -v /var/run/docker.sock:/var/run/docker.sock \ - -v "${PWD}"/../:/src \ - -w "/src/${dir}/" \ - -ti \ - "${CONU_IMAGE}" \ - ./test/run-conu - else - VERSION="${dir}" ./test/run-conu - fi - else - echo "-> conu tests are not present, skipping" - fi - fi - - if [ -n "${TEST_OPENSHIFT_4}" ]; then - if [[ -x test/run-openshift-remote-cluster ]]; then - VERSION=$dir test/run-openshift-remote-cluster - else - echo "-> Tests for OpenShift 4 are not present. Add run-openshift-remote-cluster script, skipping" - fi - fi - - if [ -n "${TEST_OPENSHIFT_MODE}" ]; then - if [[ -x test/run-openshift ]]; then - VERSION=$dir test/run-openshift - else - echo "-> OpenShift 3 tests are not present, skipping" - fi - fi - - popd > /dev/null -done diff --git a/common/tests/.gitignore b/common/tests/.gitignore deleted file mode 100644 index 3207cd6..0000000 --- a/common/tests/.gitignore +++ /dev/null @@ -1,3 +0,0 @@ -.image-id* -help.1 -check_imagestreams.py diff --git a/common/tests/check_imagestreams.sh b/common/tests/check_imagestreams.sh deleted file mode 100755 index caef12b..0000000 --- a/common/tests/check_imagestreams.sh +++ /dev/null @@ -1,9 +0,0 @@ -#!/bin/sh - -set -x - -check_imagestreams=$(dirname "$(readlink -f "$0")")/../check_imagestreams.py -"${PYTHON-python3}" "$check_imagestreams" "2.5" -test $? -eq 1 -"${PYTHON-python3}" "$check_imagestreams" "2.4" -test $? -eq 0 diff --git a/common/tests/failures/check/Makefile b/common/tests/failures/check/Makefile deleted file mode 100644 index 6d41a15..0000000 --- a/common/tests/failures/check/Makefile +++ /dev/null @@ -1,4 +0,0 @@ -# Variables are documented in common/build.sh. -BASE_IMAGE_NAME = test-container -VERSIONS = v0 -include common/common.mk diff --git a/common/tests/failures/check/common b/common/tests/failures/check/common deleted file mode 120000 index a8a4f8c..0000000 --- a/common/tests/failures/check/common +++ /dev/null @@ -1 +0,0 @@ -../../.. \ No newline at end of file diff --git a/common/tests/failures/check/v0/Dockerfile b/common/tests/failures/check/v0/Dockerfile deleted file mode 100644 index 76421aa..0000000 --- a/common/tests/failures/check/v0/Dockerfile +++ /dev/null @@ -1,2 +0,0 @@ -FROM centos/s2i-core-centos7 -LABEL name=test-image diff --git a/common/tests/failures/check/v0/README.md b/common/tests/failures/check/v0/README.md deleted file mode 100644 index 19fd6ae..0000000 --- a/common/tests/failures/check/v0/README.md +++ /dev/null @@ -1 +0,0 @@ -This is just no-op-build container. diff --git a/common/tests/failures/check/v0/test/run b/common/tests/failures/check/v0/test/run deleted file mode 100755 index 1dace69..0000000 --- a/common/tests/failures/check/v0/test/run +++ /dev/null @@ -1,3 +0,0 @@ -#! /bin/sh -x - -false this must fail the whole testsuite diff --git a/common/tests/imagestreams/testing_file.json b/common/tests/imagestreams/testing_file.json deleted file mode 100644 index adfad36..0000000 --- a/common/tests/imagestreams/testing_file.json +++ /dev/null @@ -1,53 +0,0 @@ -{ - "apiVersion": "v1", - "kind": "ImageStream", - "metadata": { - "annotations": { - "openshift.io/display-name": "Apache HTTP Server (httpd)" - }, - "name": "httpd" - }, - "spec": { - "tags": [ - { - "annotations": { - "description": "Build and serve static content via Apache HTTP Server (httpd) on RHEL 7. For more information about using this builder image, including OpenShift considerations, see https://github.com/sclorg/httpd-container/blob/master/2.4/README.md.\n\nWARNING: By selecting this tag, your application will automatically update to use the latest version of Httpd available on OpenShift, including major version updates.", - "iconClass": "icon-apache", - "openshift.io/display-name": "Apache HTTP Server (Latest)", - "openshift.io/provider-display-name": "Red Hat, Inc.", - "sampleRepo": "https://github.com/sclorg/httpd-ex.git", - "supports": "httpd", - "tags": "builder,httpd" - }, - "from": { - "kind": "ImageStreamTag", - "name": "2.4" - }, - "referencePolicy": { - "type": "Local" - }, - "name": "latest" - }, - { - "annotations": { - "description": "Build and serve static content via Apache HTTP Server (httpd) 2.4 on RHEL 7. For more information about using this builder image, including OpenShift considerations, see https://github.com/sclorg/httpd-container/blob/master/2.4/README.md.", - "iconClass": "icon-apache", - "openshift.io/display-name": "Apache HTTP Server 2.4", - "openshift.io/provider-display-name": "Red Hat, Inc.", - "sampleRepo": "https://github.com/sclorg/httpd-ex.git", - "supports": "httpd", - "tags": "builder,httpd", - "version": "2.4" - }, - "from": { - "kind": "DockerImage", - "name": "registry.redhat.io/rhscl/httpd-24-rhel7" - }, - "referencePolicy": { - "type": "Local" - }, - "name": "2.4" - } - ] - } -} diff --git a/common/tests/remote-containers.sh b/common/tests/remote-containers.sh deleted file mode 100755 index b87ebff..0000000 --- a/common/tests/remote-containers.sh +++ /dev/null @@ -1,103 +0,0 @@ -#! /bin/bash - -set -e - -declare -A IMAGES - -for image in ${TESTED_IMAGES}; do - IMAGES[$image]=master -done - -OS=centos7 - -MERGE_INTO=origin/master - -# This is the Fedora default, some users' boxes have more strict -# defaults (e.g. 0077). -umask 0022 - -die () { echo >&2 " # FATAL: $*"; exit 1; } -info () { echo " * $*"; } -error () { echo >&2 " # ERROR: $*"; } - -test -f common.mk -a -f build.sh -a -d .git \ - || die "Doesn't seem to be run from common's git directory" - -analyse_commits () -{ - # TODO: If we wanted to test "after PR merge", this needs to take some - # argument specifying how long we should look in the commit history. - git merge-base --is-ancestor "$MERGE_INTO" HEAD \ - || die "Please rebase the commit '$(git rev-parse --short HEAD)'" \ - "to allow --ff merge into '$MERGE_INTO' branch" - - while read line; do - case $line in - Required-by:\ *) - set -- $line - old_IFS=$IFS - IFS=\# - set -- $2 - IFS=$old_IFS - set -- $1 $2 - info "PR commits ask for testing $1 from PR $2" - IMAGES[$1]=$2 - ;; - esac - done < <(git log --format=%B --reverse "$MERGE_INTO"..HEAD) -} - -analyse_commits - -for image in "${!IMAGES[@]}"; do - # We don't want to remove user's WIP stuff. - test -e "$image" && die "directory '$image' exists" - - ( set -e - testdir=$PWD - cleanup () { - set -x - # Ensure the cleanup finishes! - trap '' INT - # Go back, wherever we are. - cd "$testdir" - # Try to cleanup, if available (and if needed). - make clean -C "$image" || : - # Drop the image sources. - test ! -d "$image" || rm -rf "$image" - } - trap cleanup EXIT - - info "Testing $image image" - - # Use --recursive even if we remove 'common', because there might be - # other git submodules which need to be tested. - git clone --recursive -q https://github.com/sclorg/"$image".git - cd "$image" - - revision=${IMAGES[$image]} - if ! test "$revision" = master; then - info "Fetching $image PR $revision" - git fetch origin "pull/$revision/head":PR_BRANCH - git checkout PR_BRANCH - git submodule update - fi - - # We fail if the 'common' directory doesn't exist. - test -d common - rm -rf common - info "Replacing common with PR's version" - ln -s ../ common - - # TODO: Do we have to test all $(VERSION)s? - # TODO: The PS4 hack doesn't work if we run the testsuite as UID=0. - PS4="+ [$image] " make TARGET="$OS" test - - # Cleanup. - make clean - ) - - if test $? -ne 0; then - die "Tests for $image failed" - fi -done diff --git a/common/tests/squash/.gitignore b/common/tests/squash/.gitignore deleted file mode 100644 index 9414382..0000000 --- a/common/tests/squash/.gitignore +++ /dev/null @@ -1 +0,0 @@ -Dockerfile diff --git a/common/tests/squash/squash.sh b/common/tests/squash/squash.sh deleted file mode 100755 index 4393d55..0000000 --- a/common/tests/squash/squash.sh +++ /dev/null @@ -1,24 +0,0 @@ -#! /bin/sh - -set -e - -if grep -q "Red Hat Enterprise Linux release 8" /etc/system-release; then - # No use testing squash.py on rhel8 for now as it does not work at all - echo " ! test case ignored on RHEL8 host" - exit 0 -fi - -origin=busybox -squash=$(dirname "$(readlink -f "$0")")/../../squash.py -cd "$(dirname "$0")" - -cat > Dockerfile <(xargs chmod a+rwx)`" - }, - "accessModes": [ - "ReadWriteOnce" - ] - } -} -EOF -``` - -It is recommended, however, that you use other [type of PersistentVolume](https://docs.okd.io/latest/architecture/additional_concepts/storage.html#types-of-persistent-volumes) -such as NFS. - -Now you can create a new database deployment: - -``` -$ oc new-app examples/replica/postgresql_replica.json -``` - -## How does this example work? - -### Services 'postgresql-master' and 'postgresql-slave' - -These services are the entry point for connecting to, respectively, the primary -database server and any of the standby servers. - -In your application, connect to the `postgresql-master` service for write operations, and to `postgresql-master` or `postgresql-slave` for reads. -Keep in mind that reading from a slave might return slightly outdated data. - -To get a list of endpoints for the read-only standby servers, you can do a DNS -query. From a container in the same OpenShift project: - -``` -$ dig postgresql-slave A +search +short -``` - -### DeploymentConfig 'postgresql-master' - -This resource defines a [deployment configuration](https://docs.okd.io/latest/architecture/core_concepts/deployments.html#deployments-and-deployment-configurations) -to spawn the PostgreSQL primary database server, or master. - -Once the master is started, it works as a standalone database server, fully -independent of the slaves. - -### DeploymentConfig 'postgresql-slave' - -This resource defines a [deployment configuration](https://docs.okd.io/latest/architecture/core_concepts/deployments.html#deployments-and-deployment-configurations) -to spawn PostgreSQL standby servers, the slaves. - -Upon startup, each slave waits for the master server to become available (via -DNS lookup). Once that happens, the slave connects to the master and starts -streaming the WAL. - -To check that the slave is connected and streaming changes from the master, -you can issue the following commands: - -``` -$ master_name=`oc get pods -l name=postgresql-master -t '{{ (index .items 0).metadata.name }}'` -$ oc exec $master_name -- bash -c 'psql -c "select client_addr, state from pg_stat_replication;"' -``` - -After a successful deployment, you should get an output similar to: - -``` - client_addr | state ---------------+----------- - 172.17.0.227 | streaming -(1 row) -``` - -## Scaling - -By default, the provided template creates one primary and one standby server. -Scaling in this setup means increasing the number of standby servers, -consequently increasing data redundancy and concurrent read throughput (if -reading from slaves). - -You can add more slaves using `oc scale`: - -``` -$ oc scale dc postgresql-slave --replicas=2 -``` - -Using `oc scale` with `postgresql-master` is not supported. - -After scaling, you can verify that all slaves are streaming changes from the -master with: - -``` -$ oc exec $master_name -- bash -c 'psql -c "select client_addr, state from pg_stat_replication;"' - client_addr | state ---------------+----------- - 172.17.0.227 | streaming - 172.17.0.229 | streaming -(2 rows) -``` - -There should be one row per slave (number of replicas defined via `oc scale`). - -## Changing passwords - -You can change the passwords for the database user and admin, as well as the -password used for replication, by changing the appropriate environment variables -in the deployment configurations described earlier. -No other method is supported. - -On every deploy, passwords are reset to match the values in the environment -variables of the DeploymentConfig 'postgresql-master'. - -### POSTGRESQL_PASSWORD and POSTGRESQL_ADMIN_PASSWORD - -These are, respectively, the passwords for the regular database user defined -by `POSTGRESQL_USER` and the admin user 'postgres'. - -You can change these passwords with: - -``` -$ oc env dc postgresql-master POSTGRESQL_PASSWORD=NewPassword POSTGRESQL_ADMIN_PASSWORD=NewAdminPassword -deploymentconfigs/postgresql-master -``` - -This will trigger the redeployment of the primary server. -Note that you can change one password but not the other by simply omitting one -of the arguments to `oc env` above. - -You can verify that the new password is in effect with: - -``` -$ oc exec $master_name -- bash -c 'PGPASSWORD=NewPassword psql -h postgresql-master $POSTGRESQL_DATABASE $POSTGRESQL_USER -c "select * from (select inet_server_addr()) ra cross join (select current_database()) cdb cross join (select current_user) cu"' - inet_server_addr | current_database | current_user -------------------+------------------+-------------- - 172.17.1.38 | userdb | user -(1 row) -``` - -You should also be able to connect to a slave using the new password: - -``` -$ oc exec $master_name -- bash -c 'PGPASSWORD=NewPassword psql -h postgresql-slave $POSTGRESQL_DATABASE $POSTGRESQL_USER -c "select * from (select inet_server_addr()) ra cross join (select current_database()) cdb cross join (select current_user) cu"' - inet_server_addr | current_database | current_user -------------------+------------------+-------------- - 172.17.1.35 | userdb | user -(1 row) -``` - -For completeness, here's how to verify the new admin password: - -``` -$ oc exec $master_name -- bash -c 'PGPASSWORD=NewAdminPassword psql -h postgresql-master $POSTGRESQL_DATABASE -c "select * from (select inet_server_addr()) ra cross join (select current_database()) cdb cross join (select current_user) cu"' - inet_server_addr | current_database | current_user -------------------+------------------+-------------- - 172.17.1.38 | userdb | postgres -(1 row) -``` - -``` -$ oc exec $master_name -- bash -c 'PGPASSWORD=NewAdminPassword psql -h postgresql-slave $POSTGRESQL_DATABASE -c "select * from (select inet_server_addr()) ra cross join (select current_database()) cdb cross join (select current_user) cu"' - inet_server_addr | current_database | current_user -------------------+------------------+-------------- - 172.17.1.35 | userdb | postgres -(1 row) -``` - -### POSTGRESQL_MASTER_PASSWORD - -This password is used by standby servers to connect to the primary. Both -deployment configurations in this example setup need to agree on the value of -this password to have replication working correctly. - -You can change the environment variable with the password on both deployment -configurations at once: - -``` -$ oc env dc postgresql-master postgresql-slave POSTGRESQL_MASTER_PASSWORD=NewReplicationPassword -deploymentconfigs/postgresql-master -deploymentconfigs/postgresql-slave -``` - -This will trigger the redeployment of both primary and standby servers. - -Note that, as a current limitation in this example, the standby servers store -replicated data in an an ephemeral [emptyDir](https://docs.okd.io/latest/dev_guide/volumes.html). -This means that redeploying a standby server will cause it to start replicating -again from scratch. - -After the primary and standby servers are ready, you can verify that the standby -servers are successfully connected to the primary: - -``` -$ oc exec $master_name -- bash -c 'psql -c "select client_addr, state from pg_stat_replication;"' - client_addr | state --------------+----------- - 172.17.1.35 | streaming -(1 row) -``` diff --git a/examples/replica/postgresql_replica.json b/examples/replica/postgresql_replica.json deleted file mode 100644 index 775a2dc..0000000 --- a/examples/replica/postgresql_replica.json +++ /dev/null @@ -1,370 +0,0 @@ -{ - "kind": "Template", - "apiVersion": "v1", - "metadata": { - "name": "pg-replica-example", - "annotations": { - "description": "PostgreSQL Replication Example", - "iconClass": "icon-database", - "tags": "database,postgresql,replication" - } - }, - "parameters": [ - { - "name": "POSTGRESQL_MASTER_USER", - "description": "The username used for master-slave replication", - "value": "master", - "required": true - }, - { - "name": "POSTGRESQL_MASTER_PASSWORD", - "description": "The password for the PostgreSQL replication user", - "generate": "expression", - "from": "[a-zA-Z0-9]{12}", - "required": true - }, - { - "name": "POSTGRESQL_USER", - "description": "The username that clients will use to connect to PostgreSQL server", - "value": "user", - "required": true - }, - { - "name": "POSTGRESQL_PASSWORD", - "description": "The password for the PostgreSQL master user", - "generate": "expression", - "from": "[a-zA-Z0-9]{12}", - "required": true - }, - { - "name": "POSTGRESQL_DATABASE", - "description": "The name of the database that will be created", - "value": "userdb", - "required": true - }, - { - "name": "POSTGRESQL_ADMIN_PASSWORD", - "description": "The password for the PostgreSQL administrator", - "generate": "expression", - "from": "[a-zA-Z0-9]{12}", - "required": false - }, - { - "name": "POSTGRESQL_MASTER_SERVICE_NAME", - "description": "The name of the PostgreSQL Service (used to DNS lookup, default: 'postgresql-master')", - "value": "postgresql-master", - "required": true - }, - { - "name": "POSTGRESQL_SLAVE_SERVICE_NAME", - "description": "The name of the PostgreSQL Service (used to DNS lookup, default: 'postgresql-slave')", - "value": "postgresql-slave", - "required": true - }, - { - "name": "VOLUME_CAPACITY", - "description": "Volume space available for data, e.g. 512Mi, 2Gi", - "value": "512Mi", - "required": true - }, - { - "name": "IMAGESTREAMTAG", - "displayName": "ImageStreamTag", - "description": "The OpenShift ImageStreamTag to use for PostgreSQL.", - "value": "postgresql:9.6" - }, - { - "name": "NAMESPACE", - "displayName": "Namespace", - "description": "The OpenShift Namespace where the ImageStream resides.", - "value": "openshift" - } - ], - "objects": [ - { - "kind": "PersistentVolumeClaim", - "apiVersion": "v1", - "metadata": { - "name": "postgresql-data-claim" - }, - "spec": { - "accessModes": [ - "ReadWriteOnce" - ], - "resources": { - "requests": { - "storage": "${VOLUME_CAPACITY}" - } - } - } - }, - { - "kind": "Service", - "apiVersion": "v1", - "metadata": { - "name": "${POSTGRESQL_MASTER_SERVICE_NAME}", - "labels": { - "name": "${POSTGRESQL_MASTER_SERVICE_NAME}" - } - }, - "spec": { - "ports": [ - { - "port": 5432, - "targetPort": 5432 - } - ], - "selector": { - "name": "${POSTGRESQL_MASTER_SERVICE_NAME}" - }, - "clusterIP": "None" - } - }, - { - "kind": "Service", - "apiVersion": "v1", - "metadata": { - "name": "${POSTGRESQL_SLAVE_SERVICE_NAME}", - "labels": { - "name": "${POSTGRESQL_SLAVE_SERVICE_NAME}" - } - }, - "spec": { - "ports": [ - { - "port": 5432, - "targetPort": 5432 - } - ], - "selector": { - "name": "${POSTGRESQL_SLAVE_SERVICE_NAME}" - }, - "clusterIP": "None" - } - }, - { - "kind": "DeploymentConfig", - "apiVersion": "v1", - "metadata": { - "name": "${POSTGRESQL_MASTER_SERVICE_NAME}" - }, - "spec": { - "strategy": { - "type": "Recreate" - }, - "triggers": [ - { - "type": "ImageChange", - "imageChangeParams": { - "automatic": true, - "containerNames": [ - "postgresql-master" - ], - "from": { - "kind": "ImageStreamTag", - "name": "${IMAGESTREAMTAG}", - "namespace": "${NAMESPACE}" - }, - "lastTriggeredImage": "" - } - }, - { - "type": "ConfigChange" - } - ], - "replicas": 1, - "selector": { - "name": "${POSTGRESQL_MASTER_SERVICE_NAME}" - }, - "template": { - "metadata": { - "labels": { - "name": "${POSTGRESQL_MASTER_SERVICE_NAME}" - } - }, - "spec": { - "volumes": [ - { - "name": "postgresql-data", - "persistentVolumeClaim": { - "claimName": "postgresql-data-claim" - } - } - ], - "containers": [ - { - "name": "postgresql-master", - "image": " ", - "args": [ - "run-postgresql-master" - ], - "ports": [ - { - "containerPort": 5432 - } - ], - "readinessProbe": { - "timeoutSeconds": 1, - "initialDelaySeconds": 5, - "exec": { - "command": [ "/usr/libexec/check-container" ] - } - }, - "livenessProbe": { - "timeoutSeconds": 10, - "initialDelaySeconds": 120, - "exec": { - "command": [ "/usr/libexec/check-container", "--live" ] - } - }, - "env": [ - { - "name": "POSTGRESQL_MASTER_USER", - "value": "${POSTGRESQL_MASTER_USER}" - }, - { - "name": "POSTGRESQL_MASTER_PASSWORD", - "value": "${POSTGRESQL_MASTER_PASSWORD}" - }, - { - "name": "POSTGRESQL_USER", - "value": "${POSTGRESQL_USER}" - }, - { - "name": "POSTGRESQL_PASSWORD", - "value": "${POSTGRESQL_PASSWORD}" - }, - { - "name": "POSTGRESQL_DATABASE", - "value": "${POSTGRESQL_DATABASE}" - }, - { - "name": "POSTGRESQL_ADMIN_PASSWORD", - "value": "${POSTGRESQL_ADMIN_PASSWORD}" - } - ], - "volumeMounts": [ - { - "name": "postgresql-data", - "mountPath": "/var/lib/pgsql/data" - } - ] - } - ] - } - } - } - }, - { - "kind": "DeploymentConfig", - "apiVersion": "v1", - "metadata": { - "name": "${POSTGRESQL_SLAVE_SERVICE_NAME}" - }, - "spec": { - "strategy": { - "type": "Recreate" - }, - "triggers": [ - { - "type": "ImageChange", - "imageChangeParams": { - "automatic": true, - "containerNames": [ - "postgresql-slave" - ], - "from": { - "kind": "ImageStreamTag", - "name": "${IMAGESTREAMTAG}", - "namespace": "${NAMESPACE}" - }, - "lastTriggeredImage": "" - } - }, - { - "type": "ConfigChange" - } - ], - "replicas": 1, - "selector": { - "name": "${POSTGRESQL_SLAVE_SERVICE_NAME}" - }, - "template": { - "metadata": { - "labels": { - "name": "${POSTGRESQL_SLAVE_SERVICE_NAME}" - } - }, - "spec": { - "volumes": [ - { - "name": "postgresql-data", - "emptyDir": {} - } - ], - "containers": [ - { - "name": "postgresql-slave", - "image": " ", - "args": [ - "run-postgresql-slave" - ], - "ports": [ - { - "containerPort": 5432 - } - ], - "readinessProbe": { - "timeoutSeconds": 1, - "initialDelaySeconds": 5, - "exec": { - "command": [ "/usr/libexec/check-container" ] - } - }, - "livenessProbe": { - "timeoutSeconds": 10, - "initialDelaySeconds": 120, - "exec": { - "command": [ "/usr/libexec/check-container", "--live" ] - } - }, - "env": [ - { - "name": "POSTGRESQL_MASTER_SERVICE_NAME", - "value": "${POSTGRESQL_MASTER_SERVICE_NAME}" - }, - { - "name": "POSTGRESQL_MASTER_USER", - "value": "${POSTGRESQL_MASTER_USER}" - }, - { - "name": "POSTGRESQL_MASTER_PASSWORD", - "value": "${POSTGRESQL_MASTER_PASSWORD}" - }, - { - "name": "POSTGRESQL_USER", - "value": "${POSTGRESQL_USER}" - }, - { - "name": "POSTGRESQL_PASSWORD", - "value": "${POSTGRESQL_PASSWORD}" - }, - { - "name": "POSTGRESQL_DATABASE", - "value": "${POSTGRESQL_DATABASE}" - } - ], - "volumeMounts": [ - { - "name": "postgresql-data", - "mountPath": "/var/lib/pgsql/data" - } - ] - } - ] - } - } - } - } - ] -} diff --git a/examples/s2i-dump-data/.s2i/bin/assemble b/examples/s2i-dump-data/.s2i/bin/assemble deleted file mode 100755 index 48e9e0c..0000000 --- a/examples/s2i-dump-data/.s2i/bin/assemble +++ /dev/null @@ -1,36 +0,0 @@ -#! /usr/bin/bash -x - -# fail early -set -e - -# source the convenience tooling -source "${CONTAINER_SCRIPTS_PATH}/common.sh" - -# set $PGDATA variable -set_pgdata - -# assert uninitialized data -test ! -f "$PGDATA/postgresql.conf" - -# empty config file is needed after 'initialize_database' call -touch "$POSTGRESQL_CONFIG_FILE" - -initialize_database - -# start local PostgreSQL server (wait with '-w') -pg_ctl -w start -o "-h ''" - -# load all sql files -shopt -s nullglob -for file in /tmp/src/init/*.sql; do - psql -f "$file" -done - -pg_ctl stop - -# dump the data into $PWD (in-image storage) -tar caf data.tar.xz -C "$PGDATA" . -rm -rf "$PGDATA" - -# install pre-start hook -cp -r /tmp/src/postgresql-pre-start . diff --git a/examples/s2i-dump-data/init/init.sql b/examples/s2i-dump-data/init/init.sql deleted file mode 100644 index 6cda639..0000000 --- a/examples/s2i-dump-data/init/init.sql +++ /dev/null @@ -1,2 +0,0 @@ -CREATE TABLE test (sth TEXT); -INSERT INTO test VALUES ('hello world'); diff --git a/examples/s2i-dump-data/postgresql-pre-start/10_boot.sh b/examples/s2i-dump-data/postgresql-pre-start/10_boot.sh deleted file mode 100644 index ad85bcb..0000000 --- a/examples/s2i-dump-data/postgresql-pre-start/10_boot.sh +++ /dev/null @@ -1,3 +0,0 @@ -if test ! -f "$PGDATA/postgresql.conf"; then - tar xf "$APP_DATA"/src/data.tar.xz -C "$PGDATA" -fi diff --git a/help.md b/help.md deleted file mode 120000 index 42061c0..0000000 --- a/help.md +++ /dev/null @@ -1 +0,0 @@ -README.md \ No newline at end of file diff --git a/root/help.1 b/root/help.1 deleted file mode 100644 index 9e4ad32..0000000 --- a/root/help.1 +++ /dev/null @@ -1,404 +0,0 @@ -.nh -.TH PostgreSQL 12 SQL Database Server container image -.PP -This container image includes PostgreSQL 12 SQL database server for OpenShift and general usage. -Users can choose between RHEL, CentOS and Fedora based images. -The RHEL images are available in the Red Hat Container Catalog -\[la]https://access.redhat.com/containers/\[ra], -the CentOS images are available on Docker Hub -\[la]https://hub.docker.com/r/centos/\[ra], -and the Fedora images are available in Fedora Registry -\[la]https://registry.fedoraproject.org/\[ra]\&. -The resulting image can be run using podman -\[la]https://github.com/containers/libpod\[ra]\&. - -.PP -Note: while the examples in this README are calling \fB\fCpodman\fR, you can replace any such calls by \fB\fCdocker\fR with the same arguments - -.SH Description -.PP -This container image provides a containerized packaging of the PostgreSQL postgres daemon -and client application. The postgres server daemon accepts connections from clients -and provides access to content from PostgreSQL databases on behalf of the clients. -You can find more information on the PostgreSQL project from the project Web site -(https://www.postgresql.org/). - -.SH Usage -.PP -For this, we will assume that you are using the \fB\fCrhscl/postgresql\-12\-rhel7\fR image, available via \fB\fCpostgresql:12\fR imagestream tag in Openshift. -If you want to set only the mandatory environment variables and not store the database -in a host directory, execute the following command: - -.PP -.RS - -.nf -$ podman run \-d \-\-name postgresql\_database \-e POSTGRESQL\_USER=user \-e POSTGRESQL\_PASSWORD=pass \-e POSTGRESQL\_DATABASE=db \-p 5432:5432 rhscl/postgresql\-12\-rhel7 - -.fi -.RE - -.PP -This will create a container named \fB\fCpostgresql\_database\fR running PostgreSQL with -database \fB\fCdb\fR and user with credentials \fB\fCuser:pass\fR\&. -> Note: user \fB\fCpostgres\fR is reserved for internal usage - -.PP -Port 5432 will be exposed -and mapped to the host. If you want your database to be persistent across container -executions, also add a \fB\fC\-v /host/db/path:/var/lib/pgsql/data\fR argument (see -below). This will be the PostgreSQL database cluster directory. - -.PP -The same can be achieved in an Openshift instance using templates provided by Openshift or available in examples -\[la]https://github.com/sclorg/postgresql-container/tree/master/examples\[ra]: - -.PP -.RS - -.nf -$ oc process \-f examples/postgresql\-ephemeral\-template.json \-p POSTGRESQL\_VERSION=12 \-p POSTGRESQL\_USER=user \-p POSTGRESQL\_PASSWORD=pass \-p POSTGRESQL\_DATABASE=db | oc create \-f \- - -.fi -.RE - -.PP -If the database cluster directory is not initialized, the entrypoint script will -first run \fB\fCinitdb\fR -\[la]http://www.postgresql.org/docs/12/static/app-initdb.html\[ra] -and setup necessary database users and passwords. After the database is initialized, -or if it was already present, \fB\fCpostgres\fR -\[la]http://www.postgresql.org/docs/12/static/app-postgres.html\[ra] -is executed and will run as PID 1. You can stop the detached container by running -\fB\fCpodman stop postgresql\_database\fR\&. - -.SH Environment variables and volumes -.PP -The image recognizes the following environment variables that you can set during -initialization by passing \fB\fC\-e VAR=VALUE\fR to the Docker run command. - -.PP -\fB\fB\fCPOSTGRESQL\_USER\fR\fP -.br - User name for PostgreSQL account to be created - -.PP -\fB\fB\fCPOSTGRESQL\_PASSWORD\fR\fP -.br - Password for the user account - -.PP -\fB\fB\fCPOSTGRESQL\_DATABASE\fR\fP -.br - Database name - -.PP -\fB\fB\fCPOSTGRESQL\_ADMIN\_PASSWORD\fR\fP -.br - Password for the \fB\fCpostgres\fR admin account (optional) - -.PP -Alternatively, the following options are related to migration scenario: - -.PP -\fB\fB\fCPOSTGRESQL\_MIGRATION\_REMOTE\_HOST\fR\fP -.br - Hostname/IP to migrate from - -.PP -\fB\fB\fCPOSTGRESQL\_MIGRATION\_ADMIN\_PASSWORD\fR\fP -.br - Password for the remote 'postgres' admin user - -.PP -\fB\fB\fCPOSTGRESQL\_MIGRATION\_IGNORE\_ERRORS (optional, default 'no')\fR\fP -.br - Set to 'yes' to ignore sql import errors - -.PP -The following environment variables influence the PostgreSQL configuration file. They are all optional. - -.PP -\fB\fB\fCPOSTGRESQL\_MAX\_CONNECTIONS (default: 100)\fR\fP -.br - The maximum number of client connections allowed - -.PP -\fB\fB\fCPOSTGRESQL\_MAX\_PREPARED\_TRANSACTIONS (default: 0)\fR\fP -.br - Sets the maximum number of transactions that can be in the "prepared" state. If you are using prepared transactions, you will probably want this to be at least as large as max\_connections - -.PP -\fB\fB\fCPOSTGRESQL\_SHARED\_BUFFERS (default: 32M)\fR\fP -.br - Sets how much memory is dedicated to PostgreSQL to use for caching data - -.PP -\fB\fB\fCPOSTGRESQL\_EFFECTIVE\_CACHE\_SIZE (default: 128M)\fR\fP -.br - Set to an estimate of how much memory is available for disk caching by the operating system and within the database itself - -.PP -You can also set the following mount points by passing the \fB\fC\-v /host/dir:/container/dir:Z\fR flag to Docker. - -.PP -\fB\fB\fC/var/lib/pgsql/data\fR\fP -.br - PostgreSQL database cluster directory - -.PP -\fBNotice: When mouting a directory from the host into the container, ensure that the mounted -directory has the appropriate permissions and that the owner and group of the directory -matches the user UID or name which is running inside the container.\fP - -.PP -Typically (unless you use \fB\fCpodman run \-u\fR option) processes in container -run under UID 26, so \-\- on GNU/Linux \-\- you can fix the datadir permissions -for example by: - -.PP -.RS - -.nf -$ setfacl \-m u:26:\-wx /your/data/dir -$ podman run <...> \-v /your/data/dir:/var/lib/pgsql/data:Z <...> - -.fi -.RE - -.SH Data migration -.PP -PostgreSQL container supports migration of data from remote PostgreSQL server. -You can run it like: - -.PP -.RS - -.nf -$ podman run \-d \-\-name postgresql\_database \\ - \-e POSTGRESQL\_MIGRATION\_REMOTE\_HOST=172.17.0.2 \\ - \-e POSTGRESQL\_MIGRATION\_ADMIN\_PASSWORD=remoteAdminP@ssword \\ - [ OPTIONAL\_CONFIGURATION\_VARIABLES ] - openshift/postgresql\-92\-centos7 - -.fi -.RE - -.PP -The migration is done the \fBdump and restore\fP way (running \fB\fCpg\_dumpall\fR against -remote cluster and importing the dump locally by \fB\fCpsql\fR). Because the process -is streamed (unix pipeline), there are no intermediate dump files created during -this process to not waste additional storage space. - -.PP -If some SQL commands fail during applying, the default behavior -of the migration script is to fail as well to ensure the \fBall\fP or \fBnothing\fP -result of scripted, unattended migration. In most common cases, successful -migration is expected (but not guaranteed!), given you migrate from -a previous version of PostgreSQL server container, that is created using -the same principles as this one (e.g. migration from -\fB\fCopenshift/postgresql\-92\-centos7\fR to \fB\fCcentos/postgresql\-95\-centos7\fR). -Migration from a different kind of PostgreSQL container can likely fail. - -.PP -If this \fBall\fP or \fBnothing\fP principle is inadequate for you, and you know -what you are doing, there's optional \fB\fCPOSTGRESQL\_MIGRATION\_IGNORE\_ERRORS\fR option -which does \fBbest effort\fP migration (some data might be lost, it is up to user -to review the standard error output and fix the issues manually in -post\-migration time). - -.PP -Please keep in mind that the container image provides help for users' -convenience, but fully automatic migration is not guaranteed. Thus, before you -start proceeding with the database migration, get prepared to perform manual -steps in order to get all your data migrated. - -.PP -Note that you might not use variables like \fB\fCPOSTGRESQL\_USER\fR in migration -scenario, all the data (including info about databases, roles or passwords are -copied from old cluster). Ensure that you use the same -\fB\fCOPTIONAL\_CONFIGURATION\_VARIABLES\fR as you used for initialization of the old -PostgreSQL container. If some non\-default configuration is done on remote -cluster, you might need to copy the configuration files manually, too. - -.PP -Security warning: Note that the IP communication between old and new PostgreSQL -clusters is not encrypted by default, it is up to user to configure SSL on -remote cluster or ensure security via different means. - -.SH PostgreSQL auto\-tuning -.PP -When the PostgreSQL image is run with the \fB\fC\-\-memory\fR parameter set and if there -are no values provided for \fB\fCPOSTGRESQL\_SHARED\_BUFFERS\fR and -\fB\fCPOSTGRESQL\_EFFECTIVE\_CACHE\_SIZE\fR those values are automatically calculated -based on the value provided in the \fB\fC\-\-memory\fR parameter. - -.PP -The values are calculated based on the -upstream -\[la]https://wiki.postgresql.org/wiki/Tuning_Your_PostgreSQL_Server\[ra] -formulas. For the \fB\fCshared\_buffers\fR we use 1/4 of given memory and for the -\fB\fCeffective\_cache\_size\fR we set the value to 1/2 of the given memory. - -.SH PostgreSQL admin account -.PP -The admin account \fB\fCpostgres\fR has no password set by default, only allowing local -connections. You can set it by setting the \fB\fCPOSTGRESQL\_ADMIN\_PASSWORD\fR environment -variable when initializing your container. This will allow you to login to the -\fB\fCpostgres\fR account remotely. Local connections will still not require a password. - -.SH Changing passwords -.PP -Since passwords are part of the image configuration, the only supported method -to change passwords for the database user (\fB\fCPOSTGRESQL\_USER\fR) and \fB\fCpostgres\fR -admin user is by changing the environment variables \fB\fCPOSTGRESQL\_PASSWORD\fR and -\fB\fCPOSTGRESQL\_ADMIN\_PASSWORD\fR, respectively. - -.PP -Changing database passwords through SQL statements or any way other than through -the environment variables aforementioned will cause a mismatch between the -values stored in the variables and the actual passwords. Whenever a database -container starts it will reset the passwords to the values stored in the -environment variables. - -.SH Upgrading database (by switching to newer PostgreSQL image version) -.PP -** Warning! Please, before you decide to do the data directory upgrade, always -ensure that you've carefully backed up all your data and that you are OK with -potential manual rollback! ** - -.PP -This image supports automatic upgrade of data directory created by -the PostgreSQL server version 10 (and \fIonly\fP this version) \- provided by sclorg -image. The upgrade process is designed so that you should be able to just -switch from \fIimage A\fP to \fIimage B\fP, and set the \fB\fC$POSTGRESQL\_UPGRADE\fR variable -appropriately to explicitly request the database data transformation. - -.PP -The upgrade process is internally implemented via \fB\fCpg\_upgrade\fR binary, and for -that purpose the container needs to contain two versions of PostgreSQL server -(have a look at \fB\fCman pg\_upgrade\fR for more info). - -.PP -For the \fB\fCpg\_upgrade\fR process \- and the new server version, we need to initialize -a brand new data directory. That's data directory is created automatically by -container tooling under /var/lib/pgsql/data, which is usually external -bind\-mountpoint. The \fB\fCpg\_upgrade\fR execution is then similar to dump\&restore -approach \-\- it starts both old and new PostgreSQL servers (within container) and -"dumps" the old datadir while and at the same time it "restores" it into new -datadir. This operation requires a lot of data files copying, so you can decide -what type of upgrade you'll do by setting \fB\fC$POSTGRESQL\_UPGRADE\fR appropriately: - -.PP -\fB\fB\fCcopy\fR\fP -.br - The data files are copied from old datadir to new datadir. This option has low risk of data loss in case of some upgrade failure. - -.PP -\fB\fB\fChardlink\fR\fP -.br - Data files are hard\-linked from old to the new data directory, which brings performance optimization \- but the old directory becomes unusable, even in case of failure. - -.PP -Note that because we copy data directory, you need to make sure that you have -enough space for the copy; upgrade failure because of not enough space might -lead to data loss. - -.SH Extending image -.PP -This image can be extended in Openshift using the \fB\fCSource\fR build strategy or via the standalone -source\-to\-image -\[la]https://github.com/openshift/source-to-image\[ra] application (where available). -For this, we will assume that you are using the \fB\fCrhscl/postgresql\-12\-rhel7\fR image, -available via \fB\fCpostgresql:12\fR imagestream tag in Openshift. - -.PP -For example to build customized image \fB\fCnew\-postgresql\fR -with configuration from \fB\fChttps://github.com/sclorg/postgresql\-container/tree/master/examples/extending\-image\fR run: - -.PP -.RS - -.nf -$ oc new\-app postgresql:12\~https://github.com/sclorg/postgresql\-container.git \\ - \-\-name new\-postgresql \\ - \-\-context\-dir examples/extending\-image/ \\ - \-e POSTGRESQL\_USER=user \\ - \-e POSTGRESQL\_DATABASE=db \\ - \-e POSTGRESQL\_PASSWORD=password - -.fi -.RE - -.PP -or via \fB\fCs2i\fR: - -.PP -.RS - -.nf -$ s2i build \-\-context\-dir examples/extending\-image/ https://github.com/sclorg/postgresql\-container.git rhscl/postgresql\-12\-rhel7 new\-postgresql - -.fi -.RE - -.PP -The directory passed to Openshift should contain one or more of the -following directories: - -.SS \fB\fCpostgresql\-pre\-start/\fR -.PP -Source all \fB\fC*.sh\fR files from this directory during early start of the -container. There's no PostgreSQL daemon running on background. - -.SS \fB\fCpostgresql\-cfg/\fR -.PP -Contained configuration files (\fB\fC*.conf\fR) will be included at the end of image -postgresql.conf file. - -.SS \fB\fCpostgresql\-init/\fR -.PP -Contained shell scripts (\fB\fC*.sh\fR) are sourced when the database is freshly -initialized (after successful initdb run which made the data directory -non\-empty). At the time of sourcing these scripts, the local PostgreSQL -server is running. For re\-deployments scenarios with persistent data -directory, the scripts are not sourced (no\-op). - -.SS \fB\fCpostgresql\-start/\fR -.PP -Same sematics as \fB\fCpostgresql\-init/\fR, except that these scripts are -always sourced (after \fB\fCpostgresql\-init/\fR scripts, if they exist). - -.ti 0 -\l'\n(.lu' - -.PP -During the s2i build all provided files are copied into \fB\fC/opt/app\-root/src\fR -directory in the new image. Only one -file with the same name can be used for customization and user provided files -are preferred over default files in \fB\fC/usr/share/container\-scripts/\fR\- -so it is possible to overwrite them. - -.SH Troubleshooting -.PP -At first the postgres daemon writes its logs to the standard output, so these are available in the container log. The log can be examined by running: - -.PP -.RS - -.nf -podman logs - -.fi -.RE - -.PP -Then log output is redirected to logging collector process and will appear in directory "pg\_log". - -.SH See also -.PP -Dockerfile and other sources for this container image are available on -https://github.com/sclorg/postgresql\-container. -In that repository, the Dockerfile for CentOS is called Dockerfile, the Dockerfile -for RHEL7 is called Dockerfile.rhel7, the Dockerfile for RHEL8 is called Dockerfile.rhel8, -and the Dockerfile for Fedora is called Dockerfile.fedora. diff --git a/root/usr/bin/container-entrypoint b/root/usr/bin/container-entrypoint deleted file mode 100755 index 5fc4448..0000000 --- a/root/usr/bin/container-entrypoint +++ /dev/null @@ -1,3 +0,0 @@ -#!/bin/bash - -exec "$@" diff --git a/root/usr/bin/run-postgresql b/root/usr/bin/run-postgresql deleted file mode 100755 index 2367e57..0000000 --- a/root/usr/bin/run-postgresql +++ /dev/null @@ -1,58 +0,0 @@ -#!/bin/bash - -export ENABLE_REPLICATION=${ENABLE_REPLICATION:-false} - -set -eu -export_vars=$(cgroup-limits) ; export $export_vars - -source "${CONTAINER_SCRIPTS_PATH}/common.sh" - -set_pgdata - -process_extending_files \ - "${APP_DATA}/src/postgresql-pre-start" \ - "${CONTAINER_SCRIPTS_PATH}/pre-start" - -check_env_vars -generate_passwd_file -generate_postgresql_config - -# Is this brand new data volume? -PG_INITIALIZED=false - -if [ ! -f "$PGDATA/postgresql.conf" ]; then - initialize_database - PG_INITIALIZED=: -else - try_pgupgrade -fi - -# Use insanely large timeout (24h) to ensure that the potential recovery has -# enough time here to happen (unless liveness probe kills us). Note that in -# case of server failure this command still exists immediately. -pg_ctl start -w --timeout 86400 -o "-h ''" - -# This is just a pedantic safety measure (the timeout above is unlikely to -# happen), but `pt_ctl -w` is not reliable prior to PostgreSQL v10 where it -# returns exit_status=0 even if the server is still starting. For more info -# see the issue#297 and -# https://www.postgresql.org/message-id/CAB7nPqSJs85wK9aknm%3D_jmS6GnH3SQBhpzKcqs8Qo2LhEg2etw%40mail.gmail.com -pg_isready - -if $PG_INITIALIZED ; then - process_extending_files \ - "${APP_DATA}/src/postgresql-init" \ - "${CONTAINER_SCRIPTS_PATH}/init" - migrate_db - create_users -fi - -process_extending_files \ - "${APP_DATA}/src/postgresql-start" \ - "${CONTAINER_SCRIPTS_PATH}/start" - -pg_ctl stop - -unset_env_vars -echo "Starting server..." -exec postgres "$@" diff --git a/root/usr/bin/run-postgresql-master b/root/usr/bin/run-postgresql-master deleted file mode 100755 index 79e7cc2..0000000 --- a/root/usr/bin/run-postgresql-master +++ /dev/null @@ -1,5 +0,0 @@ -#!/bin/bash - -export ENABLE_REPLICATION=true - -exec run-postgresql "$@" diff --git a/root/usr/bin/run-postgresql-slave b/root/usr/bin/run-postgresql-slave deleted file mode 100755 index 586bf5d..0000000 --- a/root/usr/bin/run-postgresql-slave +++ /dev/null @@ -1,39 +0,0 @@ -#!/bin/bash - -export ENABLE_REPLICATION=true - -set -eu -export_vars=$(cgroup-limits) ; export $export_vars - -source "$CONTAINER_SCRIPTS_PATH"/common.sh - -set_pgdata - -function initialize_replica() { - echo "Initializing PostgreSQL slave ..." - # TODO: Validate and reuse existing data? - rm -rf $PGDATA - PGPASSWORD="${POSTGRESQL_MASTER_PASSWORD}" pg_basebackup -X fetch --no-password --pgdata ${PGDATA} --host=${MASTER_FQDN} --port=5432 -U "${POSTGRESQL_MASTER_USER}" - - # PostgreSQL recovery configuration. - generate_postgresql_recovery_config - cat >> "$PGDATA/postgresql.auto.conf" <&2 <&2 "fixing permissions on '$dir' directory" - find "$dir" -exec chown "$uid:0" {} \; - find "$dir" -exec chmod "g+r$write" {} \; - find "$dir" -type d -exec chmod g+x {} + -done diff --git a/root/usr/share/container-scripts/postgresql/README.md b/root/usr/share/container-scripts/postgresql/README.md deleted file mode 100644 index be70877..0000000 --- a/root/usr/share/container-scripts/postgresql/README.md +++ /dev/null @@ -1,330 +0,0 @@ -PostgreSQL 12 SQL Database Server container image -=============================================== - -This container image includes PostgreSQL 12 SQL database server for OpenShift and general usage. -Users can choose between RHEL, CentOS and Fedora based images. -The RHEL images are available in the [Red Hat Container Catalog](https://access.redhat.com/containers/), -the CentOS images are available on [Docker Hub](https://hub.docker.com/r/centos/), -and the Fedora images are available in [Fedora Registry](https://registry.fedoraproject.org/). -The resulting image can be run using [podman](https://github.com/containers/libpod). - -Note: while the examples in this README are calling `podman`, you can replace any such calls by `docker` with the same arguments - - -Description ------------ - -This container image provides a containerized packaging of the PostgreSQL postgres daemon -and client application. The postgres server daemon accepts connections from clients -and provides access to content from PostgreSQL databases on behalf of the clients. -You can find more information on the PostgreSQL project from the project Web site -(https://www.postgresql.org/). - - -Usage ------ - -For this, we will assume that you are using the `rhscl/postgresql-12-rhel7` image, available via `postgresql:12` imagestream tag in Openshift. -If you want to set only the mandatory environment variables and not store the database -in a host directory, execute the following command: - -``` -$ podman run -d --name postgresql_database -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -p 5432:5432 rhscl/postgresql-12-rhel7 -``` - -This will create a container named `postgresql_database` running PostgreSQL with -database `db` and user with credentials `user:pass`. -> Note: user `postgres` is reserved for internal usage - -Port 5432 will be exposed -and mapped to the host. If you want your database to be persistent across container -executions, also add a `-v /host/db/path:/var/lib/pgsql/data` argument (see -below). This will be the PostgreSQL database cluster directory. - -The same can be achieved in an Openshift instance using templates provided by Openshift or available in [examples](https://github.com/sclorg/postgresql-container/tree/master/examples): - -``` -$ oc process -f examples/postgresql-ephemeral-template.json -p POSTGRESQL_VERSION=12 -p POSTGRESQL_USER=user -p POSTGRESQL_PASSWORD=pass -p POSTGRESQL_DATABASE=db | oc create -f - -``` - -If the database cluster directory is not initialized, the entrypoint script will -first run [`initdb`](http://www.postgresql.org/docs/12/static/app-initdb.html) -and setup necessary database users and passwords. After the database is initialized, -or if it was already present, [`postgres`](http://www.postgresql.org/docs/12/static/app-postgres.html) -is executed and will run as PID 1. You can stop the detached container by running -`podman stop postgresql_database`. - - - -Environment variables and volumes ---------------------------------- - -The image recognizes the following environment variables that you can set during -initialization by passing `-e VAR=VALUE` to the Docker run command. - -**`POSTGRESQL_USER`** - User name for PostgreSQL account to be created - -**`POSTGRESQL_PASSWORD`** - Password for the user account - -**`POSTGRESQL_DATABASE`** - Database name - -**`POSTGRESQL_ADMIN_PASSWORD`** - Password for the `postgres` admin account (optional) - - -Alternatively, the following options are related to migration scenario: - -**`POSTGRESQL_MIGRATION_REMOTE_HOST`** - Hostname/IP to migrate from - -**`POSTGRESQL_MIGRATION_ADMIN_PASSWORD`** - Password for the remote 'postgres' admin user - -**`POSTGRESQL_MIGRATION_IGNORE_ERRORS (optional, default 'no')`** - Set to 'yes' to ignore sql import errors - - -The following environment variables influence the PostgreSQL configuration file. They are all optional. - -**`POSTGRESQL_MAX_CONNECTIONS (default: 100)`** - The maximum number of client connections allowed - -**`POSTGRESQL_MAX_PREPARED_TRANSACTIONS (default: 0)`** - Sets the maximum number of transactions that can be in the "prepared" state. If you are using prepared transactions, you will probably want this to be at least as large as max_connections - -**`POSTGRESQL_SHARED_BUFFERS (default: 32M)`** - Sets how much memory is dedicated to PostgreSQL to use for caching data - -**`POSTGRESQL_EFFECTIVE_CACHE_SIZE (default: 128M)`** - Set to an estimate of how much memory is available for disk caching by the operating system and within the database itself - - -You can also set the following mount points by passing the `-v /host/dir:/container/dir:Z` flag to Docker. - -**`/var/lib/pgsql/data`** - PostgreSQL database cluster directory - - -**Notice: When mouting a directory from the host into the container, ensure that the mounted -directory has the appropriate permissions and that the owner and group of the directory -matches the user UID or name which is running inside the container.** - -Typically (unless you use `podman run -u` option) processes in container -run under UID 26, so -- on GNU/Linux -- you can fix the datadir permissions -for example by: - -``` -$ setfacl -m u:26:-wx /your/data/dir -$ podman run <...> -v /your/data/dir:/var/lib/pgsql/data:Z <...> -``` - - -Data migration ----------------------- - -PostgreSQL container supports migration of data from remote PostgreSQL server. -You can run it like: - -``` -$ podman run -d --name postgresql_database \ - -e POSTGRESQL_MIGRATION_REMOTE_HOST=172.17.0.2 \ - -e POSTGRESQL_MIGRATION_ADMIN_PASSWORD=remoteAdminP@ssword \ - [ OPTIONAL_CONFIGURATION_VARIABLES ] - openshift/postgresql-92-centos7 -``` - -The migration is done the **dump and restore** way (running `pg_dumpall` against -remote cluster and importing the dump locally by `psql`). Because the process -is streamed (unix pipeline), there are no intermediate dump files created during -this process to not waste additional storage space. - -If some SQL commands fail during applying, the default behavior -of the migration script is to fail as well to ensure the **all** or **nothing** -result of scripted, unattended migration. In most common cases, successful -migration is expected (but not guaranteed!), given you migrate from -a previous version of PostgreSQL server container, that is created using -the same principles as this one (e.g. migration from -`openshift/postgresql-92-centos7` to `centos/postgresql-95-centos7`). -Migration from a different kind of PostgreSQL container can likely fail. - -If this **all** or **nothing** principle is inadequate for you, and you know -what you are doing, there's optional `POSTGRESQL_MIGRATION_IGNORE_ERRORS` option -which does **best effort** migration (some data might be lost, it is up to user -to review the standard error output and fix the issues manually in -post-migration time). - -Please keep in mind that the container image provides help for users' -convenience, but fully automatic migration is not guaranteed. Thus, before you -start proceeding with the database migration, get prepared to perform manual -steps in order to get all your data migrated. - -Note that you might not use variables like `POSTGRESQL_USER` in migration -scenario, all the data (including info about databases, roles or passwords are -copied from old cluster). Ensure that you use the same -`OPTIONAL_CONFIGURATION_VARIABLES` as you used for initialization of the old -PostgreSQL container. If some non-default configuration is done on remote -cluster, you might need to copy the configuration files manually, too. - -Security warning: Note that the IP communication between old and new PostgreSQL -clusters is not encrypted by default, it is up to user to configure SSL on -remote cluster or ensure security via different means. - -PostgreSQL auto-tuning --------------------- - -When the PostgreSQL image is run with the `--memory` parameter set and if there -are no values provided for `POSTGRESQL_SHARED_BUFFERS` and -`POSTGRESQL_EFFECTIVE_CACHE_SIZE` those values are automatically calculated -based on the value provided in the `--memory` parameter. - -The values are calculated based on the -[upstream](https://wiki.postgresql.org/wiki/Tuning_Your_PostgreSQL_Server) -formulas. For the `shared_buffers` we use 1/4 of given memory and for the -`effective_cache_size` we set the value to 1/2 of the given memory. - -PostgreSQL admin account ------------------------- -The admin account `postgres` has no password set by default, only allowing local -connections. You can set it by setting the `POSTGRESQL_ADMIN_PASSWORD` environment -variable when initializing your container. This will allow you to login to the -`postgres` account remotely. Local connections will still not require a password. - - -Changing passwords ------------------- - -Since passwords are part of the image configuration, the only supported method -to change passwords for the database user (`POSTGRESQL_USER`) and `postgres` -admin user is by changing the environment variables `POSTGRESQL_PASSWORD` and -`POSTGRESQL_ADMIN_PASSWORD`, respectively. - -Changing database passwords through SQL statements or any way other than through -the environment variables aforementioned will cause a mismatch between the -values stored in the variables and the actual passwords. Whenever a database -container starts it will reset the passwords to the values stored in the -environment variables. - - -Upgrading database (by switching to newer PostgreSQL image version) -------------------------------------------------------------------- - -** Warning! Please, before you decide to do the data directory upgrade, always -ensure that you've carefully backed up all your data and that you are OK with -potential manual rollback! ** - -This image supports automatic upgrade of data directory created by -the PostgreSQL server version 10 (and _only_ this version) - provided by sclorg -image. The upgrade process is designed so that you should be able to just -switch from *image A* to *image B*, and set the `$POSTGRESQL_UPGRADE` variable -appropriately to explicitly request the database data transformation. - -The upgrade process is internally implemented via `pg_upgrade` binary, and for -that purpose the container needs to contain two versions of PostgreSQL server -(have a look at `man pg_upgrade` for more info). - -For the `pg_upgrade` process - and the new server version, we need to initialize -a brand new data directory. That's data directory is created automatically by -container tooling under /var/lib/pgsql/data, which is usually external -bind-mountpoint. The `pg_upgrade` execution is then similar to dump&restore -approach -- it starts both old and new PostgreSQL servers (within container) and -"dumps" the old datadir while and at the same time it "restores" it into new -datadir. This operation requires a lot of data files copying, so you can decide -what type of upgrade you'll do by setting `$POSTGRESQL_UPGRADE` appropriately: - -**`copy`** - The data files are copied from old datadir to new datadir. This option has low risk of data loss in case of some upgrade failure. - -**`hardlink`** - Data files are hard-linked from old to the new data directory, which brings performance optimization - but the old directory becomes unusable, even in case of failure. - - -Note that because we copy data directory, you need to make sure that you have -enough space for the copy; upgrade failure because of not enough space might -lead to data loss. - - -Extending image ----------------- - -This image can be extended in Openshift using the `Source` build strategy or via the standalone -[source-to-image](https://github.com/openshift/source-to-image) application (where available). -For this, we will assume that you are using the `rhscl/postgresql-12-rhel7` image, -available via `postgresql:12` imagestream tag in Openshift. - -For example to build customized image `new-postgresql` -with configuration from `https://github.com/sclorg/postgresql-container/tree/master/examples/extending-image` run: - -``` -$ oc new-app postgresql:12~https://github.com/sclorg/postgresql-container.git \ - --name new-postgresql \ - --context-dir examples/extending-image/ \ - -e POSTGRESQL_USER=user \ - -e POSTGRESQL_DATABASE=db \ - -e POSTGRESQL_PASSWORD=password -``` - -or via `s2i`: - -``` -$ s2i build --context-dir examples/extending-image/ https://github.com/sclorg/postgresql-container.git rhscl/postgresql-12-rhel7 new-postgresql -``` - -The directory passed to Openshift should contain one or more of the -following directories: - - -##### `postgresql-pre-start/` - -Source all `*.sh` files from this directory during early start of the -container. There's no PostgreSQL daemon running on background. - - -##### `postgresql-cfg/` - -Contained configuration files (`*.conf`) will be included at the end of image -postgresql.conf file. - - -##### `postgresql-init/` - -Contained shell scripts (`*.sh`) are sourced when the database is freshly -initialized (after successful initdb run which made the data directory -non-empty). At the time of sourcing these scripts, the local PostgreSQL -server is running. For re-deployments scenarios with persistent data -directory, the scripts are not sourced (no-op). - - -##### `postgresql-start/` - -Same sematics as `postgresql-init/`, except that these scripts are -always sourced (after `postgresql-init/` scripts, if they exist). - - ----------------------------------------------- - -During the s2i build all provided files are copied into `/opt/app-root/src` -directory in the new image. Only one -file with the same name can be used for customization and user provided files -are preferred over default files in `/usr/share/container-scripts/`- -so it is possible to overwrite them. - - -Troubleshooting ---------------- -At first the postgres daemon writes its logs to the standard output, so these are available in the container log. The log can be examined by running: - - podman logs - -Then log output is redirected to logging collector process and will appear in directory "pg_log". - - -See also --------- -Dockerfile and other sources for this container image are available on -https://github.com/sclorg/postgresql-container. -In that repository, the Dockerfile for CentOS is called Dockerfile, the Dockerfile -for RHEL7 is called Dockerfile.rhel7, the Dockerfile for RHEL8 is called Dockerfile.rhel8, -and the Dockerfile for Fedora is called Dockerfile.fedora. diff --git a/root/usr/share/container-scripts/postgresql/common.sh b/root/usr/share/container-scripts/postgresql/common.sh deleted file mode 100644 index 4b968f5..0000000 --- a/root/usr/share/container-scripts/postgresql/common.sh +++ /dev/null @@ -1,479 +0,0 @@ -# Configuration settings. -export POSTGRESQL_MAX_CONNECTIONS=${POSTGRESQL_MAX_CONNECTIONS:-100} -export POSTGRESQL_MAX_PREPARED_TRANSACTIONS=${POSTGRESQL_MAX_PREPARED_TRANSACTIONS:-0} - -# Perform auto-tuning based on the container cgroups limits (only when the -# limits are set). -# Users can still override this by setting the POSTGRESQL_SHARED_BUFFERS -# and POSTGRESQL_EFFECTIVE_CACHE_SIZE variables. -if [[ "${NO_MEMORY_LIMIT:-}" == "true" || -z "${MEMORY_LIMIT_IN_BYTES:-}" ]]; then - export POSTGRESQL_SHARED_BUFFERS=${POSTGRESQL_SHARED_BUFFERS:-32MB} - export POSTGRESQL_EFFECTIVE_CACHE_SIZE=${POSTGRESQL_EFFECTIVE_CACHE_SIZE:-128MB} -else - # Use 1/4 of given memory for shared buffers - shared_buffers_computed="$(($MEMORY_LIMIT_IN_BYTES/1024/1024/4))MB" - # Setting effective_cache_size to 1/2 of total memory would be a normal conservative setting, - effective_cache="$(($MEMORY_LIMIT_IN_BYTES/1024/1024/2))MB" - export POSTGRESQL_SHARED_BUFFERS=${POSTGRESQL_SHARED_BUFFERS:-$shared_buffers_computed} - export POSTGRESQL_EFFECTIVE_CACHE_SIZE=${POSTGRESQL_EFFECTIVE_CACHE_SIZE:-$effective_cache} -fi - -export POSTGRESQL_RECOVERY_FILE=$HOME/openshift-custom-recovery.conf -export POSTGRESQL_CONFIG_FILE=$HOME/openshift-custom-postgresql.conf - -postinitdb_actions= - -# match . files when moving userdata below -shopt -s dotglob -# extglob enables the !(userdata) glob pattern below. -shopt -s extglob - -function usage() { - if [ $# == 1 ]; then - echo >&2 "error: $1" - fi - - cat >&2 </dev/null) - # FIXME: This is for debugging (docker run) - if [ -v POSTGRESQL_MASTER_IP ]; then - endpoints=${POSTGRESQL_MASTER_IP:-} - fi - if [ -z "$endpoints" ]; then - >&2 echo "Failed to resolve PostgreSQL master IP address" - exit 3 - fi - echo -n "$(echo $endpoints | cut -d ' ' -f 1)" -} - -# Converts the version in format x.y or x.y.z to a number. -version2number () -{ - local old_IFS=$IFS - local to_print= depth=${2-3} width=${3-2} sum=0 one_part - IFS='.' - set -- $1 - while test $depth -ge 1; do - depth=$(( depth - 1 )) - part=${1-0} ; shift || : - printf "%0${width}d" "$part" - done - IFS=$old_IFS -} - -# On non-intel arches, data_sync_retry = off does not work -# Upstream discussion: https://www.postgresql.org/message-id/CA+mCpegfOUph2U4ZADtQT16dfbkjjYNJL1bSTWErsazaFjQW9A@mail.gmail.com -# Upstream changes that caused this issue: -# https://github.com/postgres/postgres/commit/483520eca426fb1b428e8416d1d014ac5ad80ef4 -# https://github.com/postgres/postgres/commit/9ccdd7f66e3324d2b6d3dec282cfa9ff084083f1 -# RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1779150 -# Special handle of data_sync_retry should handle only in some cases. -# These cases are: non-intel architectures, and version higher or equal 12.0, 10.7, 9.6.12 -# Return value 0 means the hack is needed. -function should_hack_data_sync_retry() { - [ "$(uname -p)" == 'x86_64' ] && return 1 - local version_number=$(version2number "$(pg_ctl -V | sed -e 's/^pg_ctl (PostgreSQL) //')") - # this matches all 12.x and versions of 10.x where we need the hack - [ "$version_number" -ge 100700 ] && return 0 - # this matches all 10.x that were not matched above - [ "$version_number" -ge 100000 ] && return 1 - # this matches all 9.x where need the hack - [ "$version_number" -ge 090612 ] && return 0 - # all rest should be older 9.x releases - return 1 -} - -# New config is generated every time a container is created. It only contains -# additional custom settings and is included from $PGDATA/postgresql.conf. -function generate_postgresql_config() { - envsubst \ - < "${CONTAINER_SCRIPTS_PATH}/openshift-custom-postgresql.conf.template" \ - > "${POSTGRESQL_CONFIG_FILE}" - - if [ "${ENABLE_REPLICATION}" == "true" ]; then - envsubst \ - < "${CONTAINER_SCRIPTS_PATH}/openshift-custom-postgresql-replication.conf.template" \ - >> "${POSTGRESQL_CONFIG_FILE}" - fi - - if should_hack_data_sync_retry ; then - echo "data_sync_retry = on" >>"${POSTGRESQL_CONFIG_FILE}" - fi - - ( - shopt -s nullglob - for conf in "${APP_DATA}"/src/postgresql-cfg/*.conf; do - echo include \'${conf}\' >> "${POSTGRESQL_CONFIG_FILE}" - done - ) -} - -function generate_postgresql_recovery_config() { - envsubst \ - < "${CONTAINER_SCRIPTS_PATH}/openshift-custom-recovery.conf.template" \ - > "${POSTGRESQL_RECOVERY_FILE}" -} - -# Generate passwd file based on current uid -function generate_passwd_file() { - export USER_ID=$(id -u) - export GROUP_ID=$(id -g) - grep -v -e ^postgres -e ^$USER_ID /etc/passwd > "$HOME/passwd" - echo "postgres:x:${USER_ID}:${GROUP_ID}:PostgreSQL Server:${HOME}:/bin/bash" >> "$HOME/passwd" - export LD_PRELOAD=libnss_wrapper.so - export NSS_WRAPPER_PASSWD=${HOME}/passwd - export NSS_WRAPPER_GROUP=/etc/group -} - -initdb_wrapper () -{ - # Initialize the database cluster with utf8 support enabled by default. - # This might affect performance, see: - # http://www.postgresql.org/docs/12/static/locale.html - LANG=${LANG:-en_US.utf8} "$@" -} - -function initialize_database() { - initdb_wrapper initdb - - # PostgreSQL configuration. - cat >> "$PGDATA/postgresql.conf" <> "$PGDATA/pg_hba.conf" <&2 "\n========== \$PGDATA upgrade: %s -> %s ==========\n\n" \ - "$POSTGRESQL_PREV_VERSION" \ - "$POSTGRESQL_VERSION" - - info_msg () { printf >&2 "\n===> $*\n\n" ;} - - # pg_upgrade writes logs to cwd, so go to the persistent storage first - cd "$HOME"/data - - # disable this because of scl_source, 'set +u' just makes the code ugly - # anyways - set +u - - # we need to have the old SCL enabled, otherwise the $old_pgengine is not - # working. The scl_source script doesn't pay attention to non-zero exit - # statuses, so use 'set +e'. - set +e - source scl_source enable $old_collection - set -e - - case $POSTGRESQL_UPGRADE in - copy) # we accept this - ;; - hardlink) - optimized=: - ;; - *) - echo >&2 "Unsupported value: \$POSTGRESQL_UPGRADE=$POSTGRESQL_UPGRADE" - false - ;; - esac - - # Ensure $PGDATA_new doesn't exist yet, so we can immediately remove it if - # there's some problem. - test ! -e "$PGDATA_new" - - # initialize the database - info_msg "Initialize new data directory; we will migrate to that." - initdb_cmd=( initdb_wrapper "$new_pgengine"/initdb "$PGDATA_new" ) - eval "\${initdb_cmd[@]} ${POSTGRESQL_UPGRADE_INITDB_OPTIONS-}" || \ - { rm -rf "$PGDATA_new" ; false ; } - - upgrade_cmd=( - "$new_pgengine"/pg_upgrade - "--old-bindir=$old_pgengine" - "--new-bindir=$new_pgengine" - "--old-datadir=$PGDATA" - "--new-datadir=$PGDATA_new" - ) - - # Dangerous --link option, we loose $DATADIR if something goes wrong. - ! $optimized || upgrade_cmd+=(--link) - - # User-specififed options for pg_upgrade. - eval "upgrade_cmd+=(${POSTGRESQL_UPGRADE_PGUPGRADE_OPTIONS-})" - - # On non-intel arches the data_sync_retry set to on - sed -i -e 's/data_sync_retry/#data_sync_retry/' "${POSTGRESQL_CONFIG_FILE}" - - # the upgrade - info_msg "Starting the pg_upgrade process." - - # Once we stop support for PostgreSQL 9.4, we don't need - # REDHAT_PGUPGRADE_FROM_RHEL hack as we don't upgrade from 9.2 -- that means - # that we don't need to fiddle with unix_socket_director{y,ies} option. - REDHAT_PGUPGRADE_FROM_RHEL=1 \ - "${upgrade_cmd[@]}" || { cat $(find "$PGDATA_new"/.. -name pg_upgrade_server.log) ; rm -rf "$PGDATA_new" && false ; } - - # Move the important configuration and remove old data. This is highly - # careless, but we can't do more for this over-automatized process. - info_msg "Swap the old and new PGDATA and cleanup." - mv "$PGDATA"/*.conf "$PGDATA_new" - rm -rf "$PGDATA" - mv "$PGDATA_new" "$PGDATA" - - # Get back the option we changed above - sed -i -e 's/#data_sync_retry/data_sync_retry/' "${POSTGRESQL_CONFIG_FILE}" - - info_msg "Upgrade DONE." -) - - -# Run right after container startup, when the data volume is already initialized -# (not initialized by this container run) and thus there exists a chance that -# the data was generated by incompatible PostgreSQL major version. -try_pgupgrade () -{ - local versionfile="$PGDATA"/PG_VERSION version upgrade_available - - # This file always exists. - test -f "$versionfile" - version=$(cat "$versionfile") - - # If we don't support pg_upgrade, skip. - test -z "${POSTGRESQL_PREV_VERSION-}" && return 0 - - if test "$POSTGRESQL_VERSION" = "$version"; then - # No need to call pg_upgrade. - - # Mistakenly requests upgrade? If not, just start the DB. - test -z "${POSTGRESQL_UPGRADE-}" && return 0 - - # Make _sure_ we have this safety-belt here, otherwise our users would - # just specify '-e POSTGRESQL_UPGRADE=hardlink' permanently, even for - # re-deployment cases when upgrade is not needed. Setting such - # unfortunate default could mean that pg_upgrade might (after some user - # mistake) migrate (or even destruct, especially with --link) the old data - # directory with limited rollback options, if any. - echo >&2 - echo >&2 "== WARNING!! ==" - echo >&2 "PostgreSQL server version matches the datadir PG_VERSION." - echo >&2 "The \$POSTGRESQL_UPGRADE makes no sense and you probably" - echo >&2 "made some mistake, keeping the variable set you might" - echo >&2 "risk a data loss in future!" - echo >&2 "===============" - echo >&2 - - # Exit here, but allow _really explicit_ foot-shot. - ${POSTGRESQL_UPGRADE_FORCE-false} - return 0 - fi - - # At this point in code we know that PG_VERSION doesn't match the PostgreSQL - # server major version; this might mean that user either (a) mistakenly - # deploys from a bad image, or (b) user wants to perform upgrade. For the - # upgrade we require explicit request -- just to avoid disasters in (a)-cases. - - if test -z "${POSTGRESQL_UPGRADE-}"; then - echo >&2 "Incompatible data directory. This container image provides" - echo >&2 "PostgreSQL '$POSTGRESQL_VERSION', but data directory is of" - echo >&2 "version '$version'." - echo >&2 - echo >&2 "This image supports automatic data directory upgrade from" - echo >&2 "'$POSTGRESQL_PREV_VERSION', please _carefully_ consult image documentation" - echo >&2 "about how to use the '\$POSTGRESQL_UPGRADE' startup option." - # We could wait for postgresql startup failure (there's no risk of data dir - # corruption), but fail rather early. - false - fi - - # We support pg_upgrade process only from previous version of this container - # (upgrade to N to N+1 is possible, so e.g. 9.4 to 9.5). - if test "$POSTGRESQL_PREV_VERSION" != "$version"; then - echo >&2 "With this container image you can only upgrade from data directory" - echo >&2 "of version '$POSTGRESQL_PREV_VERSION', not '$version'." - false - fi - - run_pgupgrade -} - -# get_matched_files PATTERN DIR [DIR ...] -# --------------------------------------- -# Print all basenames for files matching PATTERN in DIRs. -get_matched_files () -{ - local pattern=$1 dir - shift - for dir; do - test -d "$dir" || continue - find -L "$dir" -maxdepth 1 -type f -name "$pattern" -printf "%f\n" - done -} - -# process_extending_files DIR [DIR ...] -# ------------------------------------- -# Source all *.sh files in DIRs in alphabetical order, but if the file exists in -# more then one DIR, source only the first occurrence (first found wins). -process_extending_files() -{ - local filename dir - while read filename ; do - for dir in "$@"; do - local file="$dir/$filename" - if test -f "$file"; then - echo "=> sourcing $file ..." - source "$file" - set -e # ensure that users don't mistakenly change this - break - fi - done - done <<<"$(get_matched_files '*.sh' "$@" | sort -u)" -} diff --git a/root/usr/share/container-scripts/postgresql/openshift-custom-postgresql-replication.conf.template b/root/usr/share/container-scripts/postgresql/openshift-custom-postgresql-replication.conf.template deleted file mode 100644 index ef04eaa..0000000 --- a/root/usr/share/container-scripts/postgresql/openshift-custom-postgresql-replication.conf.template +++ /dev/null @@ -1,7 +0,0 @@ -# required on master for replication -wal_level = hot_standby # minimal, archive, hot_standby, or logical -max_wal_senders = 6 # max number of walsender processes -wal_keep_segments = 400 # in logfile segments, 16MB each; 0 disables - -# required on replicas for replication -hot_standby = on diff --git a/root/usr/share/container-scripts/postgresql/openshift-custom-postgresql.conf.template b/root/usr/share/container-scripts/postgresql/openshift-custom-postgresql.conf.template deleted file mode 100644 index 3c2bc7f..0000000 --- a/root/usr/share/container-scripts/postgresql/openshift-custom-postgresql.conf.template +++ /dev/null @@ -1,21 +0,0 @@ -# -# Custom OpenShift configuration. -# -# NOTE: This file is rewritten every time the container is started! -# Changes to this file will be overwritten. -# - -# Listen on all interfaces. -listen_addresses = '*' - -# Determines the maximum number of concurrent connections to the database server. Default: 100 -max_connections = ${POSTGRESQL_MAX_CONNECTIONS} - -# Allow each connection to use a prepared transaction -max_prepared_transactions = ${POSTGRESQL_MAX_PREPARED_TRANSACTIONS} - -# Sets the amount of memory the database server uses for shared memory buffers. Default: 32MB -shared_buffers = ${POSTGRESQL_SHARED_BUFFERS} - -# Sets the planner's assumption about the effective size of the disk cache that is available to a single query -effective_cache_size = ${POSTGRESQL_EFFECTIVE_CACHE_SIZE} diff --git a/root/usr/share/container-scripts/postgresql/openshift-custom-recovery.conf.template b/root/usr/share/container-scripts/postgresql/openshift-custom-recovery.conf.template deleted file mode 100644 index 58d8e33..0000000 --- a/root/usr/share/container-scripts/postgresql/openshift-custom-recovery.conf.template +++ /dev/null @@ -1,8 +0,0 @@ -# -# Custom OpenShift configuration. -# -# NOTE: This file is rewritten every time the container is started! -# Changes to this file will be overwritten. -# - -primary_conninfo = 'host=${MASTER_FQDN} port=5432 user=${POSTGRESQL_MASTER_USER} password=${POSTGRESQL_MASTER_PASSWORD}' diff --git a/root/usr/share/container-scripts/postgresql/scl_enable b/root/usr/share/container-scripts/postgresql/scl_enable deleted file mode 100644 index 1d967f9..0000000 --- a/root/usr/share/container-scripts/postgresql/scl_enable +++ /dev/null @@ -1,3 +0,0 @@ -# This will make scl collection binaries work out of box. -unset BASH_ENV PROMPT_COMMAND ENV -source scl_source enable $ENABLED_COLLECTIONS diff --git a/root/usr/share/container-scripts/postgresql/start/set_passwords.sh b/root/usr/share/container-scripts/postgresql/start/set_passwords.sh deleted file mode 100644 index 60d70e3..0000000 --- a/root/usr/share/container-scripts/postgresql/start/set_passwords.sh +++ /dev/null @@ -1,23 +0,0 @@ -#!/bin/bash - -_psql () { psql --set ON_ERROR_STOP=1 "$@" ; } - -if [[ ",$postinitdb_actions," = *,simple_db,* ]]; then -_psql --set=username="$POSTGRESQL_USER" \ - --set=password="$POSTGRESQL_PASSWORD" \ -<<< "ALTER USER :\"username\" WITH ENCRYPTED PASSWORD :'password';" -fi - -if [ -v POSTGRESQL_MASTER_USER ]; then -_psql --set=masteruser="$POSTGRESQL_MASTER_USER" \ - --set=masterpass="$POSTGRESQL_MASTER_PASSWORD" \ -<<'EOF' -ALTER USER :"masteruser" WITH REPLICATION; -ALTER USER :"masteruser" WITH ENCRYPTED PASSWORD :'masterpass'; -EOF -fi - -if [ -v POSTGRESQL_ADMIN_PASSWORD ]; then -_psql --set=adminpass="$POSTGRESQL_ADMIN_PASSWORD" \ -<<<"ALTER USER \"postgres\" WITH ENCRYPTED PASSWORD :'adminpass';" -fi diff --git a/s2i/bin/assemble b/s2i/bin/assemble deleted file mode 100755 index 6ed8f7a..0000000 --- a/s2i/bin/assemble +++ /dev/null @@ -1,14 +0,0 @@ -#!/bin/bash - -set -o errexit -set -o nounset -set -o pipefail - -shopt -s dotglob -echo "---> Installing application source ..." - - -mv /tmp/src/* ./ - -# Fix source directory permissions -/usr/libexec/fix-permissions --read-only ./ diff --git a/s2i/bin/run b/s2i/bin/run deleted file mode 120000 index a7f4076..0000000 --- a/s2i/bin/run +++ /dev/null @@ -1 +0,0 @@ -/usr/bin/run-postgresql \ No newline at end of file diff --git a/s2i/bin/usage b/s2i/bin/usage deleted file mode 100755 index 9f41312..0000000 --- a/s2i/bin/usage +++ /dev/null @@ -1 +0,0 @@ -groff -t -man -ETascii /help.1 diff --git a/test/check_imagestreams.py b/test/check_imagestreams.py deleted file mode 120000 index 48e6cad..0000000 --- a/test/check_imagestreams.py +++ /dev/null @@ -1 +0,0 @@ -../common/check_imagestreams.py \ No newline at end of file diff --git a/test/examples b/test/examples deleted file mode 120000 index 785887f..0000000 --- a/test/examples +++ /dev/null @@ -1 +0,0 @@ -../examples/ \ No newline at end of file diff --git a/test/pagila.sh b/test/pagila.sh deleted file mode 100755 index 2068de6..0000000 --- a/test/pagila.sh +++ /dev/null @@ -1,42 +0,0 @@ -#! /bin/sh - -set -e - -die() { echo "$*" >&2 ; exit 1; } - -test -z "$CID" && die "Please specify \$CID variable" -# test -d common || die "Please run me from git root directory" - -pagila_mirror=https://dl.fedoraproject.org/pub/epel/7/x86_64/Packages/p/ -pagila_base="pagila-0.10.1-3.el7.noarch.rpm" -pagila=$pagila_mirror$pagila_base -pagila_file="$PWD/postgresql-container-pagila.sql" -pagila_sha256sum=b968d9498d866bff8f47d9e50edf49feeff108d4164bff2aa167dc3eae802701 - -( - flock --timeout 180 9 - - # Already downloaded? - test ! -f "$pagila_file" || exit 0 - - set -o pipefail - curl -s "$pagila" > "$pagila_base" - for file in ./usr/share/pagila/pagila-schema.sql \ - ./usr/share/pagila/pagila-data.sql \ - ./usr/share/pagila/pagila-insert-data.sql ; \ - do - rpm2cpio "$pagila_base" | cpio --extract --to-stdout "$file" - done >"$pagila_file" -) 9<"$0" - -case $(sha256sum "$pagila_file") in -"$pagila_sha256sum"*) ;; -*) false ;; -esac - -# Deliberately using a separate container, otherwise the docker exec with redirection -# does not work in podman 1.6.x due to https://bugzilla.redhat.com/show_bug.cgi?id=1827324 -# This change can be reverted to the previous variant, once this BZ is fixed. -server_ip=$(docker inspect --format='{{.NetworkSettings.IPAddress}}' "$CID") -admin_pass=$(docker exec "$CID" bash -c 'echo $POSTGRESQL_ADMIN_PASSWORD') -docker run --rm -i "$IMAGE_NAME" bash -c "PGPASSWORD=$admin_pass psql -h $server_ip" <"$pagila_file" &>/dev/null diff --git a/test/pg-test-lib.sh b/test/pg-test-lib.sh deleted file mode 100644 index 70bea01..0000000 --- a/test/pg-test-lib.sh +++ /dev/null @@ -1,147 +0,0 @@ -DEBUG=false - -info () { echo >&2 " * $*" ; } -debug () { ! ${DEBUG} || echo >&2 " ~ $*" ; } -error () { echo >&2 "ERROR: $*" ; false ; } - -get_image_id () -{ - local old_IFS=$IFS - local result - - # split "$1" into "$1 $2 .." on colons - IFS=: - set -- $1 - IFS=$old_IFS - case $2 in - local) - # Default to $IMAGE_NAME if it is set since .image-id might not exist - echo "${IMAGE_NAME-$(cat "$1"/.image-id)}" - ;; - remote) - local version=${1//\./} - case $OS in - rhel7) - ns=rhscl - if test "$version" -eq 92; then - ns=openshift3 - fi - image=registry.redhat.io/$ns/postgresql-${version}-rhel7 - ;; - centos7) - ns=centos - if test "$version" -eq 92; then - ns=openshift - fi - local image=docker.io/$ns/postgresql-${1//\./}-centos7 - ;; - rhel8) - ns=rhel8 - local image=registry.redhat.io/$ns/postgresql-${version} - ;; - esac - docker pull "$image" >/dev/null - echo "$image" - ;; - esac -} - -data_pagila_create () -{ - debug "initializing pagila database" - CID="$CID" ./test/pagila.sh -} - -data_pagila_check () -{ - debug "doing pagila check" - local exp_output='28 -16 -2' - # Deliberately moving heredoc into the container, otherwise it does not work - # in podman 1.6.x due to https://bugzilla.redhat.com/show_bug.cgi?id=1827324 - local output=$(docker exec -i "$CID" bash -c "psql -tA </dev/null </dev/null || :") - case $output in - 1*) return ;; - "") ;; - *) echo "$output" ; false ;; - esac - sleep 1 - counter=$(( counter + 1 )) - done -} - - -# version2number VERSION [DEPTH] [WIDTH] -# -------------------------------------- -version2number () -{ - local old_IFS=$IFS - local to_print= depth=${2-3} width=${3-2} sum=0 one_part - IFS='.' - set -- $1 - while test $depth -ge 1; do - depth=$(( depth - 1 )) - part=${1-0} ; shift || : - printf "%0${width}d" "$part" - done - IFS=$old_IFS -} - -# container_ip CONTAINER_ID -# ------------------------- -container_ip() -{ - docker inspect --format='{{.NetworkSettings.IPAddress}}' "$1" -} - -# vi: set ft=sh diff --git a/test/run b/test/run deleted file mode 120000 index 653222b..0000000 --- a/test/run +++ /dev/null @@ -1 +0,0 @@ -run_test \ No newline at end of file diff --git a/test/run-openshift b/test/run-openshift deleted file mode 120000 index d84575f..0000000 --- a/test/run-openshift +++ /dev/null @@ -1 +0,0 @@ -run-openshift-local-cluster \ No newline at end of file diff --git a/test/run-openshift-local-cluster b/test/run-openshift-local-cluster deleted file mode 100755 index c2d1456..0000000 --- a/test/run-openshift-local-cluster +++ /dev/null @@ -1,456 +0,0 @@ -#!/bin/bash -# -# Test the Postgresql image in OpenShift. -# -# IMAGE_NAME specifies a name of the candidate image used for testing. -# The image has to be available before this script is executed. -# - -THISDIR=$(dirname ${BASH_SOURCE[0]}) -TEMPLATES="$THISDIR/examples" -REMOTE_TEMPLATES="https://raw.githubusercontent.com/openshift/origin/master/examples/db-templates" - -source "$THISDIR"/pg-test-lib.sh -source "$THISDIR"/test-lib-openshift.sh -source "$THISDIR"/test-lib-postgresql.sh - -set -exo nounset - -trap ct_os_cleanup EXIT SIGINT - -ct_os_check_compulsory_vars - -# Populate template variables if not set already -if [ -z "${EPHEMERAL_TEMPLATES:-}" ]; then - EPHEMERAL_TEMPLATES=" -$REMOTE_TEMPLATES/postgresql-ephemeral-template.json -$TEMPLATES/postgresql-ephemeral-template.json" -fi - -if [ -z "${PERSISTENT_TEMPLATES:-}" ]; then - PERSISTENT_TEMPLATES=" -$REMOTE_TEMPLATES/postgresql-persistent-template.json -$TEMPLATES/postgresql-persistent-template.json" -fi - -function assert_cmd_fails() { - if eval "$@" &>/dev/null; then - echo " FAIL" - return 1 - fi - echo " PASS" - return 0 -} - -function insert_postgresql_data() { - local image_name=$1 ; shift - local user=$1 ; shift - local pass=$1 ; shift - local database=$1 ; shift - local pod_ip=$1; shift - - : "Inserting data into the database" - local cmd="PGPASSWORD=$pass psql -c \"CREATE TABLE testing (a integer); INSERT INTO testing VALUES (42);\"" - local cmd_args="-h $pod_ip -U $user -d $database" - docker run --rm "$image_name" bash -c "$cmd $cmd_args" -} - -function check_postgresql_data() { - local image_name=$1 ; shift - local user=$1 ; shift - local pass=$1 ; shift - local database=$1 ; shift - local pod_ip=$1; shift - local timeout=${1:-60} - SECONDS=0 - - : "Checking whether the data can be accessed" - local cmd="PGPASSWORD=$pass psql -c \"select * from testing;\"" - local cmd_args="-h $pod_ip -U $user -d $database" - while true ; do - result=$(docker run --rm "$image_name" bash -c "$cmd -At $cmd_args") - if [ "$result" = "42" ]; then - echo " PASS" - return 0 - fi - echo -n "." - [ $SECONDS -gt $timeout ] && break - sleep 3 - done - echo " FAIL" - return 1 -} - - -function check_postgresql_os_service_connection() { - local util_image_name=$1 ; shift - local service_name=$1 ; shift - local user=$1 ; shift - local pass=$1 ; shift - local database=$1 ; shift - local timeout=${1:-60} ; shift || : - local pod_ip=$(ct_os_get_service_ip ${service_name}) - - : " Service ${service_name} check ..." - - local cmd="PGPASSWORD=${pass} pg_isready -t 15 -h ${pod_ip} -U ${user} -d ${database}" - local expected_value='accepting connections' - local output - local ret - SECONDS=0 - - echo -n "Waiting for ${service_name} service becoming ready ..." - while true ; do - output=$(docker run --rm ${util_image_name} bash -c "${cmd}" || :) - echo "${output}" | grep -qe "${expected_value}" && ret=0 || ret=1 - if [ ${ret} -eq 0 ] ; then - echo " PASS" - return 0 - fi - echo -n "." - [ ${SECONDS} -gt ${timeout} ] && break - sleep 3 - done - echo " FAIL" - return 1 -} - -function test_postgresql_pure_image() { - local image_name=$1 - local image_name_no_namespace=${image_name##*/} - local service_name=${image_name_no_namespace} - - ct_os_new_project - ct_os_upload_image "${image_name}" - # Create a specific imagestream tag for the image so that oc cannot use anything else - ct_os_upload_image "${image_name}" "$image_name_no_namespace:testing" - - ct_os_deploy_pure_image "$image_name_no_namespace:testing" \ - --name "${service_name}" \ - --env POSTGRESQL_ADMIN_PASSWORD=test - - ct_os_wait_pod_ready "${service_name}" 60 - check_postgresql_os_service_connection "${image_name}" "${service_name}" postgres test postgres - - ct_os_delete_project -} - -function test_postgresql_template() { - local image_name=$1; shift - local template=$1 - local image_name_no_namespace=${image_name##*/} - local service_name=${image_name_no_namespace} - - ct_os_new_project - ct_os_upload_image "${image_name}" "postgresql:$VERSION" - - ct_os_deploy_template_image "$template" \ - NAMESPACE="$(oc project -q)" \ - POSTGRESQL_VERSION="$VERSION" \ - DATABASE_SERVICE_NAME="${service_name}" \ - POSTGRESQL_USER=testu \ - POSTGRESQL_PASSWORD=testp \ - POSTGRESQL_DATABASE=testdb - - ct_os_wait_pod_ready "${service_name}" 60 - check_postgresql_os_service_connection "${image_name}" "${service_name}" testu testp testdb - - ct_os_delete_project -} - -function test_postgresql_update() { - local image_name=$1; shift - local template=$1 - local image_name_no_registry=${image_name#*/} - local service_name=${image_name_no_registry#*/} - local user="testu" pass="testp" db="testdb" - local registry="" old_image="" pod_ip="" - local version released=: - - old_image=$(get_image_id "$VERSION:remote") - - for version in $NOT_RELEASED_VERSIONS; do - case $image_name in - *$version*) - released=false - break - ;; - esac - done - - if docker pull "$old_image" 2>/dev/null; then - # Check if we do not have a stale unreleased versions list - # Fail only on rhel, on centos the image is likely already released - $released || [ "$OS" = "centos7" ] - elif $released; then - false "image '$old_image' should already be available" - else - return # not yet released image, skip - fi - - - ct_os_new_project - ct_os_upload_image "$old_image" "postgresql:$VERSION" - ct_os_deploy_template_image "$template" \ - NAMESPACE="$(oc project -q)" \ - POSTGRESQL_VERSION="$VERSION" \ - DATABASE_SERVICE_NAME="$service_name" \ - POSTGRESQL_USER="$user" \ - POSTGRESQL_PASSWORD="$pass" \ - POSTGRESQL_DATABASE="$db" - - ct_os_wait_pod_ready "${service_name}" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" "$user" "$pass" "$db" - - pod_ip=$(ct_os_get_service_ip "$service_name") - insert_postgresql_data "$image_name" "$user" "$pass" "$db" "$pod_ip" - - ct_os_upload_image "$image_name" "postgresql:$VERSION" - : "Waiting for a few seconds while the pods get restarted" - sleep 5 - - ct_os_wait_pod_ready "$service_name" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" "$user" "$pass" "$db" - - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$pod_ip" - ct_os_delete_project -} - -function test_postgresql_replication() { - local image_name=$1 - local image_name_no_namespace=${image_name##*/} - local master_service_name=${image_name_no_namespace}-master - local slave_service_name=${image_name_no_namespace}-slave - local istag="postgresql:$VERSION" - local user="testu" pass="testp" db="testdb" - local master_name="" master_ip="" slave_name="" slave_ip="" - - ct_os_new_project - ct_os_upload_image "${image_name}" "$istag" - - ct_os_deploy_template_image "$TEMPLATES/replica/postgresql_replica.json" \ - NAMESPACE="$(oc project -q)" \ - IMAGESTREAMTAG="$istag" \ - POSTGRESQL_MASTER_SERVICE_NAME="$master_service_name" \ - POSTGRESQL_SLAVE_SERVICE_NAME="$slave_service_name" \ - POSTGRESQL_USER=testu \ - POSTGRESQL_PASSWORD=testp \ - POSTGRESQL_DATABASE=testdb - - ct_os_wait_pod_ready "$master_service_name" 60 - ct_os_wait_pod_ready "$slave_service_name" 60 - - # Force unused rc removal as we do not need rollbacks during testing - oc patch "dc/$master_service_name" -p '{"spec":{"revisionHistoryLimit":0}}' - oc patch "dc/$slave_service_name" -p '{"spec":{"revisionHistoryLimit":0}}' - - master_name=$(ct_os_get_pod_name "$master_service_name") - slave_name=$(ct_os_get_pod_name "$slave_service_name") - master_ip=$(ct_os_get_pod_ip "$master_name") - slave_ip=$(ct_os_get_pod_ip "$slave_name") - insert_postgresql_data "$image_name" "$user" "$pass" "$db" "$master_ip" - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$slave_ip" - - : "Changing POSTGRESQL_PASSWORD for master and slave" - pass=redhat - oc set env "dc/$master_service_name" -e POSTGRESQL_PASSWORD="$pass" - oc set env "dc/$slave_service_name" -e POSTGRESQL_PASSWORD="$pass" - ct_os_wait_pod_ready "$master_service_name-2" 60 - ct_os_wait_pod_ready "$slave_service_name-2" 60 - # We need to get new pod names and IPs - master_name=$(ct_os_get_pod_name "$master_service_name") - slave_name=$(ct_os_get_pod_name "$slave_service_name") - master_ip=$(ct_os_get_pod_ip "$master_name") - slave_ip=$(ct_os_get_pod_ip "$slave_name") - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$master_ip" - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$slave_ip" - - : "Redeploying slave node" - oc rollout latest "$slave_service_name" - ct_os_wait_pod_ready "$slave_service_name-3" 60 - slave_name=$(ct_os_get_pod_name "$slave_service_name") - slave_ip=$(ct_os_get_pod_ip "$slave_name") - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$slave_ip" - - : "Scaling slaves to 2" - oc scale --replicas 2 "dc/$slave_service_name" - ct_os_wait_rc_ready "$slave_service_name" 60 - slave_name=$(ct_os_get_pod_name "$slave_service_name") - for slave in $slave_name; do - ct_os_wait_pod_ready "$slave" 60 - slave_ip=$(ct_os_get_pod_ip "$slave") - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$slave_ip" - done - - ct_os_delete_project -} - -function test_postgresql_persistent_redeploy() { - local image_name=$1; shift - local template=$1 - local image_name_no_namespace=${image_name##*/} - local service_name=$image_name_no_namespace - local user="testu" pass="testp" db="testdb" - local registry="" old_image="" pod_ip="" - - ct_os_new_project - ct_os_upload_image "$image_name" "postgresql:$VERSION" - - ct_os_deploy_template_image "$template" \ - NAMESPACE="$(oc project -q)" \ - POSTGRESQL_VERSION="$VERSION" \ - DATABASE_SERVICE_NAME="$service_name" \ - POSTGRESQL_USER="$user" \ - POSTGRESQL_PASSWORD="$pass" \ - POSTGRESQL_DATABASE="$db" - - ct_os_wait_pod_ready "$service_name" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" testu testp testdb - - pod_ip=$(ct_os_get_service_ip "$service_name") - insert_postgresql_data "$image_name" "$user" "$pass" "$db" "$pod_ip" - - : "Redeploying pod" - oc rollout latest "$service_name" - : "Waiting for a few seconds while the pod gets restarted" - sleep 5 - - ct_os_wait_pod_ready "$service_name" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" "$user" "$pass" "$db" - - : "This should succeed" - check_postgresql_data "$image_name" "$user" "$pass" "$db" "$pod_ip" 0 - ct_os_delete_project -} - -function test_postgresql_ephemeral_redeploy() { - local image_name=$1; shift - local template=$1 - local image_name_no_namespace=${image_name##*/} - local service_name=$image_name_no_namespace - local user="testu" pass="testp" db="testdb" - local registry="" old_image="" pod_ip="" - - ct_os_new_project - ct_os_upload_image "$image_name" "postgresql:$VERSION" - - ct_os_deploy_template_image "$template" \ - NAMESPACE="$(oc project -q)" \ - POSTGRESQL_VERSION="$VERSION" \ - DATABASE_SERVICE_NAME="$service_name" \ - POSTGRESQL_USER="$user" \ - POSTGRESQL_PASSWORD="$pass" \ - POSTGRESQL_DATABASE="$db" - - ct_os_wait_pod_ready "$service_name" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" testu testp testdb - - pod_ip=$(ct_os_get_service_ip "$service_name") - insert_postgresql_data "$image_name" "$user" "$pass" "$db" "$pod_ip" - - : "Redeploying pod" - oc rollout latest "$service_name" - : "Waiting for a few seconds while the pod gets restarted" - sleep 5 - - ct_os_wait_pod_ready "$service_name" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" "$user" "$pass" "$db" - - : "This should fail" - assert_cmd_fails check_postgresql_data "$image_name" "$user" "$pass" "$db" "$pod_ip" 0 - ct_os_delete_project -} - -function test_postgresql_configmap_start() { - local image_name=$1; shift - local template="$TEMPLATES/postgresql-ephemeral-template.json" - local image_name_no_namespace=${image_name##*/} - local service_name=$image_name_no_namespace - local user="testu" pass="testp" db="testdb" - local registry="" old_image="" pod_ip="" tmpdir="" - local test_string="" - - ct_os_new_project - ct_os_upload_image "$image_name" "postgresql:$VERSION" - - ct_os_deploy_template_image "$template" \ - NAMESPACE="$(oc project -q)" \ - POSTGRESQL_VERSION="$VERSION" \ - DATABASE_SERVICE_NAME="$service_name" \ - POSTGRESQL_USER="$user" \ - POSTGRESQL_PASSWORD="$pass" \ - POSTGRESQL_DATABASE="$db" - - ct_os_wait_pod_ready "$service_name" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" testu testp testdb - - # Create a simple configMap with a start.sh script - tmpdir=$(mktemp -d) - test_string="Start is working" - echo "echo $test_string" >> "$tmpdir/start.sh" - oc create configmap postgresql-start --from-file="$tmpdir/start.sh" - oc set volume "dc/$service_name" --add -t configmap --configmap-name=postgresql-start -m /opt/app-root/src/postgresql-start --name postgresql-start - - # Wait for redeploy - ct_os_wait_pod_ready "$service_name-2" 60 - check_postgresql_os_service_connection "$image_name" "$service_name" testu testp testdb - - # Check logs for the test string - oc logs "$(ct_os_get_pod_name "$service_name")" | grep "$test_string" - - ct_os_delete_project -} - -function run_ephemeral_tests() { - local image_name=$1 - for template in $EPHEMERAL_TEMPLATES; do - test_postgresql_ephemeral_redeploy "$image_name" "$template" - test_postgresql_template "$image_name" "$template" - done -} - -function run_persistent_tests() { - local image_name=$1 - for template in $PERSISTENT_TEMPLATES; do - test_postgresql_persistent_redeploy "$image_name" "$template" - test_postgresql_update "$image_name" "$template" - done -} - -ct_os_cluster_up -# Print oc logs on failure -ct_os_enable_print_logs - -test_postgresql_pure_image "$IMAGE_NAME" -test_postgresql_replication "$IMAGE_NAME" -run_ephemeral_tests "$IMAGE_NAME" -run_persistent_tests "$IMAGE_NAME" -test_postgresql_configmap_start "$IMAGE_NAME" - -# test with the just built image and an integrated template -echo "Running test_postgresql_integration with ${IMAGE_NAME}" -test_postgresql_integration "${IMAGE_NAME}" - -# test with a released image and an integrated template -PUBLIC_IMAGE_NAME=${PUBLIC_IMAGE_NAME:-$(ct_get_public_image_name "${OS}" "${BASE_IMAGE_NAME}" "${VERSION}")} - -# Try pulling the image first to see if it is accessible -if docker pull "${PUBLIC_IMAGE_NAME}"; then - echo "Running test_postgresql_integration with ${PUBLIC_IMAGE_NAME}" - test_postgresql_integration "${PUBLIC_IMAGE_NAME}" -else - echo "Warning: ${PUBLIC_IMAGE_NAME} could not be downloaded via 'docker'" - # ignore possible failure of this test for centos images - [ "${OS}" == "rhel7" ] && false "ERROR: Failed to pull image" -fi - -# Check the imagestream -echo "Running test_mariadb_imagestream" -test_postgresql_imagestream - -OS_TESTSUITE_RESULT=0 - -ct_os_cluster_down - -# vim: set tabstop=2:shiftwidth=2:expandtab: - diff --git a/test/run-openshift-remote-cluster b/test/run-openshift-remote-cluster deleted file mode 100755 index 55d4980..0000000 --- a/test/run-openshift-remote-cluster +++ /dev/null @@ -1,36 +0,0 @@ -#!/bin/bash -# -# Test the PostgreSQL image in OpenShift (remote cluster) -# -# IMAGE_NAME specifies a name of the candidate image used for testing. -# The image has to be available before this script is executed. -# VERSION specifies the major version of the PostgreSQL in format of X.Y -# OS specifies RHEL version (e.g. OS=rhel7) -# - -THISDIR=$(dirname ${BASH_SOURCE[0]}) - -source ${THISDIR}/test-lib-postgresql.sh - -set -eo nounset - -trap ct_os_cleanup EXIT SIGINT - -ct_os_check_compulsory_vars - -oc status || false "It looks like oc is not properly logged in." - -export CT_SKIP_NEW_PROJECT=true -export CT_SKIP_UPLOAD_IMAGE=true -export CT_NAMESPACE=openshift - -# Check the template -test_postgresql_integration "${IMAGE_NAME}" - -# Check the imagestream -test_postgresql_imagestream - -OS_TESTSUITE_RESULT=0 - -# vim: set tabstop=2:shiftwidth=2:expandtab: - diff --git a/test/run_migration_test b/test/run_migration_test deleted file mode 100755 index 654e56e..0000000 --- a/test/run_migration_test +++ /dev/null @@ -1,56 +0,0 @@ -#! /bin/bash - -set -e -. test/pg-test-lib.sh -ADMIN_PASSWORD=redhat - -test $# -eq 2 || error "two args expected: $0 FROM TO" - -cleanup() -{ - set +e - set -- $container_from $container_to - if test $# -gt 0; then - docker stop "$@" >/dev/null - docker rm -f "$@" - fi -} -trap cleanup EXIT - -from=$1 -to=$2 -image_from=$(get_image_id "$from") -image_to=$(get_image_id "$to") - -assert_migration_succeeds () -{ - info "starting PostgreSQL server v$from" - container_from=$(docker run -e POSTGRESQL_ADMIN_PASSWORD="$ADMIN_PASSWORD" -d "$image_from") - wait_for_postgres "$container_from" - - eval "CID=\$container_from data_${1}_create" - eval "CID=\$container_from data_${1}_check" - - ip=$(container_ip "$container_from") - - info "starting new PostgreSQL server v$to with migration options" - container_to=$(docker run \ - -e POSTGRESQL_MIGRATION_REMOTE_HOST="$ip" \ - -e POSTGRESQL_MIGRATION_ADMIN_PASSWORD="$ADMIN_PASSWORD" \ - -d "$image_to") - - # Prolong a waiting time here a bit since both dump and restore is done in - # uncertain environment (usually both is done on the same hardware). - wait_for_postgres "$container_to" 100 - - info "check that the migration passed" - eval "CID=\$container_to data_${1}_check" - - docker stop "$container_from" - docker rm -f "$container_from" - docker stop "$container_to" - docker rm -f "$container_to" - container_from= container_to= -} - -assert_migration_succeeds pagila diff --git a/test/run_test b/test/run_test deleted file mode 100755 index a47875e..0000000 --- a/test/run_test +++ /dev/null @@ -1,929 +0,0 @@ -#!/bin/bash -# -# Test the PostgreSQL image. -# -# IMAGE_NAME specifies the name of the candidate image used for testing. -# The image has to be available before this script is executed. -# - -set -exo nounset -shopt -s nullglob - -# library from container-common-scripts -. test/test-lib.sh - -# local library -. test/pg-test-lib.sh - -TEST_LIST="\ -run_container_creation_tests -run_general_tests -run_change_password_test -run_replication_test -run_master_restart_test -run_doc_test -run_s2i_test -run_test_cfg_hook -run_s2i_bake_data_test -run_s2i_enable_ssl_test -run_upgrade_test -run_migration_test -run_pgaudit_test -run_latest_imagestreams_test -" - -test $# -eq 1 -a "${1-}" == --list && echo "$TEST_LIST" && exit 0 -test -n "${IMAGE_NAME-}" || false 'make sure $IMAGE_NAME is defined' -test -n "${VERSION-}" || false 'make sure $VERSION is defined' -test -n "${OS-}" || false 'make sure $OS is defined' - -CIDFILE_DIR=$(mktemp --suffix=postgresql_test_cidfiles -d) - -volumes_to_clean= -images_to_clean=() -files_to_clean= -test_dir="$(readlink -f "$(dirname "$0")")" -test_short_summary='' -TESTSUITE_RESULT=1 - -_cleanup_commands_space= -_cleanup_commands= - -add_cleanup_command () -{ - local cmd= space= - for arg; do - cmd+="$space$(printf "%q" "$arg")" - space=' ' - done - _cleanup_commands+="$_cleanup_commands_space$cmd" - _cleanup_commands_space=' -' -} -function cleanup() { - # Print a big fat separator to find the error easier - echo "=================================== Cleanup begins here =============================" - - for cidfile in $CIDFILE_DIR/* ; do - CONTAINER=$(cat $cidfile) - - echo "Stopping and removing container $CONTAINER..." - docker stop $CONTAINER - exit_status=$(docker inspect -f '{{.State.ExitCode}}' $CONTAINER) - if [ "$exit_status" != "0" ]; then - echo "Dumping logs for $CONTAINER" - docker logs $CONTAINER - fi - docker rm $CONTAINER - rm $cidfile - echo "Done." - done - rmdir $CIDFILE_DIR - - ct_path_foreach "$volumes_to_clean" cleanup_volume_dir - - if test -n "${images_to_clean-}"; then - # Workaround for RHEL 7 bash bug: - # https://bugzilla.redhat.com/show_bug.cgi?id=1636393 - for image in "${images_to_clean[@]}"; do - docker rmi -f "$image" - done - fi - - ct_path_foreach "$files_to_clean" rm - - echo "$_cleanup_commands" | while read -r line; do - eval "$line" - done - - echo "$test_short_summary" - - if [ $TESTSUITE_RESULT -eq 0 ] ; then - echo "Tests for ${IMAGE_NAME} succeeded." - else - echo "Tests for ${IMAGE_NAME} failed." - fi -} -trap cleanup EXIT - -cleanup_volume_dir () -{ - test ! -d "$1" && : "WARN: cleaned $1 for some reason" && return 0 - # When we run this test script as non-root (we should?), the PostgreSQL server - # within container is still run under 'postgres' user. It means that, taking - # into account 0077 umask of PostgreSQL server, we are unable to remove files - # created by server. That's why we need to let docker escalate the privileges - # again. - local datadir=/var/lib/pgsql/data - docker run -v "$1:$datadir:Z" --rm "$IMAGE_NAME" /bin/sh -c "/bin/rm -rf $datadir/userdata" - rmdir "$1" -} - -function get_cid() { - local id="$1" ; shift || return 1 - echo $(cat "$CIDFILE_DIR/$id") -} - -function get_container_ip() { - local id="$1" ; shift - docker inspect --format='{{.NetworkSettings.IPAddress}}' $(get_cid "$id") -} - -function get_ip_from_cid() { - local cid="$1"; shift - docker inspect --format='{{.NetworkSettings.IPAddress}}' $cid -} - -function postgresql_cmd() { - docker run --rm -e PGPASSWORD="$PASS" "$IMAGE_NAME" psql "postgresql://$PGUSER@$CONTAINER_IP:5432/${DB-db}" "$@" -} - -function test_connection() { - local name=$1 ; shift - ip=$(get_container_ip $name) - echo " Testing PostgreSQL connection to $ip..." - local max_attempts=20 - local sleep_time=2 - for i in $(seq $max_attempts); do - echo " Trying to connect..." - set +e - # Don't let the code come here if neither user nor admin is able to - # connect. - if [ -v PGUSER ] && [ -v PASS ]; then - CONTAINER_IP=$ip postgresql_cmd <<< "SELECT 1;" - else - PGUSER=postgres PASS=$ADMIN_PASS CONTAINER_IP=$ip DB=postgres postgresql_cmd <<< "SELECT 1;" - fi - status=$? - set -e - if [ $status -eq 0 ]; then - echo " Success!" - return 0 - fi - sleep $sleep_time - done - return 1 -} - -function test_postgresql() { - echo " Testing PostgreSQL" - postgresql_cmd <<< "CREATE EXTENSION 'uuid-ossp';" # to test contrib package - postgresql_cmd <<< "CREATE TABLE tbl (col1 VARCHAR(20), col2 VARCHAR(20));" - postgresql_cmd <<< "INSERT INTO tbl VALUES ('foo1', 'bar1');" - postgresql_cmd <<< "INSERT INTO tbl VALUES ('foo2', 'bar2');" - postgresql_cmd <<< "INSERT INTO tbl VALUES ('foo3', 'bar3');" - postgresql_cmd <<< "SELECT * FROM tbl;" - #postgresql_cmd <<< "DROP TABLE tbl;" - echo " Success!" -} - -function create_container() { - local name=$1 ; shift - local cargs=${DOCKER_ARGS:-} - # TODO: fix all create_container() invocations so that we don't need this, - # e.g. multiline DOCKER_ARGS var should end by trailing backslashes - cargs=$(echo "$cargs" | tr '\n' ' ') - cidfile="$CIDFILE_DIR/$name" - # create container with a cidfile in a directory for cleanup - eval "docker run $cargs --cidfile \$cidfile -d \$IMAGE_NAME \"\$@\"" - echo "Created container $(cat $cidfile)" -} - - -create_volume_dir () -{ - volume_dir=`mktemp -d --tmpdir pg-testdata.XXXXX` - setfacl -m u:26:-wx "$volume_dir" - ct_path_append volumes_to_clean "$volume_dir" -} - - -create_temp_file () -{ - temp_file=`mktemp --tmpdir pg-testfile.XXXXX` - setfacl -m u:26:rw- "$temp_file" - ct_path_append files_to_clean "$temp_file" -} - - -function assert_login_access() { - local PGUSER=$1 ; shift - local PASS=$1 ; shift - local success=$1 ; shift - - echo "testing login as $PGUSER:$PASS; should_success=$success" - - if postgresql_cmd <<<'SELECT 1;' ; then - if $success ; then - echo " $PGUSER($PASS) access granted as expected" - return - fi - else - if ! $success ; then - echo " $PGUSER($PASS) access denied as expected" - return - fi - fi - echo " $PGUSER($PASS) login assertion failed" - exit 1 -} - -function assert_local_access() { - local id="$1" ; shift - docker exec -i $(get_cid "$id") bash -c psql <<< "SELECT 1;" -} - - -# Make sure the invocation of docker run fails. -function assert_container_creation_fails() { - - # Time the docker run command. It should fail. If it doesn't fail, - # postgresql will keep running so we kill it with SIGKILL to make sure - # timeout returns a non-zero value. - set +e - timeout -s 9 --preserve-status 60s docker run --rm "$@" $IMAGE_NAME - ret=$? - set -e - - # Timeout will exit with a high number. - if [ $ret -gt 30 ]; then - return 1 - fi -} - - -# assert_container_creation_succeeds NAME [ARGS] -# ---------------------------------------------- -# Chcek that 'docker run' with IMAGE_NAME succeeds with docker arguments -# specified as ARGS. -assert_container_creation_succeeds () -{ - local check_env=false - local name=pg-success-"$(ct_random_string)" - local PGUSER='' PGPASS='' DB='' ADMIN_PASS= - local docker_args= - - for arg; do - docker_args+=" $(printf "%q" "$arg")" - if $check_env; then - local env=${arg//=*/} - local val=${arg//$env=/} - case $env in - POSTGRESQL_ADMIN_PASSWORD) ADMIN_PASS=$val ;; - POSTGRESQL_USER) PGUSER=$val ;; - POSTGRESQL_PASSWORD) PGPASS=$val ;; - POSTGRESQL_DATABASE) DB=$val ;; - esac - check_env=false - elif test "$arg" = -e; then - check_env=: - fi - done - - DOCKER_ARGS=$docker_args create_container "$name" - - if test -n "$PGUSER" && test -n "$PGPASS"; then - PGUSER=$PGUSER PASS=$PGPASS DB=$DB test_connection "$name" - fi - - if test -n "$ADMIN_PASS"; then - PGUSER=postgres PASS=$ADMIN_PASS DB=$DB test_connection "$name" - fi - - docker stop "$(get_cid "$name")" -} - - -function try_image_invalid_combinations() { - assert_container_creation_fails -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass "$@" - assert_container_creation_fails -e POSTGRESQL_USER=user -e POSTGRESQL_DATABASE=db "$@" - assert_container_creation_fails -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db "$@" -} - -function run_container_creation_tests() { - echo " Testing image entrypoint usage" - try_image_invalid_combinations - try_image_invalid_combinations -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - - VERY_LONG_IDENTIFIER="very_long_identifier_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx" - assert_container_creation_fails -e POSTGRESQL_USER= -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_fails -e POSTGRESQL_USER=$VERY_LONG_IDENTIFIER -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_succeeds -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD="\"" -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_succeeds -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=9invalid -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_fails -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=$VERY_LONG_IDENTIFIER -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_succeeds -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD="\"" - echo " Success!" - - assert_container_creation_succeeds -e POSTGRESQL_ADMIN_PASSWORD="the @password" - assert_container_creation_succeeds -e POSTGRESQL_PASSWORD="the pass" -e POSTGRESQL_USER="the user" -e POSTGRESQL_DATABASE="the db" -} - -function test_config_option() { - local name=$1 ; shift - local setting=$1 ; shift - local value=$1 ; shift - - docker exec $(get_cid ${name}) grep -q "${setting} = ${value}" /var/lib/pgsql/openshift-custom-postgresql.conf -} - - -# wait_ready -# ---------- -# Wait until the PG container becomes ready -wait_ready () -{ - while ! docker exec "$(get_cid "$1")" /usr/libexec/check-container ; do - sleep 1 - done -} - - -# assert_runtime_option option value -# ---------------------------------- -assert_runtime_option () -{ - local name=$1 option=$2 value=$3 - wait_ready "$name" - set -- $(docker exec "$(get_cid "$name")" bash -c "psql -tA -c 'SHOW $option;'") - test "$value" = "$1" -} - - -function run_configuration_tests() { - local name=$1 ; shift - echo " Testing image configuration settings" - test_config_option ${name} max_connections ${POSTGRESQL_MAX_CONNECTIONS} - test_config_option ${name} max_prepared_transactions ${POSTGRESQL_MAX_PREPARED_TRANSACTIONS} - test_config_option ${name} shared_buffers ${POSTGRESQL_SHARED_BUFFERS} - echo " Success!" -} - -test_scl_usage() { - local name="$1" - local run_cmd="$2" - local expected="$3" - - echo " Testing the image SCL enable" - out=$(docker run --rm ${IMAGE_NAME} /bin/bash -c "${run_cmd}") - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[/bin/bash -c "${run_cmd}"] Expected '${expected}', got '${out}'" - return 1 - fi - out=$(docker exec $(get_cid $name) /bin/bash -c "${run_cmd}" 2>&1) - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[exec /bin/bash -c "${run_cmd}"] Expected '${expected}', got '${out}'" - return 1 - fi - out=$(docker exec $(get_cid $name) /bin/sh -ic "${run_cmd}" 2>&1) - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[exec /bin/sh -ic "${run_cmd}"] Expected '${expected}', got '${out}'" - return 1 - fi -} - - -function run_tests() { - echo " Testing general usage (run_tests) with '$1' as argument" - local name=$1 ; shift - - user_login=false - admin_login=false - envs= - # NOTE: We work wrongly with variables so please don't try to pass spaces - # within PGUSER/PASS/ADMIN_PASS variables. - [ -v PGUSER ] && envs+=" -e POSTGRESQL_USER=$PGUSER" - [ -v PASS ] && envs+=" -e POSTGRESQL_PASSWORD=$PASS" - if [ -v PGUSER ] && [ -v PASS ]; then - envs+=" -e POSTGRESQL_DATABASE=db" - user_login=: - fi - - if [ -v ADMIN_PASS ]; then - envs="$envs -e POSTGRESQL_ADMIN_PASSWORD=$ADMIN_PASS" - admin_login=: - fi - if [ -v POSTGRESQL_MAX_CONNECTIONS ]; then - envs="$envs -e POSTGRESQL_MAX_CONNECTIONS=$POSTGRESQL_MAX_CONNECTIONS" - fi - if [ -v POSTGRESQL_MAX_PREPARED_TRANSACTIONS ]; then - envs="$envs -e POSTGRESQL_MAX_PREPARED_TRANSACTIONS=$POSTGRESQL_MAX_PREPARED_TRANSACTIONS" - fi - if [ -v POSTGRESQL_SHARED_BUFFERS ]; then - envs="$envs -e POSTGRESQL_SHARED_BUFFERS=$POSTGRESQL_SHARED_BUFFERS" - fi - DOCKER_ARGS="${DOCKER_ARGS:-} $envs" create_container $name - CONTAINER_IP=$(get_container_ip $name) - test_connection $name - echo " Testing scl usage" - test_scl_usage $name 'psql --version' "$VERSION" - - echo " Testing login accesses" - assert_login_access "${PGUSER:-}" "${PASS-}" "$user_login" - assert_login_access "${PGUSER:-}" "${PASS-}_foo" false - - assert_login_access postgres "${ADMIN_PASS-}" "$admin_login" - assert_login_access postgres "${ADMIN_PASS-}_foo" false - - assert_local_access $name - run_configuration_tests $name - echo " Success!" - - if $user_login; then - test_postgresql $name - fi - - if $admin_login; then - DB=postgres PGUSER=postgres PASS=$ADMIN_PASS test_postgresql $name - fi -} - -function run_slave() { - local suffix="$1"; shift - docker run $cluster_args -e POSTGRESQL_MASTER_IP=${master_hostname} \ - -d --cidfile ${CIDFILE_DIR}/slave-${suffix}.cid $IMAGE_NAME run-postgresql-slave -} - -function run_master() { - local suffix="$1"; shift - master_args=${master_args-} - docker run $cluster_args $master_args \ - -d --cidfile ${CIDFILE_DIR}/master-${suffix}.cid $IMAGE_NAME run-postgresql-master >/dev/null -} - -function test_slave_visibility() { - local max_attempts=30 - - for slave in $slave_cids; do - slave_ip=$(get_ip_from_cid $slave) - if [ -z "$slave_ip" ]; then - echo "Failed to get IP for slave $slave." - echo "Dumping logs for $slave" - docker logs "$slave" - return 1 - fi - for i in $(seq $max_attempts); do - result="$(postgresql_cmd -c "select client_addr from pg_stat_replication;" | grep "$slave_ip" || true)" - if [[ -n "${result}" ]]; then - echo "${slave_ip} successfully registered as SLAVE for ${master_ip}" - break - fi - if [[ "${i}" == "${max_attempts}" ]]; then - echo "The ${slave_ip} failed to register in MASTER" - echo "Dumping logs for $slave" - docker logs $slave - return 1 - fi - sleep 1 - done - done -} - -function test_value_replication() { - local max_attempts=30 - - # Setup the replication data - local value - value=24 - postgresql_cmd -c "CREATE TABLE $table_name (a integer); INSERT INTO $table_name VALUES ($value);" - - # Read value from slaves and check whether it is expected - for slave in $slave_cids; do - slave_ip=$(get_ip_from_cid $slave) - CONTAINER_IP=$slave_ip - for i in $(seq $max_attempts); do - result="$(postgresql_cmd -At -c "select * from $table_name" || :)" - if [[ "$result" == "$value" ]]; then - echo "${slave_ip} successfully got value from MASTER ${master_ip}" - break - fi - if [[ "${i}" == "${max_attempts}" ]]; then - echo "The ${slave_ip} failed to see value added on MASTER" - echo "Dumping logs for $slave" - docker logs $slave - return 1 - fi - sleep 1 - done - done -} - -function setup_replication_cluster() { - # Run the PostgreSQL master - run_master "$cid_suffix" - - # Run the PostgreSQL slaves - local i - master_ip=$(get_container_ip "master-$cid_suffix.cid") - local cluster_args="$cluster_args --add-host postgresql-master:$master_ip" - local master_hostname="postgresql-master" - for i in $(seq ${slave_num:-1}); do - slave_cids="$slave_cids $(run_slave $cid_suffix-$i)" - done - -} - -function run_master_restart_test() { - local DB=postgres - local PGUSER=master - local PASS=master - - echo "Testing failed master restart" - local cluster_args="-e POSTGRESQL_ADMIN_PASSWORD=pass -e POSTGRESQL_MASTER_USER=$PGUSER -e POSTGRESQL_MASTER_PASSWORD=$PASS" - local cid_suffix="mrestart" - local table_name="t1" - local master_ip= - local slave_cids= - - create_volume_dir - local master_args="-v ${volume_dir}:/var/lib/pgsql/data:Z" - - # Setup the cluster - slave_num=2 setup_replication_cluster - - # Check if the master knows about the slaves - CONTAINER_IP=$master_ip - test_slave_visibility - - echo "Kill the master and create a new one" - local cidfile=$CIDFILE_DIR/master-$cid_suffix.cid - docker kill $(cat $cidfile) - # Don't forget to remove its .cid file - rm $cidfile - - run_master $cid_suffix - CONTAINER_IP=$(get_container_ip master-$cid_suffix.cid) - - # Update master_ip in slaves - for slave in $slave_cids; do - docker exec -u 0 $slave bash -c "sed \"s/$master_ip/$CONTAINER_IP/g\" /etc/hosts >/tmp/hosts && cp /tmp/hosts /etc/hosts" - done - master_ip=$CONTAINER_IP - # Check if the new master sees existing slaves - test_slave_visibility - - # Check if the replication works - table_name="t1" test_value_replication -} - -function run_replication_test() { - local DB=postgres - local PGUSER=master - local PASS=master - - echo "Testing master-slave replication" - local cluster_args="-e POSTGRESQL_ADMIN_PASSWORD=pass -e POSTGRESQL_MASTER_USER=$PGUSER -e POSTGRESQL_MASTER_PASSWORD=$PASS" - local cid_suffix="basic" - local master_ip= - local slave_cids= - - # Setup the cluster - setup_replication_cluster - - # Check if the master knows about the slaves - CONTAINER_IP=$master_ip - test_slave_visibility - - # Do some real work to test replication in practice - table_name="t1" test_value_replication -} - -function run_change_password_test() { - echo " Testing password change" - local name="change_password" - - local database='db' - local user='user' - local password='password' - local admin_password='adminPassword' - - create_volume_dir - local volume_options="-v ${volume_dir}:/var/lib/pgsql/data:Z" - - DOCKER_ARGS=" --e POSTGRESQL_DATABASE=${database} --e POSTGRESQL_USER=${user} --e POSTGRESQL_PASSWORD=${password} --e POSTGRESQL_ADMIN_PASSWORD=${admin_password} -$volume_options -" create_container ${name} - - # need to set these because `postgresql_cmd` relies on global variables - PGUSER=${user} - PASS=${password} - - # need this to wait for the container to start up - CONTAINER_IP=$(get_container_ip ${name}) - test_connection ${name} - - echo " Testing login" - - assert_login_access ${user} ${password} true - assert_login_access 'postgres' ${admin_password} true - - echo " Changing passwords" - - docker stop $(get_cid ${name}) - DOCKER_ARGS=" --e POSTGRESQL_DATABASE=${database} --e POSTGRESQL_USER=${user} --e POSTGRESQL_PASSWORD=NEW_${password} --e POSTGRESQL_ADMIN_PASSWORD=NEW_${admin_password} -$volume_options -" create_container "${name}_NEW" - - # need to set this because `postgresql_cmd` relies on global variables - PASS="NEW_${password}" - - # need this to wait for the container to start up - CONTAINER_IP=$(get_container_ip "${name}_NEW") - test_connection "${name}_NEW" - - echo " Testing login with new passwords" - - assert_login_access ${user} "NEW_${password}" true - assert_login_access ${user} ${password} false - - assert_login_access 'postgres' "NEW_${admin_password}" true - assert_login_access 'postgres' ${admin_password} false - - echo " Success!" -} - -run_upgrade_test () -{ - # Do not run on Fedora or RHEL8 until the upgrade script - # is fixed for non-SCL use cases - { [ "${OS}" == "fedora" ] || [ "${OS}" == "rhel8" ]; } && return 0 - - local upgrade_path="none 9.2 9.4 9.5 9.6 10 12 none" prev= act= - for act in $upgrade_path; do - if test "$act" = $VERSION; then - break - fi - prev=$act - done - test "$prev" != none - # Check if the previous image is available in the registry - docker pull "$(get_image_id "$prev:remote")" || return 0 - - # TODO: We run this script from $VERSION directory, through test/run symlink. - test/run_upgrade_test "$prev:remote" "$VERSION:local" -} - -run_migration_test () -{ - [ "${OS}" == "fedora" ] && return 0 - - local from_version - # Only test a subset of the migration path on non-intel hosts - if [ "$(uname -i)" == "x86_64" ]; then - local upgrade_path="9.2 9.4 9.5 9.6 10 12" - else - local upgrade_path="10 12" - fi - - for from_version in $upgrade_path; do - # Do not test migration from $VERSION:remote to $VERSION:local - test $(version2number $from_version) -lt $(version2number "$VERSION") \ - || break - # Skip if the previous image is not available in the registry - docker pull "$(get_image_id "$from_version:remote")" || continue - test/run_migration_test $from_version:remote $VERSION:local - done -} - -run_doc_test() { - local tmpdir=$(mktemp -d) - local f - echo " Testing documentation in the container image" - # Extract the help files from the container - for f in help.1 ; do - docker run --rm ${IMAGE_NAME} /bin/bash -c "cat /${f}" >${tmpdir}/$(basename ${f}) - # Check whether the files include some important information - for term in "POSTGRESQL\_ADMIN\_PASSWORD" volume 5432 ; do - if ! cat ${tmpdir}/$(basename ${f}) | grep -F -q -e "${term}" ; then - echo "ERROR: File /${f} does not include '${term}'." - return 1 - fi - done - done - # Check whether the files use the correct format - if ! file ${tmpdir}/help.1 | grep -q roff ; then - echo "ERROR: /help.1 is not in troff or groff format" - return 1 - fi - echo " Success!" - echo -} - -test_the_app_image () { - local container_name=$1 - local mount_opts=$2 - echo " Testing s2i app image with invalid configuration" - assert_container_creation_fails -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db - echo " Testing s2i app image with correct configuration" - - DOCKER_ARGS=" --e POSTGRESQL_DATABASE=db --e POSTGRESQL_USER=user --e POSTGRESQL_PASSWORD=password --e POSTGRESQL_ADMIN_PASSWORD=password --e POSTGRESQL_BACKUP_USER=backuser --e POSTGRESQL_BACKUP_PASSWORD=pass -${mount_opts} -" create_container "$container_name" - - # need this to wait for the container to start up - PGUSER=user PASS=password test_connection "$container_name" - PGUSER=backuser PASS=pass DB=backup test_connection "$container_name" - - docker stop "$(get_cid $container_name)" >/dev/null -} - -run_s2i_test() { - local temp_file - echo " Testing s2i usage" - ct_s2i_usage "${IMAGE_NAME}" --pull-policy=never 1>/dev/null - - echo " Testing s2i build" - - local s2i_image_name=$IMAGE_NAME-testapp_$(ct_random_string) - images_to_clean+=( "$s2i_image_name" ) - - ct_s2i_build_as_df "file://${test_dir}/test-app" "${IMAGE_NAME}" "$s2i_image_name" 1>/dev/null - IMAGE_NAME=$s2i_image_name test_the_app_image s2i_config_build "" - - echo " Testing s2i mount" - create_temp_file - cat "$test_dir"/test-app/postgresql-init/backup_user.sh >> "$temp_file" - - # Test against original image, not the s2i one. But even if so, we expect - # user mouns the directory under "s2i" direcetory $APP_DATA/src. - local mount_point=/opt/app-root/src/postgresql-init/add_backup_user.sh - test_the_app_image _s2i_test_mount "-v ${temp_file}:$mount_point:z,ro" - echo " Success!" -} - -function run_general_tests() { - PGUSER=user PASS=pass POSTGRESQL_MAX_CONNECTIONS=42 POSTGRESQL_MAX_PREPARED_TRANSACTIONS=42 POSTGRESQL_SHARED_BUFFERS=64MB run_tests no_admin - PGUSER=user1 PASS=pass1 ADMIN_PASS=r00t run_tests admin - DB=postgres ADMIN_PASS=r00t run_tests only_admin - # Test with arbitrary uid for the container - DOCKER_ARGS="-u 12345" PGUSER=user2 PASS=pass run_tests no_admin_altuid - DOCKER_ARGS="-u 12345" PGUSER=user3 PASS=pass1 ADMIN_PASS=r00t run_tests admin_altuid - DB=postgres DOCKER_ARGS="-u 12345" ADMIN_PASS=rOOt run_tests only_admin_altuid -} - -run_test_cfg_hook() -{ - local volume_dir name=pg-test-cfg-dir - volume_dir=$(mktemp -d --tmpdir pg-hook-volume.XXXXX) - add_cleanup_command /bin/rm -rf "$volume_dir" - setfacl -R -m u:26:rwx "$volume_dir" - cp -r "$test_dir"/examples/custom-config/* "$volume_dir" - setfacl -R -m u:26:rwx "$volume_dir" - - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --v $volume_dir:/opt/app-root/src:Z - " create_container "$name" - assert_runtime_option "$name" shared_buffers 111MB - - # Check that POSTGRESQL_SHARED_BUFFERS has effect. - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --e POSTGRESQL_SHARED_BUFFERS=113MB - " create_container "$name-2" - assert_runtime_option "$name-2" shared_buffers 113MB - - # Check that volume has priority over POSTGRESQL_SHARED_BUFFERS. - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --e POSTGRESQL_SHARED_BUFFERS=113MB --v $volume_dir:/opt/app-root/src:Z - " create_container "$name-3" - assert_runtime_option "$name-3" shared_buffers 111MB -} - -run_s2i_enable_ssl_test() -{ - local s2i_image_name="$IMAGE_NAME-ssl_$(ct_random_string)" - ct_s2i_build_as_df "file://$test_dir/examples/enable-ssl" "${IMAGE_NAME}" "$s2i_image_name" 1>/dev/null - images_to_clean+=( "$s2i_image_name" ) - - local container_name=enable-ssl-test - - DOCKER_ARGS="-e POSTGRESQL_ADMIN_PASSWORD=password" \ - IMAGE_NAME="$s2i_image_name" create_container "$container_name" - - wait_ready "$container_name" - CONTAINER_IP=$(get_container_ip $container_name) - - DB=postgres assert_login_access postgres password true - - docker run --rm -e PGPASSWORD="password" "$IMAGE_NAME" psql "postgresql://postgres@$CONTAINER_IP:5432/postgres?sslmode=require" || \ - false "FAIL: Did not manage to connect using SSL only." - - docker stop "$(get_cid "$container_name")" -} - -run_s2i_bake_data_test () -{ - local s2i_image_name="$IMAGE_NAME-bake_$(ct_random_string)" - ct_s2i_build_as_df "file://$test_dir/examples/s2i-dump-data" "${IMAGE_NAME}" "$s2i_image_name" 1>/dev/null - images_to_clean+=( "$s2i_image_name" ) - - local container_name=bake-data-test - - DOCKER_ARGS="-e POSTGRESQL_ADMIN_PASSWORD=password" \ - IMAGE_NAME="$s2i_image_name" create_container "$container_name" - - wait_ready "$container_name" - - test "hello world" == "$(docker exec "$(get_cid "$container_name")" \ - bash -c "psql -tA -c 'SELECT * FROM test;'")" - - docker stop "$(get_cid "$container_name")" -} - -run_pgaudit_test() -{ - # extension pgaudit is not available for older versions - case ${VERSION} in - 9.6|10|11) echo "pgaudit not expected, test skipped."; return ;; - *) ;; - esac - - local config_dir data_dir name=pg-test-pgaudit - - # create a dir for config - config_dir=$(mktemp -d --tmpdir pg-hook-volume.XXXXX) - add_cleanup_command /bin/rm -rf "$config_dir" - cp -r "$test_dir"/examples/pgaudit/* "$config_dir" - setfacl -R -m u:26:rwx "$config_dir" - - # create a dir for data - create_volume_dir - data_dir="${volume_dir}" - - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --v ${config_dir}:/opt/app-root/src:Z --v ${data_dir}:/var/lib/pgsql/data:Z - " create_container "$name" - - assert_runtime_option "$name" shared_preload_libraries pgaudit - wait_ready "$name" - - # enable the pgaudit extension - # Deliberately moving heredoc into the container, otherwise it does not work - # in podman 1.6.x due to https://bugzilla.redhat.com/show_bug.cgi?id=1827324 - docker exec -i $(get_cid "$name") bash -c "psql </dev/null || return 1 - ct_check_latest_imagestreams - result=$? - popd >/dev/null || return 1 - return $result -} - -function run_all_tests() { - for test_case in $TEST_LIST; do - : "Running test $test_case" - $test_case - done; -} - -# configuration defaults -POSTGRESQL_MAX_CONNECTIONS=100 -POSTGRESQL_MAX_PREPARED_TRANSACTIONS=0 -POSTGRESQL_SHARED_BUFFERS=32MB - -# Run the chosen tests -TEST_LIST=${TESTS:-$TEST_LIST} run_all_tests - -TESTSUITE_RESULT=0 -echo "All tests passed." diff --git a/test/run_test.orig b/test/run_test.orig deleted file mode 100755 index 7411c18..0000000 --- a/test/run_test.orig +++ /dev/null @@ -1,847 +0,0 @@ -#!/bin/bash -# -# Test the PostgreSQL image. -# -# IMAGE_NAME specifies the name of the candidate image used for testing. -# The image has to be available before this script is executed. -# - -set -exo nounset -shopt -s nullglob - -# library from container-common-scripts -. test/test-lib.sh - -# local library -. test/pg-test-lib.sh - -TEST_LIST="\ -run_container_creation_tests -run_general_tests -run_change_password_test -run_replication_test -run_master_restart_test -run_doc_test -run_s2i_test -run_test_cfg_hook -run_s2i_bake_data_test -run_s2i_enable_ssl_test -run_upgrade_test -run_migration_test -" - -test $# -eq 1 -a "${1-}" == --list && echo "$TEST_LIST" && exit 0 -test -n "${IMAGE_NAME-}" || false 'make sure $IMAGE_NAME is defined' -test -n "${VERSION-}" || false 'make sure $VERSION is defined' -test -n "${OS-}" || false 'make sure $OS is defined' - -CIDFILE_DIR=$(mktemp --suffix=postgresql_test_cidfiles -d) - -volumes_to_clean= -images_to_clean=() -files_to_clean= -test_dir="$(readlink -f "$(dirname "$0")")" - -_cleanup_commands_space= -_cleanup_commands= - -add_cleanup_command () -{ - local cmd= space= - for arg; do - cmd+="$space$(printf "%q" "$arg")" - space=' ' - done - _cleanup_commands+="$_cleanup_commands_space$cmd" - _cleanup_commands_space=' -' -} -function cleanup() { - for cidfile in $CIDFILE_DIR/* ; do - CONTAINER=$(cat $cidfile) - - echo "Stopping and removing container $CONTAINER..." - docker stop $CONTAINER - exit_status=$(docker inspect -f '{{.State.ExitCode}}' $CONTAINER) - if [ "$exit_status" != "0" ]; then - echo "Dumping logs for $CONTAINER" - docker logs $CONTAINER - fi - docker rm $CONTAINER - rm $cidfile - echo "Done." - done - rmdir $CIDFILE_DIR - - ct_path_foreach "$volumes_to_clean" cleanup_volume_dir - - if test -n "${images_to_clean-}"; then - # Workaround for RHEL 7 bash bug: - # https://bugzilla.redhat.com/show_bug.cgi?id=1636393 - for image in "${images_to_clean[@]}"; do - docker rmi -f "$image" - done - fi - - ct_path_foreach "$files_to_clean" rm - - echo "$_cleanup_commands" | while read -r line; do - eval "$line" - done -} -trap cleanup EXIT - -cleanup_volume_dir () -{ - test ! -d "$1" && : "WARN: cleaned $1 for some reason" && return 0 - # When we run this test script as non-root (we should?), the PostgreSQL server - # within container is still run under 'postgres' user. It means that, taking - # into account 0077 umask of PostgreSQL server, we are unable to remove files - # created by server. That's why we need to let docker escalate the privileges - # again. - local datadir=/var/lib/pgsql/data - docker run -v "$1:$datadir:Z" --rm "$IMAGE_NAME" /bin/sh -c "/bin/rm -rf $datadir/userdata" - rmdir "$1" -} - -function get_cid() { - local id="$1" ; shift || return 1 - echo $(cat "$CIDFILE_DIR/$id") -} - -function get_container_ip() { - local id="$1" ; shift - docker inspect --format='{{.NetworkSettings.IPAddress}}' $(get_cid "$id") -} - -function get_ip_from_cid() { - local cid="$1"; shift - docker inspect --format='{{.NetworkSettings.IPAddress}}' $cid -} - -function postgresql_cmd() { - docker run --rm -e PGPASSWORD="$PASS" "$IMAGE_NAME" psql "postgresql://$PGUSER@$CONTAINER_IP:5432/${DB-db}" "$@" -} - -function test_connection() { - local name=$1 ; shift - ip=$(get_container_ip $name) - echo " Testing PostgreSQL connection to $ip..." - local max_attempts=20 - local sleep_time=2 - for i in $(seq $max_attempts); do - echo " Trying to connect..." - set +e - # Don't let the code come here if neither user nor admin is able to - # connect. - if [ -v PGUSER ] && [ -v PASS ]; then - CONTAINER_IP=$ip postgresql_cmd <<< "SELECT 1;" - else - PGUSER=postgres PASS=$ADMIN_PASS CONTAINER_IP=$ip DB=postgres postgresql_cmd <<< "SELECT 1;" - fi - status=$? - set -e - if [ $status -eq 0 ]; then - echo " Success!" - return 0 - fi - sleep $sleep_time - done - return 1 -} - -function test_postgresql() { - echo " Testing PostgreSQL" - postgresql_cmd <<< "CREATE EXTENSION 'uuid-ossp';" # to test contrib package - postgresql_cmd <<< "CREATE TABLE tbl (col1 VARCHAR(20), col2 VARCHAR(20));" - postgresql_cmd <<< "INSERT INTO tbl VALUES ('foo1', 'bar1');" - postgresql_cmd <<< "INSERT INTO tbl VALUES ('foo2', 'bar2');" - postgresql_cmd <<< "INSERT INTO tbl VALUES ('foo3', 'bar3');" - postgresql_cmd <<< "SELECT * FROM tbl;" - #postgresql_cmd <<< "DROP TABLE tbl;" - echo " Success!" -} - -function create_container() { - local name=$1 ; shift - local cargs=${DOCKER_ARGS:-} - # TODO: fix all create_container() invocations so that we don't need this, - # e.g. multiline DOCKER_ARGS var should end by trailing backslashes - cargs=$(echo "$cargs" | tr '\n' ' ') - cidfile="$CIDFILE_DIR/$name" - # create container with a cidfile in a directory for cleanup - eval "docker run $cargs --cidfile \$cidfile -d \$IMAGE_NAME \"\$@\"" - echo "Created container $(cat $cidfile)" -} - - -create_volume_dir () -{ - volume_dir=`mktemp -d --tmpdir pg-testdata.XXXXX` - setfacl -m u:26:-wx "$volume_dir" - ct_path_append volumes_to_clean "$volume_dir" -} - - -create_temp_file () -{ - temp_file=`mktemp --tmpdir pg-testfile.XXXXX` - setfacl -m u:26:rw- "$temp_file" - ct_path_append files_to_clean "$temp_file" -} - - -function assert_login_access() { - local PGUSER=$1 ; shift - local PASS=$1 ; shift - local success=$1 ; shift - - echo "testing login as $PGUSER:$PASS; should_success=$success" - - if postgresql_cmd <<<'SELECT 1;' ; then - if $success ; then - echo " $PGUSER($PASS) access granted as expected" - return - fi - else - if ! $success ; then - echo " $PGUSER($PASS) access denied as expected" - return - fi - fi - echo " $PGUSER($PASS) login assertion failed" - exit 1 -} - -function assert_local_access() { - local id="$1" ; shift - docker exec -i $(get_cid "$id") bash -c psql <<< "SELECT 1;" -} - - -# Make sure the invocation of docker run fails. -function assert_container_creation_fails() { - - # Time the docker run command. It should fail. If it doesn't fail, - # postgresql will keep running so we kill it with SIGKILL to make sure - # timeout returns a non-zero value. - set +e - timeout -s 9 --preserve-status 60s docker run --rm "$@" $IMAGE_NAME - ret=$? - set -e - - # Timeout will exit with a high number. - if [ $ret -gt 30 ]; then - return 1 - fi -} - - -# assert_container_creation_succeeds NAME [ARGS] -# ---------------------------------------------- -# Chcek that 'docker run' with IMAGE_NAME succeeds with docker arguments -# specified as ARGS. -assert_container_creation_succeeds () -{ - local check_env=false - local name=pg-success-"$(ct_random_string)" - local PGUSER='' PGPASS='' DB='' ADMIN_PASS= - local docker_args= - - for arg; do - docker_args+=" $(printf "%q" "$arg")" - if $check_env; then - local env=${arg//=*/} - local val=${arg//$env=/} - case $env in - POSTGRESQL_ADMIN_PASSWORD) ADMIN_PASS=$val ;; - POSTGRESQL_USER) PGUSER=$val ;; - POSTGRESQL_PASSWORD) PGPASS=$val ;; - POSTGRESQL_DATABASE) DB=$val ;; - esac - check_env=false - elif test "$arg" = -e; then - check_env=: - fi - done - - DOCKER_ARGS=$docker_args create_container "$name" - - if test -n "$PGUSER" && test -n "$PGPASS"; then - PGUSER=$PGUSER PASS=$PGPASS DB=$DB test_connection "$name" - fi - - if test -n "$ADMIN_PASS"; then - PGUSER=postgres PASS=$ADMIN_PASS DB=$DB test_connection "$name" - fi - - docker stop "$(get_cid "$name")" -} - - -function try_image_invalid_combinations() { - assert_container_creation_fails -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass "$@" - assert_container_creation_fails -e POSTGRESQL_USER=user -e POSTGRESQL_DATABASE=db "$@" - assert_container_creation_fails -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db "$@" -} - -function run_container_creation_tests() { - echo " Testing image entrypoint usage" - try_image_invalid_combinations - try_image_invalid_combinations -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - - VERY_LONG_IDENTIFIER="very_long_identifier_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx" - assert_container_creation_fails -e POSTGRESQL_USER= -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_fails -e POSTGRESQL_USER=$VERY_LONG_IDENTIFIER -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_succeeds -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD="\"" -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_succeeds -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=9invalid -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_fails -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=$VERY_LONG_IDENTIFIER -e POSTGRESQL_ADMIN_PASSWORD=admin_pass - assert_container_creation_succeeds -e POSTGRESQL_USER=user -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db -e POSTGRESQL_ADMIN_PASSWORD="\"" - echo " Success!" - - assert_container_creation_succeeds -e POSTGRESQL_ADMIN_PASSWORD="the @password" - assert_container_creation_succeeds -e POSTGRESQL_PASSWORD="the pass" -e POSTGRESQL_USER="the user" -e POSTGRESQL_DATABASE="the db" -} - -function test_config_option() { - local name=$1 ; shift - local setting=$1 ; shift - local value=$1 ; shift - - docker exec $(get_cid ${name}) grep -q "${setting} = ${value}" /var/lib/pgsql/openshift-custom-postgresql.conf -} - - -# wait_ready -# ---------- -# Wait until the PG container becomes ready -wait_ready () -{ - while ! docker exec "$(get_cid "$1")" /usr/libexec/check-container ; do - sleep 1 - done -} - - -# assert_runtime_option option value -# ---------------------------------- -assert_runtime_option () -{ - local name=$1 option=$2 value=$3 - wait_ready "$name" - set -- $(docker exec "$(get_cid "$name")" bash -c "psql -tA -c 'SHOW $option;'") - test "$value" = "$1" -} - - -function run_configuration_tests() { - local name=$1 ; shift - echo " Testing image configuration settings" - test_config_option ${name} max_connections ${POSTGRESQL_MAX_CONNECTIONS} - test_config_option ${name} max_prepared_transactions ${POSTGRESQL_MAX_PREPARED_TRANSACTIONS} - test_config_option ${name} shared_buffers ${POSTGRESQL_SHARED_BUFFERS} - echo " Success!" -} - -test_scl_usage() { - local name="$1" - local run_cmd="$2" - local expected="$3" - - echo " Testing the image SCL enable" - out=$(docker run --rm ${IMAGE_NAME} /bin/bash -c "${run_cmd}") - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[/bin/bash -c "${run_cmd}"] Expected '${expected}', got '${out}'" - return 1 - fi - out=$(docker exec $(get_cid $name) /bin/bash -c "${run_cmd}" 2>&1) - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[exec /bin/bash -c "${run_cmd}"] Expected '${expected}', got '${out}'" - return 1 - fi - out=$(docker exec $(get_cid $name) /bin/sh -ic "${run_cmd}" 2>&1) - if ! echo "${out}" | grep -q "${expected}"; then - echo "ERROR[exec /bin/sh -ic "${run_cmd}"] Expected '${expected}', got '${out}'" - return 1 - fi -} - - -function run_tests() { - echo " Testing general usage (run_tests) with '$1' as argument" - local name=$1 ; shift - - user_login=false - admin_login=false - envs= - # NOTE: We work wrongly with variables so please don't try to pass spaces - # within PGUSER/PASS/ADMIN_PASS variables. - [ -v PGUSER ] && envs+=" -e POSTGRESQL_USER=$PGUSER" - [ -v PASS ] && envs+=" -e POSTGRESQL_PASSWORD=$PASS" - if [ -v PGUSER ] && [ -v PASS ]; then - envs+=" -e POSTGRESQL_DATABASE=db" - user_login=: - fi - - if [ -v ADMIN_PASS ]; then - envs="$envs -e POSTGRESQL_ADMIN_PASSWORD=$ADMIN_PASS" - admin_login=: - fi - if [ -v POSTGRESQL_MAX_CONNECTIONS ]; then - envs="$envs -e POSTGRESQL_MAX_CONNECTIONS=$POSTGRESQL_MAX_CONNECTIONS" - fi - if [ -v POSTGRESQL_MAX_PREPARED_TRANSACTIONS ]; then - envs="$envs -e POSTGRESQL_MAX_PREPARED_TRANSACTIONS=$POSTGRESQL_MAX_PREPARED_TRANSACTIONS" - fi - if [ -v POSTGRESQL_SHARED_BUFFERS ]; then - envs="$envs -e POSTGRESQL_SHARED_BUFFERS=$POSTGRESQL_SHARED_BUFFERS" - fi - DOCKER_ARGS="${DOCKER_ARGS:-} $envs" create_container $name - CONTAINER_IP=$(get_container_ip $name) - test_connection $name - echo " Testing scl usage" - test_scl_usage $name 'psql --version' "$VERSION" - - echo " Testing login accesses" - assert_login_access "${PGUSER:-}" "${PASS-}" "$user_login" - assert_login_access "${PGUSER:-}" "${PASS-}_foo" false - - assert_login_access postgres "${ADMIN_PASS-}" "$admin_login" - assert_login_access postgres "${ADMIN_PASS-}_foo" false - - assert_local_access $name - run_configuration_tests $name - echo " Success!" - - if $user_login; then - test_postgresql $name - fi - - if $admin_login; then - DB=postgres PGUSER=postgres PASS=$ADMIN_PASS test_postgresql $name - fi -} - -function run_slave() { - local suffix="$1"; shift - docker run $cluster_args -e POSTGRESQL_MASTER_IP=${master_hostname} \ - -d --cidfile ${CIDFILE_DIR}/slave-${suffix}.cid $IMAGE_NAME run-postgresql-slave -} - -function run_master() { - local suffix="$1"; shift - master_args=${master_args-} - docker run $cluster_args $master_args \ - -d --cidfile ${CIDFILE_DIR}/master-${suffix}.cid $IMAGE_NAME run-postgresql-master >/dev/null -} - -function test_slave_visibility() { - local max_attempts=30 - - for slave in $slave_cids; do - slave_ip=$(get_ip_from_cid $slave) - if [ -z "$slave_ip" ]; then - echo "Failed to get IP for slave $slave." - echo "Dumping logs for $slave" - docker logs "$slave" - return 1 - fi - for i in $(seq $max_attempts); do - result="$(postgresql_cmd -c "select client_addr from pg_stat_replication;" | grep "$slave_ip" || true)" - if [[ -n "${result}" ]]; then - echo "${slave_ip} successfully registered as SLAVE for ${master_ip}" - break - fi - if [[ "${i}" == "${max_attempts}" ]]; then - echo "The ${slave_ip} failed to register in MASTER" - echo "Dumping logs for $slave" - docker logs $slave - return 1 - fi - sleep 1 - done - done -} - -function test_value_replication() { - local max_attempts=30 - - # Setup the replication data - local value - value=24 - postgresql_cmd -c "CREATE TABLE $table_name (a integer); INSERT INTO $table_name VALUES ($value);" - - # Read value from slaves and check whether it is expected - for slave in $slave_cids; do - slave_ip=$(get_ip_from_cid $slave) - CONTAINER_IP=$slave_ip - for i in $(seq $max_attempts); do - result="$(postgresql_cmd -At -c "select * from $table_name" || :)" - if [[ "$result" == "$value" ]]; then - echo "${slave_ip} successfully got value from MASTER ${master_ip}" - break - fi - if [[ "${i}" == "${max_attempts}" ]]; then - echo "The ${slave_ip} failed to see value added on MASTER" - echo "Dumping logs for $slave" - docker logs $slave - return 1 - fi - sleep 1 - done - done -} - -function setup_replication_cluster() { - # Run the PostgreSQL master - run_master "$cid_suffix" - - # Run the PostgreSQL slaves - local i - master_ip=$(get_container_ip "master-$cid_suffix.cid") - local cluster_args="$cluster_args --add-host postgresql-master:$master_ip" - local master_hostname="postgresql-master" - for i in $(seq ${slave_num:-1}); do - slave_cids="$slave_cids $(run_slave $cid_suffix-$i)" - done - -} - -function run_master_restart_test() { - local DB=postgres - local PGUSER=master - local PASS=master - - echo "Testing failed master restart" - local cluster_args="-e POSTGRESQL_ADMIN_PASSWORD=pass -e POSTGRESQL_MASTER_USER=$PGUSER -e POSTGRESQL_MASTER_PASSWORD=$PASS" - local cid_suffix="mrestart" - local table_name="t1" - local master_ip= - local slave_cids= - - create_volume_dir - local master_args="-v ${volume_dir}:/var/lib/pgsql/data:Z" - - # Setup the cluster - slave_num=2 setup_replication_cluster - - # Check if the master knows about the slaves - CONTAINER_IP=$master_ip - test_slave_visibility - - echo "Kill the master and create a new one" - local cidfile=$CIDFILE_DIR/master-$cid_suffix.cid - docker kill $(cat $cidfile) - # Don't forget to remove its .cid file - rm $cidfile - - run_master $cid_suffix - CONTAINER_IP=$(get_container_ip master-$cid_suffix.cid) - - # Update master_ip in slaves - for slave in $slave_cids; do - docker exec -u 0 $slave bash -c "sed \"s/$master_ip/$CONTAINER_IP/g\" /etc/hosts >/tmp/hosts && cp /tmp/hosts /etc/hosts" - done - master_ip=$CONTAINER_IP - # Check if the new master sees existing slaves - test_slave_visibility - - # Check if the replication works - table_name="t1" test_value_replication -} - -function run_replication_test() { - local DB=postgres - local PGUSER=master - local PASS=master - - echo "Testing master-slave replication" - local cluster_args="-e POSTGRESQL_ADMIN_PASSWORD=pass -e POSTGRESQL_MASTER_USER=$PGUSER -e POSTGRESQL_MASTER_PASSWORD=$PASS" - local cid_suffix="basic" - local master_ip= - local slave_cids= - - # Setup the cluster - setup_replication_cluster - - # Check if the master knows about the slaves - CONTAINER_IP=$master_ip - test_slave_visibility - - # Do some real work to test replication in practice - table_name="t1" test_value_replication -} - -function run_change_password_test() { - echo " Testing password change" - local name="change_password" - - local database='db' - local user='user' - local password='password' - local admin_password='adminPassword' - - create_volume_dir - local volume_options="-v ${volume_dir}:/var/lib/pgsql/data:Z" - - DOCKER_ARGS=" --e POSTGRESQL_DATABASE=${database} --e POSTGRESQL_USER=${user} --e POSTGRESQL_PASSWORD=${password} --e POSTGRESQL_ADMIN_PASSWORD=${admin_password} -$volume_options -" create_container ${name} - - # need to set these because `postgresql_cmd` relies on global variables - PGUSER=${user} - PASS=${password} - - # need this to wait for the container to start up - CONTAINER_IP=$(get_container_ip ${name}) - test_connection ${name} - - echo " Testing login" - - assert_login_access ${user} ${password} true - assert_login_access 'postgres' ${admin_password} true - - echo " Changing passwords" - - docker stop $(get_cid ${name}) - DOCKER_ARGS=" --e POSTGRESQL_DATABASE=${database} --e POSTGRESQL_USER=${user} --e POSTGRESQL_PASSWORD=NEW_${password} --e POSTGRESQL_ADMIN_PASSWORD=NEW_${admin_password} -$volume_options -" create_container "${name}_NEW" - - # need to set this because `postgresql_cmd` relies on global variables - PASS="NEW_${password}" - - # need this to wait for the container to start up - CONTAINER_IP=$(get_container_ip "${name}_NEW") - test_connection "${name}_NEW" - - echo " Testing login with new passwords" - - assert_login_access ${user} "NEW_${password}" true - assert_login_access ${user} ${password} false - - assert_login_access 'postgres' "NEW_${admin_password}" true - assert_login_access 'postgres' ${admin_password} false - - echo " Success!" -} - -run_upgrade_test () -{ - # Do not run on Fedora or RHEL8 until the upgrade script - # is fixed for non-SCL use cases - { [ "${OS}" == "fedora" ] || [ "${OS}" == "rhel8" ]; } && return 0 - - local upgrade_path="none 9.2 9.4 9.5 9.6 10 none" prev= act= - for act in $upgrade_path; do - if test "$act" = $VERSION; then - break - fi - prev=$act - done - test "$prev" != none - # Check if the previous image is available in the registry - docker pull "$(get_image_id "$prev:remote")" || return 0 - - # TODO: We run this script from $VERSION directory, through test/run symlink. - test/run_upgrade_test "$prev:remote" "$VERSION:local" -} - -run_migration_test () -{ - [ "${OS}" == "fedora" ] && return 0 - - local from_version - # Only test a subset of the migration path on non-intel hosts - if [ "$(uname -i)" == "x86_64" ]; then - local upgrade_path="9.2 9.4 9.5 9.6 10" - else - local upgrade_path="10" - fi - - for from_version in $upgrade_path; do - # Do not test migration from $VERSION:remote to $VERSION:local - test $(version2number $from_version) -lt $(version2number "$VERSION") \ - || break - # Skip if the previous image is not available in the registry - docker pull "$(get_image_id "$from_version:remote")" || continue - test/run_migration_test $from_version:remote $VERSION:local - done -} - -run_doc_test() { - local tmpdir=$(mktemp -d) - local f - echo " Testing documentation in the container image" - # Extract the help files from the container - for f in help.1 ; do - docker run --rm ${IMAGE_NAME} /bin/bash -c "cat /${f}" >${tmpdir}/$(basename ${f}) - # Check whether the files include some important information - for term in "POSTGRESQL\_ADMIN\_PASSWORD" volume 5432 ; do - if ! cat ${tmpdir}/$(basename ${f}) | grep -F -q -e "${term}" ; then - echo "ERROR: File /${f} does not include '${term}'." - return 1 - fi - done - done - # Check whether the files use the correct format - if ! file ${tmpdir}/help.1 | grep -q roff ; then - echo "ERROR: /help.1 is not in troff or groff format" - return 1 - fi - echo " Success!" - echo -} - -test_the_app_image () { - local container_name=$1 - local mount_opts=$2 - echo " Testing s2i app image with invalid configuration" - assert_container_creation_fails -e POSTGRESQL_PASSWORD=pass -e POSTGRESQL_DATABASE=db - echo " Testing s2i app image with correct configuration" - - DOCKER_ARGS=" --e POSTGRESQL_DATABASE=db --e POSTGRESQL_USER=user --e POSTGRESQL_PASSWORD=password --e POSTGRESQL_ADMIN_PASSWORD=password --e POSTGRESQL_BACKUP_USER=backuser --e POSTGRESQL_BACKUP_PASSWORD=pass -${mount_opts} -" create_container "$container_name" - - # need this to wait for the container to start up - PGUSER=user PASS=password test_connection "$container_name" - PGUSER=backuser PASS=pass DB=backup test_connection "$container_name" - - docker stop "$(get_cid $container_name)" >/dev/null -} - -run_s2i_test() { - local temp_file - echo " Testing s2i usage" - ct_s2i_usage "${IMAGE_NAME}" --pull-policy=never 1>/dev/null - - echo " Testing s2i build" - - local s2i_image_name=$IMAGE_NAME-testapp_$(ct_random_string) - images_to_clean+=( "$s2i_image_name" ) - - ct_s2i_build_as_df "file://${test_dir}/test-app" "${IMAGE_NAME}" "$s2i_image_name" 1>/dev/null - IMAGE_NAME=$s2i_image_name test_the_app_image s2i_config_build "" - - echo " Testing s2i mount" - create_temp_file - cat "$test_dir"/test-app/postgresql-init/backup_user.sh >> "$temp_file" - - # Test against original image, not the s2i one. But even if so, we expect - # user mouns the directory under "s2i" direcetory $APP_DATA/src. - local mount_point=/opt/app-root/src/postgresql-init/add_backup_user.sh - test_the_app_image _s2i_test_mount "-v ${temp_file}:$mount_point:z,ro" - echo " Success!" -} - -function run_general_tests() { - PGUSER=user PASS=pass POSTGRESQL_MAX_CONNECTIONS=42 POSTGRESQL_MAX_PREPARED_TRANSACTIONS=42 POSTGRESQL_SHARED_BUFFERS=64MB run_tests no_admin - PGUSER=user1 PASS=pass1 ADMIN_PASS=r00t run_tests admin - DB=postgres ADMIN_PASS=r00t run_tests only_admin - # Test with arbitrary uid for the container - DOCKER_ARGS="-u 12345" PGUSER=user2 PASS=pass run_tests no_admin_altuid - DOCKER_ARGS="-u 12345" PGUSER=user3 PASS=pass1 ADMIN_PASS=r00t run_tests admin_altuid - DB=postgres DOCKER_ARGS="-u 12345" ADMIN_PASS=rOOt run_tests only_admin_altuid -} - -run_test_cfg_hook() -{ - local volume_dir name=pg-test-cfg-dir - volume_dir=$(mktemp -d --tmpdir pg-hook-volume.XXXXX) - add_cleanup_command /bin/rm -rf "$volume_dir" - setfacl -R -m u:26:rwx "$volume_dir" - cp -r "$test_dir"/examples/custom-config/* "$volume_dir" - setfacl -R -m u:26:rwx "$volume_dir" - - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --v $volume_dir:/opt/app-root/src:Z - " create_container "$name" - assert_runtime_option "$name" shared_buffers 111MB - - # Check that POSTGRESQL_SHARED_BUFFERS has effect. - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --e POSTGRESQL_SHARED_BUFFERS=113MB - " create_container "$name-2" - assert_runtime_option "$name-2" shared_buffers 113MB - - # Check that volume has priority over POSTGRESQL_SHARED_BUFFERS. - DOCKER_ARGS=" --e POSTGRESQL_ADMIN_PASSWORD=password --e POSTGRESQL_SHARED_BUFFERS=113MB --v $volume_dir:/opt/app-root/src:Z - " create_container "$name-3" - assert_runtime_option "$name-3" shared_buffers 111MB -} - -run_s2i_enable_ssl_test() -{ - local s2i_image_name="$IMAGE_NAME-ssl_$(ct_random_string)" - ct_s2i_build_as_df "file://$test_dir/examples/enable-ssl" "${IMAGE_NAME}" "$s2i_image_name" 1>/dev/null - images_to_clean+=( "$s2i_image_name" ) - - local container_name=enable-ssl-test - - DOCKER_ARGS="-e POSTGRESQL_ADMIN_PASSWORD=password" \ - IMAGE_NAME="$s2i_image_name" create_container "$container_name" - - wait_ready "$container_name" - CONTAINER_IP=$(get_container_ip $container_name) - - DB=postgres assert_login_access postgres password true - - docker run --rm -e PGPASSWORD="password" "$IMAGE_NAME" psql "postgresql://postgres@$CONTAINER_IP:5432/postgres?sslmode=require" || \ - false "FAIL: Did not manage to connect using SSL only." - - docker stop "$(get_cid "$container_name")" -} - -run_s2i_bake_data_test () -{ - local s2i_image_name="$IMAGE_NAME-bake_$(ct_random_string)" - ct_s2i_build_as_df "file://$test_dir/examples/s2i-dump-data" "${IMAGE_NAME}" "$s2i_image_name" 1>/dev/null - images_to_clean+=( "$s2i_image_name" ) - - local container_name=bake-data-test - - DOCKER_ARGS="-e POSTGRESQL_ADMIN_PASSWORD=password" \ - IMAGE_NAME="$s2i_image_name" create_container "$container_name" - - wait_ready "$container_name" - - test "hello world" == "$(docker exec "$(get_cid "$container_name")" \ - bash -c "psql -tA -c 'SELECT * FROM test;'")" - - docker stop "$(get_cid "$container_name")" -} - -function run_all_tests() { - for test_case in $TEST_LIST; do - : "Running test $test_case" - $test_case - done; -} - -# configuration defaults -POSTGRESQL_MAX_CONNECTIONS=100 -POSTGRESQL_MAX_PREPARED_TRANSACTIONS=0 -POSTGRESQL_SHARED_BUFFERS=32MB - -# Run the chosen tests -TEST_LIST=${TESTS:-$TEST_LIST} run_all_tests diff --git a/test/run_upgrade_test b/test/run_upgrade_test deleted file mode 100755 index e91282b..0000000 --- a/test/run_upgrade_test +++ /dev/null @@ -1,97 +0,0 @@ -#! /bin/bash - -set -e - -. test/pg-test-lib.sh - -ADMIN_PASSWORD=redhat - -STEPS=( "$@" ) - -quote_args () -{ - quote_args_result= - local space= - for arg; do - quote_args_result+="$space$(printf "%q" "$arg")" - space=' ' - done -} - -quote_args_print () -{ - quote_args "$@" - echo "$quote_args_result" -} - -# background_container IMAGE DOCKER_ARGS CMD -background_container () -{ - CID=$(eval "docker run -d $2 $1 $3") - test -n "$CID" -} - -# run_server DATADIR IMAGE_ID DOCKER_ARGS -run_server () -{ - local datadir=$1 - local image_id=$2 - local docker_args=$(quote_args_print -e POSTGRESQL_ADMIN_PASSWORD="$ADMIN_PASSWORD") - docker_args+=' '$(quote_args_print -v "$datadir:/var/lib/pgsql/data:Z") - docker_args+=" $3" - background_container "$image_id" "$docker_args" -} - -# init_datadir IMAGE_ID dataspec -# ------------------------------ -init_datadir () -{ - local image_id=$1 - DATADIR=$(mktemp -d) - setfacl -m u:26:rwx "$DATADIR" - run_server "$DATADIR" "$image_id" - wait_for_postgres "$CID" - - eval "data_$2_create" - eval "data_$2_check" - - docker stop "$CID" >/dev/null - docker rm -f "$CID" >/dev/null -} - - -# check_upgrade_path {hardlink|copy} dataspec -assert_upgrade_succeeds () -{ - info "Initializing datadir with $VERSION_FROM PostgreSQL" - local INIT_IMAGE=$(get_image_id "$VERSION_FROM") - local dataspec=$2 - init_datadir "$INIT_IMAGE" "$dataspec" - - info "Running upgrade '$1/$dataspec'" - - for upgrade_to in "${UPGRADE_PATH[@]}"; do - info "Upgrading to $upgrade_to" - UPGRADE_IMAGE=$(get_image_id "$upgrade_to") - run_server "$DATADIR" "$UPGRADE_IMAGE" "-e POSTGRESQL_UPGRADE=$1" - wait_for_postgres "$CID" - eval "data_${dataspec}_check" - debug "the upgrading container of version '$upgrade_to' responded" - docker stop "$CID" >/dev/null - docker rm -f "$CID" >/dev/null - - run_server "$DATADIR" "$UPGRADE_IMAGE" - wait_for_postgres "$CID" - eval "data_${dataspec}_check" - debug "restarted server of version '$upgrade_to' responded" - docker stop "$CID" >/dev/null - docker rm -f "$CID" >/dev/null - done -} - -VERSION_FROM=$1 ; shift -UPGRADE_PATH=( "$@" ) -for data in empty pagila; do - assert_upgrade_succeeds hardlink "$data" - assert_upgrade_succeeds copy "$data" -done diff --git a/test/test-app/postgresql-init/backup_user.sh b/test/test-app/postgresql-init/backup_user.sh deleted file mode 100644 index 762bef1..0000000 --- a/test/test-app/postgresql-init/backup_user.sh +++ /dev/null @@ -1,12 +0,0 @@ -# Check that user credentials for backup is set - -[[ -v POSTGRESQL_BACKUP_USER && -v POSTGRESQL_BACKUP_PASSWORD ]] || usage "You have to set all variables for user for doing backup: POSTGRESQL_BACKUP_USER, POSTGRESQL_BACKUP_PASSWORD" - -# create backup user with 'backup' database -psql --variable=user="$POSTGRESQL_BACKUP_USER" \ - --variable=password="$POSTGRESQL_BACKUP_PASSWORD" \ - <<<" -CREATE USER :user SUPERUSER password :'password'; -CREATE DATABASE backup OWNER = :user; -ALTER USER :user set default_transaction_read_only = on; -" diff --git a/test/test-lib-openshift.sh b/test/test-lib-openshift.sh deleted file mode 120000 index f48003b..0000000 --- a/test/test-lib-openshift.sh +++ /dev/null @@ -1 +0,0 @@ -../common/test-lib-openshift.sh \ No newline at end of file diff --git a/test/test-lib-postgresql.sh b/test/test-lib-postgresql.sh deleted file mode 100644 index 599507f..0000000 --- a/test/test-lib-postgresql.sh +++ /dev/null @@ -1,39 +0,0 @@ -#!/bin/bash -# -# Functions for tests for the PostgreSQL image in OpenShift. -# -# IMAGE_NAME specifies a name of the candidate image used for testing. -# The image has to be available before this script is executed. -# - -THISDIR=$(dirname ${BASH_SOURCE[0]}) - -source ${THISDIR}/test-lib.sh -source ${THISDIR}/test-lib-openshift.sh - -function test_postgresql_integration() { - local image_name=$1 - local service_name=postgresql - ct_os_template_exists postgresql-ephemeral && t=postgresql-ephemeral || t=postgresql-persistent - ct_os_test_template_app_func "${image_name}" \ - "${t}" \ - "${service_name}" \ - "ct_os_check_cmd_internal '' '${service_name}-testing' 'PGPASSWORD=testp pg_isready -t 15 -h -U testu -d testdb' 'accepting connections' 120" \ - "-p POSTGRESQL_VERSION=${VERSION} \ - -p DATABASE_SERVICE_NAME="${service_name}-testing" \ - -p POSTGRESQL_USER=testu \ - -p POSTGRESQL_PASSWORD=testp \ - -p POSTGRESQL_DATABASE=testdb" -} - -# Check the imagestream -function test_postgresql_imagestream() { - case ${OS} in - rhel7|centos7) ;; - *) echo "Imagestream testing not supported for $OS environment." ; return 0 ;; - esac - - ct_os_test_image_stream_template "${THISDIR}/../imagestreams/postgresql-${OS%[0-9]*}.json" "${THISDIR}/../examples/postgresql-ephemeral-template.json" postgresql "-p POSTGRESQL_VERSION=${VERSION}" -} - -# vim: set tabstop=2:shiftwidth=2:expandtab: diff --git a/test/test-lib.sh b/test/test-lib.sh deleted file mode 120000 index 6ddf876..0000000 --- a/test/test-lib.sh +++ /dev/null @@ -1 +0,0 @@ -../common/test-lib.sh \ No newline at end of file