Merge pull request 'fix: generate Talconfig during reconcile' (#31) from fix/reconcile-talconfig into main

Reviewed-on: #31

internal/bootstrap/bootstrap.go

@@ -1385,7 +1385,7 @@ func (r Runner) reconcileTerraform(terraformDir string) error {
 }
 
 func terraformReconcileTargets(cfg config.Config) []string {
-	targets := []string{"-target=proxmox_virtual_environment_download_file.talos_iso"}
+	targets := []string{"-target=proxmox_virtual_environment_download_file.talos_iso", "-target=local_file.talconfig"}
 	if cfg.Talos.Cluster.ManageNetworkBridges {
 		targets = append(targets, "-target=proxmox_virtual_environment_network_linux_bridge.cluster_bridge")
 	}

internal/bootstrap/bootstrap_test.go

@@ -1109,13 +1109,13 @@ func TestTerraformPlanPathIsAbsolute(t *testing.T) {
 	}
 }
 
-func TestTerraformReconcileTargetsStageImageAndManagedBridgesOnly(t *testing.T) {
-	if targets := strings.Join(terraformReconcileTargets(config.Config{}), " "); targets != "-target=proxmox_virtual_environment_download_file.talos_iso" {
+func TestTerraformReconcileTargetsStageImageTalconfigAndManagedBridgesOnly(t *testing.T) {
+	if targets := strings.Join(terraformReconcileTargets(config.Config{}), " "); targets != "-target=proxmox_virtual_environment_download_file.talos_iso -target=local_file.talconfig" {
 		t.Fatalf("default reconcile targets = %q", targets)
 	}
 	cfg := config.Config{Talos: config.TalosConfig{Cluster: config.TalosClusterConfig{ManageNetworkBridges: true}}}
 	targets := strings.Join(terraformReconcileTargets(cfg), " ")
-	if !strings.Contains(targets, "-target=proxmox_virtual_environment_download_file.talos_iso") || !strings.Contains(targets, "-target=proxmox_virtual_environment_network_linux_bridge.cluster_bridge") || strings.Contains(targets, "proxmox_virtual_environment_vm.vm") {
+	if !strings.Contains(targets, "-target=proxmox_virtual_environment_download_file.talos_iso") || !strings.Contains(targets, "-target=local_file.talconfig") || !strings.Contains(targets, "-target=proxmox_virtual_environment_network_linux_bridge.cluster_bridge") || strings.Contains(targets, "proxmox_virtual_environment_vm.vm") {
 		t.Fatalf("managed bridge reconcile targets = %q", targets)
 	}
 }
@@ -1197,7 +1197,7 @@ func TestReconcileTerraformImportsConfiguredTalosVMsBeforePlan(t *testing.T) {
 			if len(state) != 1 || state[0] != importedAddress {
 				return errors.New("VM create collision")
 			}
-			if strings.Contains(strings.Join(args, " "), "proxmox_virtual_environment_vm.vm") || !strings.Contains(strings.Join(args, " "), "-target=proxmox_virtual_environment_download_file.talos_iso") {
+			if strings.Contains(strings.Join(args, " "), "proxmox_virtual_environment_vm.vm") || !strings.Contains(strings.Join(args, " "), "-target=proxmox_virtual_environment_download_file.talos_iso") || !strings.Contains(strings.Join(args, " "), "-target=local_file.talconfig") {
 				t.Fatalf("reconcile plan did not exclude the Talos VM: %q", args)
 			}
 			events = append(events, "plan")