Thanks for volunteering.
Sent from Yahoo Mail for iPhone On Wednesday, June 26, 2019, 7:08 PM, Yikun Jiang <yikunk...@gmail.com> wrote: Yes, we will definitely help to fix ARM test failures in PRs, and the OpenLab CI will tell us which PR perhaps has ARM compatible problem, if it's an easy fix problem, I think authors can fix it by themselves. If not, we will help them to address it. For the lack of ARM environment, if the developer are interested in testing and debugging their storm patch in ARM environment but don't have ARM environment , we can also provide tmp ARM environment to them for testing. Regards, Yikun ---------------------------------------- Jiang Yikun（Kero） Mail: yikunk...@gmail.com Stig Rohde Døssing <stigdoess...@gmail.com> 于2019年6月26日周三 上午1:48写道： > It sounds pretty low risk for us, if you're volunteering to help fix any > ARM-specific CI failures that may crop up. Will this include helping to fix > ARM test failures in PRs? Most people are unlikely to have an ARM > environment they can use to test, and I'd prefer not to ask contributors to > fix ARM test failures themselves. > > Den tir. 25. jun. 2019 kl. 11.08 skrev Yikun Jiang <yikunk...@gmail.com>: > > > Sorry for late reply, actually, we have a developer team that willing to > > work on this, and I'm the owner of storm ARM CI in the OpenLab team. We > not > > only want to enable the OpenLab CI, but also want to maintain the arm CI > > job and fix the CI issue in Storm project. That means if the storm > project > > has some ARM compatible problem, I will very happy to fix it. > > > > As I mentioned before, one of the OpenLab goal is to make more open > source > > software to be more compatible for aarch64 platform. And the Storm > project > > is one of the most important one in BigData area, so we would like to > > propose to work on aarch64 related works in Storm. we plan to start the > > aarch64 related work from to add aarch64 build job for Storm, and our > > initial plan as below: > > > > 1. We first propose enable a periodical job in OpenLab project to make > sure > > the storm can compile and build successfully in arm64 env. > > 2. We hope can run the OpenLab CI in every pr to ensure that any pr after > > Step 1 will not break the arm64 build. > > 3. We can add more complex test cases on aarch64, like unit tests and > > functional tests, step by step. It's a log term works. > > > > Of course, welcome another developers join to maintain the aarch64 CI, > and > > take effort on aarch64 work together. > > > > Thanks for your attention. > > > > Regards, > > Yikun > > ---------------------------------------- > > Jiang Yikun（Kero） > > Mail: yikunk...@gmail.com > > > > > > Stig Rohde Døssing <stigdoess...@gmail.com> 于2019年6月21日周五 上午2:23写道： > > > > > I guess there is no interest in maintaining ARM compatibility, or at > > least > > > no one currently wants to take on the effort. Let's not add the Openlab > > CI > > > then, we can always do it later if someone expresses interest (and > > > willingness to maintain). > > > > > > Den ons. 12. jun. 2019 kl. 17.11 skrev Stig Rohde Døssing < > > > stigdoess...@gmail.com>: > > > > > > > Good point, let's see if there's anyone with an ARM environment. > > > > > > > > Den tir. 11. jun. 2019 kl. 23.07 skrev Jungtaek Lim < > kabh...@gmail.com > > >: > > > > > > > >> I guess the point is not related to open CI build. The point is > > whether > > > we > > > >> really want to support ARM. I'm seeing OpenLab request for other > > Apache > > > >> projects as well, so I'd rather not treat their request as > commitment > > of > > > >> putting efforts to make builds on ARM green. > > > >> > > > >> I'm a bit hesitant to add some environment on the support list > unless > > > >> there're enough engineers willing to work on maintaining. If the > work > > is > > > >> left to existing maintainers, we will be stuck on failing tests on > ARM > > > CI > > > >> but no environment to play locally. > > > >> > > > >> So I'd be +1 if there're at least two folks experienced with ARM > env. > > > >> volunteer to maintain the compatibility part. Otherwise -1 here. > > > >> > > > >> Thanks, > > > >> Jungtaek Lim (HeartSaVioR) > > > >> > > > >> On Mon, Jun 10, 2019 at 7:27 PM Stig Rohde Døssing < > > > >> stigdoess...@gmail.com> > > > >> wrote: > > > >> > > > >> > Hi, > > > >> > > > > >> > An issue was opened suggesting adding ARM CI to the Storm build, > > using > > > >> > OpenLab https://issues.apache.org/jira/browse/STORM-3401, > > associated > > > PR > > > >> > here https://github.com/apache/storm/pull/3023 > > > >> > > > > >> > As far as I can tell, we need to allow OpenLab access to the > Github > > > PRs, > > > >> > similar to how Travis has permission to access our PRs. The app > that > > > >> needs > > > >> > access is https://github.com/apps/theopenlab-ci. Note that the > app > > > will > > > >> > have access to the PRs only, not the repo code. > > > >> > > > > >> > Is anyone opposed to me asking infra to allow OpenLab access to > our > > > PRs? > > > >> > I'll let this thread sit for a week or so, if there is no > opposition > > > >> I'll > > > >> > ask infra to give the app access. > > > >> > > > > >> > > > >> > > > >> -- > > > >> Name : Jungtaek Lim > > > >> Blog : http://medium.com/@heartsavior > > > >> Twitter : http://twitter.com/heartsavior > > > >> LinkedIn : http://www.linkedin.com/in/heartsavior > > > >> > > > > > > > > > >
