Closed tdejager closed 9 months ago
@baszalmstra can you update with any progress made in resolvo
:)?
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,
cargo run openmetadata-managed-apis
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 cargo run apache-airflow
, I see rip
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
@baszalmstra
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 😄
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.