[release-1.34] Move data dir into position before creating CNI symlinks

[brandond]

Proposed Changes

Move data dir into position before creating CNI symlinks

Addresses issue where CNI bin symlinks were created with targets that were not yet valid, which caused external container runtimes that rely on filesystem notifications to detect changes to think that the bins did not exist.

Types of Changes

Bugfix

Verification

See linked issue - user will probably need to validate as this is a fairly odd configuration.

Testing

Linked Issues

• [Release-1.34] - CNI bin symlinks are initially created with invalid targets #12872

User-Facing Change

Further Comments

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.47%. Comparing base (4d1f1af) to head (d92de0b).
⚠️ Report is 2 commits behind head on release-1.34.

❗ There is a different number of reports uploaded between BASE (4d1f1af) and HEAD (d92de0b). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (4d1f1af)	HEAD (d92de0b)
e2etests	1	0

Additional details and impacted files

@@                Coverage Diff                @@
##           release-1.34   #12876       +/-   ##
=================================================
- Coverage         40.62%   20.47%   -20.16%     
=================================================
  Files               185      182        -3     
  Lines             18937    18869       -68     
=================================================
- Hits               7694     3863     -3831     
- Misses            10063    14570     +4507     
+ Partials           1180      436      -744     

Flag	Coverage Δ
e2etests	?
unittests	20.47% <ø> (+0.01%)	⬆️

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.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.