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

Compare commits

base: suse-edge:818a739343d9cf31e02b1206ea22fd6dae92aa49db0933a9dd26f369cc5bd98f
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

7 Commits
818a739343 ... 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
Marco Chiappero
83503bddb3 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 16:31:47 +00:00
Marco Chiappero
c2adfe1589 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 16:31:47 +00:00
Marco Chiappero
0571a17c87 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 generate
  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 16:31:36 +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 }}