proxmox_kvm: get_vmid fails altogether if any node is offline

[tobiasmcnulty]

SUMMARY

If a node is offline, proxmoxer returns a dict describing a VM without a name attribute, like so::

{u'status': u'unknown', u'vmid': 1002, u'type': u'qemu', u'id': u'qemu/1002', u'node': u'pve2'}

This causes get_vmid() to fail since there's no name key in this dict. Worse, this error is hidden and reported as "no such VM in cluster" for any and all VMs, no matter whether they're online or not.

ISSUE TYPE

• Bug Report

COMPONENT NAME

ANSIBLE VERSION

$ ansible --version
ansible 2.7.0
  config file = None
  configured module search path = ['/Users/tobias/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /Users/tobias/.virtualenvs/proxmox-infra/lib/python3.6/site-packages/ansible
  executable location = /Users/tobias/.virtualenvs/proxmox-infra/bin/ansible
  python version = 3.6.2 (v3.6.2:5fd33b5926, Jul 16 2017, 20:11:06) [GCC 4.2.1 (Apple Inc. build 5666) (dot 3)]

CONFIGURATION

[blank output]

[Akasurde]

!component =lib/ansible/modules/cloud/misc/proxmox_kvm.py

[ansibot]

cc @helldorado @joshainglis @karmab
click here for bot help

[LaurentDumont]

What is the status of this. I was just bitten by this when I lost a node on a 4 nodes Proxmox cluster. I knew it was working right before I lost a node so I dig up this issue.

What is required for a Merge?

[ansibot]

Thank you very much for your interest in Ansible. Ansible has migrated much of the content into separate repositories to allow for more rapid, independent development. We are closing this issue/PR because this content has been moved to one or more collection repositories.

• lib/ansible/modules/cloud/misc/proxmox_kvm.py -> https://galaxy.ansible.com/community/general

For further information, please see:
https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md