engine: resources: file: Remove Validate owner/group Checks

purpleidea / mgmt

Next generation distributed, event-driven, parallel config management!

https://purpleidea.com/tags/mgmtconfig/

GNU General Public License v3.0

3.47k stars 308 forks source link

engine: resources: file: Remove Validate owner/group Checks #763

Closed cianyleow closed 1 month ago

cianyleow commented 1 month ago

The owner/group of a file should not be validated on the host until runtime. This removes the checks in Validate() that were happening before the execution of the resource graph (and therefore bound to fail if the system was being bootstrapped).

Fixes #762

purpleidea commented 1 month ago

Patch looks perfect to me. Thank you!

(Will merge later today/tomorrow.)

If you're looking for a next patch please LMK.

purpleidea commented 1 month ago

This has been merged, thank you! As a bonus you can forever gloat that your first mgmt patch only removed code =D