Skip to content

store the CA private key unencrypted#674

Open
bochi wants to merge 1 commit into
theforeman:masterfrom
bochi:drop-ca-encryption
Open

store the CA private key unencrypted#674
bochi wants to merge 1 commit into
theforeman:masterfrom
bochi:drop-ca-encryption

Conversation

@bochi

@bochi bochi commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Why are you introducing these changes? (Problem description, related links)

The CA private key is stored encrypted, but that password is in ca.pwd next to the key, and Candlepin is handed a decrypted copy. So there is no real benefit.

Followup to #602

What are the changes introduced in this pull request?

  • Store key in an unencrypted way
  • Remove ca_key_password and candlepin_ca_key_password variables
  • migrate existing deployments in place

How to test this pull request

Steps to reproduce:

  • on initial deployment:
    -> check that the ca.key is unencrypted
  • on upgrade
    -> deploy, see that key is NOW unencrypted

Checklist

  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)

Comment thread src/roles/candlepin/tasks/certs.yml Outdated
Comment thread tests/feature/katello/candlepin_test.py Outdated
@bochi
bochi force-pushed the drop-ca-encryption branch from b107022 to 41ab82d Compare July 17, 2026 17:09
@bochi
bochi force-pushed the drop-ca-encryption branch from 41ab82d to 3c06c35 Compare July 17, 2026 17:11
@ehelms

ehelms commented Jul 17, 2026

Copy link
Copy Markdown
Member

Valid test failure for backup tests:

FAILED tests/backup_test.py::test_foremanctl_state_archive_contents - AssertionError: Archive should contain certificates-ca-password (generated during deployment)

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.

2 participants