Skip to content

Add assembly for configuring Insights#4882

Draft
Lennonka wants to merge 24 commits into
theforeman:masterfrom
Lennonka:configuring-rhlightspeeds
Draft

Add assembly for configuring Insights#4882
Lennonka wants to merge 24 commits into
theforeman:masterfrom
Lennonka:configuring-rhlightspeeds

Conversation

@Lennonka
Copy link
Copy Markdown
Contributor

@Lennonka Lennonka commented May 27, 2026

What changes are you introducing?

  • Adding a new assembly for configuring both hosted and in-Project Insights (RH Lightspeed) to Admin guide
  • Moving procedures requiring admin permissions from Managing hosts to the Admin guide

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

  • To provide users with flexibility regarding migrations between hosted Insights and IoP
  • Cleanup for personas

SAT-41488

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)
  • 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)
  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16; orcharhino 7.2 on EL9 only; orcharhino 7.3)
  • We do not accept PRs for Foreman older than 3.12.

@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 May 27, 2026
Comment thread guides/common/assembly_configuring-and-setting-up-remote-jobs.adoc
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I plan to re-use this in the Planning/Overview guide later.

Comment thread guides/common/modules/con_insights-overview.adoc
Copy link
Copy Markdown
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

Overall is a great start :) Left a few comments and suggestions

Comment thread guides/common/modules/con_enabling-hosted-insights.adoc
Comment thread guides/common/modules/con_insights-overview.adoc
Comment thread guides/common/modules/con_insights-overview.adoc Outdated
Comment thread guides/common/modules/con_insights-overview.adoc
Comment thread guides/common/modules/proc_disabling-hosted-insights.adoc Outdated
Comment thread guides/common/modules/proc_disabling-insights-iop.adoc Outdated
Comment thread guides/common/modules/proc_enabling-insights-iop.adoc Outdated
Comment thread guides/common/assembly_configuring-and-setting-up-remote-jobs.adoc
Comment thread guides/common/modules/proc_disabling-hosted-insights.adoc Outdated
Comment thread guides/common/modules/proc_disabling-hosted-insights.adoc Outdated
Comment thread guides/common/modules/proc_disabling-insights-iop.adoc Outdated
Comment thread guides/common/modules/proc_disabling-insights-iop.adoc Outdated
Comment thread guides/common/modules/proc_enabling-insights-iop.adoc Outdated
Comment thread guides/common/modules/proc_enabling-insights-iop.adoc Outdated
Comment on lines +33 to +40
. Re-register the Insights client on your hosts:
+
[options="nowrap", subs="+quotes,verbatim,attributes"]
----
# insights-client --register
----
+
You can use remote execution to run this command on your hosts.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@chris1984 They always have to reregister the Insights client when they migrate, correct?

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.

Yes that is true, because going from hosted to iop, the insights_facet that gets created in the database becomes the subscription-manager uuid instead of the insights_id from hosted

Comment on lines +42 to +43
.Verification
* TODO: Add verification steps
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@chris1984 Need verification steps.

Comment on lines +31 to +32
.Verification
* TODO: Add verification steps
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@chris1984 Need verification steps.

@Lennonka
Copy link
Copy Markdown
Contributor Author

Lennonka commented Jun 2, 2026

@chris1984 Please, take a look at the previews, especially Satellite and Katello builds, and review that the procedures make sense. Then let me know what needs to be corrected.

@Lennonka Lennonka force-pushed the configuring-rhlightspeeds branch from d936570 to 5f1d669 Compare June 4, 2026 03:06
@Lennonka
Copy link
Copy Markdown
Contributor Author

Lennonka commented Jun 4, 2026

Rebased.

@Lennonka
Copy link
Copy Markdown
Contributor Author

Lennonka commented Jun 4, 2026

@chris1984 I've simplified the chapter per our discussion. Please, address the remaining comments, including #4882 (comment)

Comment on lines +11 to +13
ifdef::satellite[]
This is enabled by default after installing {Project}.
endif::[]
Copy link
Copy Markdown
Contributor Author

@Lennonka Lennonka Jun 4, 2026

Choose a reason for hiding this comment

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

I think this is misleading because the plugin is installed but the cloud connector is not configured by default. Correct? Let's drop it?

Thefore this prerequisite shouldn't be required for Satellite?

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.

Right, the cloud connector only is configured if the user wants to remediate from console.redhat.com, otherwise it's not configured. We can drop it

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 Needs tech review Requires a review from the technical perspective Needs testing Requires functional testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants