Skip to content

Use dashes in module names for platform servers#5038

Draft
maximiliankolb wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:fix_smart_proxy_file_names
Draft

Use dashes in module names for platform servers#5038
maximiliankolb wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:fix_smart_proxy_file_names

Conversation

@maximiliankolb

Copy link
Copy Markdown
Contributor

What changes are you introducing?

This patch reduces the number of variants how to say "Foreman server" or "Smart Proxy server" in file names.

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

This helps contributors to navigate the source trees and GHA to identify wrong file names.

$ fd projectserver
$ fd smartproxyserver
$ rg projectserver
$ rg smartproxyserver

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

There are no new broken links:

$ rg -i "configuring-the-active-directory-authentication-source-on-projectserver_"
$ rg -i "configuring-the-freeipa-authentication-source-on-projectserver_"
$ rg -i "determining-whether-your-projectserver-uses-internal-or-external-databases"
$ rg -i "snip_prerequisites-configuring-smartproxyservers-for-load-balancing.adoc"

Fixes #4998 (Unify file name to use dashes)

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)
  • We do not accept PRs for Foreman older than 3.14.

@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 Jul 10, 2026
@maximiliankolb maximiliankolb removed Needs tech review Requires a review from the technical perspective Not yet reviewed Needs testing Requires functional testing labels Jul 10, 2026
@maximiliankolb maximiliankolb marked this pull request as draft July 10, 2026 06:28
@maximiliankolb maximiliankolb force-pushed the fix_smart_proxy_file_names branch from 479143c to 73d2c88 Compare July 10, 2026 06:48
This patch reduces the number of variants how to say "Foreman server" or
"Smart Proxy server" in file names.
This helps contributors to navigate the source trees and GHA to identify
wrong file names.

$ fd projectserver
$ fd smartproxyserver
$ rg projectserver
$ rg smartproxyserver

There are no new broken links:
$ rg -i "configuring-the-active-directory-authentication-source-on-projectserver_"
$ rg -i "configuring-the-freeipa-authentication-source-on-projectserver_"
$ rg -i "determining-whether-your-projectserver-uses-internal-or-external-databases"
$ rg -i "snip_prerequisites-configuring-smartproxyservers-for-load-balancing.adoc"

Fixes theforeman#4998 (Unify file name to use dashes)
@maximiliankolb maximiliankolb force-pushed the fix_smart_proxy_file_names branch from 73d2c88 to 929788f Compare July 10, 2026 07:02
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unify file names for modules that include Foreman Server or Smart Proxy (Server)

1 participant