Skip to content

Add oVirt docs - #5088

Open
maximiliankolb wants to merge 2 commits into
theforeman:masterfrom
ATIX-AG:add_ovirt_docs
Open

Add oVirt docs#5088
maximiliankolb wants to merge 2 commits into
theforeman:masterfrom
ATIX-AG:add_ovirt_docs

Conversation

@maximiliankolb

Copy link
Copy Markdown
Contributor

What changes are you introducing?

Add oVirt docs back for non-Satellite builds.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

oVirt is still a supported compute resource plugin.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

Contributor checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.19/Katello 4.21
  • Foreman 3.18/Katello 4.20 (Satellite 6.19; orcharhino 7.9)
  • Foreman 3.17/Katello 4.19
  • Foreman 3.16/Katello 4.18 (Satellite 6.18; orcharhino 7.6, 7.7, and 7.8)
  • Foreman 3.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17; orcharhino 7.4; orcharhino 7.5)
  • We do not accept PRs for Foreman older than 3.14.

@github-actions github-actions Bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Jul 21, 2026
@maximiliankolb maximiliankolb added this to the Provisioning hosts milestone Jul 21, 2026
@maximiliankolb
maximiliankolb marked this pull request as draft July 21, 2026 12:39
@maximiliankolb
maximiliankolb force-pushed the add_ovirt_docs branch 2 times, most recently from 86768b6 to afa6d8e Compare July 21, 2026 13:16
* Install oVirt plugin via foreman-installer
* Revert changes in guides/common/modules/ref_troubleshooting-virt-who.adoc
* Split proc_adding-images-to-server.adoc by UI for oVirt
* Split proc_creating-network-or-image-based-hosts.adoc by UI for oVirt
* Use path attribute when including snippets
@maximiliankolb

Copy link
Copy Markdown
Contributor Author

@fraenki As discussed on the Foreman Birthday Party: Here's a PR to bring back docs for oVirt. What do you think? Could you/your team do a review?

@fraenki fraenki left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! Thank you.

+
[options="nowrap", subs="+quotes,verbatim,attributes"]
----
# {foreman-installer} --enable-foreman-plugin-ovirt

@fraenki fraenki Jul 28, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Oh, I don't know whether or not this parameter was re-added to foreman-installer. At least I haven't added it.

EDIT: It was re-added in theforeman/foreman-installer#1047 and theforeman/foreman-installer#1045.

@maximiliankolb
maximiliankolb marked this pull request as ready for review July 30, 2026 06:06
@maximiliankolb maximiliankolb added tech review done No issues from the technical perspective and removed Needs tech review Requires a review from the technical perspective Needs testing Requires functional testing labels Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs style review Requires a review from docs style/grammar perspective tech review done No issues from the technical perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants