fix(plan): streamline add_inventory_hostnames by removing redundant code

- Remove redundant set_var calls for certname
- Combine yq command execution within the target mapping loop
- Ensure inventory file is updated correctly for each target

Author: CoMfUcIoS

spec/acceptance/peadm_spec/plans/add_inventory_hostnames.pp

@@ -4,21 +4,15 @@
   $t = get_targets('*')
   wait_until_available($t)
 
-  $fqdn_results = $t.map |$target| {
+  $t.map |$target| {
     $fqdn = run_command('hostname -f', $target).first
     if $fqdn['exit_code'] != 0 {
       fail("Failed to get FQDN for target ${target.name}: ${fqdn['stderr']}")
     }
-    $target.set_var('certname', $fqdn['stdout'].chomp)
-  }
-
-  $fqdn_results.each |$result| {
-    $command = "yq eval '(.groups[].targets[] | select(.uri == \"${uri}\").name) = \"${certname}\"' -i ${inventory_file}"
+    $command = "yq eval '(.groups[].targets[] | select(.uri == \"${target.uri}\").name) = \"${fqdn['stdout'].chomp}\"' -i ${inventory_file}"
     $result = run_command($command, 'localhost').first
     if $result['exit_code'] != 0 {
       fail("Failed to update inventory file for target ${uri}: ${result['stderr']}")
     }
-    $command = "yq eval '(.groups[].targets[] | select(.uri == \"${uri}\").name) = \"${certname}\"' -i ${inventory_file}"
-    run_command($command, 'localhost')
   }
 }