Containerize Remote Execution section

[AkshayGadhaveRH]

JIRA:wq
https://redhat.atlassian.net/browse/SAT-41055

What changes are you introducing?

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

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]

The PR preview for 13b3477 is available at theforeman-foreman-documentation-preview-pr-5093.surge.sh

The following output files are affected by this PR:

• Managing_Hosts/index-containerized-katello.html
• Managing_Hosts/index-containerized-orcharhino.html
• Managing_Hosts/index-satellite.html
• Updating_Project/index-foreman-deb.html

show diff

show diff as HTML

[adamruzicka]

Should this also address how the plugin can be enabled in containerized deployments where it is not enabled by default?

Something along the lines of an additional ifdef and {{foremanctl}} --add-feature remote-execution in guides/common/modules/proc_installing-the-remote-execution-plugin.adoc

[adamruzicka]

ping on the previous comment

[jafiala]

I think this was already addressed in https://github.com/theforeman/foreman-documentation/pull/5092/changes. Unfortunately, I did not squish the commits so the history on master is a mess :/

[adamruzicka]

Looks like documentation moves fast. I'm surprised that this pr has no conflicts.

Anyway, what about my original point, which doesn't seem to be addressed in 5092 either?

[maximiliankolb]

Looks like documentation moves fast. I'm surprised that this pr has no conflicts.

There's a merge conflict by now @AkshayGadhaveRH Please rebase your PR to HEAD of "master".

[maximiliankolb]

style-wise LGTM

[adamruzicka]

If I'm reading it correctly, then for containerized builds this would result in a list with a single ordered item. -> Should be unordered for containerized and ordered for all others.

[maximiliankolb]

nice catch!

Suggested change

	. Install the Remote Execution plugin on your {ProjectServer}:
	ifdef::containerized[]
	* Install the Remote Execution plugin on your {ProjectServer}:
	endif::[]
	ifndef::containerized[]
	. Install the Remote Execution plugin on your {ProjectServer}:
	endif::[]

[adamruzicka]

Well, I had a good teacher :)