Fix transient IT failure (#5167) #5326
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nadove-ucsc
force-pushed
the
issues/nadove-ucsc/5167-followup-transient-failure
branch
from
7cddb26 to
b69dd11
Compare
Codecov Report
@@ Coverage Diff @@
## develop #5326 +/- ##
========================================
Coverage 83.85% 83.85%
========================================
Files 152 152
Lines 18539 18539
========================================
Hits 15546 15546
Misses 2993 2993
|
|
Underlying problem this is meant to solve is when entities occur in multiple bundles. |
hannes-ucsc
reviewed
Jun 20, 2023
nadove-ucsc
force-pushed
the
issues/nadove-ucsc/5167-followup-transient-failure
branch
from
b69dd11 to
4180790
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
hannes-ucsc
force-pushed
the
issues/nadove-ucsc/5167-followup-transient-failure
branch
from
4180790 to
6d2bbc9
Compare
hannes-ucsc
approved these changes
Jun 21, 2023
achave11-ucsc
force-pushed
the
issues/nadove-ucsc/5167-followup-transient-failure
branch
from
6d2bbc9 to
453659a
Compare
achave11-ucsc
added
the
sandbox
achave11-ucsc
deleted the
issues/nadove-ucsc/5167-followup-transient-failure
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Connected issues: #5167
This PR resolves a bug introduced by the previous PR for this ticket.
Checklist
Author
developissues/<GitHub handle of author>/<issue#>-<slug>PR title matches1 that of a connected issuecomment in PR explains why they're differentpartiallabel to PR or this PR completely resolves all connected issues1 when the issue title describes a problem, the corresponding PR
title is
Fix:followed by the issue titleAuthor (reindex, API changes)
rtag to commit title or this PR does not require reindexingreindexlabel to PR or this PR does not require reindexinga(compatible changes) orA(incompatible ones) tag to commit title or this PR does not modify the Azul service APIAPIlabel to connected issues or this PR does not modify the Azul service APIAuthor (chains)
baselabel to the blocking PR or this PR is not chained to another PRchainedlabel to this PR or this PR is not chained to another PRAuthor (upgrading)
utag to commit title or this PR does not require upgradingupgradelabel to PR or this PR does not require upgradingAuthor (operator tasks)
Author (hotfixes)
Ftag to main commit title or this PR does not include permanent fix for a temporary hotfixprodbranch has no temporary hotfixes for any connected issuesAuthor (before every review)
develop, squashed old fixupsmake requirements_updateor this PR does not touch requirements*.txt, common.mk, Makefile and DockerfileRtag to commit title or this PR does not touch requirements*.txtreqslabel to PR or this PR does not touch requirements*.txtmake integration_testpasses in personal deployment or this PR does not touch functionality that could break the ITPeer reviewer (after requesting changes)
Uncheck the Author (before every review) checklists.
Peer reviewer (after approval)
Primary reviewer (after requesting changes)
Uncheck the before every review checklists. Update the
N reviewslabel.Primary reviewer (after approval)
demoorno demono demono sandboxN reviewslabel is accurateOperator (before pushing merge the commit)
reindexlabel andrcommit title tagno demoupgradedevelopdevand addedsandboxlabel or PR is labeledno sandboxanvildevor PR is labeledno sandboxsandboxdeployment or PR is labeledno sandboxanvilboxdeployment or PR is labeledno sandboxsandboxdeployment or PR is labeledno sandboxanvilboxdeployment or PR is labeledno sandboxsandboxor this PR does not remove catalogs or otherwise causes unreferenced indicesanvilboxor this PR does not remove catalogs or otherwise causes unreferenced indicessandboxor this PR does not require reindexingsandboxanvilboxor this PR does not require reindexingsandboxsandboxor this PR does not require reindexingsandboxanvilboxor this PR does not require reindexingsandboxOperator (chain shortening)
developor this PR is not labeledbasechainedlabel from the blocked PR or this PR is not labeledbasebasebaselabel from this PR or this PR is not labeledbaseOperator (after pushing the merge commit)
devor PR is labeledno sandboxanvildevor PR is labeledno sandboxanvilprodor PR is labeledno sandboxdev1dev1anvildev1anvildev1anvilprod1anvilprod1devanvildevanvilprod1 When pushing the merge commit is skipped due to the PR being
labelled
no sandbox, the next build triggered by a PR whose merge commit ispushed determines this checklist item.
Operator (reindex)
devor this PR does not remove catalogs or otherwise causes unreferenced indicesanvildevor this PR does not remove catalogs or otherwise causes unreferenced indicesanvilprodor this PR does not remove catalogs or otherwise causes unreferenced indicesdevor this PR does not require reindexinganvildevor this PR does not require reindexinganvilprodor this PR does not require reindexingdevor this PR does not require reindexinganvildevor this PR does not require reindexinganvilprodor this PR does not require reindexingdevdeployment or this PR does not require reindexinganvildevdeployment or this PR does not require reindexinganvilproddeployment or this PR does not require reindexingOperator
Shorthand for review comments
Lline is too longWline wrapping is wrongQbad quotesFother formatting problem