Merge ~mwhudson/curtin:v2-make_dname into curtin:master

[Michael Hudson-Doyle]

block_meta_v2: call make_dname when required

Yet another thing found by v2 vmtests.

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:88c7b5b38f479916fe9340fa359a4adaba814c21
https://jenkins.ubuntu.com/server/job/curtin-ci/275/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/server/job/curtin-ci/nodes=metal-amd64/275/
    SUCCESS: https://jenkins.ubuntu.com/server/job/curtin-ci/nodes=metal-arm64/275/
    SUCCESS: https://jenkins.ubuntu.com/server/job/curtin-ci/nodes=metal-ppc64el/275/
    SUCCESS: https://jenkins.ubuntu.com/server/job/curtin-ci/nodes=metal-s390x/275/

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/curtin-ci/275//rebuild

review: Approve (continuous-integration)

[Dan Bungert (dbungert)]

This looks fine but sounds worthy of a test.

[Dan Bungert (dbungert)]

oh wait vmtests found it

review: Approve

[Michael Hudson-Doyle (mwhudson)]

Yes indeed.

diff --git a/curtin/commands/block_meta_v2.py b/curtin/commands/block_meta_v2.py
index 26f5e6a..051649b 100644
--- a/curtin/commands/block_meta_v2.py
+++ b/curtin/commands/block_meta_v2.py
@@ -8,6 +8,7 @@ from curtin import (block, util)
 from curtin.commands.block_meta import (
     disk_handler as disk_handler_v1,
     get_path_to_storage_volume,
+    make_dname,
     partition_handler as partition_handler_v1,
     partition_verify_sfdisk,
     )
@@ -280,6 +281,12 @@ def disk_handler_v2(info, storage_config, handlers):
         if mode is not None:
             block.wipe_volume(block.kname_to_path(kname), mode)
 
+    # Make the names if needed
+    if 'name' in info:
+        for action in part_actions:
+            if action.get('flag') != 'extended':
+                make_dname(action['id'], storage_config)
+
 
 def partition_handler_v2(info, storage_config, handlers):
     pass