fog / fog-libvirt

libvirt provider for fog
MIT License
16 stars 42 forks source link

fix error raising for cloud-init genisoimage #31

Closed mmoll closed 7 years ago

mmoll commented 7 years ago

Also adjust the error message so it's easier to find the cause. I didn't have genisoimage installed on the host in question...

plribeiro3000 commented 7 years ago

Thanks @mmoll .

Could you use the standard format instead?

raise Fog::Errors::Error, "Couldn't generate cloud-init iso disk with genisoimage."
mmoll commented 7 years ago

@plribeiro3000 I can do that, but a quick "git grep" over fog-libvirt shows zero occourences of that format, maybe the others should get changed then, too?

plribeiro3000 commented 7 years ago

Hmm. Yeah, they can be changed but we can leave this to another PR then.

👍

plribeiro3000 commented 7 years ago

Thanks for your patience. =)