Uh oh!
There was an error while loading. Please reload this page.
Add changes to build arm64 wheels - #1108
Conversation
075b21e to
3eadf1aCompareUpdate setup tools to fix build error in `python setup.py egg_info` Update greenlet version to fix build error in `tox -e gevent_loop`
tbbharaj
commented
Jul 13, 2021
Following up on this PR addressing the issue to build arm64 wheels. Thank you! |
jdonenine
commented
Jul 16, 2021
Hi @tbbharaj thanks for the PR! We'll do our best to try to get someone to review the changes. Before we get going there ... Have you signed our CLA? https://cla.datastax.com/ -- we'll need you to do that before we can merge the PR. Thanks! |
tbbharaj
commented
Jul 16, 2021
Thank you Jeff, I haven't signed this CLA yet as I wasn't aware of it. I will work on getting approvals to get this CLA signed for merging this PR. I hope we can review the PR in the meantime. |
tbbharaj
commented
Aug 10, 2021
@jdonenine Following up to check if we can have PR to get reviewed as we are working to get CLA approved. Thanks! |
absurdfarce
commented
Aug 11, 2021
@tbbharaj First off, thanks for the submission (and apologies for not updating this PR sooner)! I am in the process of reviewing this PR now. We have a backend system which actually builds the wheels for packages that we push to pypi so there may be some changes we need to make here to account for that process. I'm having trouble convincing that backend process to correctly build ARM wheels, however, so until I get that sorted out I won't know for sure what changes we need to make on this PR. I'm working with the folks at Travis to try to resolve the issue and will try to do a better job of updating this PR as that process moves along. |
tbbharaj
commented
Aug 12, 2021
No issues, Thank you for the update @absurdfarce. I will wait for next steps! |
absurdfarce
commented
Aug 25, 2021
Just updated https://datastax-oss.atlassian.net/browse/PYTHON-1278 with info about recent work on this effort. As mentioned above this ticket will be reviewed once we complete the work described there. |
absurdfarce
commented
Sep 1, 2021
@tbbharaj A quick update; I'm still waiting on a few internal infrastructure pieces to get the wheel building process back on track. In the meantime would you mind pulling your two build fixes (the upgrades to setuptools and greenlet) out of this PR and into their own PR? I'd like to get those changes into the Travis build as soon as possible so that every PR going forward will start from a baseline of a good, working build. Thanks! |
These fixes were originally implemented by user tbbharaj in #1108. Extracting them into their own PR since 1108 is still being worked and I'd very much like to benefit from this work across _all_ PRs against python-driver. Major thanks to tbbharaj for the original work here.
These fixes were originally implemented by user tbbharaj in #1108. Extracting them into their own PR since 1108 is still being worked and I'd very much like to benefit from this work across _all_ PRs against python-driver. Major thanks to tbbharaj for the original work here.
absurdfarce
commented
Sep 15, 2021
absurdfarce
commented
Sep 17, 2021
@tbbharaj I just posted a comment to the JIRA ticket mentioning that the we now have a working public repository for wheel builds that includes the ARM platforms you wanted to add via this PR. As mentioned above I've also pulled out your fixes for the tests contained in the PR above and applied those separately. Thanks again for those; it's great to have a starting point of green test runs for future PRs. Given all of that, I think I'm going to decline this PR itself. My rationale is that this PR moves wheel builds into the Travis runs for individual PRs which isn't really what we want to do... I'd rather keep those builds focused on running tests. More importantly this process has now been superseded by the public repository for Python wheels mentioned above. Many, many thanks for pushing this process forward overall; I think we've landed in a better spot when it comes to our support for Python wheels (and the visibility and transparency of that support) and that's largely thanks to you kicking it off! |
tbbharaj
commented
Sep 22, 2021
@absurdfarce Thank you so much... apologies for the delay on my end but it's great seeing this progress. |
absurdfarce
commented
Sep 29, 2021
@tbbharaj Apologies, I've been working through a few other unrelated issues so I'm running a bit behind here. It certainly is my intent to deploy the ARM wheels to pypi.org in the near future. I'll be sure to post an update here once that's been done. Thanks! |
tbbharaj
commented
Sep 29, 2021
Thanks for the update @absurdfarce |
absurdfarce
commented
Oct 11, 2021
@tbbharaj Apologies for the delay, but I come bringing good news. I just deployed the ARM wheels for cassandra-driver to Pypi! I pushed wheels for Python 3.7 and 3.8; they should be visible at https://pypi.org/project/cassandra-driver/#files. My guess is that you have a testbed for ARM installs that's far more accessible than what I have access to at the moment so are you able to confirm that (a) Python is correctly downloading these wheels when using cassandra-driver on an ARM platform and (b) the package seems to function normally when you do so? Thanks again for your patience while we work through these changes! |
tbbharaj
commented
Oct 12, 2021
@absurdfarce Thats great news! Thank you so much...I am closing this PR for now then :) |
This PR addresses the issue to build arm64 wheels. To address this issue, I updated
travis.ymlto includearm64.Also there were two additional errors (Master Build Error) I encountered when I triggered master build w/o any changes.
Ref: https://travis-ci.com/github/tbbharaj/python-driver/jobs/515470491
Ref: https://travis-ci.com/github/tbbharaj/python-driver/jobs/515470490
To fix above two errors, I made following changes:
travis.ymlto fix build error inpython setup.py egg_info,tox.inito fix build error intox -e gevent_loopSuccessful builds Build
If there are any questions/comments - I'd be happy fix things as needed.
I opened the following JIRA issue as well Issue
Thank you
Tanveen