Skip to content

Re-run osdeploy initialize -l after /etc/hosts is repaired#57

Merged
viniciusferrao merged 2 commits into
masterfrom
fix-confluent-cert-fqdn
May 26, 2026
Merged

Re-run osdeploy initialize -l after /etc/hosts is repaired#57
viniciusferrao merged 2 commits into
masterfrom
fix-confluent-cert-fqdn

Conversation

@viniciusferrao

Copy link
Copy Markdown
Member

Summary

  • The first osdeploy initialize -l (line 905) runs before confluent2hosts and the hosts repair script have updated /etc/hosts with the cluster FQDN
  • At that point, osdeploy only sees the short hostname, so the SSH host certificate is missing the FQDN in its principals
  • This breaks host-based authentication because the client sends chost <FQDN> but the certificate doesn't include it
  • Add a second osdeploy initialize -l at the end of the Confluent install, after /etc/hosts has the correct FQDN

Confirmed on opencattus-alma10-confluent-rel1-headnode:

  • Before fix: principals = 192.168.222.10, 192.168.232.254, opencattus
  • After re-running -l: principals = 192.168.222.10, 192.168.232.254, opencattus, opencattus.cluster.example.com

Fixes feedback item #5 (certificate part).

Test plan

  • Deploy Confluent cluster and verify ssh-keygen -L -f /etc/ssh/ssh_host_ed25519_key-cert.pub includes the FQDN
  • Verify host-based auth works from headnode to compute nodes

🤖 Generated with Claude Code

viniciusferrao and others added 2 commits May 26, 2026 16:16
The first osdeploy initialize -l runs before confluent2hosts and the
hosts repair script update /etc/hosts with the cluster FQDN. At that
point osdeploy only sees the short hostname, so the SSH host
certificate is signed without the FQDN in its principals — breaking
host-based authentication.

Re-run with -l after /etc/hosts has the correct FQDN entry so the
certificate includes it.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The clang-format-check workflow also runs on the self-hosted runner
and hits the same root-owned file issue from sudo podman builds.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@viniciusferrao viniciusferrao merged commit 4405f89 into master May 26, 2026
2 checks passed
@viniciusferrao viniciusferrao deleted the fix-confluent-cert-fqdn branch May 26, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant