Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

ps: Mark VM with * when the state can't be confirmed #808

Merged
merged 1 commit into from
Mar 15, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion cmd/ignite/run/ps.go
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ func Ps(po *PsOptions) error {
if len(outdatedVMs) > 0 {
endWarnings = append(
endWarnings,
fmt.Errorf("The symbol %s on the VM status indicates that the VM manifest on disk is not up-to-date with the actual VM status from the container runtime", oldManifestIndicator),
fmt.Errorf("The symbol %s on the VM status indicates that the VM manifest on disk may not be up-to-date with the actual VM status from the container runtime", oldManifestIndicator),
)
}
if len(errList) > 0 {
Expand Down Expand Up @@ -200,6 +200,9 @@ func fetchLatestStatus(vms []*api.VM) (outdatedVMs map[string]bool, errList []er
// Inspect the VM container using the runtime client.
ir, inspectErr := vmRuntime.InspectContainer(containerID)
if inspectErr != nil {
// Failed to get the container status. Latest status can't be
// confirmed.
outdatedVMs[vm.Name] = true
errList = append(errList, errors.Wrapf(inspectErr, "failed to inspect container for VM %s", containerID))
continue
}
Expand Down