From e4651bbc7c06ddc7311283b09a24e4125b1b1604 Mon Sep 17 00:00:00 2001
From: Jonathon Anderson <janderson@ciq.com>
Date: Thu, 26 Sep 2024 14:20:07 -0600
Subject: [PATCH] Update Rocky 9 quickstart

- Closes #1179

Signed-off-by: Jonathon Anderson <janderson@ciq.com>
---
 CHANGELOG.md                               |  1 +
 internal/app/wwctl/container/imprt/root.go |  2 +-
 userdocs/quickstart/el.rst                 | 35 +++++++++++++++-------
 3 files changed, 27 insertions(+), 11 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0f496558..0121f1c6 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -70,6 +70,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - Avoid panic on container import #1244
 - make sure that warewulfd has the permission 0755 at creation time #674
 - Fix "onboot" behavior for NetworkManager, Debian networking, and Suse wicked. #1278
+- Clarified missing steps in Enterprise Linux quickstart. #1179
 
 ### Added
 
diff --git a/internal/app/wwctl/container/imprt/root.go b/internal/app/wwctl/container/imprt/root.go
index 7bf42a78..9073329b 100644
--- a/internal/app/wwctl/container/imprt/root.go
+++ b/internal/app/wwctl/container/imprt/root.go
@@ -42,7 +42,7 @@ Imported containers are used to create bootable VNFS images.`,
 func init() {
 	baseCmd.PersistentFlags().BoolVarP(&SetForce, "force", "f", false, "Force overwrite of an existing container")
 	baseCmd.PersistentFlags().BoolVarP(&SetUpdate, "update", "u", false, "Update and overwrite an existing container")
-	baseCmd.PersistentFlags().BoolVarP(&SetBuild, "build", "b", false, "Build container when after pulling")
+	baseCmd.PersistentFlags().BoolVarP(&SetBuild, "build", "b", false, "Build container after pulling")
 	baseCmd.PersistentFlags().BoolVar(&SetDefault, "setdefault", false, "Set this container for the default profile")
 	baseCmd.PersistentFlags().BoolVar(&SyncUser, "syncuser", false, "Synchronize UIDs/GIDs from host to container")
 	baseCmd.PersistentFlags().BoolVar(&OciNoHttps, "nohttps", false, "Ignore wrong TLS certificates, superseedes env WAREWULF_OCI_NOHTTPS")
diff --git a/userdocs/quickstart/el.rst b/userdocs/quickstart/el.rst
index bc23812d..e924fe99 100644
--- a/userdocs/quickstart/el.rst
+++ b/userdocs/quickstart/el.rst
@@ -13,10 +13,7 @@ v4.5.7 release on Enterprise Linux 9:
 
    dnf install https://github.com/warewulf/warewulf/releases/download/v4.5.7/warewulf-4.5.7-1.el9.x86_64.rpm
 
-Packages are available for el7, el8, and el9.
-
-.. note::
-   CentOS 7 (and el7 in general) uses ``yum``, and does not provide ``dnf``.
+Packages are available for el8 and el9.
 
 .. _GitHub releases: https://github.com/warewulf/warewulf/releases
 
@@ -36,9 +33,6 @@ If you prefer, you can also install Warewulf from source.
    make
    make install
 
-.. note::
-   CentOS 7 (and el7 in general) only provides a single ``ipxe-bootimgs`` package.
-
 Configure firewalld
 ===================
 
@@ -163,10 +157,11 @@ default running kernel from the controller node and set both in the
 
 .. code-block:: bash
 
-   wwctl container import docker://ghcr.io/warewulf/warewulf-rockylinux:9 rockylinux-9 --setdefault
+   wwctl container import docker://ghcr.io/warewulf/warewulf-rockylinux:9 rockylinux-9 --build
+   wwctl profile set default --container rockylinux-9
 
-Configure a default node profile
-================================
+Configure the default node profile
+==================================
 
 In this example, all nodes share the netmask and gateway
 configuration, so we can set them in the default profile.
@@ -195,4 +190,24 @@ overlapping names, distinguish them using dot notation.
 The full node configuration comes from both cascading profiles and
 node configurations which always supersede profile configurations.
 
+Build overlays
+==============
+
+The default configuration should cause node overlays to be built automatically
+when they are required; but you can build them explicitly, just to be sure.
+
+.. warning::
+
+   Overlay autobuild has been broken at various times prior to v4.5.6; so it's
+   a reasonable practice to rebuild overlays manually after changes to the
+   cluster.
+
+.. code-block:: bash
+
+   # you can also supply an `n1` argument to build for the specific node
+   wwctl overlay build
+
+Boot
+====
+
 Turn on your compute node and watch it boot!