refactor: use functional names for hook bash functions

Rename hook functions to describe their actions rather than when they
run. Add documentation comments for clarity. No API changes.

Signed-off-by: Carlos Eduardo Arango Gutierrez <[email protected]>

Author: ArangoGutierrez

deployments/systemd/hooks-default.yaml

@@ -3,16 +3,16 @@ hooks:
     apply-start:
     - workdir: "/etc/nvidia-mig-manager"
       command: "/bin/bash"
-      args: ["-x", "-c", "source hooks.sh; apply-start ${MIG_PARTED_SELECTED_CONFIG}"]
+      args: ["-x", "-c", "source hooks.sh; persist-config ${MIG_PARTED_SELECTED_CONFIG}"]
     pre-apply-mode:
     - workdir: "/etc/nvidia-mig-manager"
       command: "/bin/bash"
-      args: ["-x", "-c", "source hooks.sh; pre-apply-mode"]
+      args: ["-x", "-c", "source hooks.sh; stop-services-for-mode"]
     pre-apply-config:
     - workdir: "/etc/nvidia-mig-manager"
       command: "/bin/bash"
-      args: ["-x", "-c", "source hooks.sh; pre-apply-config"]
+      args: ["-x", "-c", "source hooks.sh; stop-services-for-config"]
     apply-exit:
     - workdir: "/etc/nvidia-mig-manager"
       command: "/bin/bash"
-      args: ["-x", "-c", "source hooks.sh; apply-exit"]
+      args: ["-x", "-c", "source hooks.sh; start-services"]

deployments/systemd/hooks-minimal.yaml

@@ -3,4 +3,4 @@ hooks:
     apply-start:
     - workdir: "/etc/nvidia-mig-manager"
       command: "/bin/bash"
-      args: ["-x", "-c", "source hooks.sh; apply-start ${MIG_PARTED_SELECTED_CONFIG}"]
+      args: ["-x", "-c", "source hooks.sh; persist-config ${MIG_PARTED_SELECTED_CONFIG}"]

deployments/systemd/hooks.sh

@@ -35,7 +35,9 @@ k8s_pod_images=(
 	dcgm-exporter
 )
 
-function apply-start() {
+# persist-config saves the selected MIG configuration to ensure it is
+# automatically reapplied after a system reboot.
+function persist-config() {
 	local selected_config="${1}"
 	nvidia-mig-manager::service::persist_config_across_reboot ${selected_config}
 	if [ "${?}" != "0" ]; then
@@ -44,7 +46,10 @@ function apply-start() {
 	return 0
 }
 
-function pre-apply-mode() {
+# stop-services-for-mode stops all GPU-dependent services (K8s workloads
+# and driver services) before applying MIG mode changes that require a
+# GPU reset.
+function stop-services-for-mode() {
 	stop_k8s_services
 	if [ "${?}" != "0" ]; then
 		return 1
@@ -60,7 +65,9 @@ function pre-apply-mode() {
 	return 0
 }
 
-function pre-apply-config() {
+# stop-services-for-config stops Kubernetes services and pods before
+# applying MIG device configuration changes (no GPU reset required).
+function stop-services-for-config() {
 	stop_k8s_services
 	if [ "${?}" != "0" ]; then
 		return 1
@@ -72,7 +79,9 @@ function pre-apply-config() {
 	return 0
 }
 
-function apply-exit() {
+# start-services restarts all previously stopped services (driver and K8s)
+# after MIG configuration changes have been successfully applied.
+function start-services() {
 	start_driver_services
 	if [ "${?}" != "0" ]; then
 		return 1