Merge pull request #1547 from mslacken/TaggAddWithNet

Fix panic when adding tag with existing netdev

CHANGELOG.md

@@ -132,6 +132,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - Fix "onboot" behavior for NetworkManager, Debian networking, and Suse wicked. #1278
 - Clarified missing steps in Enterprise Linux quickstart. #1179
 - Fix dhcpd.conf static template to include next-server and dhcp-range #1536
+- Fix panic when adding tag with existing netdev #1546
 
 
 ## v4.5.7, 2024-09-11

internal/app/wwctl/node/set/main_test.go

@@ -344,6 +344,50 @@ nodes:
       tag2: value2
 `,
 		},
+		{
+			name:    "single node add tag",
+			args:    []string{"--tagadd", "tag1=foobaar", "n01"},
+			wantErr: false,
+			stdout:  "",
+			inDB: `nodeprofiles:
+  default: {}
+nodes:
+  n01: {}
+`,
+			outDb: `nodeprofiles:
+  default: {}
+nodes:
+  n01:
+    tags:
+      tag1: foobaar
+`,
+		},
+		{
+			name:    "single node add tag with netdev",
+			args:    []string{"--tagadd", "tag1=foobaar", "n01"},
+			wantErr: false,
+			stdout:  "",
+			inDB: `nodeprofiles:
+  default: {}
+nodes:
+  n01:
+    network devices:
+      default:
+        ipaddr: 172.16.130.101
+
+`,
+			outDb: `nodeprofiles:
+  default: {}
+nodes:
+  n01:
+    tags:
+      tag1: foobaar
+    network devices:
+      default:
+        ipaddr: 172.16.130.101
+`,
+		},
+
 		{
 			name:    "single node set fs,part and disk",
 			args:    []string{"--fsname=var", "--fspath=/var", "--fsformat=btrfs", "--partname=var", "--partnumber=1", "--diskname=/dev/vda", "n01"},

internal/pkg/api/node/set.go

@@ -134,7 +134,7 @@ func NodeSetParameterCheck(set *wwapiv1.ConfSetParameter) (nodeDB node.NodesYaml
 				for _, key := range set.NetTagDel {
 					delete(nodePtr.NetDevs[set.Netdev].Tags, key)
 				}
-				for key, val := range set.TagAdd {
+				for key, val := range set.NetTagAdd {
 					nodePtr.NetDevs[set.Netdev].Tags[key] = val
 				}
 			}