Skip to content

Fix remove_hosts() function #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main_base
Choose a base branch
from
Open

Conversation

kdevops-kpd[bot]
Copy link

@kdevops-kpd kdevops-kpd bot commented Nov 11, 2024

Pull request for series with
subject: Fix remove_hosts() function
version: 1
url: https://patchwork.kernel.org/project/kdevops/list/?series=908579

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 11, 2024

Upstream branch: 5d9af77
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 12, 2024

Upstream branch: 45dd47e
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 12, 2024

Upstream branch: 11f72ce
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 12, 2024

Upstream branch: 39aeba4
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 18, 2024

Upstream branch: 683610d
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 23, 2024

Upstream branch: 01b0bdd
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 23, 2024

Upstream branch: b9e7570
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 26, 2024

Upstream branch: 1046d42
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Nov 26, 2024

Upstream branch: ad7a161
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Jan 2, 2025

Upstream branch: 775d123
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Mar 21, 2025

Upstream branch: 0e172a8
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 6, 2025

Upstream branch: 3521091
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 21, 2025

Upstream branch: 23c9fca
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 21, 2025

Upstream branch: ef5f294
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 7ce773b
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 1199224
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 4a0beb6
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 22, 2025

Upstream branch: 292c2a5
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 24, 2025

Upstream branch: 76b19c2
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

@kdevops-kpd
Copy link
Author

kdevops-kpd bot commented Apr 24, 2025

Upstream branch: 370657c
series: https://patchwork.kernel.org/project/kdevops/list/?series=908579
version: 1

Commit 3b6fbaaaf1df ("update_ssh_config.py: add IP address on Host
line") added an IP address at the end of each Host line. This extra
field breaks the simple host entry matching logic in the
remove_hosts() function, making "--remove" a no-op.

If the ssh_config entries are not removed, the config file grows
without bounds. If new hostnames are added that have the same name
as existing entries, ssh to those hosts fails, which makes certain
Ansible tasks hang for a bit and then fail.

Fixes: 3b6fbaaaf1df ("update_ssh_config.py: add IP address on Host line")
Signed-off-by: Chuck Lever <[email protected]>
@kdevops-kpd kdevops-kpd bot force-pushed the series/908579=>main branch from 29ce530 to d505607 Compare April 24, 2025 17:32
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