Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add vulnerability scan workflow #1261

Merged
merged 6 commits into from
Jul 12, 2024
Merged

Conversation

kian99
Copy link
Contributor

@kian99 kian99 commented Jul 4, 2024

Description

This PR adds a vulnerability scan workflow and updates some module dependencies where vulnerabilities were detected.
It also bumps the Go version to 1.22.5 to address some vulns in the stdlib.

Originally this tool would hang when run in CI - Filed an issue against the upstream golang/go#68307
Resolution: The tool consumes up to 12GB of memory, switched the job to use a self-hosted runner with 16GB of memory.

Fixes CSS-9379

@kian99 kian99 marked this pull request as ready for review July 4, 2024 08:08
@kian99 kian99 requested a review from a team as a code owner July 4, 2024 08:08
@kian99 kian99 force-pushed the vulnerability-scan branch from 4e4e56b to 8e98e5d Compare July 4, 2024 08:10
@kian99
Copy link
Contributor Author

kian99 commented Jul 4, 2024

This shouldn't be merged as govulncheck hangs in CI. Filed an issue against the upstream golang/go#68307
Resolved: The tool consumes up to 12GB of memory, switched the job to use a self-hosted runner with 16GB of memory.

@kian99 kian99 force-pushed the vulnerability-scan branch from d18dba3 to 0263286 Compare July 10, 2024 08:11
@kian99 kian99 force-pushed the vulnerability-scan branch from 4c0aab3 to e23d3c1 Compare July 10, 2024 08:29
Copy link
Contributor

@ale8k ale8k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@kian99 kian99 merged commit 4bfd2a9 into canonical:v3 Jul 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants