suse-edge/Factory
SHA256
14
0
Fork 15
Code Issues Pull Requests 4 Actions Packages Projects Releases Wiki Activity

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
Conversation 4 Commits 5 Files Changed 3 +11 -8
dbekhit commented 2025-04-18 07:17:48 +02:00
Owner
Copy Link

@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
Details
9548235fde
merge upstream b219a28ae1
changes for EIB 1.2.0-rc0
Some checks failed
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 3h20m57s
Details
954bf7b140
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
Owner
Copy Link

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
Details
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
Details
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
Details
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
Details
4259b167fd
fdegirmenci commented 2025-05-05 14:41:10 +02:00
Owner
Copy Link

@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
Owner
Copy Link

@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
Copy Link

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.
Reviewers
No Reviewers
fdegirmenci
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
amorgante atanasdinov atrendafilov dbekhit dprodanov eminguez fdegirmenci geoagriogiannis ipetrov117 jtomasek kzhelyazkov mchiappero nbelouin roxenham steven.hardy
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#126
No description provided.
Delete Branch ":main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?