Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reveal UUID of VM in API and GUI #16

Closed
dn0 opened this issue Dec 12, 2016 · 2 comments
Closed

Reveal UUID of VM in API and GUI #16

dn0 opened this issue Dec 12, 2016 · 2 comments
Assignees
Milestone

Comments

@dn0
Copy link
Member

dn0 commented Dec 12, 2016

  • add support for working with VMs with uuid as required parameter (API)
  • show uuid in GET vm_manage (/vm/(hostname)) and GET vm_define (/vm/(hostname)/define) output (read-only)
  • show uuid in VM details (GUI)
@dn0 dn0 added this to the 2.3.2 milestone Dec 12, 2016
secult pushed a commit that referenced this issue Dec 12, 2016
secult pushed a commit that referenced this issue Dec 12, 2016
Hostname is always used as a not-keyword argument so this seems to be safe
secult pushed a commit that referenced this issue Dec 13, 2016
secult pushed a commit that referenced this issue Dec 13, 2016
secult pushed a commit that referenced this issue Dec 13, 2016
secult pushed a commit that referenced this issue Dec 13, 2016
secult pushed a commit that referenced this issue Dec 14, 2016
secult pushed a commit that referenced this issue Dec 20, 2016
secult pushed a commit that referenced this issue Dec 22, 2016
… edited hostname validator #16

hostname != other uuid
secult pushed a commit that referenced this issue Dec 23, 2016
secult added a commit that referenced this issue Dec 23, 2016
Changed all the VM-related API calls to be able to handle UUID-based requests instead of only hostname. Closes  #16

* Changed get_vm function to handle also UUID #16

* Edited hostname validator to check whether we don't assign same UUID as a hostname to a different uuid #16

* Removed obsolete qs_filter function
@secult
Copy link
Contributor

secult commented Dec 23, 2016

Closed in v2.4

@secult secult closed this as completed Dec 23, 2016
@dn0
Copy link
Member Author

dn0 commented Dec 23, 2016

The doc/api/source/api/vm_*.rst files need to updated (hostname -> hostname_or_uuid)

@dn0 dn0 reopened this Dec 23, 2016
secult pushed a commit that referenced this issue Dec 23, 2016
@secult secult closed this as completed Dec 23, 2016
@secult secult reopened this Dec 23, 2016
secult pushed a commit that referenced this issue Dec 23, 2016
@secult secult closed this as completed Dec 23, 2016
dn0 added a commit that referenced this issue Dec 24, 2016
dn0 added a commit that referenced this issue Apr 20, 2017
After introduction of hostname_or_uuid for VMs (#16) the
`message_callback` had received the hostname attribute in a
`hostname_or_uuid` field. In case of a started task the hostname field
will be stored in the apiview - so the apiview.hostname should be used a
primary source of VM hostname.
dn0 added a commit that referenced this issue Apr 20, 2017
After introduction of hostname_or_uuid for VMs (#16) the
`message_callback` had received the hostname attribute in a
`hostname_or_uuid` field. In case of a started task the hostname field
is stored in the apiview - so the apiview.hostname should be used a
primary source of VM hostname.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants