Skip to content

Commit ad1fead

Browse files
committed
docs: add suggested changes
- Clean up name of tasks - Add OpenBao assertion - Update release note Signed-off-by: Jack Hodgkiss <jack@stackhpc.com>
1 parent 8ad0f8a commit ad1fead

File tree

3 files changed

+12
-4
lines changed

3 files changed

+12
-4
lines changed

doc/source/configuration/openbao.rst

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -466,6 +466,7 @@ Pulp TLS
466466
.. warning::
467467

468468
For clouds in production consider the impact of enabling TLS on specific hosts as Docker daemon will be restarted and this will disrupt deployments of Ceph Reef and older.
469+
As Vault is deprecated and will be removed in future releases this process only works for OpenBao
469470

470471
To enable TLS for Pulp we first need to generate the certificates and the proceed to configure all hosts that use Pulp to add the root CA to their truststore.
471472

@@ -481,9 +482,10 @@ To enable TLS for Pulp we first need to generate the certificates and the procee
481482
482483
kayobe playbook run $KAYOBE_CONFIG_PATH/ansible/deployment/copy-ca-to-hosts.yml
483484
484-
3. Enable TLS for Pulp in pulp.yml
485+
3. Enable TLS for Pulp
485486

486487
.. code-block::
488+
:caption: $KAYOBE_CONFIG_PATH/pulp.yml
487489
488490
# Whether to enable TLS for Pulp.
489491
pulp_enable_tls: true

etc/kayobe/ansible/pulp/pulp-generate-certificate.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,18 @@
66
openbao_api_addr: http://127.0.0.1:8200
77
openbao_intermediate_ca_name: OS-TLS-INT
88
tasks:
9+
- name: Assert that stackhpc_ca_secret_store is 'openbao'
10+
ansible.builtin.assert:
11+
that:
12+
- stackhpc_ca_secret_store == "openbao"
13+
fail_msg: "stackhpc_ca_secret_store must be 'openbao'"
14+
915
- name: Include OpenBao keys
1016
ansible.builtin.include_vars:
1117
file: "{{ kayobe_env_config_path }}/openbao/seed-openbao-keys.json"
1218
name: openbao_keys
1319

14-
- name: Issue a certificate Pulp
20+
- name: Issue Pulp certificate
1521
hashivault_pki_cert_issue: # noqa: fqcn
1622
url: "{{ openbao_api_addr }}"
1723
ca_cert: "{{ '/etc/pki/tls/certs/ca-bundle.crt' if ansible_facts.os_family == 'RedHat' else '/usr/local/share/ca-certificates/OS-TLS-ROOT.crt' }}"
@@ -23,7 +29,7 @@
2329
ip_sans: "{{ admin_oc_net_name | net_ip(inventory_hostname=groups['seed'][0]) }}"
2430
register: pulp_certificate
2531

26-
- name: Ensure pulp certificates directory exists
32+
- name: Ensure Pulp certificates directory exists
2733
ansible.builtin.file:
2834
path: "{{ kayobe_env_config_path }}/pulp/certificates"
2935
state: directory

releasenotes/notes/pulp-tls-105e47f0da602a25.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22
features:
33
- |
44
Add playbooks and configuration to enable the easy deployment of Pulp with
5-
TLS support.
5+
TLS support in combination with certificates generated via OpenBao.

0 commit comments

Comments
 (0)