-
Notifications
You must be signed in to change notification settings - Fork 2
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
Switch GitLab to stock Amazon Linux 2 AMI (#4188, #3942) #4449
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
hannes-ucsc
changed the title
Issues/hannes ucsc/4188 amazon linux
Switch GitLab to stock Amazon Linux 2 AMI (#4188)
Sep 14, 2022
64 tasks
Codecov Report
@@ Coverage Diff @@
## develop #4449 +/- ##
========================================
Coverage 82.11% 82.11%
========================================
Files 131 131
Lines 16312 16312
========================================
Hits 13394 13394
Misses 2918 2918 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
hannes-ucsc
changed the title
Switch GitLab to stock Amazon Linux 2 AMI (#4188)
Switch GitLab to stock Amazon Linux 2 AMI (#4188, #3942)
Sep 14, 2022
achave11-ucsc
force-pushed
the
issues/hannes-ucsc/4188-amazon-linux
branch
from
September 15, 2022 17:05
db720ad
to
a04d7c1
Compare
achave11-ucsc
added
the
sandbox
[process] Resolution is being verified in sandbox deployment
label
Sep 15, 2022
… to avoid deletion by the overlord.
achave11-ucsc
force-pushed
the
issues/hannes-ucsc/4188-amazon-linux
branch
from
September 20, 2022 06:22
c3479b7
to
5793e41
Compare
achave11-ucsc
approved these changes
Sep 20, 2022
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 issue: #4188, #3942
Continued from: #4443
Checklist
Author
develop
issues/<GitHub handle of author>/<issue#>-<slug>
partial
label to PR or this PR completely resolves the connected issue1 when the issue title describes a problem, the PR title is
Fix:
followed by the issue titleAuthor (reindex)
r
tag to commit title or this PR does not require reindexingreindex
label to PR or this PR does not require reindexingAuthor (chains)
base
label to the blocking PR or this PR is not chained to another PRchained
label to this PR or this PR is not chained to another PRAuthor (upgrading)
u
tag to commit title or this PR does not require upgradingupgrade
label to PR or this PR does not require upgradingAuthor (operator tasks)
Author (hotfixes)
F
tag to main commit title or this PR does not include permanent fix for a temporary hotfixprod
branchAuthor (requirements)
make requirements_update
or this PR does not touch requirements*.txt, common.mk, Makefile and DockerfileR
tag to commit title or this PR does not touch requirements*.txtreqs
label to PR or this PR does not touch requirements*.txtAuthor (rebasing, integration test)
make integration_test
passes in personal deployment or this PR does not touch functionality that could break the ITdevelop
, squashed old fixupsPeer reviewer (after requesting changes)
Uncheck the Author (requirements) and Author (rebasing, integration test)
checklists.
Peer reviewer (after approval)
Primary reviewer (after requesting changes)
Uncheck the Author (requirements) and Author (rebasing, integration test)
checklists. Update the
N reviews
label.Primary reviewer (after approval)
demo
orno demo
no demo
no sandbox
N reviews
label is accurateOperator (before pushing merge the commit)
dev.gitlab
prod.gitlab
anvildev.gitlab
reindex
label andr
commit title tagno demo
sandbox
label or PR is labeledno sandbox
no sandbox
sandbox
or this PR does not remove catalogs or otherwise causes unreferenced indicessandbox
or this PR does not require reindexingsandbox
sandbox
or this PR does not require reindexingsandbox
Operator (after pushing the merge commit)
base
no sandbox
1 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)
dev
or this PR does not remove catalogs or otherwise causes unreferenced indicesdev
or this PR does not require reindexingOperator
clamscan
andprune-images
jobs ran on the weekend after this was deployed todev.gitlab
clamscan
andprune-images
jobs ran on the weekend after this was deployed toprod.gitlab
clamscan
andprune-images
jobs ran on the weekend after this was deployed toanvildev.gitlab
Shorthand for review comments
L
line is too longW
line wrapping is wrongQ
bad quotesF
other formatting problem