Issue #7065
closedAnsible playbook examples in Pulpcore docs and Pulp Ansible installer docs differ
Description
Firstly, I was looking at the Ansible installer entry in Pulpcore docs https://docs.pulpproject.org/installation/instructions.html#ansible-installation-recommended with a view to suggesting we add a summary here and a link to the Ansible installer docs, so that the content only needs to be updated once. I think they might be slipping out of date.
Secondly, I noticed that there is a disparity between the example playbooks shown in the docs:
https://docs.pulpproject.org/installation/instructions.html#ansible-installation-recommended
vs
https://pulp-installer.readthedocs.io/en/latest/quickstart/#example-playbook-for-installing-plugins
Which is more up to date?
If you're happy for me to:
- Replace the Ansible installer section in the Pulpcore docs with just a description and redirect to the Pulp Ansible Installer docs
- Update the existing playbook if necessary
Please let me know.
Updated by dkliban@redhat.com over 4 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 76
Updated by fao89 over 4 years ago
pulpcore master: https://docs.pulpproject.org/en/master/nightly/installation/instructions.html#ansible-installation-recommended the main difference is about getting attention to set the secret and password, and I agree we should update it on pulpcore
Updated by mdepaulo@redhat.com over 4 years ago
I think we should use "<< YOUR SECRET HERE >>", as it is our newer convention, and it is more safe. It prevents users from using a simple guessable secret like that in production.
As fao89 showed, the rest of the playbook on the master branch seems to be identical other than this.
Updated by mcorr over 4 years ago
Yep. I wasn't looking at master, just the published docs. I will make that small change!
Updated by pulpbot over 4 years ago
- Status changed from NEW to POST
Added by mcorr over 4 years ago
Updated by mcorr over 4 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulpcore|e0fe2ede9a74a7592f02bada6f3190478cb262a9.
Updated by pulpbot over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Make passwd variable consistent with Ansible docs
fixes #7065