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

Force array context for VMs, hosts, vnets and images #13

Merged
merged 1 commit into from
Aug 1, 2017
Merged

Force array context for VMs, hosts, vnets and images #13

merged 1 commit into from
Aug 1, 2017

Conversation

djberg96
Copy link
Collaborator

@djberg96 djberg96 commented Aug 1, 2017

Addresses an issue where a single VM, host, vnet or image would not be interpreted in array context, which breaks our refresh parser. Add @() forces the issue. We already did this for clusters, and should do it for the other types of resources as well.

Addresses https://bugzilla.redhat.com/show_bug.cgi?id=1474404

@bronaghs bronaghs merged commit 5cfc104 into ManageIQ:master Aug 1, 2017
@bronaghs bronaghs added this to the Sprint 66 Ending Aug 7, 2017 milestone Aug 1, 2017
@bronaghs bronaghs added the bug label Aug 1, 2017
@djberg96
Copy link
Collaborator Author

djberg96 commented Aug 1, 2017

@miq-bot add_label fine/yes

@simaishi
Copy link
Contributor

simaishi commented Aug 9, 2017

Fine backport (to manageiq repo) details:

$ git log -1
commit 6d85537f04da62e2565aba929922f5a0028877cd
Author: Bronagh Sorota <bsorota@redhat.com>
Date:   Tue Aug 1 10:18:18 2017 -0400

    Merge pull request #13 from djberg96/powershell_script
    
    Force array context for VMs, hosts, vnets and images
    (cherry picked from commit 5cfc104942f4a4c5f48478e000cfc9b73f71b72f)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1479972

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants