Skip to content

Make foremanctl customizations visible#4803

Open
jafiala wants to merge 17 commits intotheforeman:masterfrom
jafiala:add-additional-config-foremanctl-SAT-43780
Open

Make foremanctl customizations visible#4803
jafiala wants to merge 17 commits intotheforeman:masterfrom
jafiala:add-additional-config-foremanctl-SAT-43780

Conversation

@jafiala
Copy link
Copy Markdown
Contributor

@jafiala jafiala commented Apr 29, 2026

What changes are you introducing?

Add additional configurations that don't require changes in foremanctl

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

To make foremanctl installations work

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

https://redhat.atlassian.net/browse/SAT-43780
This necessitates also some changes in the ifdef structure, which includes also dropping Appendix A from foremanctl builds (SAT-44660)

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.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 and 7.7)
  • 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 Apr 29, 2026
Comment on lines -84 to -90
endif::[]
ifdef::katello,orcharhino,satellite[]

include::common/assembly_configuring-an-alternate-cname.adoc[leveloffset=+2]

ifndef::foremanctl[]

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.

This extends the ifndef::foremanctl[] to also cover assembly_configuring-an-alternate-cname.adoc because this is included in the foremanctl-specific customize assembly

Comment thread guides/doc-Installing_Server_Disconnected/master.adoc Outdated
@jafiala jafiala marked this pull request as ready for review May 4, 2026 06:49
Comment thread guides/common/modules/con_customize-the-installation.adoc Outdated
Comment thread guides/common/modules/con_customize-the-installation.adoc Outdated
Comment thread guides/common/assembly_customize-the-installation.adoc Outdated
Comment thread guides/doc-Installing_Server_Disconnected/master.adoc Outdated
@pr-processor pr-processor Bot added the Waiting on contributor Requires an action from the author label May 4, 2026
Co-authored-by: Maximilian Kolb <mail@maximilian-kolb.de>
@pr-processor pr-processor Bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels May 5, 2026
@jafiala jafiala marked this pull request as draft May 5, 2026 11:12
@jafiala
Copy link
Copy Markdown
Contributor Author

jafiala commented May 5, 2026

Avoiding nesting assemblies means reverting the structure, taking it back into draft.

DONE

@jafiala jafiala marked this pull request as ready for review May 5, 2026 12:32
@evgeni
Copy link
Copy Markdown
Member

evgeni commented May 7, 2026

The following chapters might (!) not work for foremanctl:

  • 4.1.3. Configuring SELinux to ensure access to Foreman on custom ports
  • 4.1.8. Configuring HTTP proxy for PXE file downloads on Smart Proxies

@jafiala
Copy link
Copy Markdown
Contributor Author

jafiala commented May 7, 2026

The following chapters might (!) not work for foremanctl:

* 4.1.3. Configuring SELinux to ensure access to Foreman on custom ports
* 4.1.8. Configuring HTTP proxy for PXE file downloads on Smart Proxies

OK. Can we resolve it now? If not, I'll hide them from foremanctl builds for now an get back to them later.

@evgeni
Copy link
Copy Markdown
Member

evgeni commented May 7, 2026

The following chapters might (!) not work for foremanctl:

* 4.1.3. Configuring SELinux to ensure access to Foreman on custom ports
* 4.1.8. Configuring HTTP proxy for PXE file downloads on Smart Proxies

OK. Can we resolve it now? If not, I'll hide them from foremanctl builds for now an get back to them later.

As we don't have PXE for foremanctl right now, 4.1.8 is something that we couldn't come to a resolution anyway.

4.1.3 would need some testing, which I don't have the time right now, so unless someone wants to jump in, I'd just hide both.

Copy link
Copy Markdown
Member

@evgeni evgeni left a comment

Choose a reason for hiding this comment

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

tech ack, tack ack, here's my tech ack

(to be read with the voice of Tom Jones)

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.

4 participants