Skip to content

Commit fcb7d9e

Browse files
committed
Fix linter issue
Signed-off-by: Nelo-T. Wallus <[email protected]>
1 parent 388dffc commit fcb7d9e

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

sdk/apis/third_party/conditions/util/conditions/getter.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ func mirror(from Getter, targetCondition conditionsapi.ConditionType, options ..
224224
case true:
225225
condition = TrueCondition(targetCondition)
226226
case false:
227-
condition = FalseCondition(targetCondition, mirrorOpt.fallbackReason, mirrorOpt.fallbackSeverity, mirrorOpt.fallbackMessage)
227+
condition = FalseCondition(targetCondition, mirrorOpt.fallbackReason, mirrorOpt.fallbackSeverity, "%s", mirrorOpt.fallbackMessage)
228228
}
229229
}
230230

sdk/apis/third_party/conditions/util/conditions/merge.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,9 @@ func merge(conditions []localizedCondition, targetCondition conditionsapi.Condit
6464
targetReason := getReason(g, options)
6565
targetMessage := getMessage(g, options)
6666
if g.TopGroup().status == corev1.ConditionFalse {
67-
return FalseCondition(targetCondition, targetReason, g.TopGroup().severity, targetMessage)
67+
return FalseCondition(targetCondition, targetReason, g.TopGroup().severity, "%s", targetMessage)
6868
}
69-
return UnknownCondition(targetCondition, targetReason, targetMessage)
69+
return UnknownCondition(targetCondition, targetReason, "%s", targetMessage)
7070
}
7171

7272
// getConditionGroups groups a list of conditions according to status, severity values.

test/e2e/virtual/apiexport/authorizer_test.go

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

632632
// Bind to second tenant workspace so url pops up. So we use this to check tenant1 access.
633633
for _, shard := range shards.Items {
634-
tenant2Path, _ := kcptesting.NewWorkspaceFixture(t, server, orgPath, kcptesting.WithName(shard.Name+"-tenant-2"))
634+
tenant2Path, _ := kcptesting.NewWorkspaceFixture(t, server, orgPath, kcptesting.WithName("%s-tenant-2", shard.Name))
635635
kcptestinghelpers.Eventually(t, func() (bool, string) {
636636
_, err = kcpClient.Cluster(tenant2Path).ApisV1alpha2().APIBindings().Create(ctx, apiBinding, metav1.CreateOptions{})
637637
return err == nil, fmt.Sprintf("Error creating APIBinding: %v", err)

test/integration/dynamicrestmapper/dynamicrestmapper_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ func TestDynamicRestMapper(t *testing.T) {
6565
extensionsClusterClient, err := kcpapiextensionsv1client.NewForConfig(cfg)
6666
require.NoError(t, err, "error creating apiextensions cluster client")
6767

68-
workspace := kcptesting.NewLowLevelWorkspaceFixture(t, kcpClientSet, kcpClientSet, core.RootCluster.Path(), kcptesting.WithName(name))
68+
workspace := kcptesting.NewLowLevelWorkspaceFixture(t, kcpClientSet, kcpClientSet, core.RootCluster.Path(), kcptesting.WithName("%s", name))
6969

7070
logicalClusterName := logicalcluster.Name(workspace.Spec.Cluster)
7171

0 commit comments

Comments
 (0)