Closed michelle192837 closed 1 year ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: michelle192837, sultan-duisenbay
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Thanks! /hold cancel
It's possible to have a grid with a column (see https://github.com/GoogleCloudPlatform/testgrid/blob/6ad903765c1a60491ea661d268780aa0e0de71ea/pkg/tabulator/tabstate.go#L445, which keeps a column if the grid would otherwise be empty) but no rows. This is fine, but the stale alert doesn't account for this at the moment. Update it to alert when no rows exist, in addition to the other reasons a tab might be stale.