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

WIP: chart-ironic_v29.0.0 #210

Closed
mchiappero wants to merge 15 commits from mchiappero/Factory:chart-ironic_v29.0.0 into main
pull from: mchiappero/Factory:chart-ironic_v29.0.0
merge into: suse-edge:main
Conversation 4 Commits 15 Files Changed 22 +296 -185
mchiappero commented 2025-07-14 13:59:57 +02:00
Owner
Copy Link
No description provided.
mchiappero added 15 commits 2025-07-14 13:59:57 +02:00
Align to v29.0.0 6c6544a608 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Use correct paths and modules for Apache df4fce4388 
Correct the path of the Apache modules for a SUSE image.

Also keep a couple of modules disabled.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Use ironic-suse user/group in Apache for media cc08766fdf 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Use ironic-suse user/group in Apache for API dc68a1d44b 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Change GRUB path in ironic.conf 37a92b02c4 
Correct path for grub.cfg on a SUSE system.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Use arch named IPA file in IMAGE_CACHE_PREFIX db28299e63 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Use arch specific file for IPA 728354e926 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Use arch specific ESP img 589be0a21f 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Disable the network schema check to allow for nmstate definitions 909aa27611 
Bypass the OpenStack network-data format validation, to allow for the
nmstate based one we instead use (which would otherwise fail).

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Update the destination path of Jinjia templates 7da89fbe2b 
Previously .j2 files used to be copied to /etc before being
instantiated. In order to make the image potentially read only,
move the templates to /tmp.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Force the use of Python 3.11 58f9552a26 
SLE 15.6 provides Python 3.11, make sure it's enforced.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Remove unused prometheus exporter 16d2670d54 
The Prometheus exporter is effectively, not only unused, but
unusable, due to missing dependencies. Since currently we
don't have use case for it, opt for dropping the exporter
entirely from the image.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Make the new scripts executable 5b8cbc57e1 
v29.0.0 add a couple of new scripts, such as ironic-probe.sh; make sure
they have the 'executable' flag.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Align configure-nonroot.sh
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 8s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 4m3s
Details
05aa1341a9 
Try to reuse as much as possible of the upstream configure-nonroot.sh

Co-authored-by: Nicolas Belouin <nicolas.belouin@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Bump ironic-image and metal3-chart
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 17s
Details
e30fc2dc3e 
Ironic image has been updated to v29.0.0, bump the version on the Helm
Charts too.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
mchiappero commented 2025-07-14 14:01:54 +02:00
Author
Owner
Copy Link

@nbelouin How should we change the appVersion and buildtag here, with a major bump? Thanks

@nbelouin How should we change the appVersion and buildtag here, with a major bump? Thanks
nbelouin commented 2025-07-15 08:30:26 +02:00
Owner
Copy Link

@mchiappero first we change the Ironic image build tag to be 29.0.0.0 (and related labels), then for the chart, I believe we should stay with a "patch" bump, and only do the "minor" one when we upgrade the BMO (also planned for 3.4 timeframe)

@mchiappero first we change the Ironic image build tag to be `29.0.0.0` (and related labels), then for the chart, I believe we should stay with a "patch" bump, and only do the "minor" one when we upgrade the BMO (also planned for 3.4 timeframe)
👍 1
nbelouin commented 2025-07-28 11:36:53 +02:00
Owner
Copy Link

I update the versions in #214, so this one will probably not be needed.

I update the versions in #214, so this one will probably not be needed.
mchiappero commented 2025-07-28 14:04:21 +02:00
Author
Owner
Copy Link

Already included in #214, closing here.

Already included in #214, closing here.
mchiappero closed this pull request 2025-07-28 14:04:38 +02:00
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 17s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#210
No description provided.
Delete Branch "mchiappero/Factory:chart-ironic_v29.0.0"

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?