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

Upgrade to SLE 15.7 base and bump Ironic to use 2025.1 version #214

Merged
nbelouin merged 4 commits from nbelouin/Factory:15.7-upgrade into main 2025-07-29 11:20:49 +02:00
Conversation 7 Commits 4 Files Changed 19 +23 -33
nbelouin commented 2025-07-24 13:46:36 +02:00
Owner
Copy Link
No description provided.
nbelouin added 2 commits 2025-07-24 13:46:37 +02:00
Update to SLE 15.7 and OpenStack 2025.1 3a98fe8f00 
Signed-off-by: Nicolas Belouin <nicolas.belouin@suse.com>
ironic-image: remove obsolete packages
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 1s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 33m15s
Details
6251d8b670 
Signed-off-by: Nicolas Belouin <nicolas.belouin@suse.com>
nbelouin commented 2025-07-24 13:47:58 +02:00
Author
Owner
Copy Link

Here is an OBS built project for this: https://build.opensuse.org/project/show/home:nbelouin:Factory157
As the meta file is updated, automation cannot reflect that in the automatic OBS project, so you can find a proper one here.

Here is an OBS built project for this: https://build.opensuse.org/project/show/home:nbelouin:Factory157 As the meta file is updated, automation cannot reflect that in the automatic OBS project, so you can find a proper one here.
steven.hardy reviewed 2025-07-25 11:41:11 +02:00
ironic-image/Dockerfile Outdated
@@ -21,3 +21,3 @@
#!ArchExclusiveLine: x86_64
RUN if [ "$(uname -m)" = "x86_64" ];then \
zypper --installroot /installroot --non-interactive install --no-recommends syslinux python311-devel python311 python311-pip python-dracclient python311-sushy-oem-idrac python311-proliantutils python311-sushy python311-pyinotify python3-ironicclient git curl sles-release tar gzip vim gawk dnsmasq dosfstools apache2 apache2-mod_wsgi ipcalc ipmitool iproute2 procps qemu-tools sqlite3 util-linux xorriso tftp ipxe-bootimgs python311-sushy-tools crudini openstack-ironic openstack-ironic-inspector-api; \
zypper --installroot /installroot --non-interactive install --no-recommends syslinux python311-devel python311 python311-pip python311-sushy-oem-idrac python311-proliantutils python311-sushy python311-pyinotify python3-ironicclient git curl sles-release tar gzip vim gawk dnsmasq dosfstools apache2 apache2-mod_wsgi ipcalc ipmitool iproute2 procps qemu-tools sqlite3 util-linux xorriso tftp ipxe-bootimgs python311-sushy-tools crudini openstack-ironic; \
steven.hardy commented 2025-07-25 11:41:11 +02:00
Owner
Copy Link

I think we need to update the BuildTag e.g to 29.0.4.0 ?

I think we need to update the BuildTag e.g to `29.0.4.0` ?
steven.hardy commented 2025-07-25 11:42:08 +02:00
Owner
Copy Link

Also the BuildVersion is still 15.6, which I think is why we see this in the build logs?

Saving image ironic:26.1.2.5 to ironic-15.6.x86_64-3.2.tar
Also the BuildVersion is still 15.6, which I think is why we see this in the build logs? ``` Saving image ironic:26.1.2.5 to ironic-15.6.x86_64-3.2.tar ```
nbelouin commented 2025-07-25 11:47:43 +02:00
Author
Owner
Copy Link

True, regarding build version I think we should just drop these as they don't really make any sense in our usecase

True, regarding build version I think we should just drop these as they don't really make any sense in our usecase
nbelouin marked this conversation as resolved
steven.hardy requested changes 2025-07-25 11:44:06 +02:00
Dismissed
steven.hardy left a comment
Owner
Copy Link

Looks good other than the tag/version comments - I also installed the chart locally and did some minimal testing

Looks good other than the tag/version comments - I also installed the chart locally and did some minimal testing
nbelouin added 1 commit 2025-07-25 12:01:58 +02:00
Removing BuildVersion, bump ironic-image version
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 10s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 26m54s
Details
2adc97e581 
Signed-off-by: Nicolas Belouin <nicolas.belouin@suse.com>
steven.hardy commented 2025-07-25 13:25:49 +02:00
Owner
Copy Link

Looks good, but now since the ironic-image tag changed, we'll need to align metal3-chart too

Looks good, but now since the ironic-image tag changed, we'll need to align [metal3-chart too](https://src.opensuse.org/suse-edge/Factory/src/branch/main/metal3-chart/charts/ironic/values.yaml#L59)
nbelouin added 1 commit 2025-07-25 15:31:20 +02:00
Bump metal3-chart
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in -1s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 48m46s
Details
e340024904 
Signed-off-by: Nicolas Belouin <nicolas.belouin@suse.com>
nbelouin requested review from dprodanov 2025-07-28 10:31:01 +02:00
nbelouin requested review from mchiappero 2025-07-28 10:31:02 +02:00
mchiappero approved these changes 2025-07-28 14:08:06 +02:00
mchiappero left a comment
Owner
Copy Link

LGTM

LGTM
steven.hardy reviewed 2025-07-29 10:47:25 +02:00
metal3-chart/Chart.yaml Outdated
@@ -3,2 +2,3 @@
#!BuildTag: %%CHART_PREFIX%%metal3:%%CHART_MAJOR%%.0.10_up0.11.8-%RELEASE%
apiVersion: v2
appVersion: 0.11.7
appVersion: 0.11.8
steven.hardy commented 2025-07-29 10:47:25 +02:00
Owner
Copy Link

Given this is a major update to the Ironic version, we should probably bump to 0.12.0 ? I'm also thinking of how we'll handle any maintenance updates for 3.3, where I guess it will make sense to continue bumping 0.11.x (particularly in the upstream GH charts consumed by Sylva)

Given this is a major update to the Ironic version, we should probably bump to 0.12.0 ? I'm also thinking of how we'll handle any maintenance updates for 3.3, where I guess it will make sense to continue bumping 0.11.x (particularly in the upstream GH charts consumed by Sylva)
steven.hardy marked this conversation as resolved
nbelouin force-pushed 15.7-upgrade from e340024904 to dc254aa461 2025-07-29 11:02:33 +02:00 Compare
steven.hardy approved these changes 2025-07-29 11:17:24 +02:00
nbelouin merged commit e021cfa92f into main 2025-07-29 11:20:49 +02:00
nbelouin deleted branch 15.7-upgrade 2025-07-29 11:20:49 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
dprodanov
mchiappero
steven.hardy
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
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#214
No description provided.
Delete Branch "nbelouin/Factory:15.7-upgrade"

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?