Add support for OpenVox agent#11839
Conversation
|
Can one of the admins verify this patch? |
1 similar comment
|
Can one of the admins verify this patch? |
|
Thank you @binford2k for taking care of this. I'd just like to add to this issue that I'm a former puppet ce user, our test environment has successfully upgraded all agents to openvox-agent 8, the puppetdb to openvox-db 8, the puppetserver to openvox-server 8, however the foreman-installer package is no longer available due to package dependencies. The changes which were supplied in this pull request are exactly what we would need to make use of the foreman-installer again. I'm not sure yet what exactly foreman-installer will do, but the scenario answers.yaml seems to have enough options to support different package names, so should be fully useable with openvox packages. |
|
Personally I'd welcome this PR as I'm in the same boat with @deubert-it as of typing this. In any case, cheers for the effort to all involved in bringing a full OpenVox support to Foreman! |
|
[test deb] |
ogajduse
left a comment
There was a problem hiding this comment.
I see this PR as an equivalent to #12533 in the rpm/develop branch.
The patch looks sane to me. However, we still need to bump the changelog files in all packages that were changed.
- debian/bookworm/foreman-installer/changelog - bump from 3.18.0-1 to 3.18.0-2
- debian/jammy/foreman-installer/changelog - bump from 3.18.0-1 to 3.18.0-2
- plugins/puppet-agent-oauth/debian/changelog - bump from 0.5.5-1 to 0.5.5-2
- plugins/puppet-agent-puppet-strings/debian/changelog - bump from 3.0.1-2 to 3.0.1-3
- plugins/puppet-agent-rgen/debian/changelog - bump from 0.9.0-1 to 0.9.0-2
- plugins/puppet-agent-yard/debian/changelog - bump from 0.9.37-1 to 0.9.37-2
There was a problem hiding this comment.
Perhaps at this point we should switch over to openvox-strings. We may need to fix Kafo to support that too though.
|
In https://community.theforeman.org/t/foreman-installer-3-18-1-dependency-problem-on-debian-12/46189/5 a user is complaining about this. For rpm we already have this support and the installer moved to puppet modules which require >8.0.0, so @binford2k are you willing and able to update the pull request? |
793f7a3 to
594060a
Compare
594060a to
eba9307
Compare
Co-authored-by: Dirk Goetz <[email protected]>
eba9307 to
fae21a7
Compare
It might be better to start at version 8.11.0 instead. The 7.35.0 version is only best-effort supported. It's only intended to help people migrate and then upgrade.