Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore info on relevant optional fields in Register Host interface #3508

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion guides/common/modules/proc_registering-a-host.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,14 @@ endif::[]

.Procedure
. In the {ProjectWebUI}, navigate to *Hosts* > *Register Host*.
. Enter the details for how you want the registered host to be configured.
. Enter the details for how you want the registered host to be configured. Note:
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd drop "Note:". alternative:

Suggested change
. Enter the details for how you want the registered host to be configured. Note:
. Enter the details for how you want the registered host to be configured.
+
[NOTE]
====
...
====

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
. Enter the details for how you want the registered host to be configured. Note:
. Enter the details for how you want the registered host to be configured.

An alternative. I think the bulleted list is "noteable" enough.

* If you select a host group from the *Host Group* list, the *Operating system*
ifdef::katello,satellite,orcharhino[]
, *Activation Keys*
endif::[]
and *Lifecycle environment* fields inherit their values from the host group.
Copy link
Contributor

Choose a reason for hiding this comment

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

The fields are inherited from the host group but not preselected. Is that intentional? On Foreman 3.13:

image

@Thorben-D @MariaAga

ifdef::katello,satellite,orcharhino[]
* A Capsule behind a load balancer takes precedence over the Capsule selected in this UI as the host’s content source.
Copy link
Contributor

Choose a reason for hiding this comment

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

branding! "Capsule" -> "{SmartProxy}"

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* A Capsule behind a load balancer takes precedence over the Capsule selected in this UI as the host’s content source.
* A {SmartProxy} behind a load balancer takes precedence over the {SmartProxy} selected in the {ProjectWebUI} as the content source of the host."

Attributes to avoid branding issues and minor rewording to avoid possessive apostrophe.

. On the *General* tab, in the *Activation Keys* field, enter one or more activation keys to assign to your host.
endif::[]
. Click *Generate* to generate a `curl` command.
Expand Down Expand Up @@ -84,6 +90,7 @@ To verify synchronized {client-content-type} content, you can use {Project} API
For example, `\https://{foreman-example-com}/katello/api/v2/repositories/_My_Repository_ID_/gpg_key_content`.
endif::[]
endif::[]
* On the *Advanced* tab, you can configure remote execution, Red{nbsp}Hat Insights, and packages to be installed.
* On the *Advanced* tab, in the *Token lifetime (hours)* field, you can change the validity duration of the JSON Web Token (JWT) that {Project} uses for authentication.
The duration of this token defines how long the generated registration command works.
+
Expand Down