Skip to content

Add support for OpenVox agent#11839

Open
binford2k wants to merge 1 commit intotheforeman:deb/developfrom
binford2k:support_openvox_deb
Open

Add support for OpenVox agent#11839
binford2k wants to merge 1 commit intotheforeman:deb/developfrom
binford2k:support_openvox_deb

Conversation

@binford2k
Copy link
Copy Markdown

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.

@binford2k binford2k requested a review from a team as a code owner March 7, 2025 02:22
@theforeman-bot
Copy link
Copy Markdown
Member

Can one of the admins verify this patch?

1 similar comment
@theforeman-bot
Copy link
Copy Markdown
Member

Can one of the admins verify this patch?

@deubert-it
Copy link
Copy Markdown

deubert-it commented Apr 24, 2025

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.

# apt-get install foreman-installer
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 foreman-installer : Depends: puppet-agent (>= 7.0.0) but it is not going to be installed or
                              puppet (>= 7.0.0)
E: Unable to correct problems, you have held broken packages.

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.

@tyybbi
Copy link
Copy Markdown

tyybbi commented May 9, 2025

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!

@ogajduse
Copy link
Copy Markdown
Member

[test deb]

Copy link
Copy Markdown
Member

@ogajduse ogajduse left a comment

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perhaps at this point we should switch over to openvox-strings. We may need to fix Kafo to support that too though.

@dgoetz
Copy link
Copy Markdown
Member

dgoetz commented Apr 14, 2026

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?

@dgoetz dgoetz force-pushed the support_openvox_deb branch from eba9307 to fae21a7 Compare April 16, 2026 15:28
@dgoetz
Copy link
Copy Markdown
Member

dgoetz commented Apr 16, 2026

Rebased, updated and changelog added.

@ogajduse Can you re-review please?

@ekohl As debian is lacking this support now while RPM already has it, would it be ok to merge this and postpone the switch to openvox-strings?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants