Skip to content

Commit

Permalink
Merge pull request #482 from zeeke/exclude-topology-conformance-test-2
Browse files Browse the repository at this point in the history
e2e: Fix Validate ExcludeTopology test leftover
  • Loading branch information
e0ne authored Aug 2, 2023
2 parents 990648e + 79bd2e3 commit 12b2566
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/conformance/tests/test_sriov_operator.go
Original file line number Diff line number Diff line change
Expand Up @@ -1405,7 +1405,7 @@ var _ = Describe("[sriov] operator", func() {
},
}

excludeTopologyTrueResourceXXX = &sriovv1.SriovNetworkNodePolicy{
excludeTopologyFalseResourceXXX = &sriovv1.SriovNetworkNodePolicy{
ObjectMeta: metav1.ObjectMeta{
Name: "test-exclude-topology-false-res-xxx",
Namespace: operatorNamespace,
Expand All @@ -1416,7 +1416,7 @@ var _ = Describe("[sriov] operator", func() {
ResourceName: "resourceXXX",
NodeSelector: map[string]string{"kubernetes.io/hostname": node},
NicSelector: sriovv1.SriovNetworkNicSelector{
PfNames: []string{intf.Name + "#0-4"},
PfNames: []string{intf.Name + "#5-9"},
},
ExcludeTopology: false,
},
Expand Down Expand Up @@ -1467,7 +1467,7 @@ var _ = Describe("[sriov] operator", func() {
Expect(err).To(HaveOccurred())

Expect(err.Error()).To(ContainSubstring(
"excludeTopology[false] field conflicts with policy [test-exclude-topology-true].ExcludeTopology[true]" +
"excludeTopology[false] field conflicts with policy [test-exclude-topology-true-res-xxx].ExcludeTopology[true]" +
" as they target the same resource[resourceXXX]"))
})
})
Expand Down

0 comments on commit 12b2566

Please sign in to comment.