Skip to content

Reduced _get_event_summary() error and warning descriptions to ensure 100kb pydantic limit is always met. - #304

Merged
amd-etannant merged 3 commits into
developmentfrom
amd/bugfix/event_summary_shortening
Oct 1, 2026
Merged

amd-etannant merged 3 commits into
developmentfrom
amd/bugfix/event_summary_shortening

Conversation

@amd-etannant

@amd-etannant amd-etannant commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Shorten task event summaries while retaining warning and error totals.
  • Show the three most frequent error and critical descriptions.
  • Report how many additional descriptions were omitted.
  • Update affected tests for the new summary format.

Test plan

  • pytest test/unit
  • pytest test/functional (if applicable)
  • pre-commit run --all-files

Checklist

  • Added/updated tests (or explained why not)
  • Updated docs/README if behavior changed
  • No secrets or credentials committed

@amd-ansonyim amd-ansonyim left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.

@amd-etannant
amd-etannant merged commit 7926d81 into development Oct 1, 2026
7 checks passed
@amd-etannant
amd-etannant deleted the amd/bugfix/event_summary_shortening branch October 1, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants