network topology order

[Kevin Fox]

Often when three or more networks are involved, with one external, one internal attached to a router which is attached to external (aka tenant public), and one or more backend networks the view often (always?) displays in the following order:

external, private backend*, tenant public

This causes the display to look very backwards to what one might expect and lines to cross that would not normally need to.

A more intuative ordering would be:
external networks, tenant networks attached to routers attached to external networks, tenant networks with no routers.

Thanks,
Kevin

[Kevin Fox (kevin-fox-y)]

I have a patch to fix this now, but I'm trying to get it through legal. The openstack contributor agreement is proving to be a challenge.

[Kevin Fox (kevin-fox-y)]

Ok, I got through licensing. The patch listed here is under the public domain.
https://github.com/EMSL-MSC/horizon/commit/34a3218c02267a68bc34a57505e8751474797821

I'm working with the OpenStack/PNNL lawyers to get the CLA figured out, but in the mean time, we can start working on cleaning up the patch.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/79778

[Facundo Farias (facundo-farias)]

The proposed patch doesn't seem to be working. I will be trying to solve this asap.

[Facundo Farias (facundo-farias)]

Done! It's updated on the review. Thanks

[Kevin Fox (kevin-fox-y)]

ok, I just got the ok from PNNL legal so I can contribute the patch under the Individual CLA. We should be good to go now.

Thanks,
Kevin

[Akihiro Motoki (amotoki)]

The patch is under review and i think it is worth targeted to Juno-3.

[OpenStack Infra (hudson-openstack)]

Change abandoned by David Lyle (<email address hidden>) on branch: master
Review: https://review.openstack.org/79778
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/213647

[OpenStack Infra (hudson-openstack)]

Change abandoned by Aleš Křivák (<email address hidden>) on branch: master
Review: https://review.openstack.org/213647
Reason: Network topology was rewritten in liberty, this change is no longer required.

[Aleš Křivák (aleskrivak)]

This bug is no longer relevant as network topology was completly rewritten in liberty (see blueprint curvature-network-topology).