test: read and set nas.backup.incremental.enabled for the test zone

[jmsperu]

Description

nas.backup.incremental.enabled is a zone-scoped setting, but test_backup_recovery_nas.py read and updated it without a zoneid, so a zone override in the test environment would make the suite read the wrong value (and restore the wrong one in tearDownClass). This passes zoneid=cls.zone.id on the read and on both updates, the same way test_backup_recovery_veeam.py handles zone-scoped settings.

Noted by the Copilot review on the 4.22 backport #13877; this is the main-side fix so both branches stay in sync.

Types of changes

• Test change only (no functional change)

How Has This Been Tested?

Python syntax check; the change is limited to three Marvin calls in setUpClass/tearDownClass.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 19.73%. Comparing base (4e4a26b) to head (b442b73).

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #13960      +/-   ##
============================================
- Coverage     19.73%   19.73%   -0.01%     
+ Complexity    19956    19954       -2     
============================================
  Files          6371     6371              
  Lines        575765   575765              
  Branches      70478    70478              
============================================
- Hits         113642   113632      -10     
- Misses       449766   449779      +13     
+ Partials      12357    12354       -3     

Flag	Coverage Δ
uitests	3.41% <ø> (ø)
unittests	21.01% <ø> (-0.01%)	⬇️

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

☔ View full report in Codecov by Harness.
📢 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[abh1sar]

@blueorangutan package

[blueorangutan]

@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

[abh1sar]

@jmsperu
This is not an issue worth fixing.
The config is zone scoped, but it can be set at the global level as well.
If the zone scoped value is set then that will take precedence over the global value.
This is same with all config. I don't think this requires fixing. The test works with global scope value.

[blueorangutan]

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 19000