mirror of
https://github.com/k3s-io/kubernetes.git
synced 2025-08-12 05:21:58 +00:00
node: topology-mgr: resolve lint errors
Signed-off-by: Swati Sehgal <swsehgal@redhat.com>
This commit is contained in:
parent
9e1d6d5cdc
commit
8ee3558391
@ -47,11 +47,11 @@ func CheckPolicyOptionAvailable(option string) error {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if alphaOptions.Has(option) && !utilfeature.DefaultFeatureGate.Enabled(kubefeatures.TopologyManagerPolicyAlphaOptions) {
|
if alphaOptions.Has(option) && !utilfeature.DefaultFeatureGate.Enabled(kubefeatures.TopologyManagerPolicyAlphaOptions) {
|
||||||
return fmt.Errorf("Topology Manager Policy Alpha-level Options not enabled, but option %q provided", option)
|
return fmt.Errorf("topology manager policy alpha-level options not enabled, but option %q provided", option)
|
||||||
}
|
}
|
||||||
|
|
||||||
if betaOptions.Has(option) && !utilfeature.DefaultFeatureGate.Enabled(kubefeatures.TopologyManagerPolicyBetaOptions) {
|
if betaOptions.Has(option) && !utilfeature.DefaultFeatureGate.Enabled(kubefeatures.TopologyManagerPolicyBetaOptions) {
|
||||||
return fmt.Errorf("Topology Manager Policy Beta-level Options not enabled, but option %q provided", option)
|
return fmt.Errorf("topology manager policy beta-level options not enabled, but option %q provided", option)
|
||||||
}
|
}
|
||||||
|
|
||||||
return nil
|
return nil
|
||||||
|
@ -74,7 +74,7 @@ func TestNewTopologyManagerOptions(t *testing.T) {
|
|||||||
policyOptions: map[string]string{
|
policyOptions: map[string]string{
|
||||||
MaxAllowableNUMANodes: "8",
|
MaxAllowableNUMANodes: "8",
|
||||||
},
|
},
|
||||||
expectedErr: fmt.Errorf("Topology Manager Policy Beta-level Options not enabled,"),
|
expectedErr: fmt.Errorf("topology manager policy beta-level options not enabled,"),
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
description: "return empty TopologyManagerOptions",
|
description: "return empty TopologyManagerOptions",
|
||||||
@ -117,7 +117,7 @@ func TestNewTopologyManagerOptions(t *testing.T) {
|
|||||||
policyOptions: map[string]string{
|
policyOptions: map[string]string{
|
||||||
fancyBetaOption: "true",
|
fancyBetaOption: "true",
|
||||||
},
|
},
|
||||||
expectedErr: fmt.Errorf("Topology Manager Policy Beta-level Options not enabled,"),
|
expectedErr: fmt.Errorf("topology manager policy beta-level options not enabled,"),
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
description: "test alpha options success",
|
description: "test alpha options success",
|
||||||
@ -136,7 +136,7 @@ func TestNewTopologyManagerOptions(t *testing.T) {
|
|||||||
policyOptions: map[string]string{
|
policyOptions: map[string]string{
|
||||||
fancyAlphaOption: "true",
|
fancyAlphaOption: "true",
|
||||||
},
|
},
|
||||||
expectedErr: fmt.Errorf("Topology Manager Policy Alpha-level Options not enabled,"),
|
expectedErr: fmt.Errorf("topology manager policy alpha-level options not enabled,"),
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user