Skip to content

Pulp tls update #1743

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 15 commits into
base: stackhpc/2024.1
Choose a base branch
from
Open

Pulp tls update #1743

wants to merge 15 commits into from

Conversation

jackhodgkiss
Copy link
Contributor

No description provided.

technowhizz and others added 8 commits December 13, 2024 10:35
Add playbooks, config & docs for enabling pulp tls with vault
Dont start bifrost playbooks when deploying pulp tls

Co-authored-by: Matt Crees <[email protected]>
The playbook `copy-ca-to-hosts` has been refactored in a couple ways.
Firstly, the tasks for installing in either `RHEL` or `Debian` based
systems are placed in `blocks`. Secondly both the root and intermediate
certificate authority have added here to ensure the full chain is
available if required.
@jackhodgkiss jackhodgkiss requested a review from a team as a code owner July 7, 2025 14:12
@product-auto-label product-auto-label bot added size: m ansible Ansible playbooks documentation Improvements or additions to documentation labels Jul 7, 2025
This is not required as the pulp certificate can be verified with just
`OS-TLS-INT` as the intermediate is provided by the `Pulp` server. Also
the `OS-TLS-INT.pem` includes the private key.
@@ -30,7 +30,7 @@ docker_registry: "{{ stackhpc_docker_registry }}"
docker_registry_insecure: "{{ 'https' not in stackhpc_repo_mirror_url }}"

# CA of docker registry
#docker_registry_ca:
docker_registry_ca: "{{ kayobe_env_config_path ~ '/openbao/OS-TLS-INT.crt' if pulp_enable_tls | bool else '' }}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets not default to Bao in this instance. We still have many more people using vault on Caracal

@jackhodgkiss jackhodgkiss changed the base branch from update-vault-docs-dec-2024 to stackhpc/2024.1 July 24, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ansible Ansible playbooks documentation Improvements or additions to documentation size: l
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants