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

ovirt_vms: Pass correct VM entity to create method #37249

Merged
merged 1 commit into from Mar 10, 2018

Conversation

machacekondra
Copy link
Contributor

SUMMARY

When user is using following task:

ovirt_vms:
  name: myvm
  state: stopped
  serial_console: true

we fail on:

{'changed': False,
                    u'exception': u'Traceback (most recent call last):\n  File "/tmp/ansible_j18cwF/ansible_module_ovirt_vms.py", line 1909, in main\n    clone_permissions=module.par
{u'vmstop_result': {u'msg': u"'NoneType' object has no attribute 'enabled'", u'failed': True, u'exce.__doc__: "dict() -> new empty dictionary\ndict(mapping) -> new dictionary initialized from a mapping object's\n    (key, value) pairs\ndict(iterable) -> new dictionary initialized as if via:\n    d = {}\n    for k, v in iterable:\n        d[k] = v\ndict(**kwargs) -> new dictionary initialized with the name=value pairs\n    in the keyword argument list.  For example:  dict(one=1, two=2)"
{u'vmstop_result': {u'msg': u"'NoneType' object has no attribute 'enabled'", u'failed': True, u'exce.__hash__: None
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ovirt_vms

ANSIBLE VERSION
2.5.0.rc1
ADDITIONAL INFORMATION

@ansibot
Copy link
Contributor

ansibot commented Mar 9, 2018

@ansibot ansibot added bug This issue/PR relates to a bug. cloud community_review In order to be merged, this PR must follow the community review workflow. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. virt Virt community (incl. QEMU, KVM, libvirt, ovirt, RHV and Proxmox) owner_pr This PR is made by the module's maintainer. support:community This issue/PR relates to code supported by the Ansible community. labels Mar 9, 2018
@s-hertel s-hertel removed the needs_triage Needs a first human triage before being processed. label Mar 9, 2018
Copy link
Contributor

@mwperina mwperina left a comment

Choose a reason for hiding this comment

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

shipit

@mwperina
Copy link
Contributor

Btw, we should backport to 2.5.1, right?

@ansibot ansibot added automerge This PR was automatically merged by ansibot. shipit This PR is ready to be merged by Core and removed community_review In order to be merged, this PR must follow the community review workflow. labels Mar 10, 2018
@ansibot ansibot merged commit c5c78ab into ansible:devel Mar 10, 2018
gerrit-ovirt-org pushed a commit to oVirt/ovirt-hosted-engine-setup that referenced this pull request Mar 12, 2018
Backport
ansible/ansible#37249

Change-Id: I7f6e740cfd9d23157b26311545f37a8372f14d7a
Bug-Url: https://bugzilla.redhat.com/1549082
Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>
gerrit-ovirt-org pushed a commit to oVirt/ovirt-hosted-engine-setup that referenced this pull request Mar 12, 2018
Backport
ansible/ansible#37249

Change-Id: I7f6e740cfd9d23157b26311545f37a8372f14d7a
Bug-Url: https://bugzilla.redhat.com/1549082
Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>
(cherry picked from commit 92d5a2a)
privateip added a commit that referenced this pull request Mar 13, 2018
@dagwieers dagwieers added ovirt oVirt and RHV community and removed virt Virt community (incl. QEMU, KVM, libvirt, ovirt, RHV and Proxmox) labels Feb 21, 2019
@ansible ansible locked and limited conversation to collaborators Apr 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge This PR was automatically merged by ansibot. bug This issue/PR relates to a bug. cloud module This issue/PR relates to a module. ovirt oVirt and RHV community owner_pr This PR is made by the module's maintainer. shipit This PR is ready to be merged by Core support:community This issue/PR relates to code supported by the Ansible community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants