Skip to content

Commit

Permalink
Merge pull request #21149 from cgwalters/machine-debugging
Browse files Browse the repository at this point in the history
machine/qemu: A few debugging prints
  • Loading branch information
openshift-merge-bot[bot] authored Jan 4, 2024
2 parents 0587a00 + 92f3bac commit 94da639
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pkg/machine/qemu/machine.go
Original file line number Diff line number Diff line change
Expand Up @@ -483,6 +483,7 @@ func (v *MachineVM) Start(name string, opts machine.StartOptions) error {
}
doneStarting := func() {
v.Starting = false
logrus.Debug("done starting")
if err := v.writeConfig(); err != nil {
logrus.Errorf("Writing JSON file: %v", err)
}
Expand Down Expand Up @@ -531,7 +532,7 @@ func (v *MachineVM) Start(name string, opts machine.StartOptions) error {

qemuSocketConn, err = sockets.DialSocketWithBackoffs(maxBackoffs, defaultBackoff, v.QMPMonitor.Address.Path)
if err != nil {
return err
return fmt.Errorf("failed to connect to qemu monitor socket: %w", err)
}
defer qemuSocketConn.Close()

Expand Down Expand Up @@ -578,6 +579,7 @@ func (v *MachineVM) Start(name string, opts machine.StartOptions) error {
if err := runStartVMCommand(cmd); err != nil {
return err
}
logrus.Debugf("Started qemu pid %d", cmd.Process.Pid)
defer cmd.Process.Release() //nolint:errcheck

if !opts.Quiet {
Expand Down

0 comments on commit 94da639

Please sign in to comment.