-
Notifications
You must be signed in to change notification settings - Fork 120
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
PMM-12899 Use go mod cache in actions #3026
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v3 #3026 +/- ##
==========================================
- Coverage 44.63% 44.58% -0.06%
==========================================
Files 368 368
Lines 35397 35397
==========================================
- Hits 15801 15781 -20
- Misses 17971 17988 +17
- Partials 1625 1628 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -22,6 +22,13 @@ PMM_LD_FLAGS = -ldflags " \ | |||
-X 'github.com/percona/pmm/version.Branch=$(PMM_RELEASE_BRANCH)' \ | |||
" | |||
|
|||
IS_ARM := $(filter arm64,$(shell uname -m)) |
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.
Maybe we should include aarch64
in the filter list too?
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.
Good idea!
Do you know what type of hardware we could get it from? I've checked my M2 processor and only got arm64
.
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.
I think it must be RaspberryPi, right? In such case I doubt it's a good idea to run PMM Server on a Pi machine, it'd not have sufficient resources to be performant.
PMM-12899
Please check the motivation for using a docker image to persist cache before reviewing.