This issue is the same issue from rodjek's original project, rodjek#58. I'm just pulling it into yo61's fork because I agree with the original issue.

There is a PR that proposed a fix at rodjek#59

Currently the package's state is not a class parameter in logrotate::base and its default value of 'latest' doesn't seem to be justified.

If ever YUM is used by another user/process at the same time as a Puppet run or if the repositories are too slow to answer, that shows up as a failure in the run.