Bug 1511083

Summary: OSP Provider refresh fails with error: Validation failed: Name can't be blank
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: ProvidersAssignee: Sam Lucidi <slucidi>
Status: CLOSED CURRENTRELEASE QA Contact: Ido Ovadia <iovadia>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: cpelland, gblomqui, iovadia, jfrey, jhardy, maufart, obarenbo
Target Milestone: Beta 2Keywords: Regression
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.0.9 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1510067 Environment:
Last Closed: 2018-03-06 15:07:46 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Openstack Target Upstream Version:
Embargoed:
Bug Depends On: 1510067    
Bug Blocks:    

Comment 2 Greg Blomquist 2017-11-16 23:18:10 UTC
Gaprindashvili backport details:

$ git log -1
commit 7e4a43e49c0b7e0b2109cb6c3642a65c4269eb14
Author: Marek Aufart <aufi.cz>
Date:   Wed Nov 8 07:55:44 2017 +0100

    Merge pull request #146 from mansam/handle-images-with-empty-strings-for-names
    
    If an image name is "", use the image's id instead
    (cherry picked from commit 1f64915cd2de7c41303987ac9d6ee448d7694f0c)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1511083

Comment 3 CFME Bot 2017-11-29 20:46:21 UTC
New commit detected on ManageIQ/manageiq-providers-openstack/gaprindashvili:
https://github.com/ManageIQ/manageiq-providers-openstack/commit/7e4a43e49c0b7e0b2109cb6c3642a65c4269eb14

commit 7e4a43e49c0b7e0b2109cb6c3642a65c4269eb14
Author:     Marek Aufart <aufi.cz>
AuthorDate: Wed Nov 8 07:55:44 2017 +0100
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Nov 14 18:34:31 2017 -0500

    Merge pull request #146 from mansam/handle-images-with-empty-strings-for-names
    
    If an image name is "", use the image's id instead
    (cherry picked from commit 1f64915cd2de7c41303987ac9d6ee448d7694f0c)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1511083

 .../manageiq/providers/openstack/inventory/parser/cloud_manager.rb      | 2 +-
 app/models/manageiq/providers/openstack/refresh_parser_common/images.rb | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comment 4 CFME Bot 2017-11-29 20:46:26 UTC
New commit detected on ManageIQ/manageiq-providers-openstack/gaprindashvili:
https://github.com/ManageIQ/manageiq-providers-openstack/commit/8398a57d9e6fef9fa10efd243dc71be4039bec9b

commit 8398a57d9e6fef9fa10efd243dc71be4039bec9b
Author:     Samuel Lucidi <mansam.edu>
AuthorDate: Thu Nov 9 10:05:49 2017 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Nov 14 18:36:45 2017 -0500

    Merge pull request #148 from Ladas/make_sure_volume_template_do_not_have_blank_name
    
    Make sure volume template has name
    (cherry picked from commit 5a99b6f3bdd088e5ab61c1cfef551e7822ecd67a)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1511083

 .../manageiq/providers/openstack/inventory/parser/cloud_manager.rb    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comment 5 Ido Ovadia 2017-12-11 13:58:25 UTC
Verified
========
5.9.0.12