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

chore(deps): update odh-data-science-pipelines-operator-controller-v2-16 to d94ae98 #1376

Conversation

konflux-internal-p02[bot]
Copy link

This PR contains the following updates:

Package Update Change
quay.io/rhoai/odh-data-science-pipelines-operator-controller-rhel8 digest 928a610 -> d94ae98

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), 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

…-16 to d94ae98

Signed-off-by: konflux-internal-p02 <126015336+konflux-internal-p02[bot]@users.noreply.github.com>
@rhods-devops-app rhods-devops-app bot merged commit 9ae10a1 into rhoai-2.16 Dec 22, 2024
7 checks passed
zdtsw added a commit to zdtsw-forking/rhods-operator that referenced this pull request Jan 2, 2025
* update: owners (red-hat-data-services#1376)

Signed-off-by: Wen Zhou <[email protected]>
(cherry picked from commit 4a15a8f)
Signed-off-by: Wen Zhou <[email protected]>

* Dockerfile, Makefile: make CGO_ENABLED configurable (red-hat-data-services#1382)

The commit
a107703 ("feat(fips): enable GO_ENABLED in build (red-hat-data-services#1001)")

enabled CGO which makes problems for builders on non-x86 platforms.

Make it as an in the Dockerfile keeping default the same (enabled),
but make it possible to override with either environment
(`export CGO_ENABLED=0`) or make (`make CGO_ENABLED=0 image-build`)

Signed-off-by: Yauheni Kaliuta <[email protected]>
(cherry picked from commit 2d94349)

* Makefile: make USE_LOCAL overridable (red-hat-data-services#1384)

Get USE_LOCAL image build flag from makefile variable to make it
overridable with `make USE_LOCAL=true image`

Do not allow to get its value for environment be default due to
pretty generic name.

This is shorter than the old recommendation of overriding
IMAGE_BUILD_FLAGS. And since now CGO_ENABLED is also a flag, does
not mess up with it.

* USE_LOCAL=true uses existing manifests from opt/manifests for the
produced image without downloading them with get_all_manifests.sh
making it possible to both save time and make local amendments.

Signed-off-by: Yauheni Kaliuta <[email protected]>
(cherry picked from commit cea41dc)

* Dockerfile: merges manifests builder stages to one (red-hat-data-services#1381)

We can combine two build stages into one, as there is no need to
always build both images (not done by podman) to only then decide
from which one we want to copy manifests to the target
image. Instead manifests stage will either copy local manifests or
fetches using the script based on USE_LOCAL argument.

Move USE_LOCAL and OVERWIRTE_MANIFESTS args under FROM since args
have scope of the FROM they are declared in.

It requires opt/manifests directory to exist, but since it's a part
of git repo, it's fine.

Original patch from: Bartosz Majsak <[email protected]> [1]

[1] opendatahub-io#773

Signed-off-by: Yauheni Kaliuta <[email protected]>
(cherry picked from commit c1671ab)

* fix: rhoai build has extra manifests than odh's

- changes made in odh, missing manifests for rhoai
  since it is only using builder for main and others from manifests stage

Signed-off-by: Wen Zhou <[email protected]>

---------

Signed-off-by: Wen Zhou <[email protected]>
Co-authored-by: Yauheni Kaliuta <[email protected]>
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.

0 participants