Customize helm "--timeout" option format if running helm v3 91/117591/4
authorBartek Grzybowski <b.grzybowski@partner.samsung.com>
Mon, 8 Feb 2021 12:48:06 +0000 (13:48 +0100)
committerBartek Grzybowski <b.grzybowski@partner.samsung.com>
Tue, 9 Feb 2021 08:33:42 +0000 (08:33 +0000)
Helm v3 requires unit definition for 'timeout' option which is not
supported by Helm v2.

Change-Id: Ia5839e5c412be1700459c70d43d2e6cc0400633d
Issue-ID: OOM-2665
Signed-off-by: Bartek Grzybowski <b.grzybowski@partner.samsung.com>
ansible/roles/application/defaults/main.yml
ansible/roles/application/molecule/default/tests/test_default.py
ansible/test/bin/install-molecule.sh

index 6a7472a..8e630f7 100644 (file)
@@ -2,7 +2,7 @@
 helm_repository_name: local
 helm_repository_url: http://127.0.0.1:8879
 helm_extra_install_options:
-  - { opt: '--timeout 1800'}
+  - { opt: "{% if helm_version | regex_search('^v3' ) %}{{ '--timeout 1800s' }}{% else %}{{ '--timeout 1800' }}{% endif %}"}
 # Override file generation for Helm application can be customized by any role
 # given by user and found by ansible from roles_path.
 # By default override file is generated by 'application-override' role that
index be9b4cd..f57f5af 100644 (file)
@@ -29,7 +29,7 @@ repo add local http://127.0.0.1:8879
 deploy moleculetestapp local/moleculetestapp --namespace \
 moleculetestapp -f /opt/moleculetestapp/helm_charts/onap/resources/\
 overrides/onap-all.yaml -f /opt/moleculetestapp/override.yaml \
---timeout 1800"""
+--timeout 1800s"""
         expected_plugin_path = '/root/.local/share/helm/plugins/deploy/' +\
                                'deploy.sh'
     assert fc == expected_content
index ab6de43..4df0668 100755 (executable)
@@ -41,5 +41,6 @@ source ${VENV_PATH}/bin/activate
 # Install Molecule
 if [ ! -z ${VIRTUAL_ENV} ]; then
     echo "Activated virtual env in ${VIRTUAL_ENV}"
-    pip -q install molecule==2.20 ansible==2.7.8 ansible-lint==4.2.0 docker pyopenssl
+    pip -q install -U pip
+    pip -q install pyopenssl molecule==2.20 ansible==2.7.8 ansible-lint==4.2.0 docker
 fi