Add edpm_ssh_info role for retrieving EDPM SSH connectivity information by rdiazcam · Pull Request #3650 · openstack-k8s-operators/ci-framework

rdiazcam · 2026-01-31T12:34:17Z

This role queries OpenShift for OpenStackDataPlaneNodeSet resources, retrieves SSH credentials, and returns node information for external data plane management.

Assisted-By: Claude Sonnet 4.5 noreply@anthropic.com

openshift-ci · 2026-01-31T12:34:23Z

[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 danpawlik 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:

OWNERS

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

softwarefactory-project-zuul · 2026-01-31T14:47:26Z

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/9b7d9eb816dd47a0ba9b6c2bc66af485

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 12m 05s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 16m 34s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 40m 30s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 58m 35s
❌ cifmw-pod-zuul-files FAILURE in 4m 26s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 7m 51s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 59s

softwarefactory-project-zuul · 2026-02-01T12:16:00Z

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f0648324700d48078817b57caaffd192

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 10m 34s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 18m 55s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 37m 44s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 56m 43s
❌ cifmw-pod-zuul-files FAILURE in 4m 34s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 7m 56s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 57s

roles/edpm_ssh_info/tasks/main.yml

danpawlik

please run make role_molecule . Thanks

rdiazcam · 2026-02-02T15:59:23Z

please run make role_molecule . Thanks

done, thanks

softwarefactory-project-zuul · 2026-02-02T17:34:38Z

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ea9330c8d985419b9f0dfb1e29d8476c

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 57m 13s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 17m 43s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 41m 09s
❌ cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 1h 43m 28s
❌ cifmw-pod-zuul-files FAILURE in 4m 57s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 23s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 42s
✔️ cifmw-molecule-edpm_ssh_info SUCCESS in 1m 34s
✔️ cifmw-molecule-os_flavor SUCCESS in 1m 58s

danpawlik · 2026-02-03T07:12:20Z

recheck

danpawlik

please remove role os_flavor and execute again make role_molecule

evallesp

I see there's no molecule tests but I also see that might be hard to set.
LGTM in general.

roles/edpm_ssh_info/defaults/main.yml

roles/edpm_ssh_info/tasks/main.yml

roles/edpm_ssh_info/README.md

michburk

One request otherwise lgtm.

Also just want to check, have you tested this, and is everything working as expected?

Thanks!

roles/edpm_ssh_info/tasks/main.yml

This role queries OpenShift for OpenStackDataPlaneNodeSet resources, retrieves SSH credentials, and returns node information for external data plane management (SSH connectivity). Assisted-By: Claude Sonnet 4.5 <noreply@anthropic.com> Signed-off-by: Ricardo Diaz <rdiazcam@redhat.com>

danpawlik · 2026-02-16T12:32:23Z

roles/edpm_ssh_info/tasks/main.yml

+  ansible.builtin.set_fact:
+    cifmw_edpm_ssh_info:
+      ssh_key_path: "{{ cifmw_edpm_ssh_info_ssh_key_path }}"
+      nodes: >-


nit: consider to change variable name to something different, that would be helpful to recognize place, where it has been set.
In other words, other people might think that the nodes var comes from CI (Zuul), but we set the fact here. More, the var is common, so changing var to other name might be helpful in future when debug is needed.

danpawlik

One nit, that might be helpful to save some time on debug.

rdiazcam force-pushed the edpm_ssh_info branch 2 times, most recently from b950629 to e316345 Compare February 1, 2026 10:04

danpawlik reviewed Feb 2, 2026

View reviewed changes

roles/edpm_ssh_info/tasks/main.yml Outdated Show resolved Hide resolved

danpawlik reviewed Feb 2, 2026

View reviewed changes

roles/edpm_ssh_info/tasks/main.yml Outdated Show resolved Hide resolved

danpawlik reviewed Feb 2, 2026

View reviewed changes

rdiazcam force-pushed the edpm_ssh_info branch from e316345 to 172626d Compare February 2, 2026 15:36

danpawlik previously approved these changes Feb 3, 2026

View reviewed changes

danpawlik requested a review from a team February 3, 2026 07:12

danpawlik reviewed Feb 3, 2026

View reviewed changes

rdiazcam dismissed danpawlik’s stale review via 1ac8102 February 3, 2026 08:16

rdiazcam force-pushed the edpm_ssh_info branch from 172626d to 1ac8102 Compare February 3, 2026 08:16

github-actions bot added the Ready For Review label Feb 3, 2026

evallesp reviewed Feb 6, 2026

View reviewed changes

rdiazcam force-pushed the edpm_ssh_info branch from 1ac8102 to 6139153 Compare February 6, 2026 13:20

github-actions bot added Ready For Review and removed Ready For Review labels Feb 6, 2026

rdiazcam force-pushed the edpm_ssh_info branch from 6139153 to 691543e Compare February 6, 2026 19:46

github-actions bot added Ready For Review and removed Ready For Review labels Feb 6, 2026

rdiazcam force-pushed the edpm_ssh_info branch from 691543e to ad9fb5e Compare February 7, 2026 08:46

github-actions bot removed the Ready For Review label Feb 7, 2026

rdiazcam force-pushed the edpm_ssh_info branch from ad9fb5e to e6a99f0 Compare February 7, 2026 09:30

github-actions bot added the Ready For Review label Feb 7, 2026

evallesp previously approved these changes Feb 9, 2026

View reviewed changes

michburk reviewed Feb 12, 2026

View reviewed changes

roles/edpm_ssh_info/tasks/main.yml Outdated Show resolved Hide resolved

rdiazcam dismissed evallesp’s stale review via e2fbd98 February 16, 2026 08:00

rdiazcam force-pushed the edpm_ssh_info branch from e6a99f0 to e2fbd98 Compare February 16, 2026 08:00

github-actions bot added Ready For Review and removed Ready For Review labels Feb 16, 2026

danpawlik reviewed Feb 16, 2026

View reviewed changes

danpawlik approved these changes Feb 16, 2026

View reviewed changes

Conversation

rdiazcam commented Jan 31, 2026

Uh oh!

openshift-ci bot commented Jan 31, 2026

Uh oh!

softwarefactory-project-zuul bot commented Jan 31, 2026

Uh oh!

softwarefactory-project-zuul bot commented Feb 1, 2026

Uh oh!

Uh oh!

Uh oh!

danpawlik left a comment

Choose a reason for hiding this comment

Uh oh!

rdiazcam commented Feb 2, 2026

Uh oh!

softwarefactory-project-zuul bot commented Feb 2, 2026

Uh oh!

danpawlik commented Feb 3, 2026

Uh oh!

danpawlik left a comment

Choose a reason for hiding this comment

Uh oh!

evallesp left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

michburk left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

danpawlik Feb 16, 2026

Choose a reason for hiding this comment

Uh oh!

danpawlik left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants