dd-trace-go icon indicating copy to clipboard operation
dd-trace-go copied to clipboard

ci: Run Datadog SCA in CI

Open harmonherring-pro opened this issue 3 months ago • 7 comments

What does this PR do?

Adds a new Gitlab CI job that dogfoods the Datadog SCA product

Motivation

@DataDog/software-integrity-and-trust partners with @DataDog/static-analysis to dogfood their SCA product and secure Datadog's supply chain.

Reviewer's Checklist

  • [ ] Changed code has unit tests for its functionality at or near 100% coverage.
  • [ ] System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
  • [ ] There is a benchmark for any new code, or changes to existing code.
  • [ ] If this interacts with the agent in a new way, a system test has been added.
  • [ ] Add an appropriate team label so this PR gets put in the right place for the release notes.
  • [ ] Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

harmonherring-pro avatar Mar 28 '24 20:03 harmonherring-pro

Benchmarks

Benchmark execution time: 2024-03-28 21:04:16

Comparing candidate commit f92faae97847b3652eec395d7c857ae476a87d11 in PR branch harmon.herring/sint-1892-deploy-sca with baseline commit aaf8af5710fd94c17f3c5d78930b0dcd927d0557 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 44 metrics, 0 unstable metrics.

pr-commenter[bot] avatar Mar 28 '24 21:03 pr-commenter[bot]

Cool to see this initiative poping here ;-)

Is the docker image publicly accessible? If so, I suggest we do a github workflow instead, as this is our main CI tool. (You could even provide a simple reusable github workflow or action to hide all these details).

Julio-Guerra avatar Mar 28 '24 21:03 Julio-Guerra

Cool to see this initiative poping here ;-)

Is the docker image publicly accessible? If so, I suggest we do a github workflow instead, as this is our main CI tool. (You could even provide a simple reusable github workflow or action to hide all these details).

It's not. If you want to use a GitHub action, you can use the SCA GitHub action.

juli1 avatar Mar 29 '24 01:03 juli1

Cool to see this initiative poping here ;-) Is the docker image publicly accessible? If so, I suggest we do a github workflow instead, as this is our main CI tool. (You could even provide a simple reusable github workflow or action to hide all these details).

It's not. If you want to use a GitHub action, you can use the SCA GitHub action.

So why not integrate this here? We are an open-source library, so the more open we are, the better ;-) So far GitLab is used only for the benchmark platform which relies on gitlab runners maintained by the related backend team, everything else we have and rely on ourselves runs on GitHub Workflow.

Still open to starting your early integration this way.

Julio-Guerra avatar Mar 29 '24 12:03 Julio-Guerra

I was planning to apply this at some point. Thanks for taking the lead, @harmonherring-pro! As @Julio-Guerra stated, I think too that we should go with the SCA GitHub Action.

If you don't mind, I can take care of this and convert the GitLab YAML to its GitHub equivalent in this same PR. WDYT?

darccio avatar Apr 03 '24 10:04 darccio

I was planning to apply this at some point. Thanks for taking the lead, @harmonherring-pro! As @Julio-Guerra stated, I think too that we should go with the SCA GitHub Action.

If you don't mind, I can take care of this and convert the GitLab YAML to its GitHub equivalent in this same PR. WDYT?

Thanks for pinging me! I was wrapped up in some other work for a few days and didn't realize the activity this PR had 😅 that sounds good to me, let me know if you need anything from me!

harmonherring-pro avatar Apr 03 '24 16:04 harmonherring-pro

This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Apr 25 '24 01:04 github-actions[bot]