search

chainguard-dev / malcontent

#supply #chain #attack #detection
Apache License 2.0
446 stars 31 forks source link

Use 8-core runners for tests and updating third-party rules #633

Closed egibs closed 2 days ago

egibs commented 2 days ago

With all of our tests using t.Parallel() appropriately, we can leverage a larger runner to further cut down on how long our tests take.

Based on https://github.com/chainguard-dev/malcontent/pull/632, 8-core runners offer a sizable performance uplift over the default 4-core runners and anything larger hits the realm of diminishing returns pretty quickly.

image This definitely beats the 12-13+ minute Workflow runs we were seeing originally.

Bonus third-party rule Workflow timing after the change (ten minutes faster!): image