🔥 Remove non-root directory logic
All Pull Requests:
Check all that apply:
- [x] Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
- [ ] Is this PR to correct an issue?
- [x] Is this PR an enhancement?
Complete Description of Additions/Changes:
The Insights Client can only be run as root. There is a condition that prevents it from running for non-root users. The logic determining log and lib directories for a non-root user is never used.
Looks good, but keep in mind we do want to be able to run as non-root someday. Until then though, :+1:
If non-root ever turns up as a requirement again, we'll need to take a new, holistic approach to it anyway, so I'm fine dropping unused code in the interim.
Hi, @Glutexo! We've identified that setup pipeline is failing for RHEL6 on "QE Pull Request Testing", but it is working as expected for RHEL79, RHEL84 and RHEL90.
This issue has been reported and is being analyzed here: RHCLOUD-17392.
Can one of the admins verify this patch?