Skip to content
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.

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

landing_job: add landed_revisions field (bug 1834885) #318

Merged
merged 5 commits into from
May 26, 2023

Conversation

zzzeid
Copy link
Contributor

@zzzeid zzzeid commented May 25, 2023

  • add diff_id to revision_landing_job table
  • add method to set diff IDs and call it at landing time
  • add landed_revisions helper method to fetch revision_to_diff_ids
  • add a test that lands two versions of a revision
  • fix variable names in transplants.warning_previously_landed

@zzzeid zzzeid changed the base branch from main to zeid/bug-1834868-fix-key-error-revisions May 25, 2023 02:04
@zzzeid zzzeid changed the title Zeid/bug 1834885 track diff id on landing receipt landing_job: add landed revisions receipt (bug 1834885) May 25, 2023
Base automatically changed from zeid/bug-1834868-fix-key-error-revisions to main May 25, 2023 12:53
@zzzeid zzzeid force-pushed the zeid/bug-1834885-track-diff-id-on-landing-receipt branch 3 times, most recently from 45ab553 to cbaf7ce Compare May 25, 2023 15:41
@zzzeid zzzeid marked this pull request as ready for review May 25, 2023 15:42
@zzzeid zzzeid requested a review from cgsheeh May 25, 2023 15:42
@zzzeid zzzeid changed the title landing_job: add landed revisions receipt (bug 1834885) landing_job: add landed revisions field (bug 1834885) May 25, 2023
@zzzeid zzzeid marked this pull request as draft May 25, 2023 16:20
@zzzeid zzzeid force-pushed the zeid/bug-1834885-track-diff-id-on-landing-receipt branch from cbaf7ce to f2d488c Compare May 25, 2023 16:24
- add diff_id to revision_landing_job table
- add method to set diff IDs and call it at landing time
- add landed_revisions helper method to fetch revision_to_diff_ids
- add a test that lands two versions of a revision
- fix variable names in transplants.warning_previously_landed
@zzzeid zzzeid force-pushed the zeid/bug-1834885-track-diff-id-on-landing-receipt branch from f2d488c to 843c134 Compare May 25, 2023 16:26
@zzzeid zzzeid marked this pull request as ready for review May 25, 2023 16:28
landoapi/models/landing_job.py Outdated Show resolved Hide resolved
landoapi/transplants.py Outdated Show resolved Hide resolved
landoapi/models/landing_job.py Outdated Show resolved Hide resolved
tests/test_transplants.py Outdated Show resolved Hide resolved
tests/test_transplants.py Outdated Show resolved Hide resolved
tests/test_transplants.py Outdated Show resolved Hide resolved
tests/test_transplants.py Outdated Show resolved Hide resolved
zzzeid and others added 2 commits May 26, 2023 12:07
Co-authored-by: Connor Sheehan <cosheehan@mozilla.com>
@zzzeid zzzeid requested a review from cgsheeh May 26, 2023 16:13
@zzzeid zzzeid changed the title landing_job: add landed revisions field (bug 1834885) landing_job: add landed_revisions field (bug 1834885) May 26, 2023
@zzzeid zzzeid merged commit e84e171 into main May 26, 2023
1 check passed
@zzzeid zzzeid deleted the zeid/bug-1834885-track-diff-id-on-landing-receipt branch May 26, 2023 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants