Skip to content

Commit

Permalink
CI: remove the daemon running check
Browse files Browse the repository at this point in the history
Quick workaround for not having to rebump frr-k8s version and upgrade
metallb again.
The daemon running check picks the wrong labels and the wrong namespace,
disabling it.

Signed-off-by: Federico Paolinelli <[email protected]>
  • Loading branch information
fedepaol committed Jul 21, 2024
1 parent c027d08 commit 17d06e0
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -379,9 +379,16 @@ jobs:
kubectl create ns frr-k8s-system
cd ${GITHUB_WORKSPACE}/frr-k8s
mkdir -p /tmp/kind_logs
# TODO: fix the e2e test so it checks for the current labels too and the proper namespace.
sed -i '/FRRK8isDaemonSetReady/{N;N;d;}' e2etests/e2etest_suite_test.go
sed -i '/github.com\/metallb\/frrk8stests\/pkg\/k8s\"/d' e2etests/e2etest_suite_test.go
# TODO: remove skipping the conflict test, curerntly broken for multiple namespaces
KUBECONFIG_PATH=${HOME}/.kube/config GINKGO_ARGS="--focus Webhooks --skip conflict" TEST_ARGS="--report-path=/tmp/kind_logs" make e2etests
kubectl delete ns frr-k8s-system
#TODO remove when the webhook tests do not leave resources behind
kubectl delete frrconfiguration --all -n metallb-system
- name: MetalLB E2E Tests
run: |
Expand Down

0 comments on commit 17d06e0

Please sign in to comment.