Updates for EIB 1.2.0-rc1 #126

dbekhit · 2025-04-18T07:17:48+02:00

dbekhit commented

2025-04-18 07:17:48 +02:00

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

dbekhit added 1 commit 2025-04-18 07:17:48 +02:00

Upgrade Hauler to v1.2.1 and add version to build

Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -1s

Details

9548235fde

dbekhit force-pushed main from 954bf7b140 to 9ebd6dce35

2025-04-18 07:21:12 +02:00

Compare

atanasdinov commented

2025-04-22 08:45:18 +02:00

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 force-pushed main from d90c6cbc01 to f28dfa589e

2025-04-29 16:50:50 +02:00

Compare

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

Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -47s

Details

4259b167fd

fdegirmenci commented

2025-05-05 14:41:10 +02:00

@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 ```

fdegirmenci commented

2025-05-05 14:50:17 +02:00

@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

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

nbelouin referenced this issue from a commit

2025-05-06 14:36:46 +02:00

Merge pull request 'Updates for EIB 1.2.0-rc1' (#126) from dbekhit/Factory:main into main

Sign in to join this conversation.

4 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: suse-edge/Factory#126