dashboard requires L3 extension to be enabled else throws an error while launching instance

Neutron without L3 extension is a possible case and it is better to take into account it.

I'm also facing this problem, related here:

http://www.marshut.com/itsqiz/help-with-ml2-flat-network-errors-on-logs-while-trying-it-instance-not-created.html

Cheers!
Thiago

I am working on this but it is not in time for icehouse initial release.
If the fix lands by icehouse stable update, we would like to consider the backport.

After Neutron ML2 plugin is shipped, we sometimes receive this report.
It is because ML2 plugin can be used without L3 feature.

Horizon Neutron API interface already proposes a mechanism to cache and check if an extension is supported by the Neutron API [1].
I think we should add that test for the L3 extension.
And I also found another extension not tested: agent extension. That extension is used in the admin dashboard to list Neutron agents [2]. If that extension is not supported, Horizon raised a 404 error.

I think we need to check if a Neutron API request is dependent to an extension.
Perhaps, add a decorator to add that check?

[1] https://github.com/openstack/horizon/blob/master/openstack_dashboard/api/neutron.py#L831
[2] https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/info/tables.py#L128

L3 extension check is a bit tricky compared to other neutron extensions because floating IP support also depends on L3 extension.
I am working on this bug as a blueprint hide-router-panel-by-config.

Another reason is that Horizon does not have a way to control enabling/disabling panel/dashboard based on extension list from neutron. Dynamic control mechanism is being implemented as a part of blueprint separate-identity-dash and the mechanism can be used to control Neutron related panels.

> And I also found another extension not tested: agent extension. That extension is used in the admin dashboard to list Neutron agents [2]. If that extension is not supported, Horizon raised a 404 error.

Regarding agent extension, we already check agent extension is enabled based on extension list from Neutron.
Please check NetworkAgentsTab.allowed() and if you have still an issue please file it as a separate bug.

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

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

Reviewed: https://review.openstack.org/113336
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=11001691308b633abf6cac378ae926677ad0d903
Submitter: Jenkins
Branch: master

commit 11001691308b633abf6cac378ae926677ad0d903
Author: Akihiro Motoki <email address hidden>
Date: Sat Aug 9 09:45:56 2014 +0900

    Add a config option to disable Router panel

    Neutron setup allows operators not to enable Router feautre
    (by not using Router service plugin in neutron.conf) and
    some Neutron plugin does not support Router feature.
    According to bug 1350639 there is also a case where operators
    want to disable Router panel from their policy.
    It seems nice we have a option to control Router panel.

    This commit add a new option 'enable_router' to OPENSTACK_NEUTRON_NETWORK.
    The default is True since router feature is enabled in most deployments
    and it is the current default behavior of Horizon.
    If this option is False, Router panel will disappear.

    blueprint hide-router-panel-by-config
    Partial-Bug: #1292022
    Closes-Bug: #1350639
    Change-Id: If90517bd35c5b18173c54c14a1ba229f97ba4797

Reviewed: https://review.openstack.org/114088
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=936fc59aa8c9b1a45aaefc9779741c84ed3fea90
Submitter: Jenkins
Branch: master

commit 936fc59aa8c9b1a45aaefc9779741c84ed3fea90
Author: Akihiro Motoki <email address hidden>
Date: Sat Aug 9 15:08:40 2014 +0900

    Disable Floating IP features if Neutron router is disabled

    If the config option 'enable_router' is set to False,
    Floating IP features are disabled when Neutron is enabled.
    It does not affect when Neutron is disabled.

    It also adds unit tests for api.network.servers_update_addresses
    which is affected by this change.

    Completes blueprint hide-router-panel-by-config
    Closes-Bug: #1292022
    Change-Id: Ib63c6a0e7bb5661d4a60d10a1722fdad978b50bb

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/189459

Change abandoned by Tracy Jones (<email address hidden>) on branch: stable/icehouse
Review: https://review.openstack.org/189459
Reason: don't need it