Skip to content

Conversation

red-hat-konflux[bot]
Copy link

@red-hat-konflux red-hat-konflux bot commented Mar 20, 2025

This PR contains the following updates:

Package Change
quay.io/konflux-ci/tekton-catalog/task-ecosystem-cert-preflight-checks 50668ba -> 00b13d0
quay.io/konflux-ci/tekton-catalog/task-rpms-signature-scan 1ed16be -> 60ec0c9
quay.io/konflux-ci/tekton-catalog/task-sast-coverity-check-oci-ta 32c4d3e -> cd7e802
quay.io/konflux-ci/tekton-catalog/task-sast-shell-check-oci-ta 4b704fa -> a591675
quay.io/konflux-ci/tekton-catalog/task-sast-snyk-check-oci-ta 916d718 -> ec2495e

  • If you want to rebase/retry this PR, check this box

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

Copy link

openshift-ci bot commented Mar 20, 2025

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

I'm waiting for a opendatahub-io 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.

@openshift-ci openshift-ci bot added the size/xs label Mar 20, 2025
@codecov-commenter
Copy link

codecov-commenter commented Mar 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.87%. Comparing base (2ccfe4d) to head (9b4441d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #567   +/-   ##
=======================================
  Coverage   55.87%   55.87%           
=======================================
  Files           9        9           
  Lines        2339     2339           
=======================================
  Hits         1307     1307           
  Misses        930      930           
  Partials      102      102           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jstourac
Copy link
Member

/override ci/prow/images
/override ci/prow/kf-notebook-controller-pr-image-mirror
/override ci/prow/odh-notebook-controller-e2e
/override ci/prow/odh-notebook-controller-pr-image-mirror

Copy link

openshift-ci bot commented Mar 21, 2025

@jstourac: Overrode contexts on behalf of jstourac: ci/prow/images, ci/prow/kf-notebook-controller-pr-image-mirror, ci/prow/odh-notebook-controller-e2e, ci/prow/odh-notebook-controller-pr-image-mirror

In response to this:

/override ci/prow/images
/override ci/prow/kf-notebook-controller-pr-image-mirror
/override ci/prow/odh-notebook-controller-e2e
/override ci/prow/odh-notebook-controller-pr-image-mirror

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.

@jstourac
Copy link
Member

The golangci-lint failures shouldn't be relevant to this change, I created this issue #568 to track it and get rid of it eventually.

/override "Code static analysis / golangci-lint (components/notebook-controller) (push)"
/override "Code static analysis / golangci-lint (components/odh-notebook-controller) (push)"

Copy link

openshift-ci bot commented Mar 21, 2025

@jstourac: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • Code static analysis / golangci-lint (components/notebook-controller) (push)
  • Code static analysis / golangci-lint (components/odh-notebook-controller) (push)

Only the following failed contexts/checkruns were expected:

  • Red Hat Konflux
  • ci/prow/images
  • ci/prow/kf-notebook-controller-pr-image-mirror
  • ci/prow/odh-notebook-controller-e2e
  • ci/prow/odh-notebook-controller-pr-image-mirror
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

The golangci-lint failures shouldn't be relevant to this change, I created this issue #568 to track it and get rid of it eventually.

/override "Code static analysis / golangci-lint (components/notebook-controller) (push)"
/override "Code static analysis / golangci-lint (components/odh-notebook-controller) (push)"

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.

@jstourac
Copy link
Member

/retest

@jiridanek
Copy link
Member

Code static analysis / golangci-lint (components/notebook-controller) (push)

It's not a failure on the PR proper, it's failure on the branch from which the PR is created...

@jiridanek
Copy link
Member

/ok-to-test

@jstourac
Copy link
Member

/retest

@jstourac
Copy link
Member

/retest kf-notebook-controller-on-pull-request

Copy link

openshift-ci bot commented Mar 21, 2025

@jstourac: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

/test images
/test kf-notebook-controller-pr-image-mirror
/test kf-notebook-controller-unit
/test odh-notebook-controller-e2e
/test odh-notebook-controller-pr-image-mirror
/test odh-notebook-controller-unit

Use /test all to run the following jobs that were automatically triggered:

pull-ci-opendatahub-io-kubeflow-main-images
pull-ci-opendatahub-io-kubeflow-main-kf-notebook-controller-pr-image-mirror
pull-ci-opendatahub-io-kubeflow-main-odh-notebook-controller-e2e
pull-ci-opendatahub-io-kubeflow-main-odh-notebook-controller-pr-image-mirror

In response to this:

/retest kf-notebook-controller-on-pull-request

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.

@red-hat-konflux red-hat-konflux bot changed the title chore(deps): update konflux references to 00b13d0 chore(deps): update konflux references Mar 21, 2025
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch from 4546ff0 to 6530e19 Compare March 21, 2025 13:07
@openshift-ci openshift-ci bot added size/s and removed size/xs labels Mar 21, 2025
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch from 6530e19 to 72b5774 Compare March 21, 2025 20:54
@openshift-ci openshift-ci bot added size/m and removed size/s labels Mar 21, 2025
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 72b5774 to 9b4441d Compare March 24, 2025 11:03
@openshift-ci openshift-ci bot added size/m and removed size/m labels Mar 24, 2025
@openshift-ci openshift-ci bot added the size/m label Mar 24, 2025
@jstourac
Copy link
Member

/lgtm

@jstourac
Copy link
Member

/override ci/prow/images ci/prow/kf-notebook-controller-pr-image-mirror ci/prow/odh-notebook-controller-e2e

Copy link

openshift-ci bot commented Mar 24, 2025

@jstourac: Overrode contexts on behalf of jstourac: ci/prow/images, ci/prow/kf-notebook-controller-pr-image-mirror, ci/prow/odh-notebook-controller-e2e

In response to this:

/override ci/prow/images ci/prow/kf-notebook-controller-pr-image-mirror ci/prow/odh-notebook-controller-e2e

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.

@jstourac
Copy link
Member

@jiridanek the fail seem to be quite consistent for the past couple of days for these konflux failed CI runs:

[2/2] STEP 2/9: RUN microdnf install -y shadow-utils &&    microdnf clean all
Downloading metadata...
Downloading metadata...
Downloading metadata...
Downloading metadata...
error: cannot update repo 'rhel-8-for-x86_64-baseos-rpms': Cannot download repomd.xml: Curl error (77): Problem with the SSL CA cert (path? access rights?) for https://cdn.redhat.com/content/dist/rhel8/8/x86_64/baseos/os/repodata/repomd.xml [error setting certificate verify locations:
  CAfile: /etc/rhsm/ca/redhat-uep.pem
  CApath: none]; Last error: Curl error (77): Problem with the SSL CA cert (path? access rights?) for https://cdn.redhat.com/content/dist/rhel8/8/x86_64/baseos/os/repodata/repomd.xml [error setting certificate verify locations:
  CAfile: /etc/rhsm/ca/redhat-uep.pem
  CApath: none]
