Skip to content

Conversation

@cruizen
Copy link
Contributor

@cruizen cruizen commented Jun 14, 2022

Create trivy.yml GitHub action to enable license scan with Trivy

ISSUE(S):

SUMMARY

ISSUE TYPE

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that may cause existing functionality to not work as expected)
  • This change requires a documentation update

TESTING DONE

Manual

License check is performed as part of this PR build

Reviewers

/cc @anupbarve, @AnirudhPokala, @devidask27

Enable license scan with Trivy
@cruizen cruizen marked this pull request as draft June 14, 2022 14:17
@cruizen cruizen changed the title Create trivy.yml Enable vulnerability scanning with Trivy Jun 14, 2022
@cruizen cruizen marked this pull request as ready for review January 27, 2023 09:26
@cruizen cruizen marked this pull request as draft January 27, 2023 09:31
@cruizen
Copy link
Contributor Author

cruizen commented Jan 27, 2023

TODO: This will need a Dockerfile.

@cruizen cruizen requested review from a team and gauravgavhane05 March 13, 2024 08:44
@cruizen cruizen requested review from NeelavaChatterjee and mridulgain and removed request for devidask27 February 26, 2025 08:25
contents: read # for actions/checkout to fetch code
security-events: write # for github/codeql-action/upload-sarif to upload SARIF results
name: Build
runs-on: "ubuntu-18.04"

Choose a reason for hiding this comment

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

is there a requirement for this to be u18 can we use a higher one like u24?

# The branches below must be a subset of the branches above
branches: [ "master" ]
schedule:
- cron: '31 6 * * 3'

Choose a reason for hiding this comment

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

This seems to be 6:31 every Wed, so does it need to be every week? We are already doing it one during PRs so maybe once a month would suffice?

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