Bug 770445

Summary: "undefined method `name' for nil:NilClass " : When created a catalog entry with wrong hwp
Product: [Retired] CloudForms Cloud Engine Reporter: Shveta <ssachdev>
Component: aeolus-conductorAssignee: Petr Blaho <pblaho>
Status: CLOSED ERRATA QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, athomas, deltacloud-maint, morazi, slinaber, ssachdev
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-15 21:30:51 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
err
none
hwp_does_not_exist none

Description Shveta 2011-12-26 16:17:09 UTC
Created attachment 549607 [details]
err

Description of problem:


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


How reproducible:


Steps to Reproduce:
1. [root@hp-bl260cg5-01 ~]# aeolus-image import --provider_account ssachdev --id ami-31d41658

Image: 349e9e79-db0e-4ae2-bd6b-7cdeeeee883b
Build: 8ae6a9a9-804f-4eee-83dd-7b7046ca3af0
Target Image: 222e2ee6-cc0c-4a17-a780-d3c03eb04fbb
Provider Image: 200998b0-9c91-4a5d-94c4-555570d3442e
Status: COMPLETE

2. create a deployable def, in this definition use a hwp that does not exist in
the system
3. Go to Monitor Page and click on the catalog link 
"undefined method `name' for nil:NilClass "
  
Actual results:


Expected results:


Additional info:

rpm -qa|grep aeolus
aeolus-conductor-doc-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-configure-2.5.0-0.20111222173430git17b704a.el6.noarch
rubygem-aeolus-image-0.3.0-0.20111222173411gitc13b654.el6.noarch
rubygem-aeolus-cli-0.3.0-0.20111222173356git3cd6277.el6.noarch
aeolus-all-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-conductor-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-conductor-daemons-0.8.0-0.20111222233342gitd98cb57.el6.noarch

Comment 1 wes hayutin 2012-01-10 17:10:44 UTC
adding to ce-sprint-next

Comment 2 wes hayutin 2012-01-10 17:13:33 UTC
adding to ce-sprint-next

Comment 3 wes hayutin 2012-01-12 16:32:32 UTC
adding to ce-sprint

Comment 4 wes hayutin 2012-01-12 16:39:23 UTC
removing ce-sprint-next tracker

Comment 5 Petr Blaho 2012-01-23 16:24:46 UTC
I have discovered another bug in displaying assemblies for deployment during writing test for this bug.

When there are two assemblies defined in deployable xml document with the same image id the launch from catalog page will show the first assembly twice.

There some bad code in the view - consulted with jprovazn.

I will work on this too. I will post solution soon.

Comment 6 Petr Blaho 2012-01-25 12:54:34 UTC
commit 8bcb08553ce4d0a6f40f8a5ce81eda1954bc8ac8
Author: Petr Blaho <pblaho>
Date:   Tue Jan 24 15:32:06 2012 +0100

    Correct displaying of assemblies
    
    on launch from catalog page
    
    https://bugzilla.redhat.com/show_bug.cgi?id=770445

Comment 7 Steve Linabery 2012-01-31 22:18:36 UTC
8bcb08553ce4d0a6f40f8a5ce81eda1954bc8ac8 in aeolus-conductor-0.8.0-17

Comment 8 Shveta 2012-02-01 09:52:06 UTC
Created attachment 558782 [details]
hwp_does_not_exist

Comment 9 Shveta 2012-02-01 09:53:01 UTC
Verified in

rpm -qa|grep aeolus
aeolus-conductor-daemons-0.8.0-17.el6.noarch
aeolus-configure-2.5.0-11.el6.noarch
aeolus-conductor-doc-0.8.0-17.el6.noarch
rubygem-aeolus-image-0.3.0-7.el6.noarch
aeolus-all-0.8.0-17.el6.noarch
aeolus-conductor-0.8.0-17.el6.noarch
rubygem-aeolus-cli-0.3.0-7.el6.noarch

Comment 10 errata-xmlrpc 2012-05-15 21:30:51 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-0583.html