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
Support mariner via github action [MOD-5872] #1516
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1516 +/- ##
=======================================
Coverage 89.02% 89.02%
=======================================
Files 35 35
Lines 5834 5834
=======================================
Hits 5194 5194
Misses 640 640 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done!
A few comments
.github/workflows/mariner2.yml
Outdated
uses: actions/checkout@v4 | ||
with: | ||
submodules: 'recursive' | ||
set-safe-directory: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can checkout get true here? I was under the impression it takes a path (or * for all).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, when I tried with "'*'"
(as I saw you did) I got an error...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and without specifying set-safe-directory
does it work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, because I have git config --global --add safe.directory $PWD
in mariner2.sh
Support mariner via github action [MOD-5872]
Support mariner via github action [MOD-5872]
Support mariner2 OS
Use github actions rather than circle-ci and a compact workflow.
Also, since setting the required configs in redis via RLTest to ensure that we can run debug commands and change protected configs in redis (previously this has been done via patch in
getredis
script)MOD-5872