Skip to content

Conversation

@apinnick
Copy link
Contributor

Issue: foreman-documentation issue #3140

Additional information:

@apinnick
Copy link
Contributor Author

Review requested in forum-ccs-style channel

Copy link

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

It should also be "Capsule" in favor of "Capsule Server" because you can also use Satellite Server itself.

ALso, I'd drop "roles" because you typically use Ansible roles during configuration management; and Ansible Playbooks for REX.

@apinnick apinnick force-pushed the remote-execution-term branch from 9a03104 to 0dd9a9e Compare September 17, 2025 12:47
Copy link

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

Makes sense to me.

I wonder if "(noun, adjective)" is correct. Shouldn't this be "(noun)" only?

@Lennonka
Copy link

I wonder if "(noun, adjective)" is correct.

@maximiliankolb Yes, it is correct. When used as "remote execution job", it's an adjective.

@apinnick
Copy link
Contributor Author

I wonder if "(noun, adjective)" is correct. Shouldn't this be "(noun)" only?

@maximiliankolb
In the phrase "remote execution job," "remote execution" is a compound adjective that modifies the noun "job".

@apinnick apinnick force-pushed the remote-execution-term branch from ce55ddc to 374df74 Compare September 17, 2025 13:29
@apinnick apinnick force-pushed the remote-execution-term branch from 374df74 to a7c4dc2 Compare September 17, 2025 14:05
@apinnick
Copy link
Contributor Author

apinnick commented Oct 5, 2025

@bergerhoffer @dfitzmau @mportman12
Could someone review this PR please?

Copy link
Collaborator

@IngridT1 IngridT1 left a comment

Choose a reason for hiding this comment

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

I'm happy to approve this after you make the changes.

@apinnick apinnick requested a review from IngridT1 October 15, 2025 12:19
@apinnick
Copy link
Contributor Author

I'm happy to approve this after you make the changes.

Thanks @IngridT1! I made the changes.

@IngridT1 IngridT1 merged commit 5c861a3 into redhat-documentation:main Oct 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants