Skip to content

Conversation

@hweawer
Copy link
Collaborator

@hweawer hweawer commented Jul 14, 2025

Enhanced Origin Service Logging

  • Improved error reporting with component-specific identifiers
  • Better debug information for origin blob operations
  • Standardized log message formatting consistent with agent improvements

Command Structure Refactoring

  • Cleaner command organization in origin service
  • Improved error handling and validation
  • Better configuration management

Code Quality Enhancements

  • Bug fixes and stability improvements
  • Consistent coding patterns with the agent service
  • Enhanced maintainability
    🎯 Benefits
  • Unified logging experience across agent and origin services
  • Better debugging capabilities for origin-specific operations
  • Improved operational visibility into blob serving and caching

@hweawer hweawer self-assigned this Jul 14, 2025
@hweawer hweawer force-pushed the improve-kraken-origin-logging branch from f1f5948 to 4c3f228 Compare July 16, 2025 09:19
@hweawer hweawer changed the base branch from master to improve-kraken-agent-logging July 16, 2025 09:19
@hweawer hweawer force-pushed the improve-kraken-agent-logging branch from 159aea2 to 7247d25 Compare July 16, 2025 12:36
@hweawer hweawer force-pushed the improve-kraken-origin-logging branch from 4c3f228 to 58bfdd6 Compare July 16, 2025 12:36
@hweawer hweawer force-pushed the improve-kraken-agent-logging branch from 7247d25 to f4bd0d9 Compare July 17, 2025 08:14
@hweawer hweawer force-pushed the improve-kraken-origin-logging branch from 58bfdd6 to 28f4b6f Compare July 17, 2025 08:14
Copy link
Collaborator

@Anton-Kalpakchiev Anton-Kalpakchiev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please split into smaller PRs, as currently I can only give a rubber stamp, as it's hard for me to understand all aspects of the proposed changes.

DuplicateWriteBackStagger time.Duration `yaml:"duplicate_write_back_stagger"`

// Timeout configurations
DownloadTimeout time.Duration `yaml:"download_timeout"`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we adding the custom timeout configuration?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants