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
apache-airflow resolution #75
Comments
@baszalmstra can you update with any progress made in |
I spent half a day debugging the issue. It looks like at some point something is not properly propagated but I cant figure out what causes it. I'll have to get back to it later. |
I think we should:
I also looked into it and could not yet figure out the root cause. |
Not sure if this helps,
which has dependency of apache-airflow>=2.2 works and installs apache-airflow version 2.6.3 which is valid since all the newer versions need opentelemetry-api and opentelemetry-exporter-otlp which cannot be installed as one of their dependency (opentelemetry-semantic-conventions) has only pre-release versions available From the logs on the
This should ideally backtracks to next version of 2.7.2 and so on. Wish I could dig more on this, still at early days of Rust and understanding SAT solvers. |
I added a minimal integration test on resolvo which replicates this, https://github.com/sumanth-manchala/resolvo/blob/airflow-int-test/tests/solver.rs#L462 |
Hi @sumanth-manchala this is amazing. I also tried to make a reproducer the other day, but didn't manage so I am really happy to see that you managed to do it! Did you – perhaps – also work on a fix? This issue is certainly one that we would like to fix sooner rather than later :) |
PS: do you want to make a PR with your changes? We could mark it as "expected fail" for now, too. Would love to give you credit for the test! |
Thanks @wolfv. Have added the integration test to expect panic, will ping here if I could figure out the fix |
Yeah that's absolutely great work, @sumanth-manchala :) Jumping head first into such a gnarly problem couldn't have been easy! Thanks a lot 😄 |
Relates to prefix-dev/rip#75 --------- Co-authored-by: Bas Zalmstra <bas@prefix.dev>
Just in case anyone out there is trying to debug this further, I'm pretty sure I've found the root cause of the issue and hope to have a fix soon. |
When running
we get the following error message.
Some impossible statement is being hit, is being hit inside of resolvo.
The text was updated successfully, but these errors were encountered: