(#15446) Improve handling of user/group removal on rpm removal
(#15446) Improve handling of user/group removal on rpm removal

Prior to this commit, we had a few bugs in our handling of
user/group removal during rpm removal:

1. We were not conditioning the calls to groupdel / userdel to
   avoid running them during an upgrade, which meant that we
   were trying to delete them even during upgrades... which would
   have been bad.
2. We had an || where we needed an &&, so we weren't actually
   calling the groupdel / userdel commands.
3. We were hard-coding the user's home dir to a bad path.
4. We had some '-r' flags that were wrong and/or unnecessary.

This commit fixes those issues.

johnduarte/puppetdb,puppetlabs/puppetdb,waynr/puppetdb,rbrw/puppetdb,wkalt/puppetdb,kbrezina/puppetdb,puppetlabs/puppetdb,ajroetker/puppetdb,kbrezina/puppetdb,shrug/puppetdb,grimradical/puppetdb,ajroetker/puppetdb,highb/puppetdb,mullr/puppetdb,nfagerlund/puppetdb,grimradical/puppetdb,nfagerlund/puppetdb,senior/puppetdb,senior/puppetdb,shrug/puppetdb,mullr/puppetdb,kbarber/puppetdb,rbrw/puppetdb,puppetlabs/puppetdb,melissa/puppetdb,kbrezina/puppetdb,waynr/puppetdb,senior/puppetdb,mullr/puppetdb,waynr/puppetdb,senior/puppetdb,jantman/puppetdb,cprice404/puppetdb,wkalt/puppetdb,shrug/puppetdb,github/puppetlabs-puppetdb,highb/puppetdb,wkalt/puppetdb,melissa/puppetdb,kbarber/puppetdb,johnduarte/puppetdb,kbrezina/puppetdb,johnduarte/puppetdb,nfagerlund/puppetdb,highb/puppetdb,ajroetker/puppetdb,github/puppetlabs-puppetdb,melissa/puppetdb,johnduarte/puppetdb,wkalt/puppetdb,cprice404/puppetdb,puppetlabs/puppetdb,puppetlabs/puppetdb,grimradical/puppetdb,rbrw/puppetdb,mullr/puppetdb,ajroetker/puppetdb,shrug/puppetdb,rbrw/puppetdb,rbrw/puppetdb,waynr/puppetdb,jantman/puppetdb,kbarber/puppetdb,grimradical/puppetdb,github/puppetlabs-puppetdb,highb/puppetdb,jantman/puppetdb,mullr/puppetdb,cprice404/puppetdb,kbarber/puppetdb