Skip to content

Commit

Permalink
ci: explicitly wait for cilium status after enabling Hubble relay on GKE
Browse files Browse the repository at this point in the history
We're occasionally encountering #918 on GKE too, e.g.
https://github.com/cilium/cilium-cli/runs/7421447399?check_suite_focus=true

Apply the same workaround as already added for EKS in commit
a0faf25 ("ci: explicitly wait for cilium status after enabling
Hubble relay") and for AKS in commit 7176ce5 ("ci: explicitly wait
for cilium status after enabling Hubble relay on AKS").

Signed-off-by: Tobias Klauser <tobias@cilium.io>
  • Loading branch information
tklauser committed Jul 20, 2022
1 parent 8b805d6 commit 60bc657
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/in-cluster-test-scripts/external-workloads-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ cilium install \
# Enable Relay
cilium hubble enable

# Wait for cilium and hubble relay to be ready
# NB: necessary to work against occassional flakes due to https://github.com/cilium/cilium-cli/issues/918
cilium status --wait

# Enable cluster mesh
cilium clustermesh enable

Expand Down
4 changes: 4 additions & 0 deletions .github/in-cluster-test-scripts/gke.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ cilium install \
# Enable Relay
cilium hubble enable

# Wait for cilium and hubble relay to be ready
# NB: necessary to work against occassional flakes due to https://github.com/cilium/cilium-cli/issues/918
cilium status --wait

# Port forward Relay
cilium hubble port-forward&
sleep 10s
Expand Down
5 changes: 5 additions & 0 deletions .github/in-cluster-test-scripts/multicluster.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,11 @@ cilium install \
cilium --context "${CONTEXT1}" hubble enable
cilium --context "${CONTEXT2}" hubble enable --relay=false

# Wait for cilium and hubble relay to be ready
# NB: necessary to work against occassional flakes due to https://github.com/cilium/cilium-cli/issues/918
cilium --context "${CONTEXT1}" status --wait
cilium --context "${CONTEXT2}" status --wait

# Enable cluster mesh
cilium --context "${CONTEXT1}" clustermesh enable
cilium --context "${CONTEXT2}" clustermesh enable
Expand Down

0 comments on commit 60bc657

Please sign in to comment.