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

Compare commits

base: suse-edge:44a6a19b563c3db8f046757d6cd749a2733fda735bd8ea0e5d9e0dbbc7555f3d
Branches Tags
suse-edge:main suse-edge:3.3 suse-edge:3.2 suse-edge:devel
..
compare: suse-edge:419e8a0814f1f1e90cf50d22c61842744b165616b77de772c6f3344a21dba420
Branches Tags
suse-edge:main suse-edge:3.3 suse-edge:3.2 suse-edge:devel

4 Commits
44a6a19b56 ... 419e8a0814

Author SHA256 Message Date
Marco Chiappero
419e8a0814 Update the URL for the BMO to connect to Ironic
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 14s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in 25m17s
Details 
The BMO should now connect via the provisioningHostname if set or an IP
address. Add a helper that returns the ironic hostname or correctly
formatted IP to define the ironicApiHost variable in the BMO configmap.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
 2025-08-05 16:31:47 +00:00
Marco Chiappero
4a70b1ac21 Include the hostname for SAN in Certificates 
Recently provisioningHostname has been introduced as an alternative way
to configure the IPs to bind and respond to. This however requires that
the Certificates for HTTPS also include a dnsNames section whenver such
value is present.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
 2025-08-05 16:31:47 +00:00
Marco Chiappero
73e69b34f8 Introduce metal3.provisioningIP template and deprecate ironicIP 
So far ironicIP has been part of values.yaml under the global section,
however this is very misleading: this variable is internal to the Ironic
startup scripts and should not be set, moreover it conflicts with
provisioningIP, which is instead a public configuration variable for the
purpose.

This commits thus introduces the following changes:
- removes the creation of IRONIC_IP in the Ironic configmap
- does not yet remove ironicIP from values.yaml to avoid breaking
  forward compatibility
- introduces a utility function to perform input validation while still
  prioritizing ironicIP if present

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
 2025-08-05 16:31:47 +00:00
Marco Chiappero
3cc7dbe1e7 Allow to change the LISTEN_ALL_INTERFACE variable for Ironic 
It should be possible to enable or disable the environment variable
LISTEN_ALL_INTERFACE in the Ironic configmap, as it allows to the way
Ironic binds to socket, especially in combination with the changes
introduced in v29.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
 2025-08-05 16:31:47 +00:00
1 changed files with 1 additions and 1 deletions
Expand all files Collapse all files

2
metal3-chart/charts/ironic/templates/configmap.yaml
View File

@@ -40,7 +40,7 @@ data:
IRONIC_URL_HOSTNAME: {{ .Values.global.provisioningHostname }} IRONIC_URL_HOSTNAME: {{ .Values.global.provisioningHostname }}
{{- end }} {{- end }}
IRONIC_FAST_TRACK: "true" IRONIC_FAST_TRACK: "true"
LISTEN_ALL_INTERFACES: {{ .Values.global.listenOnAll }} LISTEN_ALL_INTERFACES: {{ .Values.listenOnAll }}
{{- if ( .Values.global.enable_tls ) }} {{- if ( .Values.global.enable_tls ) }}
RESTART_CONTAINER_CERTIFICATE_UPDATED: "true" RESTART_CONTAINER_CERTIFICATE_UPDATED: "true"
IRONIC_KERNEL_PARAMS: {{ .Values.global.ironicKernelParams }} tls.enabled=true IRONIC_KERNEL_PARAMS: {{ .Values.global.ironicKernelParams }} tls.enabled=true