[fr4] Adjust nova04delta T-O, add DNS and BMO layers#662
Merged
softwarefactory-project-zuul[bot] merged 1 commit intoopenstack-k8s-operators:18.0-fr4from Dec 1, 2025
Merged
Conversation
1) Add DNS config and ctlplane NAD for T-O When T-O uses ctlplane net for metrics scrapping on EDPM hosts, additional Openshift DNS servers must be added to let it resolving FQDN of EDPM hosts on controlplane network. Note that cifmw addresses this differently, however, nova04delta is also used outside of cifmw. Enable metricStorage/dashboard. Also, as documented for T-O, add ctlplane NAD for metrics storage template spec in oscp CR. 2) Simplify bmh kustzomiation and extract BMO into a separate kustomization for allowing it in parallel steps in CI. Create a separate kustomization for BMH and provisioning to make it applicable independently of nodeset. Adjust automation to create CP and BMH in parallel. The stages will now execute as follows: 1. nncp-configuration (networking setup) 2. network-configuration (MetalLB setup) 3. control-plane and baremetalhosts-configuration (run in parallel) 4. edpm-nodeset (depends on both control plane and BMH being ready) 5. edpm-deployment (final deployment) 3) Align nvidia-vfio VA with underlying DT changes 4) nova04/vfio: fix storage class overriding The storage lib component relies on network-values data for setting control plane SC values, historically. The replacement of telemetry data overrides this in the nova04delta and vfio DT/VA. Fixing that by re-applying the storage component logic locally in the DT/VA as well, while repurposing it to control-plane data instead of network-values didn't work in CI (however, it works locally with kustomize build). Fix that by simply providing pvcStorageClass directly in DT/VA. We would not have to fix that, if T-O had been taking the top scope storageClass for its metricStorage template (that fix will be in FR5 likely). Unrelated change: * update gitignore for asdf .tool-versions to simplify local development when kustomize is installed with asdf. CI impact: rework extracted BMO values for baremetalhosts-configuration Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
Contributor
Author
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abays, bogdando The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Contributor
|
Build succeeded (gate pipeline). ✔️ noop SUCCESS in 0s |
softwarefactory-project-zuul
Bot
merged commit 237103c
into
openstack-k8s-operators:18.0-fr4
4 checks passed
rabi
reviewed
Dec 1, 2025
| namespace: openstack | ||
| annotations: | ||
| inspect.metal3.io: _replaced_ | ||
| inspect.metal3.io: "" |
Contributor
There was a problem hiding this comment.
YOu can get rid of this if you don't want inspection to be disabled. No point hardcoding iit.
Contributor
Author
There was a problem hiding this comment.
good to know, thanks! so it would work as default value?
| @@ -0,0 +1,2 @@ | |||
| username=admin | |||
| password=password No newline at end of file | |||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When T-O uses ctlplane net for metrics scrapping on EDPM hosts, additional Openshift DNS servers must be added to let it resolving FQDN of EDPM hosts on controlplane network. Note that cifmw addresses this differently, however, nova04delta is also used outside of cifmw.
Enable metricStorage/dashboard.
Also, as documented for T-O, add ctlplane NAD for metrics storage template spec in oscp CR.
Create a separate kustomization for BMH and provisioning to make it applicable independently of nodeset.
Adjust automation to create CP and BMH in parallel.
The stages will now execute as follows:
Align nvidia-vfio VA with underlying DT changes
nova04/vfio: fix storage class overriding
The storage lib component relies on network-values data for setting control plane SC values, historically. The replacement of telemetry data overrides this in the nova04delta and vfio DT/VA.
Fixing that by re-applying the storage component logic locally in the DT/VA as well, while repurposing it to control-plane data instead of network-values didn't work in CI (however, it works locally with kustomize build).
Fix that by simply providing pvcStorageClass directly in DT/VA.
We would not have to fix that, if T-O had been taking the top scope storageClass for its metricStorage template (that fix will be in FR5 likely).
Unrelated change:
CI impact: rework extracted BMO values for baremetalhosts-configuration
Jira: OSPRH-21650