build(deps): bump github.com/vmware/govmomi from 0.27.4 to 0.28.0

Bumps [github.com/vmware/govmomi](https://github.com/vmware/govmomi) from 0.27.4 to 0.28.0.
- [Release notes](https://github.com/vmware/govmomi/releases)
- [Changelog](https://github.com/vmware/govmomi/blob/master/CHANGELOG.md)
- [Commits](https://github.com/vmware/govmomi/compare/v0.27.4...v0.28.0)

---
updated-dependencies:
- dependency-name: github.com/vmware/govmomi
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

vendor/github.com/vmware/govmomi/govc/flags/datacenter.go

@@ -203,6 +203,10 @@ func (flag *DatacenterFlag) ManagedObjects(ctx context.Context, args []string) (
 			continue
 		}
 
+		if !strings.Contains(arg, "/") {
+			return nil, fmt.Errorf("%q must be qualified with a path", arg)
+		}
+
 		elements, err := finder.ManagedObjectList(ctx, arg)
 		if err != nil {
 			return nil, err

vendor/github.com/vmware/govmomi/govc/flags/debug.go

@@ -31,7 +31,7 @@ import (
 	"text/tabwriter"
 	"time"
 
-	"github.com/kr/pretty"
+	"github.com/dougm/pretty"
 
 	"github.com/vmware/govmomi/vim25/debug"
 	"github.com/vmware/govmomi/vim25/soap"

vendor/github.com/vmware/govmomi/govc/flags/output.go

@@ -29,7 +29,7 @@ import (
 	"sync"
 	"time"
 
-	"github.com/kr/pretty"
+	"github.com/dougm/pretty"
 
 	"github.com/vmware/govmomi/task"
 	"github.com/vmware/govmomi/vim25/progress"

vendor/github.com/vmware/govmomi/govc/importx/spec.go

@@ -107,7 +107,7 @@ func (cmd *spec) Run(ctx context.Context, f *flag.FlagSet) error {
 			return fmt.Errorf("invalid file extension %s", path.Ext(fpath))
 		}
 
-		if isRemotePath(fpath) {
+		if isRemotePath(f.Arg(0)) {
 			client, err := cmd.Client()
 			if err != nil {
 				return err

vendor/github.com/vmware/govmomi/govc/vm/change.go

@@ -77,6 +77,7 @@ type change struct {
 	extraConfig     extraConfig
 	extraConfigFile extraConfigFile
 	Latency         string
+	hwUpgradePolicy string
 }
 
 func init() {
@@ -105,6 +106,28 @@ func (cmd *change) setLatency() error {
 	return fmt.Errorf("latency must be one of: %s", strings.Join(latencyLevels, "|"))
 }
 
+var hwUpgradePolicies = []string{
+	string(types.ScheduledHardwareUpgradeInfoHardwareUpgradePolicyOnSoftPowerOff),
+	string(types.ScheduledHardwareUpgradeInfoHardwareUpgradePolicyNever),
+	string(types.ScheduledHardwareUpgradeInfoHardwareUpgradePolicyAlways),
+}
+
+// setHwUpgradePolicy validates hwUpgradePolicy if set
+func (cmd *change) setHwUpgradePolicy() error {
+	if cmd.hwUpgradePolicy == "" {
+		return nil
+	}
+	for _, l := range hwUpgradePolicies {
+		if l == cmd.hwUpgradePolicy {
+			cmd.ScheduledHardwareUpgradeInfo = &types.ScheduledHardwareUpgradeInfo{
+				UpgradePolicy: string(types.ScheduledHardwareUpgradeInfoHardwareUpgradePolicy(cmd.hwUpgradePolicy)),
+			}
+			return nil
+		}
+	}
+	return fmt.Errorf("Hardware upgrade policy must be one of: %s", strings.Join(hwUpgradePolicies, "|"))
+}
+
 // setAllocation sets *info=nil if none of the fields have been set.
 // We need non-nil fields for use with flag.FlagSet, but we want the
 // VirtualMachineConfigSpec fields to be nil if none of the related flags were given.
@@ -155,6 +178,8 @@ func (cmd *change) Register(ctx context.Context, f *flag.FlagSet) {
 	f.Var(flags.NewOptionalBool(&cmd.MemoryHotAddEnabled), "memory-hot-add-enabled", "Enable memory hot add")
 	f.Var(flags.NewOptionalBool(&cmd.MemoryReservationLockedToMax), "memory-pin", "Reserve all guest memory")
 	f.Var(flags.NewOptionalBool(&cmd.CpuHotAddEnabled), "cpu-hot-add-enabled", "Enable CPU hot add")
+
+	f.StringVar(&cmd.hwUpgradePolicy, "scheduled-hw-upgrade-policy", "", fmt.Sprintf("Schedule hardware upgrade policy (%s)", strings.Join(hwUpgradePolicies, "|")))
 }
 
 func (cmd *change) Description() string {
@@ -174,7 +199,8 @@ Examples:
   vmware-rpctool "info-get guestinfo.vmname"
   govc vm.change -vm $vm -latency high
   govc vm.change -vm $vm -latency normal
-  govc vm.change -vm $vm -uuid 4139c345-7186-4924-a842-36b69a24159b`
+  govc vm.change -vm $vm -uuid 4139c345-7186-4924-a842-36b69a24159b
+  govc vm.change -vm $vm -scheduled-hw-upgrade-policy always`
 }
 
 func (cmd *change) Process(ctx context.Context) error {
@@ -206,6 +232,10 @@ func (cmd *change) Run(ctx context.Context, f *flag.FlagSet) error {
 		return err
 	}
 
+	if err = cmd.setHwUpgradePolicy(); err != nil {
+		return err
+	}
+
 	task, err := vm.Reconfigure(ctx, cmd.VirtualMachineConfigSpec)
 	if err != nil {
 		return err

vendor/github.com/vmware/govmomi/govc/vm/destroy.go

@@ -101,7 +101,9 @@ func (cmd *destroy) Run(ctx context.Context, f *flag.FlagSet) error {
 			return err
 		}
 
-		return task.Wait(ctx)
+		if err = task.Wait(ctx); err != nil {
+			return err
+		}
 	}
 
 	return nil

vendor/github.com/vmware/govmomi/govc/vm/guest/auth.go

@@ -56,9 +56,9 @@ func (flag *AuthFlag) Register(ctx context.Context, f *flag.FlagSet) {
 	value := os.Getenv(env)
 	err := flag.Set(value)
 	if err != nil {
-		fmt.Printf("couldn't set guest login values: %v", err)
+		fmt.Printf("could not set guest login values: %v", err)
 	}
-	usage := fmt.Sprintf("Guest VM credentials [%s]", env)
+	usage := fmt.Sprintf("Guest VM credentials (<user>:<password>) [%s]", env)
 	f.Var(flag, "l", usage)
 	if flag.proc {
 		f.BoolVar(&flag.auth.GuestAuthentication.InteractiveSession, "i", false, "Interactive session")
@@ -66,6 +66,10 @@ func (flag *AuthFlag) Register(ctx context.Context, f *flag.FlagSet) {
 }
 
 func (flag *AuthFlag) Process(ctx context.Context) error {
+	if flag.auth.Username == "" {
+		return fmt.Errorf("guest login username must not be empty")
+	}
+
 	return nil
 }