[GitHub] [storm] liusheng commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources

[GitBox <gi...@apache.org>]

liusheng commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r539881557



##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
   include:
     - arch: s390x
       jdk: openjdk11
+    - dist: focal
+      group: edge

Review comment:
       sorry, this is unnecessary, my mistake.

##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
   include:
     - arch: s390x
       jdk: openjdk11
+    - dist: focal
+      group: edge
+      arch: arm64-graviton2

Review comment:
       Hmm,  we have tried using the common `arm64` resources, but many modules(especially Integration-Test) are unstable and easy to fail. the new added `arm64-graviton` is anounced a better performance ARM resources, so switch to use it. 
   Maybe we can try back to use the `arm64` resources as we have migrated from `travis-ci.org` to `travis-ci.com` to see if the problems still occur.

##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
   include:
     - arch: s390x
       jdk: openjdk11
+    - dist: focal

Review comment:
       Currently, the "arm64-graviton2" resources are only available in `Xenial`, `Bonic`, `Focal` dists of Ubuntu, please see the [Travis doc](https://docs.travis-ci.com/user/reference/overview#which-one-do-i-use)， I just picked a lastest version.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org