Right, I thought something like this, but misread it. Both solutions have their benefits, duplication vs coupling. Since we already require foreman for ```foreman::install::repos```, I think this may not be a bad solution but I'm still slightly leaning toward the cache_data construct but no strong preference.