Bug 1450527 - get-inventory.ps is returning SCVMM internal temporary templates in addition to actual templates
Summary: get-inventory.ps is returning SCVMM internal temporary templates in addition ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.9.0
Assignee: Daniel Berger
QA Contact: Leo Khomenko
URL:
Whiteboard: provider:scvmm:stats
Depends On:
Blocks: 1458947
TreeView+ depends on / blocked
 
Reported: 2017-05-12 22:41 UTC by Jeff Teehan
Modified: 2018-03-06 14:37 UTC (History)
7 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1458947 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:37:22 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jeff Teehan 2017-05-12 22:41:39 UTC
Description of problem:
This line:
https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/microsoft/infra_manager/ps_scripts/get_inventory.ps1#L34

has the -All flag set, which tells SCVMM to return transient temporary templates as well as actual templates.  I removed the -All and it worked fine, but maybe we just want to parse out any templates that have GUIDs as names.

Version-Release number of selected component (if applicable):
5.8.0.x

How reproducible:
Always

Steps to Reproduce:
1. Compare SCVMM templates in SCVMM vs CFME
2.
3.

Actual results:
6 more than expected.  All temporary templates

Expected results:
Matching numbers.

Additional info:

I modified the appliance code myself by removing the -All flag, and Updated relationships.  It worked as expected after that.

Comment 2 Daniel Berger 2017-05-16 15:43:13 UTC
https://github.com/ManageIQ/manageiq/pull/15106


Note You need to log in before you can comment on or make changes to this bug.