Skip to content

docs: fix and modernize dev_running_from_source.markdown#309

Merged
tuxmea merged 1 commit into
OpenVoxProject:masterfrom
miharp:docs/issue-165-docker-agent
Jun 8, 2026
Merged

docs: fix and modernize dev_running_from_source.markdown#309
tuxmea merged 1 commit into
OpenVoxProject:masterfrom
miharp:docs/issue-165-docker-agent

Conversation

@miharp

@miharp miharp commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

What

Fixes and modernizes dev_running_from_source.markdown. Started as a fix for the broken Docker agent section and expanded into related corrections found while testing the guide end-to-end against a from-source server.

Changes

Docker agent section (the original issue)

  • Connect via --add-host puppet:host-gateway with the default --server puppet, instead of the Docker host IP (which fails TLS hostname validation).
  • Persist the agent SSL dir in a named volume so keys/certs survive re-runs.
  • Autosign by default; document the manual puppetserver-ca sign path and its requirement that the server name resolve on the host.

Quick Start (was "Step 0")

  • Renamed; the remaining steps are 1–4.
  • Replaced the no-op rm -rf ~/.puppetserver with a recoverable, timestamped mv backup.
  • Run the source agent through Bundler (the old RUBYLIB/PATH approach double-loads and breaks when a puppet/openvox gem is installed).
  • Added a pointer to the Step 1 prerequisites.

Prerequisites (Step 1)

  • Added Ruby 3.1+ (needed to run a source agent on the host). Java 17/21 confirmed correct against the source CI matrix; Leiningen version wording fixed.

Step 3 / OpenVoxDB / conventions

  • Enable autosign with puppet config set autosign true --section server.
  • Deprecated jruby-puppet.master-*server-*; [master][server]; routes master:server:.
  • Nest the Step 4c gotchas as ###; "puppet master" → "Puppet Server"; convert terminal bash fences to console per CONTRIBUTING.

Testing

Verified end-to-end against a from-source server: the Docker agent (autosign + docker start re-run), the full Quick Start sequence, both server start methods (REPL (go) and lein run -c), and the installed/source host agents all apply a catalog. Passes markdownlint, jekyll build, and the internal link check.

Not exercised end-to-end: the Running OpenVoxDB section (requires an OpenVoxDB source checkout) and the manual puppetserver-ca sign path (requires the server name to resolve on the host, which the doc now notes).

Closes #165

@miharp miharp marked this pull request as ready for review June 7, 2026 17:48
@miharp miharp requested a review from a team as a code owner June 7, 2026 17:48
@miharp miharp force-pushed the docs/issue-165-docker-agent branch from 055213f to c20bafd Compare June 8, 2026 17:39
Rewrite the Docker agent section to the actual working flow and apply
related corrections found while testing the guide end-to-end against a
from-source server.

- Docker agent: connect via `--add-host puppet:host-gateway` (default
  `--server puppet`), persist a named SSL volume, autosign by default,
  and document the manual `puppetserver-ca` path and its requirement
  that the server name resolve on the host.
- Quick Start: rename from "Step 0"; replace the no-op
  `rm -rf ~/.puppetserver` with a recoverable timestamped `mv` backup;
  run the source agent through Bundler; point to the Step 1 prerequisites.
- Prerequisites: add Ruby 3.1+; reword the stale Leiningen version.
- Step 3: enable autosign with `puppet config set ... --section server`;
  update deprecated `jruby-puppet.master-*` settings to `server-*`.
- OpenVoxDB section: `[master]` -> `[server]`, routes `master:` ->
  `server:`, grammar fix.
- Structure/conventions: nest the Step 4c gotchas as `###`; "puppet
  master" -> "Puppet Server"; convert terminal `bash` fences to `console`
  per CONTRIBUTING.

Signed-off-by: Michael Harp <mike@mikeharp.com>
Assisted-by: Claude Opus 4.8
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@miharp miharp force-pushed the docs/issue-165-docker-agent branch from c20bafd to 4541afd Compare June 8, 2026 17:55
@tuxmea tuxmea merged commit e194d56 into OpenVoxProject:master Jun 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix and modernize dev_running_from_source.markdown (Docker agent section + related corrections)

2 participants