From 8c37ac7f60f9af968112383e7241d72271fe5f6d Mon Sep 17 00:00:00 2001 From: Felipe Monteiro Date: Fri, 29 Jun 2018 18:30:16 -0400 Subject: [PATCH] trivial: Add orientation='reverse' to find_cycle in layering This patchset adds orientation='reverse' when looking for substitution cycles in the layering module in order to work with newer versions of networkx which throw: networkx.exception.NetworkXUnfeasible: Graph contains a cycle. This corrects the issue. Change-Id: I854a10ed524daf45bef9ad856938ffedb3e76baf --- deckhand/engine/layering.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deckhand/engine/layering.py b/deckhand/engine/layering.py index d89278cf..ab2c1fe9 100644 --- a/deckhand/engine/layering.py +++ b/deckhand/engine/layering.py @@ -337,7 +337,7 @@ class DocumentLayering(object): g.add_edge(document.meta, src.meta) try: - cycle = find_cycle(g) + cycle = find_cycle(g, orientation='reverse') except networkx.exception.NetworkXNoCycle: pass else: