gwdg/rOCCI — pull request #17 by arax (2012-10-28 22:23:40+00:00)
Title: Fixed OVA/OVF parser and tests

Commits

* 8bdda2f970
* 00b4e5f867
* 5dbcfbf6fb

broke OVA/OVF parser and rspec tests. They also changed parser's behavior when parsing OVA/OVF. GBs are used as memory units when parsing OVA/OVF and MBs when parsing OCCI JSON. This is confusing since rspec tests now have to use

compute_resources.first.attributes.occi!.compute!.memory.should == 0.25

for OVA/OVF parsing and

compute_resources.first.attributes.occi!.compute!.memory.should == 256

for OCCI JSON parsing ... not to mention the confusion it causes on the upper levels.

This pull request includes a quick fix, but we should definitely avoid such changes in the future (at least without proper pull requests). How should we proceed?