Skip to content

Conversation

@tylertitsworth
Copy link
Contributor

@tylertitsworth tylertitsworth commented May 28, 2024

Description

Ensure Sarif upload does not run on a dispatch, otherwise scan-container will fail

Related Issue

n/a

Changes Made

Title

  • The code follows the project's coding standards.
  • No Intel Internal IP is present within the changes.
  • The documentation has been updated to reflect any changes in functionality.

Validation

https://github.com/intel/ai-containers/actions/runs/9277915007

  • I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

@github-actions
Copy link

github-actions bot commented May 28, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

@tylertitsworth tylertitsworth disabled auto-merge May 28, 2024 22:35
@tylertitsworth tylertitsworth changed the title add security events perms Add Security Settings to Scan Upload May 28, 2024
@tylertitsworth tylertitsworth changed the title Add Security Settings to Scan Upload Sarif Upload does not work on dispatch May 29, 2024
@tylertitsworth tylertitsworth self-assigned this May 29, 2024
@tylertitsworth tylertitsworth enabled auto-merge May 29, 2024 15:37
auto-merge was automatically disabled May 30, 2024 16:56

Pull request was closed

@tylertitsworth tylertitsworth deleted the tylertitsworth/weekly-scan-uploads branch May 30, 2024 16:56
@tylertitsworth tylertitsworth restored the tylertitsworth/weekly-scan-uploads branch May 30, 2024 23:56
Signed-off-by: Tyler Titsworth <tyler.titsworth@intel.com>
@tylertitsworth tylertitsworth merged commit b193d07 into main May 31, 2024
@tylertitsworth tylertitsworth deleted the tylertitsworth/weekly-scan-uploads branch June 3, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants