Bug 767068

Summary: Require validation for aeolus-cli delete --image
Product: [Retired] CloudForms Cloud Engine Reporter: Aziza Karol <akarol>
Component: rubygem-aeolus-cliAssignee: Martyn Taylor <mtaylor>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0   
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-08-30 17:13:35 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Aziza Karol 2011-12-13 06:02:41 UTC
Description of problem:


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


How reproducible:


Steps to Reproduce:
Try deleting an invalid image.

[root@dell-pe1950-01 templates]# aeolus-cli delete --image lkjlkj



  
Actual results:
No validation done.


Expected results:

ERROR:  ImageStatusNotFound => Could not find status for image lkjlkj 
should be displayed for invalid image.

Additional info:
[root@dell-pe1950-01 templates]# rpm  -qa | grep aeolus
rubygem-aeolus-image-0.2.0-1.el6.noarch
aeolus-conductor-doc-0.7.0-4.el6.noarch
aeolus-configure-2.4.0-3.el6.noarch
rubygem-aeolus-cli-0.2.0-3.el6.noarch
aeolus-all-0.7.0-4.el6.noarch
aeolus-conductor-0.7.0-4.el6.noarch
aeolus-conductor-daemons-0.7.0-4.el6.noarch

Comment 1 Martyn Taylor 2011-12-20 16:21:53 UTC
commit 769a8355cd36eed8527581aa8708f153119b7ef0
Author: Martyn Taylor <mtaylor>
Date:   Fri Dec 16 15:33:21 2011 +0000

    BZ#767068 Fixed Error handling on delete

Comment 2 Aziza Karol 2012-01-02 05:30:28 UTC
Validation for aeolus-image delete done.

[root@dell-pe1950-01 templates]# aeolus-image delete --image esfjsdfds

ERROR:  ImageDeleteFailure => Could not find Image esfjsdfds


verified on:
[root@dell-pe1950-01 templates]# rpm -qa | grep aeolus
aeolus-conductor-daemons-0.8.0-0.20111222233342gitd98cb57.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-configure-2.5.0-0.20111222173430git17b704a.el6.noarch
aeolus-conductor-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-conductor-doc-0.8.0-0.20111222233342gitd98cb57.el6.noarch

Comment 3 wes hayutin 2012-01-03 16:09:35 UTC
bugs in verified, removing from ce-sprint-next