Skip to content

Commit 2abb738

Browse files
committed
fixup! runtime/secrets: add package for consolidated secret handling
PullSecretsFromServiceAccount: remove unnecessary if statement Remove the unnecessary check for empty ImagePullSecrets as make() with zero capacity returns the same result and improves code readability. Signed-off-by: cappyzawa <[email protected]>
1 parent c79f432 commit 2abb738

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

runtime/secrets/reader.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159,10 +159,6 @@ func PullSecretsFromServiceAccount(ctx context.Context, c client.Client, name, n
159159
return nil, formatResourceError("serviceaccount", namespace, name, err)
160160
}
161161

162-
if len(serviceAccount.ImagePullSecrets) == 0 {
163-
return []corev1.Secret{}, nil
164-
}
165-
166162
secrets := make([]corev1.Secret, 0, len(serviceAccount.ImagePullSecrets))
167163
for _, imagePullSecret := range serviceAccount.ImagePullSecrets {
168164
secret, err := getSecret(ctx, c, imagePullSecret.Name, namespace)

0 commit comments

Comments
 (0)