Skip to content

Backport: 5e419ec (#8282, PR #8262) - #8292

Merged
nadove-ucsc merged 1 commit into
developfrom
backports/5e419ec
Sep 9, 2026
Merged

Backport: 5e419ec (#8282, PR #8262)#8292
nadove-ucsc merged 1 commit into
developfrom
backports/5e419ec

Conversation

@hannes-ucsc

@hannes-ucsc hannes-ucsc commented Sep 9, 2026

Copy link
Copy Markdown
Member

Linked issues: #8282

Checklist

Author

  • PR is assigned to the author
  • Status of PR is In progress
  • Target branch is develop
  • Name of PR branch matches backports/<7-digit SHA1 of most recent backported commit>
  • PR is linked to the issues it backports
  • Status of linked issues is Stable
  • PR title contains the 7-digit SHA1 of the backported commits
  • PR title references the issues relating to the backported commits
  • PR title references the PRs that introduced the backported commits

Author (before every review)

  • PR branch is up to date (if not, merge develop into PR branch to integrate upstream changes)
  • Ran make requirements_update or this PR does not modify pyproject.toml
  • Added R tag to commit title or this PR does not modify uv.lock
  • This PR is labeled reqs or does not modify uv.lock
  • PR is not a draft
  • PR is awaiting requested review from system administrator
  • Status of PR is Review requested
  • PR is assigned to only the system administrator and the author

System administrator (after approval)

  • Actually approved the PR
  • Decided if PR can be labeled no sandbox
  • PR title is appropriate as title of merge commit
  • N reviews label is accurate
  • Status of PR is Approved
  • PR is assigned to only the operator and the author

Operator

  • Sanity-checked history
  • Pushed PR branch to GitHub

Operator (sandbox build)

  • Added sandbox label or PR is labeled no sandbox
  • Pushed PR branch to GitLab dev or PR is labeled no sandbox
  • Pushed PR branch to GitLab anvildev or PR is labeled no sandbox
  • Build passes in sandbox deployment or PR is labeled no sandbox
  • Build passes in anvilbox deployment or PR is labeled no sandbox
  • Reviewed build logs for anomalies in sandbox deployment or PR is labeled no sandbox
  • Reviewed build logs for anomalies in anvilbox deployment or PR is labeled no sandbox
  • In sandbox, deleted the catalogs specified in the notes or this PR is missing either the reindex:partial or the reindex:dev label, or both
  • In anvilbox, deleted the catalogs specified in the notes or this PR is missing either the reindex:partial or the reindex:anvildev label, or both
  • In sandbox, deindexed the sources sepcified in the notes or this PR is missing either the reindex:partial or the reindex:dev label, or both
  • In anvilbox, deindexed the sources sepcified in the notes or this PR is missing either the reindex:partial or the reindex:anvildev label, or both
  • In sandbox, indexed the sources specified in the notes or this PR is missing either the reindex:partial or the reindex:dev label, or both
  • In anvilbox, indexed the sources specified in the notes or this PR is missing either the reindex:partial or the reindex:anvildev label, or both
  • In sandbox, indexed the catalogs specified in the notes or this PR is missing either the reindex:partial or the reindex:dev label, or both
  • In anvilbox, indexed the catalogs specified in the notes or this PR is missing either the reindex:partial or the reindex:anvildev label, or both
  • Started full reindex in sandbox or this PR is not labeled reindex:dev or it is labeled reindex:partial
  • Started full reindex in anvilbox or this PR is not labeled reindex:anvildev or it is labeled reindex:partial
  • Checked for failures in sandbox or this PR is not labeled reindex:dev
  • Checked for failures in anvilbox or this PR is not labeled reindex:anvildev
  • Started mirroring in sandbox or this PR is not labeled mirror:dev
  • Started mirroring in anvilbox or this PR is not labeled mirror:anvildev
  • Checked for failures in sandbox or this PR is not labeled mirror:dev
  • Checked for failures in anvilbox or this PR is not labeled mirror:anvildev

Operator (merge the branch)

  • All status checks passed and the PR is mergeable
  • The title of the merge commit starts with the title of this PR
  • Added PR # reference (to this PR) to merge commit title
  • Collected commit title tags in merge commit title but excluded any p tags Use [H], the only tag carried by the sole backported commit, 5e419ec, giving [H] Backport: 5e419ec (#8282, PR #8262, PR #8292).
  • Pushed merge commit to GitHub
  • Status of PR is Merged lower

Operator (main build)

  • Pushed merge commit to GitLab dev
  • Pushed merge commit to GitLab anvildev
  • Build passes on GitLab dev
  • Reviewed build logs for anomalies on GitLab dev
  • Build passes on GitLab anvildev
  • Reviewed build logs for anomalies on GitLab anvildev
  • Deleted PR branch from GitHub
  • PR is assigned to only the operator
  • Deleted PR branch from GitLab dev
  • Deleted PR branch from GitLab anvildev
  • Status of linked issues is Stable

Operator

  • PR is assigned to no one

Shorthand for review comments

  • L line is too long
  • W line wrapping is wrong
  • Q bad quotes
  • F other formatting problem

@hannes-ucsc hannes-ucsc self-assigned this Sep 9, 2026
@coveralls

coveralls commented Sep 9, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 84.87% (-0.001%) from 84.871% — backports/5e419ec into develop

@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.79%. Comparing base (fc54279) to head (8ad76f1).

Files with missing lines Patch % Lines
src/azul/terra.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8292      +/-   ##
===========================================
- Coverage    84.79%   84.79%   -0.01%     
===========================================
  Files          168      168              
  Lines        25196    25201       +5     
===========================================
+ Hits         21365    21369       +4     
- Misses        3831     3832       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hannes-ucsc hannes-ucsc mentioned this pull request Sep 9, 2026
7 tasks
…ng (#8282)

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@nadove-ucsc nadove-ucsc added the sandbox [process] Resolution is being verified in sandbox deployment label Sep 9, 2026
@nadove-ucsc
nadove-ucsc self-requested a review September 9, 2026 06:54

@nadove-ucsc nadove-ucsc left a comment

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.

Pro-forma review

@nadove-ucsc
nadove-ucsc merged commit 7899ae5 into develop Sep 9, 2026
10 checks passed
@nadove-ucsc
nadove-ucsc deleted the backports/5e419ec branch September 9, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sandbox [process] Resolution is being verified in sandbox deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redundant TDR requests for snapshot info during indexing

3 participants