onsi / ginkgo

A Modern Testing Framework for Go
http://onsi.github.io/ginkgo/
MIT License
8.09k stars 643 forks source link

Can't set custom GracePeriod on uninterruptible clean up node #1344

Closed eugenenosenko closed 4 months ago

eugenenosenko commented 5 months ago

Scenario:

I tried setting GracePeriod for ReportAfterSuite but I'm getting:

  [ReportAfterSuite] was passed NodeTimeout, SpecTimeout, or GracePeriod but
  does not have a callback that accepts a SpecContext or context.Context.  You
  must accept a context to enable timeouts and grace periods

It seems also that there is no way to pass Context to ReportAfterSuite so it's uninterruptable and I can't set custom GracePeriod.

Any suggestions on how to fix this?

Update: I created a draft PR #1345

onsi commented 5 months ago

hey thanks for the PR - I left some comments over there.

eugenenosenko commented 4 months ago

fixed in #1345