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.
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
aarch64 CI test #829
base: master
Are you sure you want to change the base?
aarch64 CI test #829
Changes from all commits
b3da8e8
dd3f464
0d865ad
7dd2ae3
06ac8a0
d7caa06
ef0f2cd
1604ee5
d971d94
d293c70
a877bf8
1f5258e
c23a3f6
604f76b
d0b6e7f
5dbc3d1
51a9819
3a62302
edfa4f8
cfc3cb4
b633cdc
9582dbe
e507d6c
9f0e893
a9ac503
89388c1
2815817
80ffc3c
960e7a4
52b9ffb
586e2e6
9f72251
33eacc3
5524ea5
ceeb6f6
36c6c0f
3437f88
c316a3b
4eb9433
b279df1
74f6e20
46ec913
8497de2
68bdeef
04354c7
d3b5e9c
a3811c8
356bc6f
0b1fc9b
4353bac
1617c95
7672787
ccef714
8985c8f
5045998
9767996
fdd6084
0288e87
e258e9a
b0420e7
75e716c
3664265
620b8d0
00437e3
b93f1b3
c724215
2e1dd7f
f56031c
81d8897
c6c61ae
fa85bd9
84834c2
145fe0b
daac8bc
0ee2fa1
0bb6425
0370890
da0ff0d
f07d9a1
f9607af
c2dc530
0c04986
236ddfa
95019ca
3f1f2a3
e3bfa84
8546a66
217a428
e6894c8
077788f
3a1b3a1
b9b98b2
4bbd140
bd135c4
5eac8c7
47dbaa8
deed41b
1d82edf
2369a97
2322175
9fe633d
c88bd12
e9137e1
227edbb
0167c94
78b9974
3a45b06
3d90538
307441e
20863a8
ab49a8c
e251f4e
8d8f71c
7c7af02
346d983
f481476
5552817
5b29cda
83887a0
f5ecbd2
db31537
283dccf
0ca35c8
82c0681
487ea66
eaaf31c
c10ee6c
70976d7
363a4bf
00a1106
f13f92b
62a72b2
39036f6
6c53d00
9f68cf7
c1b8a96
a4c35dd
9888f76
deb7715
cd27bc8
7fd7d14
0e76c21
fa4ce53
933546b
8bbde84
4111253
ac7e9ab
8115d7b
bff0e6a
bed61f3
f5c86af
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Need sudo?
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.
"Download the image from previous job", no?
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.
After you follow my suggestion from https://github.com/Granulate/gprofiler/pull/829/files#r1314022332 you can also simplify the code here to use the gProfiler executable instead of image - so it can run the exe and don't need the docker, the mappings etc, just like the x86_64 job.
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.
This comment is not relevant here, remove pls
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.
The x86_64 tests run on 3.8, 3.9, 3.10.
I'm actually planning to deprecate 3.8 and 3.9 soon so there's no need to test all 3 - let's focus on 3.10.
Can you use the GH action to install python? i.e
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.
You know what, let's actually do it with
strategy
like done in the x86_64 tests:this is extendable for a future where we'll want to add 3.11, and also shows up nicely in the job name (gonna be "test-container-aarch64 (3.10)" like we have for the x86 ones)
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.
Need it?