subprocess exited with status 1
subprocess exited with status 1
Error: building at STEP "RUN microdnf install -y shadow-utils &&    microdnf clean all": exit status 1

Are you aware of any known issue with regards to this one? I'm not sure whether the changes in this PR could be causing it - the image tasks that are updated seem to be quite irrelevant to cause this.

@jstourac
Copy link
Member

Looks like the konflux failures aren't related to changes in this PR as we can see the very same failure also for the #571.

/approve

Copy link

openshift-ci bot commented Mar 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jstourac

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

The pull request process is described here

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

@jstourac
Copy link
Member

/override kf-notebook-controller-on-pull-request
/override odh-notebook-controller-on-pull-request

Copy link

openshift-ci bot commented Mar 24, 2025

@jstourac: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • kf-notebook-controller-on-pull-request
  • odh-notebook-controller-on-pull-request

Only the following failed contexts/checkruns were expected:

  • Red Hat Konflux / kf-notebook-controller-on-pull-request
  • Red Hat Konflux / kubeflow-enterprise-contract / kf-notebook-controller
  • Red Hat Konflux / kubeflow-enterprise-contract / odh-notebook-controller
  • Red Hat Konflux / odh-notebook-controller-on-pull-request
  • ci/prow/images
  • ci/prow/kf-notebook-controller-pr-image-mirror
  • ci/prow/odh-notebook-controller-e2e
  • ci/prow/odh-notebook-controller-pr-image-mirror
  • pull-ci-opendatahub-io-kubeflow-main-images
  • pull-ci-opendatahub-io-kubeflow-main-kf-notebook-controller-pr-image-mirror
  • pull-ci-opendatahub-io-kubeflow-main-odh-notebook-controller-e2e
  • pull-ci-opendatahub-io-kubeflow-main-odh-notebook-controller-pr-image-mirror
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override kf-notebook-controller-on-pull-request
/override odh-notebook-controller-on-pull-request

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.

@jstourac
Copy link
Member

/override "Red Hat Konflux / kf-notebook-controller-on-pull-request" "Red Hat Konflux / kubeflow-enterprise-contract / kf-notebook-controller" "Red Hat Konflux / kubeflow-enterprise-contract / odh-notebook-controller" "Red Hat Konflux / odh-notebook-controller-on-pull-request"

Copy link

openshift-ci bot commented Mar 24, 2025

@jstourac: Overrode contexts on behalf of jstourac: Red Hat Konflux / kf-notebook-controller-on-pull-request, Red Hat Konflux / kubeflow-enterprise-contract / kf-notebook-controller, Red Hat Konflux / kubeflow-enterprise-contract / odh-notebook-controller, Red Hat Konflux / odh-notebook-controller-on-pull-request

In response to this:

/override "Red Hat Konflux / kf-notebook-controller-on-pull-request" "Red Hat Konflux / kubeflow-enterprise-contract / kf-notebook-controller" "Red Hat Konflux / kubeflow-enterprise-contract / odh-notebook-controller" "Red Hat Konflux / odh-notebook-controller-on-pull-request"

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.

@openshift-merge-bot openshift-merge-bot bot merged commit f242ca0 into main Mar 24, 2025
23 of 27 checks passed
@openshift-merge-bot openshift-merge-bot bot deleted the konflux/references/main branch March 24, 2025 12:44
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.

3 participants