In general, I think it'd be more readable to have one big ```if /^(hyperv|windows)/ =~ os``` block in which we set the different attributes, instead of several small such blocks.