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.

Sign up for GitHub

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

Jump to bottom

chore(deps): update konflux references #845

Merged
merged 1 commit into from
Aug 30, 2024
Merged

chore(deps): update konflux references #845

merged 1 commit into from
Aug 30, 2024

Conversation

red-hat-konflux[bot]
Copy link
Contributor

@red-hat-konflux red-hat-konflux bot commented Aug 21, 2024 •
edited
Loading

This PR contains the following updates:

Package Change Notes
quay.io/redhat-appstudio-tekton-catalog/task-buildah afd98cb -> 60cd734
quay.io/redhat-appstudio-tekton-catalog/task-prefetch-dependencies 492db3c -> 1b75828
quay.io/redhat-appstudio-tekton-catalog/task-sast-snyk-check 0.1 -> 0.2 ⚠️migration⚠️
quay.io/redhat-appstudio-tekton-catalog/task-source-build 929bf55 -> 14b91ad
  • If you want to rebase/retry this PR, check this box

To execute skipped test pipelines write comment /ok-to-test

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Aug 21, 2024

Hi @red-hat-konflux[bot]. Thanks for your PR.

I'm waiting for a konflux-ci member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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-sigs/prow repository.

@kasemAlem
Copy link
Contributor

/ok-to-test

@red-hat-konflux red-hat-konflux bot changed the title chore(deps): update konflux references to 1726e55 chore(deps): update konflux references Aug 21, 2024
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 3 times, most recently from 5f5cc4f to 11c24e3 Compare August 21, 2024 18:18
@kasemAlem
Copy link
Contributor

/retest

@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 3 times, most recently from 9c4cc61 to 6c53c9f Compare August 27, 2024 10:33
dirgim
dirgim approved these changes Aug 28, 2024
View reviewed changes
Copy link
Collaborator

@dirgim dirgim left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM, the migration is just a warning about increased snyk check coverage and doesn't require any changes on our side.

@openshift-ci openshift-ci bot added the lgtm label Aug 28, 2024
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch from 6c53c9f to ae2992a Compare August 28, 2024 12:32
@openshift-ci openshift-ci bot removed the lgtm label Aug 28, 2024
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Aug 28, 2024

New changes are detected. LGTM label has been removed.

@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 4 times, most recently from 4cc14ac to ea6690b Compare August 29, 2024 16:04
@red-hat-konflux
chore(deps): update konflux references
9fcd88f 
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch from 5c49c15 to 9fcd88f Compare August 30, 2024 10:06
@dirgim dirgim merged commit a577072 into main Aug 30, 2024
11 checks passed
@rh-tap-build-team rh-tap-build-team bot mentioned this pull request Aug 30, 2024
Merged
@hongweiliu17 hongweiliu17 mentioned this pull request Sep 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kasemAlem kasemAlem kasemAlem approved these changes

@dirgim dirgim dirgim approved these changes

@hongweiliu17 hongweiliu17 Awaiting requested review from hongweiliu17 hongweiliu17 is a code owner

@jsztuka jsztuka Awaiting requested review from jsztuka jsztuka is a code owner

@Josh-Everett Josh-Everett Awaiting requested review from Josh-Everett Josh-Everett is a code owner

@sonam1412 sonam1412 Awaiting requested review from sonam1412 sonam1412 is a code owner

@dheerajodha dheerajodha Awaiting requested review from dheerajodha dheerajodha is a code owner

@14rcole 14rcole Awaiting requested review from 14rcole 14rcole is a code owner

@jencull jencull Awaiting requested review from jencull jencull is a code owner

@chipspeak chipspeak Awaiting requested review from chipspeak chipspeak is a code owner

Assignees

@dirgim dirgim

@kasemAlem kasemAlem

Labels
ok-to-test size: XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kasemAlem @dirgim @openshift-merge-robot