Here is my feature branch.

https://github.com/teyo/puppet-cloudpack/tree/feature%2Fmaster%2F7896-add-support-for-image-size

Sent a pull request.

Merged into master.

I’m a little worried that the addition of the required option breaks other spec tests, and I jumped the gun merging this into master, but I’ll go through and sort them out.

I have code in a branch that fixes the tests and addresses the issues that Randall brought up. I’ll review that and send a pull request ASAP.

-teyo

Teyo,

I’ve already fixed the spec tests.

What issues did Randall bring up? I don’t see any mention of them here in the ticket and I didn’t see anything on the pull request.

I’m about to merge the spec test fixes into the master branch. Please rebase whatever other issues Randall brought up against master.

-Jeff

Fixed spec tests merged into master¶

commit 64a29eb2e7ee38a25445d408fb940175295e4716
Merge: 7e49217 aad6758
Author: Jeff McCune 
Date:   Wed Jun 29 18:59:32 2011 -0700

    Merge branch 'ticket/master/7896_fix_rspec_tests'
    
    * ticket/master/7896_fix_rspec_tests:
      (#7896) Fix spec tests broken by new required option

commit aad67589be72b7ff2f4ef021d260895327d3763e
Author: Jeff McCune 
Date:   Wed Jun 29 18:54:31 2011 -0700

    (#7896) Fix spec tests broken by new required option
    
    The patch added in #7896 broke some of the existing rspec tests by
    introducing a new required option.
    
    This change adds the required option to the existing spec tests.  All of
    them now pass.