Closed achave11-ucsc closed 3 weeks ago
Attention: Patch coverage is 42.85714%
with 12 lines
in your changes missing coverage. Please review.
Project coverage is 85.34%. Comparing base (
54e12dd
) to head (35c9573
). Report is 30 commits behind head on prod.
Files with missing lines | Patch % | Lines |
---|---|---|
test/integration_test.py | 0.00% | 12 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Connected issue: #6665
Checklist
Author
prod
promotions/yyyy-mm-dd-prod
Promotion yyyy-mm-dd
prod
Author (reindex, API changes)
reindex:prod
or the changes introduced by it will not require reindexing ofprod
reindex:partial
and its description documents the specific reindexing procedure forprod
or requires a full reindex or is not labeledreindex:prod
Author (upgrading deployments)
upgrade
or does not require upgrading deploymentsdeploy:shared
or does not modifydocker_images.json
, and does not require deploying theshared
component for any other reasondeploy:gitlab
or does not require deploying thegitlab
componentdeploy:runner
or does not require deploying therunner
imageSystem administrator (after approval)
no sandbox
Operator (before pushing merge the commit)
_select prod.shared && CI_COMMIT_REF_NAME=prod make -C terraform/shared apply
or this PR is not labeleddeploy:shared
prod
(see operator manual for details) or this PR is not labeledbackup:gitlab
_select prod.gitlab && CI_COMMIT_REF_NAME=prod make -C terraform/gitlab apply
or this PR is not labeleddeploy:gitlab
deploy:gitlab
deploy:gitlab
System administrator
prod.gitlab
are complete or this PR is not labeleddeploy:gitlab
Operator (before pushing merge the commit)
_select prod.gitlab && make -C terraform/gitlab/runner
or this PR is not labeleddeploy:runner
p
tagsOperator (after pushing the merge commit)
prod
prod
prod
Operator (reindex)
prod
or this PR is neither labeledreindex:partial
norreindex:prod
prod
or this PR is neither labeledreindex:partial
norreindex:prod
prod
or this PR is neither labeledreindex:partial
norreindex:prod
prod
or this PR does not require reindexingprod
prod
or this PR does not require reindexingprod
prod
or this PR does not require reindexingprod
Operator
System administrator
Shorthand for review comments
L
line is too longW
line wrapping is wrongQ
bad quotesF
other formatting problem