Skip to content

test(e2e): poll for grpcapi service deletion instead of kubectl wait - #509

Closed
Dav-14 wants to merge 1 commit into
mainfrom
test/grpcapi-deletion-wait
Closed

test(e2e): poll for grpcapi service deletion instead of kubectl wait#509
Dav-14 wants to merge 1 commit into
mainfrom
test/grpcapi-deletion-wait

Conversation

@Dav-14

@Dav-14 Dav-14 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

kubectl wait --for=delete can ignore --timeout and hang until chainsaw's
step timeout kills the script (signal: killed), which flaked the
gateway-grpcapi-sync deletion step on the K8s 1.31 runner. Replace it
with a bounded kubectl get poll so the assertion fails legibly if the
service is genuinely not garbage-collected.


Stack created with GitHub Stacks CLIGive Feedback 💬

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8ab9910e-729b-4e9c-9835-e04f4e857319

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

kubectl wait --for=delete can ignore --timeout and hang until chainsaw's
step timeout kills the script (signal: killed), which flaked the
gateway-grpcapi-sync deletion step on the K8s 1.31 runner. Replace it
with a bounded kubectl get poll so the assertion fails legibly if the
service is genuinely not garbage-collected.
@Dav-14
Dav-14 force-pushed the test/grpcapi-deletion-wait branch from 2fd150a to b4c413f Compare August 3, 2026 10:10
@Dav-14
Dav-14 changed the base branch from fix/stacks-netpol to main August 3, 2026 10:10
@Dav-14 Dav-14 closed this Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant