Skip to content

Commit c2563bc

Browse files
mcbenjemaawikkyk
authored andcommitted
Improve the template and error logs
1 parent 5131aa2 commit c2563bc

File tree

4 files changed

+6
-7
lines changed

4 files changed

+6
-7
lines changed

internal/service/vmservice/bootstrap.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ func injectCloudInit(ctx context.Context, machineScope *scope.MachineScope, boot
102102
injector := getISOInjector(machineScope.VirtualMachine, bootstrapData, metadata, network)
103103
if err := injector.Inject(ctx, inject.CloudConfigFormat); err != nil {
104104
conditions.MarkFalse(machineScope.ProxmoxMachine, infrav1alpha1.VMProvisionedCondition, infrav1alpha1.VMProvisionFailedReason, clusterv1.ConditionSeverityWarning, "%s", err)
105-
return errors.Wrap(err, "cloud-init iso inject failed")
105+
return err
106106
}
107107
return nil
108108
}
@@ -123,7 +123,7 @@ func injectIgnition(ctx context.Context, machineScope *scope.MachineScope, boots
123123
injector := getIgnitionISOInjector(machineScope.VirtualMachine, metadata, enricher)
124124
if err := injector.Inject(ctx, inject.IgnitionFormat); err != nil {
125125
conditions.MarkFalse(machineScope.ProxmoxMachine, infrav1alpha1.VMProvisionedCondition, infrav1alpha1.VMProvisionFailedReason, clusterv1.ConditionSeverityWarning, "%s", err)
126-
return errors.Wrap(err, "ignition iso inject failed")
126+
return err
127127
}
128128
return nil
129129
}

internal/service/vmservice/bootstrap_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ func TestReconcileBootstrapData_BadInjector(t *testing.T) {
130130

131131
requeue, err := reconcileBootstrapData(context.Background(), machineScope)
132132
require.Error(t, err)
133-
require.Contains(t, err.Error(), "cloud-init iso inject failed: bad FakeISOInjector")
133+
require.Contains(t, err.Error(), "failed to inject bootstrap data: bad FakeISOInjector")
134134
require.False(t, requeue)
135135
require.True(t, conditions.Has(machineScope.ProxmoxMachine, infrav1alpha1.VMProvisionedCondition))
136136
require.Nil(t, machineScope.ProxmoxMachine.Status.BootstrapDataProvided)

test/e2e/config/proxmox-dev.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,17 +82,17 @@ providers:
8282
- sourcePath: "${PWD}/test/e2e/data/infrastructure-proxmox/cluster-template-flatcar.yaml"
8383

8484
variables:
85-
KUBERNETES_VERSION: "${KUBERNETES_VERSION:-v1.31.6}"
85+
KUBERNETES_VERSION: "${KUBERNETES_VERSION:-v1.32.2}"
8686
CONTROL_PLANE_MACHINE_COUNT: "${CONTROL_PLANE_MACHINE_COUNT:-1}"
8787
WORKER_MACHINE_COUNT: "${WORKER_MACHINE_COUNT:-2}"
8888
EXP_CLUSTER_RESOURCE_SET: "true"
8989
# Cluster Addons
9090
CNI: "${PWD}/test/e2e/data/cni/calico.yaml"
9191

9292
PROXMOX_SOURCENODE: "${PROXMOX_SOURCENODE:-stg-ceph01}"
93-
TEMPLATE_VMID: "${TEMPLATE_VMID:-164}"
93+
TEMPLATE_VMID: "${TEMPLATE_VMID:-187}"
9494
TEMPLATE_VMID_FLATCAR: "${TEMPLATE_VMID_FLATCAR:-202}"
95-
VM_SSH_KEYS: ""
95+
VM_SSH_KEYS: "ssh-ed25519 ..."
9696
NODE_IP_RANGES: "[10.4.18.104-10.4.18.125]"
9797
GATEWAY: "10.4.18.1"
9898
IP_PREFIX: "25"

test/e2e/data/infrastructure-proxmox/cluster-template-ci.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,6 @@ spec:
153153
owner: root:root
154154
permissions: "0700"
155155
preKubeadmCommands:
156-
- echo "127.0.0.1 localhost kubernetes {{ ds.meta_data.hostname }}" >>/etc/hosts
157156
- /etc/kube-vip-prepare.sh
158157
initConfiguration:
159158
nodeRegistration:

0 commit comments

Comments
 (0)