Skip to content

Reduce Redundant Error-Level Logging #421

Closed

Description

@avirajsingh7

Is your feature request related to a problem?

The codebase uses logger.error in places where the situation isn't an actual error — for example, when a resource is simply not found.
This creates log noise and makes it harder to identify genuine errors.

if not credentials:
logger.error(f"No credentials found for project {project_id}")

Describe the solution you'd like

Define clear guidelines for log levels:
- error: Unexpected exceptions, failures, or situations where the system cannot proceed
- warning: Recoverable conditions, user-caused validation errors
- info: Expected conditions that are worth noting
- No log: Entirely expected behavior (e.g., resource legitimately doesn't exist)

Action items:

  • Review all logger.error usages across the codebase
  • Downgrade to warning, info, or remove based on context
  • Document logging conventions for future development

Activity

  1. ankit-mehta07 commented on Feb 6, 2026

    @ankit-mehta07

    @avirajsingh7 @nishika26 can you please assign this work to me?

  2. self-assigned this
    on Apr 28, 2026
  3. moved this from To Do to In Progress in Kaapi-devon Apr 29, 2026
  4. added theissue type on Apr 30, 2026
  5. moved this from In Progress to Closed in Kaapi-devon May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

Projects

  • Status
    Closed

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions