Fix K3S_DATA_DIR when running as non-root user

[unguiculus]

Proposed Changes

When running the generated kill and uninstall scripts as non-root user, the environment variable K3S_DATA_DIR got lost, because sudo by default doesn't preserve the environment. This is fixed by adding the --preserve-env flag.

Types of Changes

Bugfix

Verification

• Set environment variable K3S_DATA_DIR.
• Run the generated k3s-uninstall.sh script as non-root user and check the log output.
• Verify that the $K3S_DATA_DIR directory got cleaned.

Fixes: #11386

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.52%. Comparing base (0e15269) to head (329d73f).
Report is 1 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (0e15269) and HEAD (329d73f). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (0e15269)	HEAD (329d73f)
e2etests	7	6

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #11378      +/-   ##
==========================================
- Coverage   46.73%   41.52%   -5.21%     
==========================================
  Files         179      179              
  Lines       18600    18600              
==========================================
- Hits         8693     7724     -969     
- Misses       8553     9677    +1124     
+ Partials     1354     1199     -155     

Flag	Coverage Δ
e2etests	32.30% <ø> (-9.63%)	⬇️
inttests	18.82% <ø> (ø)
unittests	13.82% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[brandond]

Thanks for the fix! Can you open an issue describing the problem this fixes, so our QA team can validate that the issue is resolved by this change?

[brandond]

I'd prefer to pass through only this one necessary env var, instead of passing through ALL of them.