Updates for EIB 1.2.0-rc1 #126

Merged
nbelouin merged 5 commits from :main into main 2025-05-06 14:36:44 +02:00
Owner

@nbelouin Did I do it right? Not sure how the versioning is supposed to look when it comes to RCs.

@nbelouin Did I do it right? Not sure how the versioning is supposed to look when it comes to RCs.
dbekhit added 3 commits 2025-04-18 07:17:48 +02:00
Upgrade Hauler to v1.2.1 and add version to build
All checks were successful
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -1s
9548235fde
changes for EIB 1.2.0-rc0
Some checks failed
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 3h20m57s
954bf7b140
dbekhit force-pushed main from 954bf7b140 to 9ebd6dce35 2025-04-18 07:21:12 +02:00 Compare
Owner

I believe it should be 1.2.0.rc0 instead but you also need to bump the RPM itself.

I believe it should be `1.2.0.rc0` instead but you also need to bump the RPM itself.
dbekhit added 1 commit 2025-04-29 16:46:00 +02:00
modify '-' to '~'
Some checks failed
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 3h10m49s
d90c6cbc01
dbekhit force-pushed main from d90c6cbc01 to f28dfa589e 2025-04-29 16:50:50 +02:00 Compare
nbelouin added 1 commit 2025-04-30 10:16:19 +02:00
Fix the _service file to use the defined pre-release version
Some checks failed
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after -13s
0da3e55977
nbelouin added 1 commit 2025-04-30 10:56:01 +02:00
EIB v1.2.0-rc0 needs golang 1.124
Some checks failed
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 5m27s
d5218ab378
nbelouin force-pushed main from d5218ab378 to 49405f41f9 2025-04-30 16:26:28 +02:00 Compare
dbekhit changed title from Updates for EIB 1.2.0-rc0 to Updates for EIB 1.2.0-rc1 2025-05-02 17:11:57 +02:00
dbekhit added 1 commit 2025-05-02 17:16:43 +02:00
update to v1.2.0-rc1
All checks were successful
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -47s
4259b167fd
Owner

@dbekhit @nbelouin When I attempt to build an image using the EIB built based on this PR, I see the error below.

OCI artifact references (e.g. tags) do not support the plus sign (+). To support
storing semantic versions, Helm adopts the convention of changing plus (+) to
an underscore (_) in chart version tags when pushing to a registry and back to
a plus (+) when pulling from a registry.
command: /usr/bin/helm pull oci://registry.opensuse.org/isv/suse/edge/containers/charts/metallb-chart --version 999.0.0+up0.14.9 --destination /builds/t3_SvjZ84/0/edge-engineering/stack-validation/eib-temp/_build/build-May05_12-11-56/helm/metallb-chart
Error: registry.opensuse.org/isv/suse/edge/containers/charts/metallb-chart:999.0.0_up0.14.9: not found

Please note the path to metallb-chart which is containers/charts instead of factory/charts.

I couldn't locate where this issue is coming from in the repo but when I run the container, I see artifacts.yaml contents as below.

# podman run -it --entrypoint /bin/bash registry.opensuse.org/isv/suse/edge/factory/staging/pr-126/images/edge-image-builder:1.2.0-rc1
e45c0778e07b:/ # cat /artifacts.yaml
metallb:
  chart: metallb-chart
  repository: "oci://registry.opensuse.org/isv/suse/edge/containers/charts/"
  version: "999.0.0+up0.14.9"
endpoint-copier-operator:
  chart: endpoint-copier-operator-chart
  repository: "oci://registry.opensuse.org/isv/suse/edge/containers/charts/"
  version: "999.0.0+up0.2.1"
kubernetes:
  k3s:
    selinuxPackage: k3s-selinux-1.6-1.slemicro.noarch
    selinuxRepository: https://rpm.rancher.io/k3s/stable/common/slemicro/noarch
  rke2:
    selinuxPackage: rke2-selinux
    selinuxRepository: https://rpm.rancher.io/rke2/stable/common/slemicro/noarch
@dbekhit @nbelouin When I attempt to build an image using the EIB built based on this PR, I see the error below. ``` OCI artifact references (e.g. tags) do not support the plus sign (+). To support storing semantic versions, Helm adopts the convention of changing plus (+) to an underscore (_) in chart version tags when pushing to a registry and back to a plus (+) when pulling from a registry. command: /usr/bin/helm pull oci://registry.opensuse.org/isv/suse/edge/containers/charts/metallb-chart --version 999.0.0+up0.14.9 --destination /builds/t3_SvjZ84/0/edge-engineering/stack-validation/eib-temp/_build/build-May05_12-11-56/helm/metallb-chart Error: registry.opensuse.org/isv/suse/edge/containers/charts/metallb-chart:999.0.0_up0.14.9: not found ``` Please note the path to `metallb-chart` which is `containers/charts` instead of `factory/charts`. I couldn't locate where this issue is coming from in the repo but when I run the container, I see `artifacts.yaml` contents as below. ``` # podman run -it --entrypoint /bin/bash registry.opensuse.org/isv/suse/edge/factory/staging/pr-126/images/edge-image-builder:1.2.0-rc1 e45c0778e07b:/ # cat /artifacts.yaml metallb: chart: metallb-chart repository: "oci://registry.opensuse.org/isv/suse/edge/containers/charts/" version: "999.0.0+up0.14.9" endpoint-copier-operator: chart: endpoint-copier-operator-chart repository: "oci://registry.opensuse.org/isv/suse/edge/containers/charts/" version: "999.0.0+up0.2.1" kubernetes: k3s: selinuxPackage: k3s-selinux-1.6-1.slemicro.noarch selinuxRepository: https://rpm.rancher.io/k3s/stable/common/slemicro/noarch rke2: selinuxPackage: rke2-selinux selinuxRepository: https://rpm.rancher.io/rke2/stable/common/slemicro/noarch ```
Owner

@dbekhit @nbelouin you can ignore my previous comment as the EIB registry.opensuse.org/isv/suse/edge/factory/staging/pr-126/test_manifest_images/edge-image-builder:1.2.0-rc1 has the correct chart location. Thanks Nicolas!

@dbekhit @nbelouin you can ignore my previous comment as the EIB `registry.opensuse.org/isv/suse/edge/factory/staging/pr-126/test_manifest_images/edge-image-builder:1.2.0-rc1` has the correct chart location. Thanks Nicolas!
fdegirmenci approved these changes 2025-05-05 18:51:26 +02:00
fdegirmenci left a comment
Owner

This passed the validation in SV so it would be good if we can get this in for us to switch to this one.

This passed the validation in SV so it would be good if we can get this in for us to switch to this one.
nbelouin merged commit e71339ae00 into main 2025-05-06 14:36:44 +02:00
Sign in to join this conversation.
No Reviewers
No Label
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#126
No description provided.