att-comdev / armada

This project has moved to OpenStack.
https://www.airshipit.org/
Apache License 2.0
78 stars 35 forks source link

Manifest dependency cycle does not generate a clear error. #178

Open jezogwza opened 6 years ago

jezogwza commented 6 years ago

Is this a bug report or feature request? (choose one): BUG

Python Version (output of python --version):

Development or Deployment Environment?:

Release Tag or Master:

Expected Behavior:

What Actually Happened:

How to Reproduce the Issue (as minimally as possible):

Any Additional Comments:

gardlt commented 6 years ago

could you provide the error you are getting?

mark-burnett commented 6 years ago

This can be easily reproduced by having a chart definition depend on itself. If you don't want to implement your own cycle detection, you may want to consider using https://networkx.github.io/ for that.

This is exposed by buggy configuration referred to in att-comdev/promenade#47