Skip to content

Run ensure_beaker_ip_on before FakeCA collects cert SAN addresses - #285

Merged
silug merged 1 commit into
masterfrom
el10-ip-before-fakeca
Jul 3, 2026
Merged

Run ensure_beaker_ip_on before FakeCA collects cert SAN addresses#285
silug merged 1 commit into
masterfrom
el10-ip-before-fakeca

Conversation

@silug

@silug silug commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #284. run_fake_pki_ca_on embeds the SUTs' live IPs in cert subjectAltNames — but it can run (e.g. from a before(:suite) hook, as pupmod-simp-libreswan does) before the before(:all) hook that applies ensure_beaker_ip_on. On EL10/Vagrant SUTs the certs were then cut from the transient DHCP address; after remediation, IKE peer IDs no longer matched the cert SANs and libreswan 5 (strict ID-vs-SAN matching) rejected auth with AUTHENTICATION_FAILED.

Fix: correct the address inside run_fake_pki_ca_on, right next to the existing activate_interfaces call, before SAN data is collected. Version 3.1.1 + CHANGELOG.

Diagnosed on a live left/right almalinux10 pair: cert SAN said IP:10.254.16.168 (pre-fix DHCP) while the wire IP was the corrected 10.254.188.154.

🤖 Generated with Claude Code

run_fake_pki_ca_on embeds the SUTs' live IP addresses in certificate
subjectAltNames. On SUTs where the static private-network IP was never
applied (EL10 under Vagrant), the certs were cut from the transient DHCP
address; once ensure_beaker_ip_on corrected the interface, IKE peer IDs
no longer matched the cert SANs and strict validators (libreswan 5)
rejected authentication (AUTHENTICATION_FAILED). Correct the address
before collecting SAN data, next to the existing activate_interfaces
call.
@silug
silug merged commit 027ba8c into master Jul 3, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from New to Done in Org Triage Jul 3, 2026
@silug
silug deleted the el10-ip-before-fakeca branch July 3, 2026 20:03
@silug silug removed this from Org Triage Jul 7, 2026
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.

3 participants