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

Docs fix: ID is in .metadata.uid, not .metadata.name #219

Merged
merged 1 commit into from
Jul 18, 2019

Conversation

silenceshell
Copy link
Contributor

root@pve:~# ignite inspect vm ${VM_NAME} | jq -r .metadata.uid
062ef93e37ee96ae
root@pve:~# ignite inspect vm ${VM_NAME} | jq -r .metadata.name
my-vm

Here we want to get the uid of the vm, so we should use .metadata.uid.

```root@pve:~# ignite inspect vm ${VM_NAME} | jq -r .metadata.uid
062ef93e37ee96ae
root@pve:~# ignite inspect vm ${VM_NAME} | jq -r .metadata.name
my-vm
```

here we want to get the uid of the vm, so we should use `.metadata.uid`
Copy link
Contributor

@luxas luxas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohh nice catch @silenceshell! Thanks!

LGTM 👍

@luxas luxas self-assigned this Jul 18, 2019
@luxas luxas added this to the v0.5.0 milestone Jul 18, 2019
@luxas luxas merged commit fd4837c into weaveworks:master Jul 18, 2019
@silenceshell silenceshell deleted the patch-1 branch July 19, 2019 02:03
@luxas luxas added the kind/documentation Categorizes issue or PR as related to documentation. label Aug 6, 2019
@luxas luxas changed the title vm id is in .metadata.uid, not .metadata.name Docs fix: ID is in .metadata.uid, not .metadata.name Aug 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/documentation Categorizes issue or PR as related to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants