Skip to content

chore(ci): Remove Trivy Action#3389

Merged
google-oss-prow[bot] merged 1 commit intokubeflow:masterfrom
andreyvelich:remove-trivy-action
Mar 25, 2026
Merged

chore(ci): Remove Trivy Action#3389
google-oss-prow[bot] merged 1 commit intokubeflow:masterfrom
andreyvelich:remove-trivy-action

Conversation

@andreyvelich
Copy link
Copy Markdown
Member

As per our discussion with the security team, we suggested to remove Trivy action from our CI at the moment.

/assign @akshaychitneni @astefanutti @tenzen-y @kramaranya @Fiona-Waters @franciscojavierarceo @juliusvonkohout @thesuperzapper

Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the Trivy vulnerability scanning GitHub Actions workflow from CI, aligning the repository’s workflow configuration with the current direction agreed with the security team.

Changes:

  • Deleted the .github/workflows/trivy-scan.yaml workflow that ran Trivy filesystem scans and uploaded SARIF results to the GitHub Security tab.

@andreyvelich andreyvelich changed the title Remove Trivy Action chore(ci): Remove Trivy Action Mar 25, 2026
Copy link
Copy Markdown
Contributor

@franciscojavierarceo franciscojavierarceo left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@google-oss-prow google-oss-prow Bot added the lgtm label Mar 25, 2026
@astefanutti
Copy link
Copy Markdown
Contributor

Thanks @andreyvelich!

/lgtm
/approve

@google-oss-prow
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: astefanutti, franciscojavierarceo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@andreyvelich
Copy link
Copy Markdown
Member Author

/retest

1 similar comment
@andreyvelich
Copy link
Copy Markdown
Member Author

/retest

@andreyvelich
Copy link
Copy Markdown
Member Author

@jaiakash Could you check why GPU runners are not created?

@google-oss-prow google-oss-prow Bot merged commit 41f4962 into kubeflow:master Mar 25, 2026
42 of 47 checks passed
@google-oss-prow google-oss-prow Bot added this to the v2.2 milestone Mar 25, 2026
@andreyvelich andreyvelich deleted the remove-trivy-action branch March 26, 2026 00:15
@andreyvelich
Copy link
Copy Markdown
Member Author

/cherry-pick release-2.1
/cherry-pick release-2.2

@google-oss-robot
Copy link
Copy Markdown

@andreyvelich: #3389 failed to apply on top of branch "release-2.1":

Applying: Remove Trivy Action
Using index info to reconstruct a base tree...
M	.github/workflows/trivy-scan.yaml
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): .github/workflows/trivy-scan.yaml deleted in Remove Trivy Action and modified in HEAD. Version HEAD of .github/workflows/trivy-scan.yaml left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Remove Trivy Action

Details

In response to this:

/cherry-pick release-2.1
/cherry-pick release-2.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@andreyvelich
Copy link
Copy Markdown
Member Author

/cherry-pick release-2.1

@google-oss-robot
Copy link
Copy Markdown

@andreyvelich: #3389 failed to apply on top of branch "release-2.1":

Applying: Remove Trivy Action
Using index info to reconstruct a base tree...
M	.github/workflows/trivy-scan.yaml
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): .github/workflows/trivy-scan.yaml deleted in Remove Trivy Action and modified in HEAD. Version HEAD of .github/workflows/trivy-scan.yaml left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Remove Trivy Action

Details

In response to this:

/cherry-pick release-2.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

andreyvelich added a commit to andreyvelich/trainer that referenced this pull request Mar 26, 2026
andreyvelich added a commit to andreyvelich/trainer that referenced this pull request Mar 26, 2026
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
andreyvelich added a commit to andreyvelich/trainer that referenced this pull request Mar 26, 2026
andreyvelich added a commit to andreyvelich/trainer that referenced this pull request Mar 26, 2026
andreyvelich added a commit to andreyvelich/trainer that referenced this pull request Mar 26, 2026
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
andreyvelich added a commit to andreyvelich/trainer that referenced this pull request Mar 26, 2026
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
@jaiakash
Copy link
Copy Markdown
Member

@jaiakash Could you check why GPU runners are not created?
I will check it, its getting stuck at lot of other PRs as well.

google-oss-prow Bot pushed a commit that referenced this pull request Mar 26, 2026
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
andreyvelich added a commit that referenced this pull request Mar 26, 2026
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
digvijay-y pushed a commit to digvijay-y/trainer that referenced this pull request Apr 11, 2026
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
Signed-off-by: DIGVIJAY <144053736+digvijay-y@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.