Improve logging in cpuallocator

Add little more info to error message and also print actual error in domainmgr.

Signed-off-by: Pramodh Pallapothu <pramodh@zededa.com>

Author: Pramodh Pallapothu

pkg/pillar/cmd/domainmgr/domainmgr.go

@@ -1389,7 +1389,7 @@ func assignCPUs(ctx *domainContext, config *types.DomainConfig, status *types.Do
 	if config.VmConfig.CPUsPinned { // Pin the CPU
 		cpusToAssign, err := ctx.cpuAllocator.Allocate(config.UUIDandVersion.UUID, config.VCpus)
 		if err != nil {
-			return errors.New("failed to allocate necessary amount of CPUs")
+			return err
 		}
 		for _, cpu := range cpusToAssign {
 			status.VmConfig.CPUs = append(status.VmConfig.CPUs, cpu)
@@ -1680,7 +1680,7 @@ func doActivate(ctx *domainContext, config types.DomainConfig,
 
 	if ctx.cpuPinningSupported {
 		if err := assignCPUs(ctx, &config, status); err != nil {
-			log.Warnf("failed to assign CPUs for %s", config.DisplayName)
+			log.Warnf("failed to assign CPUs for %s err %v", config.DisplayName, err)
 			errDescription := types.ErrorDescription{Error: err.Error()}
 			status.SetErrorDescription(errDescription)
 			publishDomainStatus(ctx, status)

pkg/pillar/cpuallocator/cpuallocator.go

@@ -101,8 +101,8 @@ func (cpuAllocator *CPUAllocator) getFree(numCPUsRequested int) ([]uint32, error
 		}
 	}
 	if found < numCPUsRequested {
-		return []uint32{}, fmt.Errorf("looking for %d CPUs only found %d",
-			numCPUsRequested, found)
+		return []uint32{}, fmt.Errorf("looking for %d CPUs only found %d reserved for EVE %d total %d",
+			numCPUsRequested, found, cpuAllocator.numReservedForEVE, cpuAllocator.totalCPUs)
 	}
 	return result, nil
 }