ci: DRY_RUN variable is a string not a bool

[CecileRobertMichon]

Reason for Change:

Azure pipelines treats the variable DRY_RUN as a string (did not find a way to specify it as a bool), so using string comparison. Previously not('False') was evaluating as False.

Issue Fixed:

Requirements:

• uses conventional commit messages
• includes documentation
• adds unit tests
• tested upgrade from previous version

Notes:

[codecov]

Codecov Report

Merging #601 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #601      +/-   ##
==========================================
- Coverage    56.7%   56.68%   -0.02%     
==========================================
  Files          91       91              
  Lines       13898    13902       +4     
==========================================
  Hits         7881     7881              
- Misses       5352     5355       +3     
- Partials      665      666       +1

[jackfrancis]

/lgtm

[CecileRobertMichon]

Need to validate

/hold

[CecileRobertMichon]

This is ready. The VHD pipeline failed because our SA delete requests are being throttled (I was doing some heavy manual clean-up in the background at the same time) but skipping/not skipping the classic SA copy works as expected.

/hold cancel

[jackfrancis]

/lgtm

[acs-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon, jackfrancis

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [CecileRobertMichon,jackfrancis]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment