Skip to content

Conversation

@holser
Copy link
Contributor

@holser holser commented Jan 28, 2026

(multiple) Fix SSH cleanup

Update clean_layout.yml to use cifmw_target_host variable for SSH
jumper cleanup instead of hardcoding localhost. This ensures cleanup
runs on the correct host in environments where the target host differs
from localhost.

Add sshd reload handler and notify it during SSH config cleanup tasks.

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com
Signed-off-by: Sergii Golovatiuk sgolovat@redhat.com

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 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 valkyrie00 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

@softwarefactory-project-zuul
Copy link

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/5e5002f40b0c401ca0e2eaf16ad224a4

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 05m 54s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 26m 06s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 49m 23s
cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 48m 06s
✔️ cifmw-pod-zuul-files SUCCESS in 5m 01s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 13s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 08s
cifmw-molecule-libvirt_manager FAILURE in 16m 36s
✔️ cifmw-molecule-reproducer SUCCESS in 15m 43s

@holser holser force-pushed the fix_ssh_cleanup branch 3 times, most recently from b612114 to 55b2911 Compare January 29, 2026 09:10
@softwarefactory-project-zuul
Copy link

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/f4a63b8366f742e6ad5dadd7874bc862

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 28m 25s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 22m 39s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 47m 44s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 12m 23s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 21s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 36s
cifmw-pod-pre-commit TIMED_OUT in 30m 56s
✔️ cifmw-molecule-libvirt_manager SUCCESS in 41m 13s
✔️ cifmw-molecule-reproducer SUCCESS in 15m 04s
✔️ cifmw-molecule-ssh_jumper SUCCESS in 4m 33s

@holser
Copy link
Contributor Author

holser commented Jan 29, 2026

recheck

Update clean_layout.yml to use cifmw_target_host variable for SSH
jumper cleanup instead of hardcoding localhost. This ensures cleanup
runs on the correct host in environments where the target host differs
from localhost.

Add sshd reload handler and notify it during SSH config cleanup tasks.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Sergii Golovatiuk <sgolovat@redhat.com>
@holser holser changed the title [libvirt_manager] Fix SSH cleanup to respect target host configuration (multiple) Fix SSH cleanup to respect target host configuration Jan 29, 2026
@holser holser changed the title (multiple) Fix SSH cleanup to respect target host configuration (multiple) Fix SSH cleanup Jan 29, 2026
@softwarefactory-project-zuul
Copy link

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/acf9d2b867734ab1a1cc384a7eb48683

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 58m 49s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 25m 26s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 41m 15s
cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 43m 54s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 22s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 27s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 05s
✔️ cifmw-molecule-libvirt_manager SUCCESS in 40m 08s
✔️ cifmw-molecule-reproducer SUCCESS in 15m 54s
✔️ cifmw-molecule-ssh_jumper SUCCESS in 4m 12s

@holser
Copy link
Contributor Author

holser commented Jan 30, 2026

recheck

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