Skip to content

Commit

Permalink
Merge pull request #1354 from FabianKramm/main
Browse files Browse the repository at this point in the history
test: only deploy metrics proxy if needed
  • Loading branch information
FabianKramm authored Nov 15, 2023
2 parents e61c61b + cd7784a commit 2cb24f4
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions .github/workflows/e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,10 @@ jobs:
if [ ${{ matrix.multinamespace-mode }} == "true" ]; then
extraArgs+=( -f ./test/multins_values.yaml )
fi
if [ ${{ matrix.test-suite-path }} == "./test/e2e_metrics_proxy" ]; then
helm repo add metrics-server https://kubernetes-sigs.github.io/metrics-server/
helm upgrade --install metrics-server metrics-server/metrics-server --set args={--kubelet-insecure-tls} --set containerPort=4443 -n kube-system
fi
if [ ${{ matrix.test-suite-path }} == "./test/e2e_target_namespace" ]; then
kubectl apply -f ${{ matrix.test-suite-path }}/role.yaml
Expand All @@ -209,13 +213,6 @@ jobs:
-f ${{ matrix.test-suite-path }}/values.yaml \
"${extraArgs[@]}"
continue-on-error: true

- name: Install metrics server in host cluster
id: install-metrics-server
run: |-
set -x
helm repo add metrics-server https://kubernetes-sigs.github.io/metrics-server/
helm upgrade --install metrics-server metrics-server/metrics-server --set args={--kubelet-insecure-tls} --set containerPort=4443 -n kube-system

- name: Wait until vcluster is ready
id: wait-until-vcluster-is-ready
Expand Down

0 comments on commit 2cb24f4

Please sign in to comment.