Skip to content

Commit e668700

Browse files
committed
Changed machineset tests to be serial
1 parent c34e593 commit e668700

File tree

2 files changed

+32
-7
lines changed

2 files changed

+32
-7
lines changed

test/e2e/util.go

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import (
1010
"github.com/openshift/api/machine/v1beta1"
1111
configclient "github.com/openshift/client-go/config/clientset/versioned"
1212
machinesetclient "github.com/openshift/client-go/machine/clientset/versioned/typed/machine/v1beta1"
13+
v1 "k8s.io/api/core/v1"
1314
"k8s.io/apimachinery/pkg/api/errors"
1415
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
1516
"k8s.io/apimachinery/pkg/runtime"
@@ -176,6 +177,13 @@ func CreateMachine(ctx context.Context, cfg *rest.Config, mc *machinesetclient.M
176177
ProviderSpec: v1beta1.ProviderSpec{
177178
Value: provider,
178179
},
180+
Taints: []v1.Taint{
181+
{
182+
Effect: v1.TaintEffectNoSchedule,
183+
Key: "mapi-e2e",
184+
Value: "yes",
185+
},
186+
},
179187
},
180188
}
181189

@@ -223,6 +231,13 @@ func CreateMachineSet(ctx context.Context, cfg *rest.Config, mc *machinesetclien
223231
ProviderSpec: v1beta1.ProviderSpec{
224232
Value: provider,
225233
},
234+
Taints: []v1.Taint{
235+
{
236+
Effect: v1.TaintEffectNoSchedule,
237+
Key: "mapi-e2e",
238+
Value: "yes",
239+
},
240+
},
226241
},
227242
},
228243
},

test/e2e/vsphere/machines.go

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@ import (
1919
)
2020

2121
const (
22-
machineRole = "e2e-test"
22+
machineRole = "feature-gate-test"
2323
machineReadyTimeout = time.Minute * 6
2424
)
2525

26-
var _ = Describe("[sig-cluster-lifecycle][OCPFeatureGate:VSphereMultiDisk][platform:vsphere] Managed cluster should", func() {
26+
var _ = Describe("[sig-cluster-lifecycle][OCPFeatureGate:VSphereMultiDisk][platform:vsphere][Disruptive] Managed cluster should", Label("Conformance"), Label("Serial"), func() {
2727
defer GinkgoRecover()
2828
ctx := context.Background()
2929

@@ -46,7 +46,7 @@ var _ = Describe("[sig-cluster-lifecycle][OCPFeatureGate:VSphereMultiDisk][platf
4646
Expect(err).NotTo(HaveOccurred())
4747
})
4848

49-
It("create machines with data disks [apigroup:machine.openshift.io]", func() {
49+
It("create machines with data disks [apigroup:machine.openshift.io][Suite:openshift/conformance/serial]", func() {
5050
machineName := "machine-multi-test"
5151
dataDisks := []v1beta1.VSphereDisk{
5252
{
@@ -151,33 +151,43 @@ var _ = Describe("[sig-cluster-lifecycle][OCPFeatureGate:VSphereMultiDisk][platf
151151
err = util.ScaleMachineSet(cfg, ddMachineSet.Name, 0)
152152
Expect(err).NotTo(HaveOccurred())
153153

154+
// Verify / wait for machine is removed
155+
By("verifying machine is destroyed")
156+
Eventually(func() (int32, error) {
157+
ms, err := mc.MachineSets(util.MachineAPINamespace).Get(ctx, ddMachineSet.Name, metav1.GetOptions{})
158+
if err != nil {
159+
return -1, err
160+
}
161+
return ms.Status.ReadyReplicas, nil
162+
}, machineReadyTimeout).Should(BeEquivalentTo(0))
163+
154164
// Delete machineset
155165
By("deleting the machineset")
156166
err = mc.MachineSets(util.MachineAPINamespace).Delete(ctx, ddMachineSet.Name, metav1.DeleteOptions{})
157167
Expect(err).NotTo(HaveOccurred())
158168
},
159-
Entry("with thin data disk [apigroup:machine.openshift.io]", "ms-thin-test", []v1beta1.VSphereDisk{
169+
Entry("with thin data disk [apigroup:machine.openshift.io][Suite:openshift/conformance/serial]", "ms-thin-test", []v1beta1.VSphereDisk{
160170
{
161171
Name: "thickDataDisk",
162172
SizeGiB: 1,
163173
ProvisioningMode: v1beta1.ProvisioningModeThick,
164174
},
165175
}),
166-
Entry("with thick data disk [apigroup:machine.openshift.io]", "ms-thick-test", []v1beta1.VSphereDisk{
176+
Entry("with thick data disk [apigroup:machine.openshift.io][Suite:openshift/conformance/serial]", "ms-thick-test", []v1beta1.VSphereDisk{
167177
{
168178
Name: "thickDataDisk",
169179
SizeGiB: 1,
170180
ProvisioningMode: v1beta1.ProvisioningModeThick,
171181
},
172182
}),
173-
Entry("with eagerly zeroed data disk [apigroup:machine.openshift.io]", "ms-zeroed-test", []v1beta1.VSphereDisk{
183+
Entry("with eagerly zeroed data disk [apigroup:machine.openshift.io][Suite:openshift/conformance/serial]", "ms-zeroed-test", []v1beta1.VSphereDisk{
174184
{
175185
Name: "zeroedDataDisk",
176186
SizeGiB: 1,
177187
ProvisioningMode: v1beta1.ProvisioningModeEagerlyZeroed,
178188
},
179189
}),
180-
Entry("with a data disk using each provisioning mode [apigroup:machine.openshift.io]", "ms-multi-test", []v1beta1.VSphereDisk{
190+
Entry("with a data disk using each provisioning mode [apigroup:machine.openshift.io][Suite:openshift/conformance/serial]", "ms-multi-test", []v1beta1.VSphereDisk{
181191
{
182192
Name: "thinDataDisk",
183193
SizeGiB: 1,

0 commit comments

Comments
 (0)