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

Compare commits

base: suse-edge:419e8a0814f1f1e90cf50d22c61842744b165616b77de772c6f3344a21dba420
Branches Tags
suse-edge:main suse-edge:3.3 suse-edge:3.2 suse-edge:devel
..
compare: suse-edge:818a739343d9cf31e02b1206ea22fd6dae92aa49db0933a9dd26f369cc5bd98f
Branches Tags
suse-edge:main suse-edge:3.3 suse-edge:3.2 suse-edge:devel

7 Commits
419e8a0814 ... 818a739343

Author SHA256 Message Date
Marco Chiappero
818a739343 Update the URL for the BMO to connect to Ironic
Some checks failed
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) Failing after 3h10m43s
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 15:11:02 +00:00
Marco Chiappero
545dece0a6 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 15:11:00 +00:00
Marco Chiappero
3ce36b9df7 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 15:10:30 +00:00
Marco Chiappero
20215df6b3 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 15:10:30 +00:00
Marco Chiappero
02f460f969 Introduce the provisioningHostname env variable in Ironic 
Create a new provisioningHostname value in values.yaml in order to set
the new IRONIC_URL_HOSTNAME, that allows to set the address(es) Ironic
will bind to.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
 2025-08-05 15:10:30 +00:00
Marco Chiappero
f4ca20b40d Remove unused env and helm variables 
Since currently we can only define the provisioning network and the
external HTTP host, remove some clutter generating unused variables.

Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
 2025-08-05 15:10:29 +00:00
Marco Chiappero
52cb04d1f5 Allow control over IRONIC_EXTERNAL_HTTP_URL via values.yaml 
The purpose of this commit is to:
- avoid providing IRONIC_EXTERNAL_HTTP_URL by default, as the Ironic
  startup scripts will be able to derive the value from other variables
- define a new global value under the top values.yaml to define
  IRONIC_EXTERNAL_HTTP_URL when actually needed
- make sure that the input, which can either be a hostname or an IP
  address, is correctly formatted in case of an IPv6.

This change also allows subsequent cleanups of the whole Configmap
template for Ironic.

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

2
metal3-chart/charts/baremetal-operator/templates/_helpers.tpl
View File

@@ -69,7 +69,7 @@ Create the URL to use for connecting to the Ironic servers (e.g. API, cache)
{{- $ironicIP := include "metal3.provisioningIP" . -}}
{{- with .Values.global }}
{{- if .provisioningHostname }}
{{- .provisioningHostname }}
{{- .provisioningHostname
{{- else if regexMatch ".*:.*" $ironicIP}}
{{- print "[" $ironicIP "]" }}
{{- else }}