Skip to content

Commit

Permalink
test(cyclonus): [WIN-NPM] fix consistent failure from not sleeping lo…
Browse files Browse the repository at this point in the history
…ng enough (#2174)

* test(cyclonus): [WIN-NPM] fix consistent failure from not sleeping long enough

* build: fix syntax

* fix: specify namespace in kubectl

* fix: wait timeout=5m

Signed-off-by: Hunter Gregory <42728408+huntergregory@users.noreply.github.com>

---------

Signed-off-by: Hunter Gregory <42728408+huntergregory@users.noreply.github.com>
  • Loading branch information
huntergregory committed Sep 1, 2023
1 parent 3f313ad commit e102891
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
6 changes: 4 additions & 2 deletions .pipelines/npm/npm-conformance-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -442,8 +442,10 @@ jobs:
# swap azure-npm image with one built during run
./kubectl --kubeconfig=./kubeconfig set image daemonset/azure-npm-win -n kube-system azure-npm=$IMAGE_REGISTRY/azure-npm:windows-amd64-ltsc2022-$(TAG)
echo "sleeping 3 minutes to allow NPM pods to restart"
sleep 180
echo "sleeping and waiting for NPM pods to be ready..."
sleep 1m
./kubectl --kubeconfig=./kubeconfig wait --for=condition=Ready pod -n kube-system -l k8s-app=azure-npm --timeout=5m
echo "sleeping 3 more minutes to let windows NPM finish bootup phase"
./kubectl --kubeconfig=./kubeconfig get po -n kube-system -owide -A
Expand Down
10 changes: 2 additions & 8 deletions test/cyclonus/test-cyclonus-windows.sh
Original file line number Diff line number Diff line change
@@ -1,13 +1,7 @@
LOG_FILE=cyclonus-$CLUSTER_NAME

# curl -fsSL github.com/mattfenwick/cyclonus/releases/latest/download/cyclonus_linux_amd64.tar.gz | tar -zxv
# ./cyclonus_linux_amd64/cyclonus generate \

# this branch has double the amount of workers
git clone https://github.com/huntergregory/cyclonus.git --branch=slowness --depth=1
cd cyclonus
make cyclonus
cmd/cyclonus/cyclonus generate \
curl -fsSL github.com/mattfenwick/cyclonus/releases/latest/download/cyclonus_linux_amd64.tar.gz | tar -zxv
./cyclonus_linux_amd64/cyclonus generate \
--noisy=true \
--retries=7 \
--ignore-loopback=true \
Expand Down

0 comments on commit e102891

Please sign in to comment.