Skip to content

DOCTEAM-2206 Issue in "SETTING UP THE MAIN grub.cfg FILE"#676

Open
sounix000 wants to merge 2 commits into
mainfrom
ssarkar/DOCTEAM-2206
Open

DOCTEAM-2206 Issue in "SETTING UP THE MAIN grub.cfg FILE"#676
sounix000 wants to merge 2 commits into
mainfrom
ssarkar/DOCTEAM-2206

Conversation

@sounix000

@sounix000 sounix000 commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

PR creator: Description

  • Fixed missing config variable in grub.cfg
  • Standardized menu entry names, added architecture-specific grub2-mknetdir commands with --directory option
  • Clarified shim package scope to supported architectures
  • Added note on HTTP loading of kernel and initrd, and added &selnx; file context fix for /srv/install.

PR creator: Are there any relevant issues/feature requests?

PR reviewer: Checklist for editorial review

Apart from the usual checks, please double-check also the following:

@mtomaschewski mtomaschewski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cannot see if it's correct in the output or not.

Comment on lines +229 to +232
<screen>menuentry 'SLES-16.0 Online Installation' {
linux (http,pxe.example.net)/boot/images/SLES-16.0/${arch}/linux showopts root=live:http://pxe.example.net/boot/images/SLES-16.0/${arch}/squashfs.img ${ipcfg} ${sconsole} ${autoinstall}
initrd (http,pxe.example.net)/boot/images/SLES-16.0/${arch}/initrd
}</screen>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, you can do it, but this is IMO not required and counterproductive, because it always breaks one of the protocols. Grub automatically prepends the (http,...) or (tfp,...) root to the paths.

<screen>&prompt.sudo;<command>cat &gt; /srv/tftpboot/boot/config/aa:bb:cc:dd:ee:ff/grub.cfg &lt;&lt; 'EOF'</command>
# Machine-specific configuration for aa:bb:cc:dd:ee:ff
set default='SLES-16.0 Full Installation'
set default='SLES-16.0 Local Installation'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Without the resulting output, cannot see if this is correct or not. It must match the menuentry title or an ID assigned to it, see info grub.
I were able to locate a menuentry 'SLES-16.0 Auto-Installation' only.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment on lines +251 to +264
If nginx still cannot serve files from <filename>/srv/install</filename>, the
directory may require additional &selnx; file context configuration. Run the
following commands to equivalence <filename>/srv/install</filename> to
<filename>/srv/www</filename> and restore the file context:
</para>
<screen>&prompt.sudo;<command>semanage fcontext -e /srv/www -a /srv/install</command></screen>
<screen>&prompt.sudo;<command>restorecon -RFv /srv/install/</command></screen>
<note>
<title>&selnx; file context for <filename>/srv/install</filename></title>
<para>
This step may be required in addition to
<command>setsebool -P httpd_serve_cobbler_files=1</command>. Use
<command>ausearch -m avc -ts recent</command> to check for &selnx; denials
before and after applying these changes.

@mtomaschewski mtomaschewski Jun 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please just adjust to copy/rsync the installation files to
/srv/www/htdocs/install instead of the custom /srv/install from my initial setup as we've agreed in https://bugzilla.suse.com/show_bug.cgi?id=1246539#c22 ff.

That is, just sed -e "s|/srv/install|/srv/www/htdocs/install|g" -i *.xml or similar.

-> No localtion /install {} in nginx.conf and no custom selinux policy needed any more.

@sounix000 sounix000 force-pushed the ssarkar/DOCTEAM-2206 branch from 18e537d to 7e32c0b Compare June 6, 2026 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants