Merge pull request #59 from digitalrebar/have-sledgehammer-add-machin…

…e-HardwareAddrs-if-missing

Have Sledgehammer populate HardwareAddrs if it is not populated.

content/bootenvs/discovery.yml

@@ -73,7 +73,7 @@ Templates:
               [[ -f $nic/type && -f $nic/address && $(cat "$nic/type") == 1 ]] || continue
               maclist="$maclist,\"$(cat "$nic/address")\""
           done
-          printf '[%s]', "${maclist#,}"
+          printf '[%s]' "${maclist#,}"
       }
 
       # Stuff from sledgehammer file that makes this command debuggable
@@ -111,9 +111,10 @@ Templates:
       # See if we have already been created.
       if [[ $(cat /proc/cmdline) =~ $host_re ]]; then
           RS_UUID="${BASH_REMATCH[1]}"
+          json="$(drpcli machines show "$RS_UUID")"
           # If we did not get a hostname from DHCP, get it from DigitalRebar Provision.
           if [[ ! $HOSTNAME ]]; then
-              HOSTNAME="$(drpcli machines show "$RS_UUID" |jq -r '.Name')"
+              HOSTNAME="$(jq -r '.Name' <<< "$json")"
           fi
       else
           # If we did not get a hostname from DHCP, generate one for ourselves.

content/bootenvs/sledgehammer.yml

@@ -100,6 +100,24 @@ Templates:
       # used below.  Reset the token to the longer machine token.
       export RS_TOKEN="{{.GenerateToken}}"
 
+      # If we do not have hardware addresses set on the machine, set them now.
+      get_macs() {
+          local maclist=""
+          local nic=""
+          for nic in /sys/class/net/*; do
+              [[ -f $nic/type && -f $nic/address && $(cat "$nic/type") == 1 ]] || continue
+              maclist="$maclist,\"$(cat "$nic/address")\""
+          done
+          printf '[%s]' "${maclist#,}"
+      }
+
+      json="$(drpcli machines show "$RS_UUID")"
+      # The machine does not have hardware addresses set, so set them
+      if [[ "$(jq '.HardwareAddrs | length' <<< "$json")" = 0 ]]; then
+          drpcli machines update "$RS_UUID" "{\"HardwareAddrs\": $(get_macs)}"
+      fi
+      unset get_macs json
+
       {{template "reset-workflow.tmpl" .}}
       {{template "runner.tmpl" .}}