From d56c172d5a6d72742532064e0099cea9988f4087 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Ollivier?= Date: Mon, 8 Apr 2024 14:57:46 +0200 Subject: [PATCH 1/2] Set helm_install_namespace in all example cnf-testsuite.yml MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit example-cnfs/coredns fails passing default_namespace by default and fails in gates too [1]. It forces now deployment coredns-coredns to be in cnfspace. Please note it sets helm_install_namespace in all example cnf-testsuite.yml when missing. [1] https://github.com/cnti-testcatalog/testsuite/actions/runs/8564278589/job/23493678803#step:11:1170 close: #1952 Signed-off-by: Cédric Ollivier --- example-cnfs/coredns/cnf-testsuite.yml | 1 + example-cnfs/envoy/cnf-testsuite.yml | 1 + example-cnfs/ip-forwarder/cnf-testsuite.yml | 1 + example-cnfs/nsm/cnf-testsuite.yml | 1 + example-cnfs/pantheon-nsm-nat/cnf-testsuite.yml | 1 + example-cnfs/vpp-3c2n-csp-use-case/cnf-testsuite.yml | 1 + 6 files changed, 6 insertions(+) diff --git a/example-cnfs/coredns/cnf-testsuite.yml b/example-cnfs/coredns/cnf-testsuite.yml index c805c4605..fb8e59cf8 100644 --- a/example-cnfs/coredns/cnf-testsuite.yml +++ b/example-cnfs/coredns/cnf-testsuite.yml @@ -7,3 +7,4 @@ helm_repository: # CONFIGURATION OF HELM REPO - ONLY NEEDED WHEN USING helm_char #helm_directory: coredns # PATH_TO_CNFS_HELM_CHART ; or #manifest_directory: coredns # PATH_TO_DIRECTORY_OF_CNFS_MANIFEST_FILES ; or release_name: coredns # DESIRED_HELM_RELEASE_NAME +helm_install_namespace: cnfspace diff --git a/example-cnfs/envoy/cnf-testsuite.yml b/example-cnfs/envoy/cnf-testsuite.yml index 26bfcce8a..9283377f6 100644 --- a/example-cnfs/envoy/cnf-testsuite.yml +++ b/example-cnfs/envoy/cnf-testsuite.yml @@ -6,3 +6,4 @@ helm_repository: repo_url: https://cncf.gitlab.io/stable helm_chart: stable/envoy allowlist_helm_chart_container_names: [nginx, envoy, calico-node, kube-proxy, nginx-proxy, node-cache] +helm_install_namespace: cnfspace diff --git a/example-cnfs/ip-forwarder/cnf-testsuite.yml b/example-cnfs/ip-forwarder/cnf-testsuite.yml index 3ca7a6b3b..c57f6b760 100644 --- a/example-cnfs/ip-forwarder/cnf-testsuite.yml +++ b/example-cnfs/ip-forwarder/cnf-testsuite.yml @@ -4,3 +4,4 @@ release_name: vpp service_name: "" rolling_update_test_tag: latest allowlist_helm_chart_container_names: [nginx, calico-node, kube-proxy, nginx-proxy, node-cache, kube-multus] +helm_install_namespace: cnfspace diff --git a/example-cnfs/nsm/cnf-testsuite.yml b/example-cnfs/nsm/cnf-testsuite.yml index de2d20b07..63cb48823 100644 --- a/example-cnfs/nsm/cnf-testsuite.yml +++ b/example-cnfs/nsm/cnf-testsuite.yml @@ -6,3 +6,4 @@ helm_repository: name: stable repo_url: https://cncf.gitlab.io/stable allowlist_helm_chart_container_names: [] +helm_install_namespace: cnfspace diff --git a/example-cnfs/pantheon-nsm-nat/cnf-testsuite.yml b/example-cnfs/pantheon-nsm-nat/cnf-testsuite.yml index 0f8fb0ba0..f82415242 100644 --- a/example-cnfs/pantheon-nsm-nat/cnf-testsuite.yml +++ b/example-cnfs/pantheon-nsm-nat/cnf-testsuite.yml @@ -3,3 +3,4 @@ helm_directory: nat-cnf release_name: cnf-nat service_name: allowlist_helm_chart_container_names: [node-cache, nginx, coredns, calico-node, kube-proxy, nginx-proxy, kube-multus] +helm_install_namespace: cnfspace diff --git a/example-cnfs/vpp-3c2n-csp-use-case/cnf-testsuite.yml b/example-cnfs/vpp-3c2n-csp-use-case/cnf-testsuite.yml index 6701914e2..5a37d0fc7 100644 --- a/example-cnfs/vpp-3c2n-csp-use-case/cnf-testsuite.yml +++ b/example-cnfs/vpp-3c2n-csp-use-case/cnf-testsuite.yml @@ -2,3 +2,4 @@ helm_directory: csp release_name: ip-forwarder-csp allowlist_helm_chart_container_names: [nginx, calico-node, kube-proxy, nginx-proxy, node-cache] +helm_install_namespace: cnfspace From 16e7f494cb29e47b8aee7143cb6fe5a544f388b7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Ollivier?= Date: Thu, 11 Apr 2024 09:23:55 +0200 Subject: [PATCH 2/2] Add Rolling configurations in coredns/cnf-testsuite.yml MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit example-cnfs/coredns fails passing CNF Rollback because rollback_from_tag is undefined. Here are the error messages ␛[31mPlease add the container name coredns and a corresponding rollback_from_tag into your cnf-testsuite.yml under container names␛[0m ␛[31m✖️ FAILED: CNF Rollback Failed ␛[0m ␛[32m✔️ PASSED: CNF Rollback Passed ␛[0m Signed-off-by: Cédric Ollivier --- example-cnfs/coredns/cnf-testsuite.yml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/example-cnfs/coredns/cnf-testsuite.yml b/example-cnfs/coredns/cnf-testsuite.yml index fb8e59cf8..1eb073fb3 100644 --- a/example-cnfs/coredns/cnf-testsuite.yml +++ b/example-cnfs/coredns/cnf-testsuite.yml @@ -8,3 +8,9 @@ helm_repository: # CONFIGURATION OF HELM REPO - ONLY NEEDED WHEN USING helm_char #manifest_directory: coredns # PATH_TO_DIRECTORY_OF_CNFS_MANIFEST_FILES ; or release_name: coredns # DESIRED_HELM_RELEASE_NAME helm_install_namespace: cnfspace +container_names: + - name: coredns + rolling_update_test_tag: "1.8.0" + rolling_downgrade_test_tag: 1.6.7 + rolling_version_change_test_tag: 1.8.0 + rollback_from_tag: 1.8.0