hi Ravindra, Can we document this (just by copy-pasting what you wrote) on the wiki or someplace for future work that may touch the manylinux package builds? This might be a bit more discoverable than going through the email logs
Thanks! Wes On Fri, Mar 1, 2019 at 9:58 PM Ravindra Pindikura <ravin...@dremio.com> wrote: > > Thanks Uwe. > > For the record (in case someone needs to do it again), these are the steps : > > 1. Make the change in build_boost.sh > > 2. Setup an account on quay.io <http://quay.io/> and link to your GitHub > account > > 3. In quay.io <http://quay.io/>, Add a new repository using : > > A. Link to GitHub repository push > B. Trigger build on changes to a specific branch (eg. myquay) of the repo > (eq. pravindra/arrow) > C. Set Dockerfile location to "/python/manylinux1/Dockerfile-x86_64_base” > D. Set Context location to "/python/manylinux1” > > 4. Push change (in step 1) to the branch specified in step 3B > > This should trigger a build in quay.io <http://quay.io/>, the build takes > about 2 hrs to finish. > > 5. Add a tag “latest” to the build after step 4 finishes, save the URL of the > build (eg. quay.io/pravindra > <http://quay.io/pravindra/arrow_manylinux1_x86_64_base:latest>/arrow_manylinux1_x86_64_base:latest > <http://quay.io/pravindra/arrow_manylinux1_x86_64_base:latest>) > > 6. In your arrow PR, > > - include the change from 1. > - update travis_script_manylinux.sh to point to the location from step 5. > > Thanks & regards, > Ravindra. > > > On Feb 27, 2019, at 3:02 PM, Uwe L. Korn <uw...@xhochy.com> wrote: > > > > Hello Ravindra, > > > > simplest thing would be when you open a pull request and I can then pick > > this up and push it to my personal fork. Then a new image is built on > > quay.io. Otherwise, you can also activate quay.io on your fork to get the > > docker image to build. > > > > Uwe > > > > On Wed, Feb 27, 2019, at 8:41 AM, Krisztián Szűcs wrote: > >> Hi Ravindra! > >> > >> You'll need to rebuild the docker image and change this line accordingly: > >> https://github.com/apache/arrow/blob/master/ci/travis_script_manylinux.sh#L57 > >> > >> On Wed, Feb 27, 2019 at 8:29 AM Ravindra Pindikura <ravin...@dremio.com> > >> wrote: > >> > >>> Hi, > >>> > >>> I added an include for boost header file in gandiva. This compiles on > >>> ubuntu/Mac/windows, but fails with the manylinux CI entry. > >>> > >>> I’m getting a compilation failure : > >>> > >>> https://travis-ci.org/apache/arrow/jobs/498718755 < > >>> https://travis-ci.org/apache/arrow/jobs/498718755> > >>> /arrow/cpp/src/gandiva/decimal_xlarge.cc:29:44: fatal error: > >>> boost/multiprecision/cpp_int.hpp: No such file or directory > >>> #include "boost/multiprecision/cpp_int.hpp" > >>> ^ > >>> compilation terminated. > >>> > >>> > >>> > >>> @xhocy and @kszucs pointed out the manylinux1 image has a very minimal > >>> boost, and doesn’t include the multi precision files. So, the script that > >>> builds boost for manylinux1 needs to be updated for this. > >>> > >>> > >>> > >>> https://github.com/apache/arrow/blob/master/python/manylinux1/scripts/build_boost.sh#L38 > >>> < > >>> https://github.com/apache/arrow/blob/master/python/manylinux1/scripts/build_boost.sh#L38 > >>>> > >>> > >>> After making change, the manylinux1 build still fails with the same error > >>> :(. > >>> > >>> https://travis-ci.org/apache/arrow/jobs/498847622 > >>> > >>> Looks like the CI run downloads a prebuilt docker image. Do I need to > >>> update the docker image ? If yes, can you please point out the > >>> instructions > >>> for this ? > >>> > >>> Thanks & regards, > >>> Ravindra. > >> >