-
Notifications
You must be signed in to change notification settings - Fork 108
Boot loader ref file changes #4505
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
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideReorganizes and consolidates boot loader reference documentation by replacing the ipxe-specific reference module with more generic HTTP and shared TFTP/HTTP root reference modules and wiring them into the main bootloader binary location assembly. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
The PR preview for cdfce8a is available at theforeman-foreman-documentation-preview-pr-4505.surge.sh The following output files are affected by this PR: |
Co-authored-by: Maximilian Kolb <[email protected]>
|
Ready for review. I see the conflict, but it doesn't show in the web editor, and I don't see it on my command line either. Not sure why it's a conflict either, I renamed the |
Lennonka
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left you a couple of suggestions.
guides/common/modules/con_bootloader-binary-location-overview.adoc
Outdated
Show resolved
Hide resolved
guides/common/modules/con_bootloader-binary-location-overview.adoc
Outdated
Show resolved
Hide resolved
…adoc Co-authored-by: Lena Ansorgová <[email protected]>
…adoc Co-authored-by: Lena Ansorgová <[email protected]>
Co-authored-by: Lena Ansorgová <[email protected]>
This patch also drops "Optional:" to avoid telling users that security is optional.
Co-authored-by: Arvind Jangir <[email protected]>
Users should never clone and edit templates. Instead, if they really need to run custom code before or after registration or provisioning workflows, they should use "custom pre" and "custom post" snippets as documented. This way they do not lose support by the community/vendor. Also, there's configuration management. Fixes theforeman#4256
This patch drops the procedure to add KernelCare clients for EL and instead moves all necessary information for EL and Deb to tables. Add KernelCare Clients for EL10 Refs 29eda6f
Follow-up PR to theforeman#4565.
This patch introduces attributes for the major and minor version of SLES. It also bumps the version to SLES 16.0. SLES 16 does not use Service Packs anymore to indicate minor releases: $ podman run --rm -it registry.suse.com/bci/bci-base:16.0 cat /etc/os-release | rg "^VERSION=|^VERSION_ID=|^PRETTY_NAME=" | sort -u PRETTY_NAME="SUSE Linux Enterprise Server 16.0" VERSION="16.0" VERSION_ID="16.0" $ podman run --rm -it registry.suse.com/bci/bci-base:15.7 cat /etc/os-release | rg "^VERSION=|^VERSION_ID=|^PRETTY_NAME=" | sort -u PRETTY_NAME="SUSE Linux Enterprise Server 15 SP7" VERSION="15-SP7" VERSION_ID="15.7"
* Show enabling REX Pull Provider for Katello-like builds Foreman+Katello and Katello-like builds have REX on Foreman Server enabled by default. * Show REX on Smart Proxy Servers for orcharhino builds * Require REX to enable REX Pull Provider On Satellite, REX is always enabled on Foreman Server and Smart Proxy Servers by default. * fixup! Require REX to enable REX Pull Provider
* Fix broken xrefs * Fix titles * Hide links for orcharhino builds * Hide xrefs for orcharhino builds * Remove broken xrefs
Reproducer to verify that there are no new broken links: $ rg -i "Creating_a_Custom_File_Type_Repository_" $ rg -i "Creating_a_Local_Source_for_a_Custom_File_Type_Repository_" $ rg -i "Creating_a_Remote_Source_for_a_Custom_File_Type_Repository_" $ rg -i "Downloading_Files_to_a_Host_from_a_Custom_File_Type_Repository_" $ rg -i "Managing_Custom_File_Type_Content_" $ rg -i "Uploading_Files_To_a_Custom_File_Type_Repository_" $ rg -i "cli-creating-a-custom-file-type-repository_"
* Split helper app modules * Add abstracts * Minor fixes * Ensure continuous procedure steps to avoid error Snippets with steps sometimes cause asciidoctor-dita-vale to think the procedure is not a continuous series of steps. * Fix capitalization to address Vale error --------- Co-authored-by: Maximilian Kolb <[email protected]>
Reproducer to verify that there are no new broken links: $ rg -i "Downloading_Files_to_a_Host_from_a_Python_Type_Repository_" $ rg -i "Installing_Python_Packages_from_" $ rg -i "Managing_Python_Type_Content_" $ rg -i "Synchronizing_Python_Repositories_" $ rg -i "Uploading_Files_To_a_Python_Type_Repository_"
* Show RHUI ACS for orcharhino builds * Hide bootstrap.py for orcharhino builds * Move appendix to introduction for orcharhino builds * Fix attributes for discovery for orcharhino builds * Show discovery via Smart Proxy Servers for orcharhino builds If you have multiple Smart Proxy Servers and want to perform boot disk-based host provisioning, you need to apply this workaround. * Show latest two Deb major versions In upstream docs, show the last two Debian and Ubuntu releases. * Only show module once for orcharhino builds In downstream orcharhino builds, the module is included only once but adjusted by Client OS-specific attributes.
Update IPA Kerberos auth source for foremanctl Update AD external auth source for foremanctl Update procedure on resetting external auth Move reset to its own module Update ext auth reset with additional details Drop equal signs Define project-package-install for foremanctl Clarify auth sources affected by auth config reset Update PAM service option Add ext auth overview to foremanctl Configuring User Auth Clarify that auth source are mutually exclusive Add verification step for resetting Kerberos SSO Apply suggestions from style review Co-authored-by: Maximilian Kolb <[email protected]> Call out a f-ctl prereq explicitly Ensure standalone Hammer scenario is covered Drop an internal comment about why a prereq is here Reference --add-feature hammer Apply easy fixes from style review Co-authored-by: Lena Ansorgová <[email protected]> Co-authored-by: Maximilian Kolb <[email protected]> Move stdout outside of a command block
Refs theforeman#4578 (Adjust assemblies for orcharhino builds)
|
Closing due to conflicts that persist even after rebase. Opened #4604 with the same changes on a clean slate. Apologies for the inconvenience. |
What changes are you introducing?
Further adjust newly added files for downstream docs
Why are you introducing these changes? (Explanation, links to references, issues, etc.)
Conclusion of #4422
Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)
Contributor checklists
Please cherry-pick my commits into:
Summary by Sourcery
Update bootloader reference documentation for HTTP and shared TFTP/HTTP usage and remove obsolete iPXE-specific module.
Documentation: