Skip to content

Commit 186d409

Browse files
authored
Merge pull request #139 from kube-logging/fix/panic-on-log
2 parents 97c1c12 + 1389ef9 commit 186d409

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Diff for: internal/controller/telemetry/manager/tenant_resource_manager.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,7 @@ func (t *TenantResourceManager) ValidateSubscriptionOutputs(ctx context.Context,
203203
// Ensure the output belongs to the same tenant
204204
if checkedOutput.Status.Tenant != subscription.Status.Tenant {
205205
t.Logger.Error(errors.New("output and subscription tenants mismatch"),
206+
"output and subscription tenants mismatch",
206207
"output", checkedOutput.NamespacedName().String(),
207208
"output's tenant", checkedOutput.Status.Tenant,
208209
"subscription", subscription.NamespacedName().String(),
@@ -216,7 +217,7 @@ func (t *TenantResourceManager) ValidateSubscriptionOutputs(ctx context.Context,
216217
}
217218

218219
if len(invalidOutputs) > 0 {
219-
t.Logger.Error(errors.New("some outputs are invalid"), "invalidOutputs", invalidOutputs, "subscription", subscription.NamespacedName().String())
220+
t.Logger.Error(errors.New("some outputs are invalid"), "some outputs are invalid", "invalidOutputs", invalidOutputs, "subscription", subscription.NamespacedName().String())
220221
}
221222

222223
return validOutputs

0 commit comments

Comments
 (0)