Yeah, I was discussing that with Ash and just posted the same message :-D On Mon, Jun 24, 2019 at 1:35 PM Jarek Potiuk <jarek.pot...@polidea.com> wrote:
> OK. After some checks it turns out it's not only Sphinx really but also > sphinx-autoapi (v 1.1.0 released yesterday). > > J. > > > On Mon, Jun 24, 2019 at 9:29 AM Jarek Potiuk <jarek.pot...@polidea.com> > wrote: > > > Kamil prepared a fix https://github.com/apache/airflow/pull/5468 > (thanks!) and > > when I tested it on a clean pip-install with latest sphinx it seems to > work > > fine (Running Travis CI build > > <https://travis-ci.org/apache/airflow/builds/549600100>now). Let's see > if > > it helps. > > > > BTW. The AIP-10 Docker image last step (pending Kubernetes fix) should > > solve most of the dependency issues, however I think this particular > > problem could be missed (it's an interesting one). > > > > I will soon start another thread (after dockerfile merging) about finally > > looking into making dependencies more manageable and less transient-deps > > problems. I already have some ideas how we can approach the problem that > > Airflow is both: library (no-dependency pinning recommended) and > > end-product (strong pinning recommended). > > > > J. > > > > On Mon, Jun 24, 2019 at 8:48 AM Ash Berlin-Taylor <a...@apache.org> > wrote: > > > >> Any cyclic imports we have aren't actually a problem and are handled > fine > >> by python 3 (obviously, else it wouldn't work) so I think this is a big > in > >> sphinx with it being over zealous. > >> > >> This only affects tests and development, not our users in this case. > >> > >> If it's difficult to fix I would go with pinning - I want to get a > 1.10.4 > >> beta 1 out today. > >> > >> -ash > >> > >> On 24 June 2019 00:37:57 BST, Jarek Potiuk <jarek.pot...@polidea.com> > >> wrote: > >>> > >>> Hello Everyone, > >>> > >>> We have yet another case where relesing a package (Sphinx 2.1.2 in this > >>> case) - started to cause master to fail. > >>> > >>> Should we pin Sphinx to an earlier version or maybe someone can help > and > >>> figure out and fix it quickly (seems that there are some cyclic imports > >>> that probably need solving anyway if Sphinx is right about them) ? > @Kamil > >>> BreguĊa <kamil.breg...@polidea.com> @Fokko Driesprong > <fo...@driesprong.frl> > >>> ? WDYT ? > >>> > >>> I created an issue for that > >>> https://issues.apache.org/jira/browse/AIRFLOW-4841 > >>> > >>> J. > >>> > >>> > > > > -- > > > > Jarek Potiuk > > Polidea <https://www.polidea.com/> | Principal Software Engineer > > > > M: +48 660 796 129 <+48660796129> > > [image: Polidea] <https://www.polidea.com/> > > > > > > -- > > Jarek Potiuk > Polidea <https://www.polidea.com/> | Principal Software Engineer > > M: +48 660 796 129 <+48660796129> > [image: Polidea] <https://www.polidea.com/> > -- *Kaxil Naik* *Big Data Consultant | DevOps Data Engineer* *Certified *Google Cloud Data Engineer | *Certified* Apache Spark & Neo4j Developer *LinkedIn*: https://www.linkedin.com/in/kaxil