Skip to content

Log SessionId in params#57

Merged
PaulJKathmann merged 4 commits intodevelopfrom
pk/replica-level-logs
Jan 28, 2026
Merged

Log SessionId in params#57
PaulJKathmann merged 4 commits intodevelopfrom
pk/replica-level-logs

Conversation

@PaulJKathmann
Copy link
Copy Markdown
Contributor

@PaulJKathmann PaulJKathmann commented Jan 27, 2026

Before this PR

Users cannot filter logs by sessionId / replicas

After this PR

Adding SessionId to logs into the new params field and moving PID and Job Id into params field from message string for better readability.

Possible downsides?

@changelog-app
Copy link
Copy Markdown

changelog-app Bot commented Jan 27, 2026

Generate changelog in changelog/@unreleased

Type (Select exactly one)

  • Feature (Adding new functionality)
  • Improvement (Improving existing functionality)
  • Fix (Fixing an issue with existing functionality)
  • Break (Creating a new major version by breaking public APIs)
  • Deprecation (Removing functionality in a non-breaking way)
  • Migration (Automatically moving data/functionality to a new system)

Description

allow users to filter logs by sessionId

Check the box to generate changelog(s)

  • Generate changelog entry

@changelog-app
Copy link
Copy Markdown

changelog-app Bot commented Jan 27, 2026

Successfully generated changelog entry!

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.


📋Changelog Preview

💡 Improvements

  • allow users to filter logs by sessionId (#57)

@PaulJKathmann PaulJKathmann changed the title draft Log SessionId in params Jan 28, 2026
@PaulJKathmann PaulJKathmann marked this pull request as ready for review January 28, 2026 16:02
Copy link
Copy Markdown

@apastrana6 apastrana6 left a comment

Choose a reason for hiding this comment

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

Looks good!

@PaulJKathmann PaulJKathmann merged commit 759709e into develop Jan 28, 2026
11 checks passed
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.

2 participants