Repository navigation
Support and default to OpenVox 9 - #178
Merged
Merged
Conversation
This was referenced Oct 23, 2026
silug
force-pushed
the
openvox9-server
branch
from
October 23, 2026 16:50
457277b to
fa3b899
Compare
OpenVox 9 no longer falls back to `puppet` when `server` is unset, so the `role` plan now sets `server` in the `main` section of `puppet.conf` on every target right after the agent is installed. It defaults to the FQDN of `server_target` and can be overridden with the new `openvox_server` plan parameter. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Now that the openvox 9 gem is released, replace the "OpenVox 8 + Ruby 4.0" preview entry in the spec matrix with OpenVox 9 on Ruby 4.0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
silug
force-pushed
the
openvox9-server
branch
from
October 23, 2026 16:55
fa3b899 to
406a460
Compare
Choria before 0.30.3 finds Puppet's ssldir by running `puppet apply --configprint ssldir`. OpenVox 9 removed `--configprint`, so choria-server and choria-broker exit at startup. Setting `plugin.choria.ssldir` stops Choria from asking Puppet. The fix (choria-io/go-choria#2362) is in v0.30.3, but the newest package in the Choria yum repo is still 0.30.2. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Change the default OPENVOX_RELEASE from 8 to 9 in the Vagrantfile and the serverspec tests, and update the README. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
server setting for OpenVox 9
silug
marked this pull request as ready for review
October 23, 2026 17:57
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
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.
This makes the Vagrant environment work with OpenVox 9 and makes OpenVox 9 the default.
Changes
serversetting. OpenVox 9 no longer falls back topuppetwhenserveris unset (9.0.0 release notes). Theroleplan now runspuppet config set server <server> --section mainon every target right afterapply_prep, before any agent run. A new optionalopenvox_serverplan parameter defaults to the FQDN ofserver_target(puppet.vagrant).REFERENCE.mdis updated to match.ssldir. Choria before 0.30.3 runspuppet apply --configprint ssldir, and OpenVox 9 removed--configprint(Error: Could not parse application options: invalid option: --configprint). As a result,choria-serverandchoria-brokerexit at startup and keep restarting.data/common.yamlnow setsplugin.choria.ssldir(viachoria::server_config, which is deep-merged) andchoria::broker::ssldirto/etc/puppetlabs/puppet/ssl, so Choria never asks Puppet. The upstream fix (openvox: replace deprecatedpuppet applyto get config values choria-io/go-choria#2362) is in v0.30.3, but the newest package in the Choria yum repo is still 0.30.2. The workaround can be removed once 0.30.3 is packaged.OPENVOX_RELEASEnow defaults to9in theVagrantfileandspec/puppet/sample_spec.rb, and the README table is updated.openvox ~> 9, Ruby 4.0). Both modules' specs pass locally with openvox 9.2.1.Notes
choria enrolluses Choria's ownpuppetserver_hostsetting, which still resolves via thepuppethost alias, so theserverchange doesn't affect it.reportsfromstoretonone. Nothing here depends on stored YAML reports, so I left that alone.choria-servercan't connect. That isn't specific to OpenVox 9 (OpenVox 8 has it too) and is fixed separately in Open the Choria broker port on the server #184.Testing
vagrant up(libvirt,generic/centos9s) withOPENVOX_RELEASE=9, fromproductionwith this branch merged:puppet config print serverreturnspuppet.vagranton both nodes.choria-brokerandchoria-serverare active on both nodes. Second agent runs on both nodes exit 0, and PuppetDB lists both nodes.choria pingandchoria req rpcutil pingreach both nodes.bundle exec rake spec(serverspec): 28 examples, 0 failures.Regression run with
OPENVOX_RELEASE=8from the same branch:serverispuppet.vagranton both nodes.ssldirsetting is harmless on OpenVox 8). Second agent runs on both nodes exit 0.bundle exec rake spec(serverspec): 28 examples, 0 failures.🤖 Generated with Claude Code