chore(kongstate) discard logger argument

Discard the logger argument to kongstate.FillConsumersAndCredentials(). It is not currently used, but removing it from the signature would require a breaking change, with a decent chance we'd re-add it in the future.
Kong · Apr 15, 2024 · 56d527c · 56d527c
1 parent 60f790f
commit 56d527c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/internal/dataplane/kongstate/kongstate.go b/internal/dataplane/kongstate/kongstate.go
@@ -68,7 +68,7 @@ func (ks *KongState) SanitizedCopy() *KongState {
 }
 
 func (ks *KongState) FillConsumersAndCredentials(
-	logger logr.Logger,
+	_ logr.Logger,
 	s store.Storer,
 	failuresCollector *failures.ResourceFailuresCollector,
 ) {

diff --git a/test/envtest/admission_webhook_envtest_test.go b/test/envtest/admission_webhook_envtest_test.go
@@ -760,7 +760,6 @@ func TestAdmissionWebhook_KongConsumers(t *testing.T) {
 					},
 				},
 				StringData: map[string]string{
-
 					"username": "electron",
 					"password": "testpass",
 				},