Skip to content

[multiple] nova05epsilon: adjust Ceph for BM SNO DCN cases#3773

Draft
bogdando wants to merge 2 commits into
openstack-k8s-operators:mainfrom
bogdando:nova05epsilon
Draft

[multiple] nova05epsilon: adjust Ceph for BM SNO DCN cases#3773
bogdando wants to merge 2 commits into
openstack-k8s-operators:mainfrom
bogdando:nova05epsilon

Conversation

@bogdando
Copy link
Copy Markdown
Contributor

@bogdando bogdando commented Mar 17, 2026

  1. Adjust ceph.yml post_stage_run hook for DCN conventions

    The ceph.yml post_stage_run hook (via cifmw_ceph_client role) writes Ceph config files to
    cifmw_ceph_client_fetch_dir (default /tmp/). This template reads those files and provides them as base64-
    encoded values under data.ceph_conf (DCN convention).

  2. Allow overriding ssh and storage_mgmt

    To allow BM SNO with ceph using custom ceph CIDR values,
    make ssh_network_range and storage_mgmt_network_range overridable via
    cifmw_ceph_ssh_network_range and cifmw_ceph_storage_mgmt_network_range.
    Both are set in set_fact which clobbers extra vars, so we use the
    cifmw_ indirection with default() to preserve original defaults.

    NOTE: storage_network_range also needs this treatment.
    It use to be commented out in set_fact, and this change needs
    extra testing with Ceph ci jobs perhaps.

    Also gather network facts for IP-to-host mapping.

  3. Add computes to zuul inventory for reproducer

    Ceph hooks require edpm compute hosts in zuul inventory.
    If explicitely requested by a cifmw job config,
    add those to zuul inventory.

Jira: OSPRH-27641
Generated-by: claude-4.6-opus-high

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 17, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2026

This PR is stale because it has been for over 15 days with no activity.
Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Apr 2, 2026
@github-actions github-actions Bot closed this Apr 9, 2026
@bogdando bogdando reopened this Apr 23, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 23, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign brjackma for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bogdando bogdando changed the title nova05: populate ceph_conf from files by ceph.yml [ci_gen_kustomize_values] nova05epsilon: ceph conf Apr 23, 2026
@bogdando bogdando removed the Stale label Apr 23, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

This PR is stale because it has been for over 15 days with no activity.
Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label May 9, 2026
@bogdando bogdando removed the Stale label May 11, 2026
The ceph.yml post_stage_run hook (via cifmw_ceph_client role) writes
Ceph config files to cifmw_ceph_client_fetch_dir (default /tmp/).
This template reads those files and provides them as base64-encoded
values under data.ceph_conf (DCN convention).

Generated-by: claude-4.6-opus-high
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@bogdando bogdando changed the title [ci_gen_kustomize_values] nova05epsilon: ceph conf [multiple] nova05epsilon: adjust Ceph for BM SNO cases May 14, 2026
@bogdando bogdando changed the title [multiple] nova05epsilon: adjust Ceph for BM SNO cases [multiple] nova05epsilon: adjust Ceph for BM SNO DCN cases May 14, 2026
@bogdando bogdando requested review from danpawlik, fmount and fultonj May 14, 2026 08:27
@bogdando bogdando force-pushed the nova05epsilon branch 3 times, most recently from 84314e0 to f9d684c Compare May 14, 2026 09:32
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/188c70ab54dc4f6bb3ab99792c2ac3c4

openstack-k8s-operators-content-provider NODE_FAILURE Node(set) request 100-0000095747 failed in 0s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ podified-multinode-hci-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 4m 49s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 03s
✔️ cifmw-pod-k8s-snippets-source SUCCESS in 5m 00s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 08s
✔️ cifmw-architecture-validate-hci SUCCESS in 4m 47s
✔️ cifmw-molecule-ci_gen_kustomize_values SUCCESS in 6m 22s
✔️ cifmw-molecule-reproducer SUCCESS in 14m 04s

To allow BM SNO with ceph using custom ceph CIDR values,
make ssh_network_range and storage_mgmt_network_range overridable via
cifmw_ceph_ssh_network_range and cifmw_ceph_storage_mgmt_network_range.
Both are set in set_fact which clobbers extra vars, so we use the
cifmw_ indirection with default() to preserve original defaults.

NOTE: storage_network_range also needs this treatment.
It use to be commented out in set_fact, and this change needs
extra testing with Ceph ci jobs perhaps.

Also gather network facts for IP-to-host mapping.

Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/6347082e77b543c9b6f59af6ea456921

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 50m 26s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 29m 07s
cifmw-crc-podified-edpm-baremetal FAILURE in 39m 01s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 52m 02s
✔️ podified-multinode-hci-deployment-crc SUCCESS in 1h 51m 40s
✔️ cifmw-pod-zuul-files SUCCESS in 6m 13s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 11m 34s
✔️ cifmw-pod-k8s-snippets-source SUCCESS in 7m 25s
✔️ cifmw-pod-pre-commit SUCCESS in 10m 37s
✔️ cifmw-architecture-validate-hci SUCCESS in 5m 43s
✔️ cifmw-molecule-ci_gen_kustomize_values SUCCESS in 11m 49s

@bogdando
Copy link
Copy Markdown
Contributor Author

recheck cifmw-crc-podified-edpm-baremetal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant