Use Github Actions for postcommit CI is fine. But it is not a solution for precommit CI. Because the Github Actions runs on the forks' account. It is not a solution for contributors. Unless I missed something here.
- Sijie On Thu, Oct 10, 2019 at 9:14 AM Ali Ahmed <alahmed...@gmail.com> wrote: > Yes that's what I am referring to. > > -Ali > > On Wed, Oct 9, 2019 at 11:50 PM Sijie Guo <guosi...@gmail.com> wrote: > > > > I recommend we ask apache infra to disable them and enable the github > > cicd > > integration. > > > > What is Github CI/CD? You mean Github Actions? > > > > - Sijie > > > > On Thu, Oct 10, 2019 at 6:13 AM Ali Ahmed <alahmed...@gmail.com> wrote: > > > > > I have added some support to apache jenkins jobs as an experiment to > skip > > > documentation changes and retry failed tests. > > > > > > They are operational but it seems on different runs it picks up > different > > > job configurations. > > > reasons unclear. > > > > > > Also the infra seems to be unstable now jobs get stuck and fail midway > > > during compilation. > > > > > > eg: > > > > > > [ 34%] Building CXX object > > > lib/CMakeFiles/pulsarShared.dir/Log4cxxLogger.cc.o > > > [ 34%] Building CXX object > lib/CMakeFiles/pulsarShared.dir/LogUtils.cc.o > > > [ 37%] Building CXX object lib/CMakeFiles/pulsarShared.dir/Message.cc.o > > > [ 37%] Building CXX object > > > lib/CMakeFiles/pulsarShared.dir/MessageBatch.cc.o > > > [ 39%] Building CXX object > > > lib/CMakeFiles/pulsarShared.dir/MessageBuilder.cc.o > > > [ 39%] Building CXX object > > > lib/CMakeFiles/pulsarShared.dir/MessageCrypto.cc.o > > > [ 41%] Building CXX object > lib/CMakeFiles/pulsarShared.dir/MessageId.cc.o > > > [ 41%] Building CXX object > > lib/CMakeFiles/pulsarShared.dir/MessageImpl.cc.o > > > [ 44%] Building CXX object > > > lib/CMakeFiles/pulsarShared.dir/MessageRouterBase.cc.o > > > > > > > > > I recommend we ask apache infra to disable them and enable the github > > cicd > > > integration. It's still in beta but there some limitations, but it > should > > > help unblock the backlog of pr's. > > > > > >
