Skip to content

NO-ISSUE: Update Konflux references #7581

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

Open
wants to merge 15 commits into
base: cloud_hotfix_releases
Choose a base branch
from

Conversation

red-hat-konflux[bot]
Copy link
Contributor

This PR contains the following updates:

Package Change Notes
quay.io/konflux-ci/tekton-catalog/task-buildah a16437e -> ad68143
quay.io/konflux-ci/tekton-catalog/task-clamav-scan dd7b678 -> 11b1684
quay.io/konflux-ci/tekton-catalog/task-sast-coverity-check 0.2 -> 0.3 ⚠️migration⚠️
quay.io/konflux-ci/tekton-catalog/task-sast-shell-check b416734 -> 188a4f6
quay.io/konflux-ci/tekton-catalog/task-sast-snyk-check 0.3 -> 0.4 ⚠️migration⚠️
quay.io/konflux-ci/tekton-catalog/task-sast-unicode-check 0.1 -> 0.2 ⚠️migration⚠️
quay.io/konflux-ci/tekton-catalog/task-source-build 5619046 -> 23877bc

Configuration

📅 Schedule: Branch creation - "after 5am on saturday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


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

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

This PR has been generated by MintMaker (powered by Renovate Bot).

adriengentil and others added 15 commits February 24, 2025 18:12
Signed-off-by: Riccardo Piccoli <[email protected]>
Co-authored-by: Riccardo Piccoli <[email protected]>
…works (#7379)

With the recent addition of the external LB feature in CMN, enabling multiple machine networks, this PR adjusts the MTU validation to accommodate scenarios with multiple machine networks.

Co-authored-by: Linoy Hadad <[email protected]>
…dle (#7363)

* NO-ISSUE: Add jq tool

This patch adds a tool to simplify running _jq_ queries.

Signed-off-by: Juan Hernandez <[email protected]>

* MGMT-20060: Add OpenShift AI AMD operator bundle

This patch adds a new _Openshift AI (AMD)_ operator bundle that contains
the OpenShift AI operator and the AMD GPU operator.

The patch also adds the KMM operator, which is a dependency of the AMD
GPU operator.

The KMM operator builds the modules inside the cluster, using the image
stream mechanism, so it includes a setup job to enable the internal
image registry.

Related: https://issues.redhat.com/browse/MGMT-20060
Signed-off-by: Juan Hernandez <[email protected]>

---------

Signed-off-by: Juan Hernandez <[email protected]>
Co-authored-by: Juan Hernandez <[email protected]>
This isn't needed, it is only needed by the OpenShift AI AMD bundle, was
added by accident.

Signed-off-by: Juan Hernandez <[email protected]>
Co-authored-by: Juan Hernandez <[email protected]>
We implement the following metrics:
Record cache hits and misses

The purpose of this metric is to measure efficiency of the cache. This metric should be a counter. The total sum should be the equivalent to the sum of hits+misses (each request served). Errors should be logged instead.

assisted_installer_release_cache{hit=(true|false),release="\d.\d"}

Labels

    hit: hit should either be true or false
    release: release is major and minor openshift version, so that cardinality is contained. if we can't retrive major/minor, we label it as `unknown`

Record cache evictions

This metric is a counter. The purpose of this metric is to count the number of eviction attempts. For each eviction attempt we should just count one with success=true label if the operation evicted at least one file, false if evicted 0 files). Individual file deletion and possible errors should be logged instead.

assisted_installer_release_cache_eviction{success=(true|false)}

Labels

    success: is true if the operation removed at least one file, false otherwise

Co-authored-by: Paul Maidment <[email protected]>
…if all reports are checked (#7387)

This PR fixes a bug introduced in the previous PR (#7384), where the validation success return was placed too early. As a result, there's a chance that a failure report appears after a success report, causing failed reports to be missed.

Co-authored-by: Linoy Hadad <[email protected]>
* MGMT-20245: Add kernel args for raid (Intel VROC) (#7508)

Intel VROC supports raid creation from BIOS and creates
raid devices on booted os.
When raid is installation disk, reboot fails because device not detectd.
Enable kernel args - rd.md=1 and rd.auto=1

Co-authored-by: root <[email protected]>

* MGMT-20324:  Disable feature validation when creating and updating assisted objects due to an error when user is trying to add hosts on day2 with P/Z arch

* MGMT-20302: Remove root=/dev/disk/by-label/dm-mpath-root from kargs added when installation disk is multipath iSCSI (#7507)

coreos-installer automatically generates the root=UUID kernel argument, which makes adding a root karg via assisted-service unnecessary. Additionally, it's been found that using root with a by-label value works fine when there's only a single multipath device. However, with multiple multipath devices, it can lead to issues — since the initramfs will scan all available devices for a filesystem with the matching label, potentially causing a race condition.

---------

Co-authored-by: Benny Kopilov <[email protected]>
Co-authored-by: root <[email protected]>
Co-authored-by: Elior Erez <[email protected]>
Co-authored-by: Linoy Hadad <[email protected]>
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
@red-hat-konflux red-hat-konflux bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. konflux lgtm Indicates that a PR is ready to be merged. labels Apr 19, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 19, 2025
@openshift-ci-robot
Copy link

@red-hat-konflux[bot]: This pull request explicitly references no jira issue.

In response to this:

This PR contains the following updates:

Package Change Notes
quay.io/konflux-ci/tekton-catalog/task-buildah a16437e -> ad68143
quay.io/konflux-ci/tekton-catalog/task-clamav-scan dd7b678 -> 11b1684
quay.io/konflux-ci/tekton-catalog/task-sast-coverity-check 0.2 -> 0.3 ⚠️migration⚠️
quay.io/konflux-ci/tekton-catalog/task-sast-shell-check b416734 -> 188a4f6
quay.io/konflux-ci/tekton-catalog/task-sast-snyk-check 0.3 -> 0.4 ⚠️migration⚠️
quay.io/konflux-ci/tekton-catalog/task-sast-unicode-check 0.1 -> 0.2 ⚠️migration⚠️
quay.io/konflux-ci/tekton-catalog/task-source-build 5619046 -> 23877bc

Configuration

📅 Schedule: Branch creation - "after 5am on saturday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


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

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

This PR has been generated by MintMaker (powered by Renovate Bot).

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 19, 2025
@openshift-ci openshift-ci bot requested review from ori-amizur and pastequo April 19, 2025 10:12
Copy link

openshift-ci bot commented Apr 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: red-hat-konflux[bot]

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

1 similar comment
Copy link

openshift-ci bot commented Apr 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: red-hat-konflux[bot]

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

@danmanor danmanor force-pushed the cloud_hotfix_releases branch from 0af8f80 to 91165b3 Compare April 20, 2025 07:07
Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 20, 2025
@openshift-merge-robot
Copy link

PR needs rebase.

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.

Copy link

openshift-ci bot commented Apr 22, 2025

@red-hat-konflux[bot]: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-metal-assisted-4-19 8b93c43 link false /test e2e-metal-assisted-4-19
ci/prow/e2e-metal-assisted-ai-nvidia-4-18 8b93c43 link true /test e2e-metal-assisted-ai-nvidia-4-18
ci/prow/e2e-metal-assisted-ai-amd-4-18 8b93c43 link true /test e2e-metal-assisted-ai-amd-4-18

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. konflux lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants