Skip to content

Conversation

guettli
Copy link
Collaborator

@guettli guettli commented May 7, 2025

Before using the ssh Rescue Client, ensure the rescue system is active.

This uses ssh not an API. So rate-limit is not an issue.

It is better to double-check that.

Additionally, Events created when rebooting machines will be written to the logs, so that debugging failing e2e is easier. Unfortunately this means ctx needs to be added to a lot of functions.

@github-actions github-actions bot added size/L Denotes a PR that changes 200-800 lines, ignoring generated files. area/code Changes made in the code directory labels May 7, 2025
@guettli guettli requested a review from janiskemper May 7, 2025 12:29
@github-actions github-actions bot added size/M Denotes a PR that changes 50-200 lines, ignoring generated files. and removed size/L Denotes a PR that changes 200-800 lines, ignoring generated files. labels May 7, 2025
janiskemper
janiskemper previously approved these changes May 7, 2025
@github-actions github-actions bot added size/L Denotes a PR that changes 200-800 lines, ignoring generated files. and removed size/M Denotes a PR that changes 50-200 lines, ignoring generated files. labels May 9, 2025
@guettli guettli changed the title 🌱 Before using the ssh Rescue Client, ensure the rescue system is active. 🌱 Before using the ssh Rescue Client, ensure the rescue system is active. Log reboot events May 9, 2025
@guettli guettli requested a review from janiskemper May 9, 2025 15:24
@guettli guettli requested a review from janiskemper May 12, 2025 10:43
@github-actions github-actions bot added size/M Denotes a PR that changes 50-200 lines, ignoring generated files. and removed size/L Denotes a PR that changes 200-800 lines, ignoring generated files. labels May 13, 2025
@guettli guettli requested a review from janiskemper May 13, 2025 12:29
@guettli guettli requested a review from janiskemper May 14, 2025 08:01
janiskemper
janiskemper previously approved these changes May 14, 2025
@guettli
Copy link
Collaborator Author

guettli commented May 14, 2025

Strange error in e2e test:

https://github.com/syself/cluster-api-provider-hetzner/actions/runs/15015378781/job/42192031779?pr=1598

W0514 08:22:37.884145 1 reflector.go:547] k8s.io/[email protected]/tools/cache/reflector.go:232: failed to list *v1.CertificateSigningRequest: Get "https://162.55.157.123:443/apis/certificates.k8s.io/v1/certificatesigningrequests?resourceVersion=871": tls: failed to verify certificate: x509: certificate is valid for 10.96.0.1, 136.243.69.167, 167.235.106.155, not 162.55.157.123

136.243.69.167 is severID=1670788

162.55.157.123 is controlPlaneEndpoint and apiServerEndpoint in KubeadmConfig

@guettli guettli merged commit 893d9f6 into main May 14, 2025
7 checks passed
@guettli guettli deleted the tg/check-hostname-before-pre-provision-command branch May 14, 2025 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/code Changes made in the code directory size/M Denotes a PR that changes 50-200 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants