Extra LDAP groups for chassis as well.

pkg/butler/configure/chassis.go

@@ -114,6 +114,18 @@ func (b *Cmc) Apply() { //nolint: gocyclo
 			}
 		case "ldap_group":
 			if b.config.LdapGroups != nil && b.config.Ldap != nil {
+				k, err := b.config.LdapGroups.GetExtraGroups(b.asset.Serial, b.asset.Vendor)
+				if err != nil {
+					b.logger.WithFields(logrus.Fields{
+						"Vendor":    b.vendor,
+						"Model":     b.model,
+						"Serial":    b.serial,
+						"IPAddress": b.ip,
+						"Error":     err,
+						"K":         k,
+						"Groups":    b.config.LdapGroups.Groups,
+					}).Warn("Trying to fetch more LDAP groups has failed.")
+				}
 				err = b.configure.LdapGroups(b.config.LdapGroups.Groups, b.config.Ldap)
 			}
 		case "license":

pkg/butler/execute.go

@@ -52,23 +52,25 @@ func (b *Butler) executeCommand(command string, asset *asset.Asset) (err error)
 			log.WithFields(logrus.Fields{
 				"component":          component,
 				"Serial":             asset.Serial,
-				"AssetType":          asset.Type,
+				"Asset Type":         asset.Type,
 				"Vendor":             asset.Vendor, //at this point the vendor may or may not be known.
 				"Location":           asset.Location,
+				"IP Address":         asset.IPAddress,
 				"Command":            command,
-				"Command successful": success,
+				"Command Successful": success,
 				"Error":              err,
 				"Output":             output,
 			}).Warn("Command execute returned error.")
 		} else {
 			log.WithFields(logrus.Fields{
 				"component":          component,
 				"Serial":             asset.Serial,
-				"AssetType":          asset.Type,
+				"Asset Type":         asset.Type,
 				"Vendor":             asset.Vendor,
 				"Location":           asset.Location,
+				"IP Address":         asset.IPAddress,
 				"Command":            command,
-				"Command successful": success,
+				"Command Successful": success,
 				"Output":             output,
 			}).Debug("Command successfully executed.")
 		}