Skip to content

[ENG-9854][ENG-10025][ENG-10040] NR Post-release model migration + model dependent bug-fixes #11540

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Ostap-Zherebetskyi wants to merge 5 commits into
base: hotfix/26.1.5
Choose a base branch
from
Open

[ENG-9854][ENG-10025][ENG-10040] NR Post-release model migration + model dependent bug-fixes #11540

Ostap-Zherebetskyi wants to merge 5 commits into from
+109 −16

Conversation

@Ostap-Zherebetskyi
Copy link
Collaborator

@Ostap-Zherebetskyi Ostap-Zherebetskyi commented Jan 8, 2026 •
edited by cslzchen
Loading

Ticket

https://openscience.atlassian.net/browse/ENG-9854
https://openscience.atlassian.net/browse/ENG-10025
https://openscience.atlassian.net/browse/ENG-10040

Purpose

  • NR Post-release model migration
  • NR Post-release bug-fixes that requires migration

Changes

  • Duplicate notifications: add constraints to prevent duplicates
  • Add/Tweak status for email tasks (e.g. partially success, auto-fixed, etc.)
  • Distinguish between none-frequency sent and actual sent

Side Effects

N/A

QE Notes

N/A

CE Notes

N/A

Documentation

N/A

@Ostap-Zherebetskyi Ostap-Zherebetskyi force-pushed the hotfix/NR_model_migration branch from e72dbb2 to 1d70ead Compare January 8, 2026 13:04
@Ostap-Zherebetskyi Ostap-Zherebetskyi marked this pull request as ready for review January 8, 2026 13:32
@Ostap-Zherebetskyi Ostap-Zherebetskyi changed the title [ENG-9854][ENG-10025][ENG-10039][ENG-10040] NR model migration [ENG-9854][ENG-10025][ENG-10039][ENG-10040][ENG-10024] NR model migration Jan 8, 2026
@Ostap-Zherebetskyi Ostap-Zherebetskyi force-pushed the hotfix/NR_model_migration branch from 1d70ead to 724cdda Compare January 8, 2026 13:49
cslzchen
cslzchen approved these changes Jan 8, 2026
View reviewed changes
Copy link
Collaborator

@cslzchen cslzchen left a comment

Choose a reason for hiding this comment

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

LGTM ⭐

Note: let's take out three hotfix pieces and make a different PR, and I will re-target this PR to feature/notifications-refactor-post-release once 26.1.5 is released.

@cslzchen cslzchen mentioned this pull request Jan 8, 2026
Merged
@cslzchen cslzchen changed the title [ENG-9854][ENG-10025][ENG-10039][ENG-10040][ENG-10024] NR model migration [ENG-9854][ENG-10025][ENG-10040] NR Post-release model migration Jan 8, 2026
@cslzchen cslzchen changed the title [ENG-9854][ENG-10025][ENG-10040] NR Post-release model migration [ENG-9854][ENG-10025][ENG-10040] NR Post-release model migration + model dependent bug-fixes Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cslzchen cslzchen cslzchen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Ostap-Zherebetskyi @cslzchen