Actually, while this may not throw and error, I suspect that is because we overwrite these default string values in seeds.rb.  I think unless there are not any defaults on the underlying puppet we call here, it may not be of any benefit to have these overrides, just let the controller pass in what is needed, as we do now,