You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by pc...@apache.org on 2022/04/18 09:18:11 UTC

[camel-k] 08/12: Refactors proxy reset to its own script

This is an automated email from the ASF dual-hosted git repository.

pcongiusti pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-k.git

commit d2d0965c9b6c3d9da4133b3a3cf5cce9e581031c
Author: phantomjinx <p....@phantomjinx.co.uk>
AuthorDate: Fri Mar 25 18:53:15 2022 +0000

    Refactors proxy reset to its own script
    
    * Makes pre-flight test reset proxy as
      well as cleanup
---
 .github/actions/kamel-cleanup/cleanup.sh        | 32 +-------------
 .github/actions/kamel-cleanup/reset-proxy.sh    | 56 +++++++++++++++++++++++++
 .github/actions/kamel-preflight-test/action.yml |  8 ++++
 3 files changed, 65 insertions(+), 31 deletions(-)

diff --git a/.github/actions/kamel-cleanup/cleanup.sh b/.github/actions/kamel-cleanup/cleanup.sh
index da4ef64aa..b6cacae93 100755
--- a/.github/actions/kamel-cleanup/cleanup.sh
+++ b/.github/actions/kamel-cleanup/cleanup.sh
@@ -53,37 +53,7 @@ shift $((OPTIND-1))
 # which would require a catalogsource
 #
 if [ -n "${BUILD_CATALOG_SOURCE}" ]; then
-  set +e
-
-#
-# Remove values altogther
-#
-  PATCH=$(mktemp /tmp/proxy-patch.XXXXXXXX)
-  cat >${PATCH} <<EOF
-[
-  {
-    "op": "replace",
-    "path": "/spec",
-    "value": {}
-  },
-  {
-    "op": "replace",
-    "path": "/status",
-    "value": {}
-  }
-]
-EOF
-
-  kubectl patch --type='json' Proxy cluster --patch-file "${PATCH}"
-
-  if [ $? != 0 ]; then
-    echo "Error: Failed to reset the Proxy"
-    exit 1
-  fi
-
-  rm -f "${PATCH}"
-
-  set -e
+  ./.github/actions/kamel-cleanup/reset-proxy.sh
 fi
 
 if [ "${SAVE_NAMESPACES}" == "true" ]; then
diff --git a/.github/actions/kamel-cleanup/reset-proxy.sh b/.github/actions/kamel-cleanup/reset-proxy.sh
new file mode 100755
index 000000000..07f031970
--- /dev/null
+++ b/.github/actions/kamel-cleanup/reset-proxy.sh
@@ -0,0 +1,56 @@
+#!/bin/bash
+
+# ---------------------------------------------------------------------------
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You 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.
+# ---------------------------------------------------------------------------
+
+set +e
+
+kubectl get Proxy cluster >& /dev/null
+if [ $? != 0 ]; then
+  echo "Cluster does not have proxy called 'cluster'. Nothing to do."
+  exit 0
+fi
+
+#
+# Remove values altogther
+#
+PATCH=$(mktemp /tmp/proxy-patch.XXXXXXXX)
+cat >${PATCH} <<EOF
+[
+{
+  "op": "replace",
+  "path": "/spec",
+  "value": {}
+},
+{
+  "op": "replace",
+  "path": "/status",
+  "value": {}
+}
+]
+EOF
+
+kubectl patch --type='json' Proxy cluster --patch-file "${PATCH}"
+
+if [ $? != 0 ]; then
+  echo "Error: Failed to reset the Proxy"
+  exit 1
+fi
+
+rm -f "${PATCH}"
+
+set -e
diff --git a/.github/actions/kamel-preflight-test/action.yml b/.github/actions/kamel-preflight-test/action.yml
index 5cf1041ec..b5cd3e2d6 100644
--- a/.github/actions/kamel-preflight-test/action.yml
+++ b/.github/actions/kamel-preflight-test/action.yml
@@ -42,6 +42,14 @@ runs:
   using: "composite"
 
   steps:
+  - id: proxy-reset
+    name: Ensure Proxy has been reset
+    shell: bash
+    run: |
+      if [ -n "${{ inputs.build-catalog-source }}" ]; then
+        ./.github/actions/kamel-cleanup/reset-proxy.sh
+      fi
+
   - id: preflight-test
     name: Execute Preflight Test
     shell: bash