I've posted my notes on gerrit. Short version is: deploy-ovirt-experimental user's homedir has "repodata" (and a few gigs of other strange files) inside it so when variables are empty the script will still work but against / and cause the issue we saw.

I don't know if it's necessary, because my code needs to run after an if statement if there is a directory with repodata.
what happened was that it ignored my code till the end of the function so i don't know if we can use it to protect us from that..

As far as I understand, the root cause is still undefined yet we haven't seen failures in test mode.

The partition is still filling up regularly so I think we should re-add the cleanup yet add some kind of protection. Currently the path is composed as "$pkg_dst/$dir" which is what caused the issue when values were empty. Maybe we can define this as a variable and only continue when its length is more than 16 symbols? Or do some other check to avoid running outside the boundary? Same needs to be done before passing the output to "rm" IMHO - i.e. ensure all files have "$pkg_dst/$dir" in them (use grep?)

logs are good atm. not a single failure

monitoring the log files under /home/deploy-ovirt-experimental/"date".log