Drop options.ext.loadimpact references

[andrewslotin]

What?

A follow-up PR to use options.cloud instead of options.ext.loadimpact.

Why?

The legacy options.ext.loadimpact is superseded by options.cloud.

Checklist

• I have performed a self-review of my code.
• I have run linter locally (make lint) and all checks pass.
• I have run tests locally (make tests) and all tests pass.
• I have commented on my code, particularly in hard-to-understand areas.

Related PR(s)/Issue(s)

#3348

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b5a6feb) 73.55% compared to head (85197d6) 73.51%.

❗ Current head 85197d6 differs from pull request most recent head 277d276. Consider uploading reports for the commit 277d276 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3602      +/-   ##
==========================================
- Coverage   73.55%   73.51%   -0.05%     
==========================================
  Files         277      275       -2     
  Lines       20228    20223       -5     
==========================================
- Hits        14879    14866      -13     
- Misses       4401     4405       +4     
- Partials      948      952       +4     

Flag	Coverage Δ
macos	73.48% <100.00%> (+0.01%)	⬆️
ubuntu	73.51% <100.00%> (+0.03%)	⬆️
windows	?

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.

[olegbespalov]

@andrewslotin, is there a need for this PR since we already have the changes as part of the #3407?

[andrewslotin]

@olegbespalov, I've missed that one, but you might consider adding 277d276 there.

[olegbespalov]

I've cherry-picked the 277d276 to the #3407 👍

[andrewslotin]

Good, this PR then has no use 👍