Sorry it's taken so long to get this tested.
I was able to set it up to work with our current code base without too
much effort, which is good.
Not sure if it was intended or if I set it up wrong, but I ran into a
small issue with trying to provision a non-imaging load. Around line 356
when adding the storage medium I couldn't get it to work without
explicitly listing the hdd file.
--medium $image_repository_path/$requestedimagename", "root")
Overall it works and looks really good and I think we can go ahead and
add it to the repository. Then it will be easier to maintain it and make
any improvements. I'll go ahead and add it to the repository today.
Since you have the ICLA on file, you're interested in becoming a
committer, and have shown that you can add great value to the community,
we'll start the next phase, which is a voting process with the mentors
and current committers.
On 12/15/10 4:31 PM, David Hutchins wrote:
Thanks for the info, I will make those changes to the module.
On Wed, Dec 15, 2010 at 4:23 PM, Andy Kurth<andy_ku...@ncsu.edu> wrote:
Thanks for contributing this.
I took a quick look at it. The utils.pm patch can be avoided by changing
the call from _getcurrentimage to $self->os->get_current_image_name(). The
OS.pm::get_current_image_name subroutine takes its place and should return
the same value.
All calls to utils.pm::_getcurrentimage should really be changed and this
subroutine should be removed. It contains code that was used before the
OS-specific code was modularized.
On 12/10/2010 8:34 AM, Aaron Peeler wrote:
Cool. Look forward to testing it out.
On 12/9/10 5:26 PM, David Hutchins wrote:
Provisioning and capture are working. The module only supports localdisk
mode at the moment, but network storage support should be trivial at this
point. If anyone wants to test this, feel free to contact me with
The code is currently attached to JIRA issue VCL-417, at
https://issues.apache.org/jira/browse/VCL-417 . vcl.pm will have to be
patched with the two patches, provisioning patch then capture patch. and
utils.pm will need the provided patch.
Virtual Computing Lab
NC State University