-
Notifications
You must be signed in to change notification settings - Fork 413
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Azure AD Auth fails on ARM64 #2475
Comments
@george-zubrienko are you able to test the connection from non-arm environments? |
@ion-elgreco yes if I change machine type to |
I'm building a new image rn with 0.17.3, fyi this was the wheel used by ARM64 build: Same result with 0.17.3. I assume the error is coming from Rust code, not python as it is reported as OSError |
Could you do a bisect, to see which release this problem started occurring for you? |
We just upgraded from 0.8.1 to 0.17.* and I got the error. I can try to narrow it down a little. In case this adds anything, we are loading data like this: https://github.com/SneaksAndData/adapta/blob/main/adapta/storage/delta_lake/_functions.py#L39-L90 |
@ion-elgreco it works up to 0.16.1, 0.16.2 breaks it |
@george-zubrienko hmm strange, nothing implies there is a change that could have caused this in that release. Maybe its the rust version which it gets compiled with, are you able to compile with some older version and check that |
Arent 0.16.2 and 0.16.1 compiled with the same version? I can try to force this package to compile from source rather than use the wheel |
So, using this compiler version:
Running this:
Everything works. So it is a problem with the wheel? Kernel: |
@george-zubrienko ok that's helpful! I think it's because of the rust compiler.
You seem to get it working when it's compiled with 1.78.0 |
That is the case! Any chance we can get compiler bumped for 0.17.4? 🙏 |
Probably next release. Rust 1.78 got released today and our release just missed that version |
I'm also facing same issue with aws sts, using latest deltalake version |
@ion-elgreco tag me here once you release the latest version, right now we are reverting back to amd |
I would be surprised if this was the compiler. It is more likely that a dependency for which we have a loose version range specifier. Either way, I figure the next release should take care of this for ya 😄 |
@rtyler we are also facing same with aws as well |
@george-zubrienko can you check against v0.18.0 please? |
Will check on Monday! |
Bit delayed - deploying this as of now, in case I don't get to actually run the validation, will ping tomorrow |
@ion-elgreco I just tried with deltalake 0.18.0 on ARM machine and I'm getting the same error (library installed from wheel, I can try to build from source tomorrow if you want to confirm it still works in that case) |
@george-zubrienko you might give 0.18.1 a shot which includes latest object store version, not sure if it's going to have any impact though |
Will try, but... isn't this strange a bit? I've checked the Mac issue as well and I see people also resolve it with |
Yeah the Mac issue got fixed by just bumping the os version of the runners |
@ion-elgreco we just tested with 0.18.1 for both Azure and AWS, the error is gone |
I believe this can be closed as we rolled 0.18.1 on prod with mostly ARM machines and I do not see any failures :) |
@george-zubrienko thanks for the update! |
Environment
Amazon Linux 2023, ARM64 arch host, container based on python3.11-slim-bookworm
Delta-rs version:
0.17.2
Binding:
python
Environment:
Bug
What happened:
When trying to load the table using AZURE_CLIENT_ID etc credentials, getting this:
Stack trace:
What you expected to happen:
Table reads as before on 0.8.1 version on the same host/container
How to reproduce it:
Run table read on ARM64/AL2023 vm with Azure auth against az://... table path
More details:
N/A
The text was updated successfully, but these errors were encountered: