DiskIdentifier: Add FriendlyName as a disk identifier to Disk, DiskAccessPath and Wai…

[dshelb]

Pull Request (PR) description

This PR introduces support for using disk friendly name as a disk identifier in disk-related resources Disk, DiskAccessPath and WaitForDisk. It also pins the build agent vmAgent to ubuntu-20.04.

This Pull Request (PR) fixes the following issues
Fixes #268
Fixes #270

Task list

• Added an entry to the change log under the Unreleased section of the
  file CHANGELOG.md. Entry should say what was changed and how that
  affects users (if applicable), and reference the issue being resolved
  (if applicable).
• Resource documentation added/updated in README.md.
• Resource parameter descriptions added/updated in README.md, schema.mof
  and comment-based help.
• Comment-based help added/updated.
• Localization strings added/updated in all localization files as appropriate.
• Examples appropriately added/updated.
• [ x] Unit tests added/updated. See DSC Community Testing Guidelines.
• Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
• New/changed code adheres to DSC Community Style Guidelines.

---

This change is 

[PlagueHO]

Once the build passes, this is

Reviewed 13 of 13 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dshelb)

[PlagueHO]

Reviewed 1 of 2 files at r2, 2 of 2 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dshelb)

---

CHANGELOG.md line 24 at r3 (raw file):

- Added support to use disk FriendlyName as a disk identifer - Fixes [Issue #268](https://github.com/dsccommunity/StorageDsc/issues/268).
- Pin Azure build agent vmImage to ubuntu-20.04  - Fixes [Issue #270] (https://github.com/dsccommunity/StorageDsc/issues/270).
- Remove confirmation prompt when Clear-Disk is called

Nit: can you add a full stop at the end of this line?

[codecov]

Codecov Report

Merging #269 (ba349ef) into main (7aa733b) will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           main   #269   +/-   ##
===================================
  Coverage    95%    95%           
===================================
  Files         7      7           
  Lines       882    882           
===================================
  Hits        842    842           
  Misses       40     40           

Impacted Files	Coverage Δ
...sources/DSC_DiskAccessPath/DSC_DiskAccessPath.psm1	89% <ø> (ø)
.../DSCResources/DSC_WaitForDisk/DSC_WaitForDisk.psm1	100% <ø> (ø)
...e/Modules/StorageDsc.Common/StorageDsc.Common.psm1	100% <ø> (ø)
source/DSCResources/DSC_Disk/DSC_Disk.psm1	98% <100%> (ø)

[PlagueHO]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[PlagueHO]

Reviewed 2 of 3 files at r4, 1 of 1 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dshelb)