Skip to content

feat: support importing merge automation data#2027

Merged
bhearsum merged 2 commits into
mozilla-releng:mainfrom
bhearsum:feat-support-importing-merge-a
Jul 24, 2026
Merged

feat: support importing merge automation data#2027
bhearsum merged 2 commits into
mozilla-releng:mainfrom
bhearsum:feat-support-importing-merge-a

Conversation

@bhearsum

Copy link
Copy Markdown
Contributor

I'm working on some enhancements to merge automation; being able to import this data from production would be helpful.

I didn't implement any limitations on how many can be important because:

  1. These don't build up very quickly (in ~2 months we've had 19 rows added)
  2. They're cheap to import
  3. They're all returned at the same time anyways; limiting the imports only saves work on the client, not the server

@bhearsum
bhearsum force-pushed the feat-support-importing-merge-a branch from ad13e05 to a1eeb99 Compare July 21, 2026 14:16
I'm working on some enhancements to merge automation; being able to import this data from production would be helpful.

I didn't implement any limitations on how many can be important because:
1) These don't build up very quickly (in ~2 months we've had 19 rows added)
2) They're cheap to import
3) They're all returned at the same time anyways; limiting the imports only saves work on the client, not the server
@bhearsum
bhearsum force-pushed the feat-support-importing-merge-a branch from a1eeb99 to acc225f Compare July 21, 2026 14:24
@bhearsum
bhearsum marked this pull request as ready for review July 21, 2026 14:52
@bhearsum
bhearsum requested a review from a team as a code owner July 21, 2026 14:52
Comment thread api/src/shipit_api/admin/cli.py Outdated
project=merge["project"],
)
session.add(m)
session.commit()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we batch them instead of committing one at a time?

@bhearsum
bhearsum merged commit 55e2962 into mozilla-releng:main Jul 24, 2026
7 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