Bug 1549667 - [RFE] SCVMM provider refresh error message issue if provider user doesn't have access to VMM service
Summary: [RFE] SCVMM provider refresh error message issue if provider user doesn't hav...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.9.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: GA
: 5.10.0
Assignee: Adam Grare
QA Contact: Tasos Papaioannou
URL:
Whiteboard: scvmm
Depends On:
Blocks: 1560004
TreeView+ depends on / blocked
 
Reported: 2018-02-27 15:37 UTC by Leo Khomenko
Modified: 2019-02-11 14:07 UTC (History)
5 users (show)

Fixed In Version: 5.10.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1560004 (view as bug list)
Environment:
Last Closed: 2019-02-11 14:07:00 UTC
Category: Feature
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 2 Adam Grare 2018-03-06 20:17:17 UTC
Leo can you include the backtrace from the refresh showing which line the exception came from?

Comment 4 CFME Bot 2018-03-23 13:53:16 UTC
New commit detected on ManageIQ/manageiq-providers-scvmm/master:

https://github.com/ManageIQ/manageiq-providers-scvmm/commit/a2504adebc24957f439c3af4c6c11295a1803b35
commit a2504adebc24957f439c3af4c6c11295a1803b35
Author:     Adam Grare <agrare>
AuthorDate: Thu Mar 22 17:28:41 2018 -0400
Commit:     Adam Grare <agrare>
CommitDate: Thu Mar 22 17:28:41 2018 -0400

    Handle nil ems inventory from credential issues

    If the provided credentials do not have enough permissions to access the
    VMM management server the following is printed to stderr:
    "You cannot contact the VMM management server.
    The credentials provided have insufficient privileges on localhost."

    but an inventory hash is still returned that contains:
    {
      "ems": nil,
      "vms": [],
      "hosts": [],
      ...
    }

    before continuing with the refresh this checks if the ems data returned
    is blank to catch these issues.

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1549667

 app/models/manageiq/providers/microsoft/infra_manager/refresh_parser.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 7 CFME Bot 2018-03-26 19:27:58 UTC
New commit detected on ManageIQ/manageiq-providers-scvmm/master:

https://github.com/ManageIQ/manageiq-providers-scvmm/commit/08895dbaea79a092332b898a838e98ac9a8aa3ce
commit 08895dbaea79a092332b898a838e98ac9a8aa3ce
Author:     Adam Grare <agrare>
AuthorDate: Fri Mar 23 12:13:48 2018 -0400
Commit:     Adam Grare <agrare>
CommitDate: Fri Mar 23 12:13:48 2018 -0400

    Check connection to VMM when verifying credentials

    When verifying credentials connect to the VMM service to confirm that
    the credentials are privileged enough.

    With this now verify fails with:
    Credential validation was not successful:
    Unable to connect: You cannot contact the VMM management server.
    The credentials provided have insufficient privileges on DEV-SCVMM2K12\Administrator.

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

 app/models/manageiq/providers/microsoft/infra_manager.rb | 16 +-
 app/models/manageiq/providers/microsoft/infra_manager/ps_scripts/test_connection.ps1 | 12 +
 spec/models/manageiq/providers/microsoft/infra_manager_spec.rb | 24 +-
 3 files changed, 43 insertions(+), 9 deletions(-)

Comment 9 Tasos Papaioannou 2018-10-01 15:03:13 UTC
Verified on 5.10.0.16.


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