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 <[email protected]>: > 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: [email protected] > > > Stig Rohde Døssing <[email protected]> 于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 < > > [email protected]>: > > > > > Good point, let's see if there's anyone with an ARM environment. > > > > > > Den tir. 11. jun. 2019 kl. 23.07 skrev Jungtaek Lim <[email protected] > >: > > > > > >> 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 < > > >> [email protected]> > > >> 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 > > >> > > > > > >
