You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2020/11/09 09:12:29 UTC
[GitHub] [storm] liusheng opened a new pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
liusheng opened a new pull request #3348:
URL: https://github.com/apache/storm/pull/3348
In # apache/storm/pull/3344, we have enabled ARM CI with all Storm
modules enabled and switched to use the "arm64-graviton2" resources, but
I have made a mistak that the ARM CI actually still run on x86 than
ARM even the CI job shows the label "arm64-graviton2", details can be
found in the CI job logs[1]. Because the "arm64-graviton2" requires
explicit virt: [lxd|vm] tag, see[2].
Addtionally, we have skip the increasing swap space statement in
intergration test because it will raise error on ARM. And we only
enabled openjdk11 for ARM jobs because the jdk8 is unsupported, see[3].
[1] https://travis-ci.org/github/apache/storm/jobs/741780900
[2] https://docs.travis-ci.com/user/multi-cpu-architectures#testing-on-multiple-cpu-architectures
[3] https://travis-ci.com/github/liusheng/storm/jobs/430934514
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm merged pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm merged pull request #3348:
URL: https://github.com/apache/storm/pull/3348
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-729313128
Hi @Ethanlm ,
Thanks a lot for paying attention for this, it looks like now the Storm projects has been migrated to travis-ci.com, lets test this again.
----------------------------------------------------------------
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
[GitHub] [storm] liusheng edited a comment on pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng edited a comment on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-745102182
Hi @Ethanlm ,
Thanks for your help, I have tried some times in https://github.com/apache/storm/pull/3361, now we can run X86 and ARM64 sucessfully by using `xenial` dist of Ubuntu, and now the common `arm64` resources is OK for ARM CI running. because Ubuntu xenial cannot support `oraclejdk8`, so have changed to use `openjdk8`.
I have updated the changes to this PR, please help to review again. thank you.
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-742269189
Thank for your review @Ethanlm , let me try to use the "arm64" resources on ARM CI to see if it is OK now.
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-745102182
Hi @Ethanlm ,
Thanks for your help, I have tried some times in https://github.com/apache/storm/pull/3361, now we can run X86 and ARM64 sucessfully by using `xenial` dist of Ubuntu, and now the common `arm64` resources is OK for ARM CI running, I have updated the changes to this PR, please help to review again. thank you.
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r544767959
##########
File path: integration-test/run-it.sh
##########
@@ -101,3 +104,7 @@ for i in {1..20} ; do
done
list_storm_processes
mvn test -DfailIfNoTests=false -DskipTests=false -Dstorm.version=${STORM_VERSION} -Dui.url=http://localhost:8744
+
+# Kill all storm processes after tests running, otherwise the Travis CI(ARM) will still wait and then fail by
+# timeout error even though all the tests passed
+sudo pkill -9 -u storm
Review comment:
Thanks
----------------------------------------------------------------
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
[GitHub] [storm] liusheng closed pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng closed pull request #3348:
URL: https://github.com/apache/storm/pull/3348
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-724119848
I wonder how PR build passed last time.
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-730287933
Hi @Ethanlm ,
Thanks for your previous help about ARM CI, currently, Storm CI has been migrated to use `travis-ci.com`, in this patch, I have fixed in the .travis.yml and made small changes to make the ARM CI can run successfully in `travis-ci.com`. Could you please help to review again ? Thanks a lot.
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r540258663
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
+ group: edge
+ arch: arm64-graviton2
Review comment:
Can you link to the announcement? It looks like they all should work as indicated in the https://docs.travis-ci.com/user/reference/overview/#linux-travisyml-examples and I don't see performance concerns.
The arm64 configuration seems more clean and simple
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r544378159
##########
File path: integration-test/run-it.sh
##########
@@ -101,3 +104,7 @@ for i in {1..20} ; do
done
list_storm_processes
mvn test -DfailIfNoTests=false -DskipTests=false -Dstorm.version=${STORM_VERSION} -Dui.url=http://localhost:8744
+
+# Kill all storm processes after tests running, otherwise the Travis CI(ARM) will still wait and then fail by
+# timeout error even though all the tests passed
+sudo pkill -9 -u storm
Review comment:
Saw this in your test job https://travis-ci.com/github/apache/storm/jobs/460903786
```
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
The build has been terminated
```
Is this what you saw before?
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r544374213
##########
File path: integration-test/run-it.sh
##########
@@ -36,10 +36,13 @@ then
chmod o+rx /home/travis
fi
list_storm_processes || true
-# increasing swap space so we can run lots of workers
-sudo dd if=/dev/zero of=/swapfile.img bs=4096 count=1M
-sudo mkswap /swapfile.img
-sudo swapon /swapfile.img
+if [ "$(uname -m)" != aarch64 ]; then
Review comment:
Thanks for validating.
For future reference, the log is
```
+sudo dd if=/dev/zero of=/swapfile.img bs=4096 count=1M
1048576+0 records in
1048576+0 records out
4294967296 bytes (4.3 GB, 4.0 GiB) copied, 27.2215 s, 158 MB/s
+sudo mkswap /swapfile.img
Setting up swapspace version 1, size = 4 GiB (4294963200 bytes)
no label, UUID=ba655d1c-230e-4feb-a06a-e342c114d7eb
+sudo swapon /swapfile.img
swapon: /swapfile.img: insecure permissions 0644, 0600 suggested.
swapon: /swapfile.img: skipping - it appears to have holes.
The command "if [[ $(uname -m) != 's390x' ]]; then /bin/bash ./dev-tools/travis/travis-script.sh `pwd` $MODULES; fi" exited with 255.
```
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm removed a comment on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm removed a comment on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-724119848
I wonder how the PR build passed last time.
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r539420089
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
+ group: edge
Review comment:
What does `group` mean here?
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
+ group: edge
+ arch: arm64-graviton2
Review comment:
Is it possible to use arm64 directly?
https://docs.travis-ci.com/user/reference/overview/#linux-travisyml-examples
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
Review comment:
The default dist configured above we use is `trusty`. Should we upgrade the default dist to `focal`? (it can be a separate work)
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm edited a comment on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm edited a comment on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-724119848
I wonder how the PR build passed last time.
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-723944226
Hmm. looks like this change still cannot correctly use "arm64-graviton2" ARM resources in Travis. according to the [CI results](https://travis-ci.org/github/apache/storm/builds/742395332) of this change, it still use **x86** than **ARM** in the ARM job. that is because currently the "arm64-graviton2" is only available in **travis-ci.com**, not available in **travis-ci.org**, and Storm projects use **travis-ci.org**, please see the [Travis ci docs](https://docs.travis-ci.com/user/multi-cpu-architectures#multi-cpu-availaibility).
Addtionally, it is recomended to use **travis-ci.com** than **travis-ci.org** and the latter will be closed down in the future. please see: https://devops.stackexchange.com/questions/1201/whats-the-difference-between-travis-ci-org-and-travis-ci-com
So, maybe we can promote to migrate to use "travis-ci.com" in Storm project ? or there is already a plan ?
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r543452375
##########
File path: integration-test/run-it.sh
##########
@@ -36,10 +36,13 @@ then
chmod o+rx /home/travis
fi
list_storm_processes || true
-# increasing swap space so we can run lots of workers
-sudo dd if=/dev/zero of=/swapfile.img bs=4096 count=1M
-sudo mkswap /swapfile.img
-sudo swapon /swapfile.img
+if [ "$(uname -m)" != aarch64 ]; then
Review comment:
Is this change still needed after the change to arm64 and xenial
##########
File path: integration-test/run-it.sh
##########
@@ -101,3 +104,7 @@ for i in {1..20} ; do
done
list_storm_processes
mvn test -DfailIfNoTests=false -DskipTests=false -Dstorm.version=${STORM_VERSION} -Dui.url=http://localhost:8744
+
+# Kill all storm processes after tests running, otherwise the Travis CI(ARM) will still wait and then fail by
+# timeout error even though all the tests passed
+sudo pkill -9 -u storm
Review comment:
Is this change still needed after the change to arm64 and xenial
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r544009169
##########
File path: integration-test/run-it.sh
##########
@@ -36,10 +36,13 @@ then
chmod o+rx /home/travis
fi
list_storm_processes || true
-# increasing swap space so we can run lots of workers
-sudo dd if=/dev/zero of=/swapfile.img bs=4096 count=1M
-sudo mkswap /swapfile.img
-sudo swapon /swapfile.img
+if [ "$(uname -m)" != aarch64 ]; then
Review comment:
yes, there still have problems of swap support on Travis ARM64 resources, you can take a look the error:
https://travis-ci.com/github/apache/storm/jobs/460842259
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by 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
[GitHub] [storm] liusheng closed pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng closed pull request #3348:
URL: https://github.com/apache/storm/pull/3348
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r543094435
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
+ group: edge
+ arch: arm64-graviton2
Review comment:
There is not annoucement, just be cause I have tested many times, the ARM CI jobs are very easy to fail due to timeout errors previously. I have tested again in the PR https://github.com/apache/storm/pull/3361, it looks like now the common `arm64` resource is OK to run the ARM CI job :), so let's switch to use the `arm64` resources
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r540256904
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
Review comment:
Yeah, I was thinking maybe we can simply this by updating https://github.com/apache/storm/blob/fe2f7102e244336e288d26f2dde8089198ee4c33/.travis.yml#L28
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r544739539
##########
File path: integration-test/run-it.sh
##########
@@ -101,3 +104,7 @@ for i in {1..20} ; do
done
list_storm_processes
mvn test -DfailIfNoTests=false -DskipTests=false -Dstorm.version=${STORM_VERSION} -Dui.url=http://localhost:8744
+
+# Kill all storm processes after tests running, otherwise the Travis CI(ARM) will still wait and then fail by
+# timeout error even though all the tests passed
+sudo pkill -9 -u storm
Review comment:
yes, I have tried again in #3361, you can take a look the CI logs: https://travis-ci.com/github/apache/storm/jobs/460903786
at the end of the log, even all the intgration tests ran sucessfully, but the job didn't exit immediately and then failed due to timeout.
```
0.11s$ rm -rf $HOME/.m2/repository/org/apache/storm
cache.2
store build cache
Done. Your build exited with 0.
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
The build has been terminated
```
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-723880742
Hi @Ethanlm ,
I am so sorry I have made a mistake, the https://github.com/apache/storm/pull/3344 has introduced a wrong usage of Travis "arm64-graviton2" resources, it shows the "arm64-graviton2" label on the job, but in the CI logs, we can find it actually use x86 resources. So here we need to fix.
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on a change in pull request #3348: [STORM-3711] Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on a change in pull request #3348:
URL: https://github.com/apache/storm/pull/3348#discussion_r543093339
##########
File path: .travis.yml
##########
@@ -41,6 +40,48 @@ matrix:
include:
- arch: s390x
jdk: openjdk11
+ - dist: focal
Review comment:
OK, let's me try
----------------------------------------------------------------
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
[GitHub] [storm] liusheng commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
liusheng commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-736178566
Hi @Ethanlm ,
Sorry to disturb you again, could you please help to take a look this PR ? This change try to fix the ARM CI of Strom to running on real `arm64-graviton2` resource. Thank you.
----------------------------------------------------------------
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
[GitHub] [storm] Ethanlm commented on pull request #3348: Fix ARM CI with wrong usage of "arm64-graviton2" resources
Posted by GitBox <gi...@apache.org>.
Ethanlm commented on pull request #3348:
URL: https://github.com/apache/storm/pull/3348#issuecomment-724160281
Thanks for looking into it. I was not careful enough to see it actually ran on amd64 instead of arm64. Also thanks for the links. I will see if it makes sense to switch to travis-ci.com. It seems to be still on beta though
----------------------------------------------------------------
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