Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Minor tweaks to docs: #304

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rutson
Copy link

@rutson rutson commented Dec 9, 2020

  • facts.pp - removal of referal to historic use of facter command in setup section
  • control_repository.pp - addition of -p to mkdir command issed on existing direction to prevent distracting error
  • control_repository.pp - update to reflect UI change in PE
  • troubleshooting.md - correction of 404'ing URL

* facts.pp - removal of referal to historic use of `facter` command in setup section
* control_repository.pp - addition of `-p` to `mkdir` command issed on existing direction to prevent distracting error
* control_repository.pp - update to reflect UI change in PE
* troubleshooting.md - correction of 404'ing URL
@@ -417,7 +417,7 @@ use to authenticate itself with Gitea.

Begin by creating a new `ssh` subdirectory in `/etc/puppetlabs/puppetserver`:

mkdir /etc/puppetlabs/puppetserver/ssh
mkdir -p /etc/puppetlabs/puppetserver/ssh
Copy link
Contributor

Choose a reason for hiding this comment

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

@rutson Does the command fail without the -p option? Can you provide some more details here?

Copy link
Author

Choose a reason for hiding this comment

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

It's a soft error, but it made me pause and double check.
image

@CLAassistant
Copy link

CLAassistant commented Feb 24, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants