You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2020/11/04 07:27:37 UTC

[GitHub] [zookeeper] odidev opened a new pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

odidev opened a new pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433


   1. Added ARM64 support with jdk 11
   2. Installed maven if architecture is ARM64
   
   Signed-off-by: odidev <od...@puresoftware.com>


----------------------------------------------------------------
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] [zookeeper] eolivelli closed pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli closed pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433


   


----------------------------------------------------------------
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] [zookeeper] odidev commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
odidev commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-752461617


   Hi Team,
   This PR aims to add Linux/ARM64 support to the Travis-CI only. It is affecting neither the source code nor the amd64 build at all.
    And Travis-CI is successfully passing as well. It would be great to merge this PR.


----------------------------------------------------------------
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] [zookeeper] emiliofernandes commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
emiliofernandes commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-732140695


   +1 for ARM64 !


----------------------------------------------------------------
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] [zookeeper] ctubbsii commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-721591730


   > closed/reopened in order to kickstart Jenkins
   
   I understand it's the ZK team's expectation to get a "green" build on Jenkins for PRs before merging, but in this case, there's literally nothing of value a Jenkins build could offer for this PR, since the only file changed is Travis CI config file, which has no impact on anything else in the repository other than the Travis builds... which *did* get a green build.
   
   The only thing that can be accomplished by kicking off more Jenkins builds for this change set is to waste shared Jenkins build resources and to delay ARM64 builds from being activated for other PRs. I recommend merging on the basis of the green Travis build.


----------------------------------------------------------------
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] [zookeeper] odidev commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
odidev commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-721537672


   Hi Team, can you please review this PR ?


----------------------------------------------------------------
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] [zookeeper] eolivelli commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-752757329


   +1 I will merge as soon as possible (probably tomorrow)


----------------------------------------------------------------
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] [zookeeper] odidev commented on pull request #1433: Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
odidev commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-679762829


   Hi @eolivelli 
   I have raised the JIRA for adding ARM64 support to Travis-CI. Kindly find the link : https://issues.apache.org/jira/browse/ZOOKEEPER-3919


----------------------------------------------------------------
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] [zookeeper] odidev commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
odidev commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-696643611


   Hi Team,
   Can you please review this PR ?


----------------------------------------------------------------
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] [zookeeper] eolivelli closed pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli closed pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433


   


----------------------------------------------------------------
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] [zookeeper] eolivelli commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-752892912


   @odidev Please take care of checking that it is now working properly.
   It would be great If you can announce this new QA test to dev@zookkeeper.apache.org


----------------------------------------------------------------
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] [zookeeper] martin-g commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
martin-g commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-752884088


   > +1 I will merge as soon as possible (probably tomorrow)
   
   That would be a great finish of the year! :fireworks: 


----------------------------------------------------------------
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] [zookeeper] ctubbsii commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-752733587


   > Hi Team,
   > This PR aims to add Linux/ARM64 support to the Travis-CI only. It is affecting neither the source code nor the amd64 build at all.
   > And Travis-CI is successfully passing as well. It would be great to merge this PR.
   
   I agree. This should be merged.


----------------------------------------------------------------
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] [zookeeper] odidev commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
odidev commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-696643611


   Hi Team,
   Can you please review this PR ?


----------------------------------------------------------------
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] [zookeeper] martin-g commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
martin-g commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-753123663


   The first build with ARM64 job has passed successfully: https://travis-ci.com/github/apache/zookeeper/builds/211393928 !


----------------------------------------------------------------
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] [zookeeper] eolivelli commented on pull request #1433: Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-678366533


   can you please create a JIRA ?


----------------------------------------------------------------
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] [zookeeper] eolivelli commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-721560920


   closed/reopened in order to kickstart Jenkins


----------------------------------------------------------------
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] [zookeeper] eolivelli commented on pull request #1433: ZOOKEEPER-3919 : Added ARM64 support to Travis-CI

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #1433:
URL: https://github.com/apache/zookeeper/pull/1433#issuecomment-752892971


   committed to master branch !
   thank you @odidev 


----------------------------------------------------------------
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