Remove --no-creds

The --no-creds breaks the ssh command. Ssh will unconditionally use
the ssh key path from the VM json config, passing --no-creds sets
the path to "".

Since we don't have any open issue about it, let's remove this
option until someone ask for it.

Signed-off-by: German Maglione <gmaglione@redhat.com>

Author: germag

cmd/run.go

@@ -19,7 +19,6 @@ type osVmConfig struct {
 	CloudInitDir    string
 	KsFile          string
 	Background      bool
-	NoCredentials   bool
 	RemoveVm        bool // Kill the running VM when it exits
 	RemoveDiskImage bool // After exit of the VM, remove the disk image
 	Quiet           bool
@@ -47,7 +46,6 @@ func init() {
 	runCmd.Flags().StringVar(&vmConfig.CloudInitDir, "cloudinit", "", "--cloudinit <cloud-init data directory>")
 
 	runCmd.Flags().StringVar(&diskImageConfigInstance.Filesystem, "filesystem", "", "Override the root filesystem (e.g. xfs, btrfs, ext4)")
-	runCmd.Flags().BoolVar(&vmConfig.NoCredentials, "no-creds", false, "Do not inject default SSH key via credentials; also implies --background")
 	runCmd.Flags().BoolVarP(&vmConfig.Background, "background", "B", false, "Do not spawn SSH, run in background")
 	runCmd.Flags().BoolVar(&vmConfig.RemoveVm, "rm", false, "Remove the VM and it's disk when the SSH session exits. Cannot be used with --background")
 	runCmd.Flags().BoolVar(&vmConfig.Quiet, "quiet", false, "Suppress output from bootc disk creation and VM boot console")
@@ -108,7 +106,6 @@ func doRun(flags *cobra.Command, args []string) error {
 	err = bootcVM.Run(vm.RunVMParameters{
 		Cmd:           cmd,
 		CloudInitDir:  vmConfig.CloudInitDir,
-		NoCredentials: vmConfig.NoCredentials,
 		CloudInitData: flags.Flags().Changed("cloudinit"),
 		RemoveVm:      vmConfig.RemoveVm,
 		Background:    vmConfig.Background,

pkg/vm/vm.go

@@ -54,7 +54,6 @@ type NewVMParameters struct {
 type RunVMParameters struct {
 	VMUser        string //user to use when connecting to the VM
 	CloudInitDir  string
-	NoCredentials bool
 	CloudInitData bool
 	SSHIdentity   string
 	SSHPort       int

pkg/vm/vm_darwin.go

@@ -84,14 +84,6 @@ func (b *BootcVMMac) Run(params RunVMParameters) (err error) {
 	b.vmUsername = params.VMUser
 	b.sshIdentity = params.SSHIdentity
 
-	if params.NoCredentials {
-		b.sshIdentity = ""
-		if !b.background {
-			fmt.Print("No credentials provided for SSH, using --background by default")
-			b.background = true
-		}
-	}
-
 	execPath, err := os.Executable()
 	if err != nil {
 		return fmt.Errorf("getting executable path: %w", err)

pkg/vm/vm_linux.go

@@ -123,14 +123,6 @@ func (v *BootcVMLinux) Run(params RunVMParameters) (err error) {
 	v.vmUsername = params.VMUser
 	v.sshIdentity = params.SSHIdentity
 
-	if params.NoCredentials {
-		v.sshIdentity = ""
-		if !v.background {
-			fmt.Print("No credentials provided for SSH, using --background by default")
-			v.background = true
-		}
-	}
-
 	if v.domain != nil {
 		isRunning, err := v.IsRunning()
 		if err != nil {

pkg/vm/vm_test.go

@@ -96,7 +96,6 @@ func runTestVM(bootcVM vm.BootcVM) {
 	err := bootcVM.Run(vm.RunVMParameters{
 		VMUser:        "root",
 		CloudInitDir:  "",
-		NoCredentials: false,
 		CloudInitData: false,
 		SSHPort:       22,
 		Cmd:           []string{},