WIP: Align config files and scripts with upstream (v30.0.0) #202

Closed
mchiappero wants to merge 14 commits from mchiappero/Factory:alignment-v30.0.0 into main
Owner

Copy the configuration files and scripts from upstream to align with v30.0.0. Leave any SUSE specific customization to separate commits that could be re-applied in the future.

This PR depends on #201.

Copy the configuration files and scripts from upstream to align with v30.0.0. Leave any SUSE specific customization to separate commits that could be re-applied in the future. This PR depends on #201.
mchiappero added 15 commits 2025-06-30 18:14:43 +02:00
Drop inotify-tools and switch to pyinotify
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 22s
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 20s
09c8fb927a
No longer inotifywait use and move to python pyinotify.

See https://github.com/metal3-io/ironic-image/issues/605 for
more details.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Restore the upstream directory structure
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 14s
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -1m15s
b3a46f16f1
It is now possible to bring back the original directory structure for
config (/ironic-config) files and scripts (/scripts). This will make
updates to re-align with upstream easier.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Having a completely different image and build system the Dockerfile will
always differ quite significantly from the upstream one. Nonetheless,
it's still useful to make the commont parts to look alike as much as
possible to ease maintanance and update.

Note, this is just a small set of changes that are possible, more effort
into this direction may follow at later stage.
Two entries pointing to old and unesed files for non UEFI servers seem
to be unused both here and upstream, so this commit removes them.
This commit move the installation of packages required for EFI booting
from Dockerfile to prepare-efi.sh, in order to align with upstream.

Also, remove some spaces between some commands in the script to match
upstream.
Move the copy of Jinja templates to resemble upstream
Some checks failed
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 13s
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 1m12s
fec5b28494
Follow the same location for the COPY command moving the Jinja template
files in the image.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Drop all the config and script files into their respective directory
from the upstream release v30.0.0. Any required customization will
follow in subsequent commits, which could potentially be reused for
future realignments.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
update the destination path for jinjia templates
Some checks failed
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 31s
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 3h15m0s
4a9e341f07
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
mchiappero changed title from Align config files and scripts with upstream to WIP: Align config files and scripts with upstream 2025-06-30 18:15:30 +02:00
mchiappero requested review from amorgante 2025-06-30 18:19:07 +02:00
mchiappero force-pushed alignment-v30.0.0 from 4a9e341f07 to 51804fea1d 2025-07-02 10:28:15 +02:00 Compare
mchiappero requested review from dprodanov 2025-07-02 10:30:37 +02:00
mchiappero requested review from nbelouin 2025-07-02 10:30:37 +02:00
mchiappero requested review from steven.hardy 2025-07-02 10:30:37 +02:00
mchiappero force-pushed alignment-v30.0.0 from 51804fea1d to 4bc7e416a8 2025-07-02 15:50:05 +02:00 Compare
mchiappero force-pushed alignment-v30.0.0 from 4bc7e416a8 to 318765a043 2025-07-03 14:50:43 +02:00 Compare
steven.hardy reviewed 2025-07-04 15:24:45 +02:00
Owner

As mentioned in #200 - we can't remove this as it's required to allow Ironic to accept the nmstate network-data format instead of enforcing the openstack schema

We should perhaps push a change to the upstream ironic-image so this can be enabled/disabled via a variable as our use-case isn't unique and the idea of disabling the schema was previously discussed upstream so I expect such a change may be acceptable

As mentioned in #200 - we can't remove this as it's required to allow Ironic to accept the nmstate network-data format instead of enforcing the openstack schema We should perhaps push a change to the upstream ironic-image so this can be enabled/disabled via a variable as our use-case isn't unique and the idea of disabling the schema was previously [discussed upstream]([url](https://kubernetes.slack.com/archives/CHD49TLE7/p1700735069775069?thread_ts=1700663721.985989&cid=CHD49TLE7)) so I expect such a change may be acceptable
steven.hardy reviewed 2025-07-04 15:40:23 +02:00
@@ -0,0 +3,4 @@
# Ramdisk logs path
LOG_DIR="/shared/log/ironic/deploy"
mkdir -p "${LOG_DIR}"
Owner

This may reintroduce the spurious restart I fixed in https://build.opensuse.org/package/rdiff/isv:SUSE:Edge:Metal3:Ironic:2024.2/ironic-image?linkrev=base&rev=10 - at the time I couldn't reproduce the same issue with the upstream environment but it would probably be reasonable to push the change upstream anyway?

This may reintroduce the spurious restart I fixed in https://build.opensuse.org/package/rdiff/isv:SUSE:Edge:Metal3:Ironic:2024.2/ironic-image?linkrev=base&rev=10 - at the time I couldn't reproduce the same issue with the upstream environment but it would probably be reasonable to push the change upstream anyway?
steven.hardy reviewed 2025-07-04 15:46:40 +02:00
@@ -0,0 +12,4 @@
<Directory ~ "/shared/html">
Order deny,allow
deny from all
Owner

This reverts the fix I made for https://gitlab.com/sylva-projects/sylva-core/-/issues/2147 but I think it's probably OK since the uefi bootloader is now access via file:// instead of the vmedia webserver

This reverts the fix I made for https://gitlab.com/sylva-projects/sylva-core/-/issues/2147 but I think it's probably OK since the uefi bootloader is now access via `file://` instead of the vmedia webserver
mchiappero added 1 commit 2025-07-04 17:13:48 +02:00
Disable the network schema check to allow for nmstate definitions
Some checks failed
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in -1m46s
Build PR in OBS / Build PR in OBS (pull_request_target) Failing after 3h23m7s
a9551f2fcb
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>
mchiappero force-pushed alignment-v30.0.0 from a9551f2fcb to 0efa970293 2025-07-04 17:24:19 +02:00 Compare
mchiappero changed title from WIP: Align config files and scripts with upstream to WIP: Align config files and scripts with upstream (v30.0.0) 2025-07-08 13:07:15 +02:00
Author
Owner

I'm going to drop this PR since in the meantime upstream has move forward with newer releases.

I'm going to drop this PR since in the meantime upstream has move forward with newer releases.
mchiappero closed this pull request 2025-09-05 13:28:16 +02:00
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 10s
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 12s

Pull request closed

Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

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