feat: add correct qemu inspect status method

[fatelei]

fix #125

[AkihiroSuda]

qemnu

typo

[AkihiroSuda]

The commit message has the typo too

[fatelei]

The commit message has the typo too

fixed

[afbjorklund]

Note that when returning "", the store will check the PID files. So it is the default implementation.

StatusUnknown       Status = ""

        if status == "" {
                inspectStatusWithPIDFiles(instDir, inst, y)
                return
        }

current code:

        var err error
        inst.DriverPID, err = ReadPIDFile(filepath.Join(instDir, filenames.PIDFile(*y.VMType)))
        if err != nil {
                inst.Status = limatype.StatusBroken
                inst.Errors = append(inst.Errors, err)
        }

        if inst.Status == limatype.StatusUnknown {
                switch {
                case inst.HostAgentPID > 0 && inst.DriverPID > 0:
                        inst.Status = limatype.StatusRunning
                case inst.HostAgentPID == 0 && inst.DriverPID == 0:
                        inst.Status = limatype.StatusStopped
                case inst.HostAgentPID > 0 && inst.DriverPID == 0:
                        inst.Errors = append(inst.Errors, errors.New("host agent is running but driver is not"))
                        inst.Status = limatype.StatusBroken
                default:
                        inst.Errors = append(inst.Errors, fmt.Errorf("%s driver is running but host agent is not", inst.VMType))
                        inst.Status = limatype.StatusBroken
                }
        }

this PR:

	qemuPIDPath := filepath.Join(instance.Dir, filenames.PIDFile(limatype.QEMU))
	if _, statErr := os.Stat(qemuPIDPath); statErr != nil {
		if errors.Is(statErr, os.ErrNotExist) {
			return ""
		}
		return limatype.StatusBroken
	}
	pid, err := store.ReadPIDFile(qemuPIDPath)
	if err != nil {
		return limatype.StatusBroken
	}

	if pid == 0 {
		return limatype.StatusBroken
	}

[afbjorklund]

I don't think this is needed, since it does the same as the default/fallback?