Modernize services_master_puppetserver as services_puppetserver#151
Merged
tuxmea merged 4 commits intoOpenVoxProject:masterfrom May 5, 2026
Merged
Conversation
- Rename to services_puppetserver.markdown (remove legacy 'master' from filename) - Update title, branding, and terminology throughout for OpenVox 8.x - Replace all puppet.com external links with internal openvox equivalents - Update supported platforms and Java version (17 or 21, not 1.8/11) - Remove historical 'classic Puppet master' framing (removed in Puppet 6) - Rename 'Puppet API Service' section to 'OpenVox API Service' - Fix CA credentials path to cadir (/etc/puppetlabs/puppetserver/ca/) - Add all three log files (puppetserver.log, -access.log, -status.log) - Simplify User section (OpenVox only, RPM and Debian paths) - Remove masterport puppet.conf reference (no longer relevant) - Fix broken external_ca_configuration link -> intermediate_ca - Update nav label and index.markdown reference Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Michael Harp <mike@mikeharp.com>
- Lead intro with what OpenVox Server does before implementation details - Add systemctl commands alongside service for modern systemd platforms - Remove vague 'from a user's perspective' run environment paragraph - Make Admin API code-deploy use case more concrete - Link ca.conf from CA service section - Link logback.xml directly from Logging section Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Michael Harp <mike@mikeharp.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Michael Harp <mike@mikeharp.com>
miharp
added a commit
to miharp/openvox-docs
that referenced
this pull request
May 5, 2026
- Remove explicit "Index" nav entry (landing page accessed via top nav) - Rename "About Puppet Server" → "About OpenVox Server", pointing to services_puppetserver.html (created in PR OpenVoxProject#151) - Rebrand "Configuring/Using and extending Puppet Server" section headings - Fix broken external_ca_configuration.html link (file does not exist); remove the duplicate nav entry since "Intermediate CA" already covers it - Fix "External SSL germination" typo → "External SSL termination" - Update remaining "Puppet Server" labels to "OpenVox Server" Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Michael Harp <mike@mikeharp.com>
5 tasks
tuxmea
approved these changes
May 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
services_master_puppetserver.markdowntoservices_puppetserver.markdown(removes legacy 'master' from filename, consistent withservices_agent_unix.markdownconvention)puppet.comexternal links with internal openvox equivalentsinstall_pre.html; corrects Java version to 17 or 21cadir(/etc/puppetlabs/puppetserver/ca/) — original incorrectly pointed tossldirpuppetserver.log,puppetserver-access.log,puppetserver-status.log)masterportpuppet.confreferenceexternal_ca_configurationreference link →intermediate_caindex.markdownreferenceSee #150 for follow-on work to modernize the index landing page.
Test plan
services_puppetserver.htmlpresent in_site/, old filename absent