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

fix(ci): allow trivy to fail (temporary) #309

Merged
merged 2 commits into from
Nov 11, 2024
Merged

fix(ci): allow trivy to fail (temporary) #309

merged 2 commits into from
Nov 11, 2024

Conversation

DerekRoberts
Copy link
Member

@DerekRoberts DerekRoberts commented Nov 11, 2024

Trivy has been having problems, so it has been removed from Analysis Results needs in analysis.yml. Ideally this can be restored when they're no longer having issues. To allow this failure reusable-tests-repo.yml had to be rolled into analysis.yml.

  results:
    name: Analysis Results
    if: always()
    needs: [..., trivy] # Restore trivy when/if fixed

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

@DerekRoberts DerekRoberts self-assigned this Nov 11, 2024
@DerekRoberts DerekRoberts added the bug Something isn't working label Nov 11, 2024
@DerekRoberts DerekRoberts merged commit 4bbe0d8 into main Nov 11, 2024
13 of 16 checks passed
@DerekRoberts DerekRoberts deleted the fix/ci/trivy branch November 11, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant