Bug 1394284

Summary: [RFE] Routers do not allow you to add/remove interfaces
Product: Red Hat CloudForms Management Engine Reporter: Tzu-Mainn Chen <tzumainn>
Component: ProvidersAssignee: Tzu-Mainn Chen <tzumainn>
Status: CLOSED CURRENTRELEASE QA Contact: Ola Pavlenko <opavlenk>
Severity: high Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, dajohnso, iovadia, jfrey, jhardy, obarenbo
Target Milestone: GAKeywords: FutureFeature, TestOnly, ZStream
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: openstack:sdn
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1413212 (view as bug list) Environment:
Last Closed: 2017-06-12 17:32:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Openstack Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1398911, 1413212    

Description Tzu-Mainn Chen 2016-11-11 14:58:21 UTC
Adding/removing interfaces are a major part of router management.

Comment 2 Tzu-Mainn Chen 2016-11-28 01:25:35 UTC
*** Bug 1398915 has been marked as a duplicate of this bug. ***

Comment 3 CFME Bot 2016-12-06 17:25:44 UTC
New commit detected on ManageIQ/manageiq-gems-pending/master:
https://github.com/ManageIQ/manageiq-gems-pending/commit/509f96dbfe15aed2ba0f31540acb352afb5ba74f

commit 509f96dbfe15aed2ba0f31540acb352afb5ba74f
Author:     Scott Seago <sseago>
AuthorDate: Thu Nov 17 12:08:56 2016 -0500
Commit:     Scott Seago <sseago>
CommitDate: Fri Nov 18 12:03:49 2016 -0500

    Added exception clases for router add/remove interfaces
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1394284

 lib/gems/pending/util/miq-exception.rb | 2 ++
 1 file changed, 2 insertions(+)

Comment 7 CFME Bot 2016-12-22 18:46:01 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/4db629b0a7f144b32ad4e59811797b3301619f49

commit 4db629b0a7f144b32ad4e59811797b3301619f49
Author:     Scott Seago <sseago>
AuthorDate: Tue Dec 6 23:26:49 2016 -0500
Commit:     Scott Seago <sseago>
CommitDate: Wed Dec 7 00:07:23 2016 -0500

    model for add/remove interface on network router
    
    This commit adds model code for adding and removing router interfaces.
    
    BZ for this is:
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1394284
    
    Euwe backport depends on merging https://github.com/ManageIQ/manageiq/pull/13005

 .../openstack/network_manager/network_router.rb    | 63 +++++++++++++++++++++-
 app/models/mixins/supports_feature_mixin.rb        |  2 +
 app/models/network_router.rb                       |  1 +
 config/settings.yml                                |  2 +
 .../OpenStack.class/router.interface.create.yaml   | 12 +++++
 .../OpenStack.class/router.interface.delete.yaml   | 12 +++++
 db/fixtures/miq_product_features.yml               |  8 +++
 ...ers-openstack-network_manager-network_router.rb |  4 ++
 .../miq_ae_service_network_router.rb               |  3 --
 ...penstack-network_manager-network_router_spec.rb | 19 +++++++
 .../miq_ae_service_network_router_spec.rb          |  8 ---
 11 files changed, 122 insertions(+), 12 deletions(-)
 create mode 100644 db/fixtures/ae_datastore/ManageIQ/System/Event/EmsEvent/OpenStack.class/router.interface.create.yaml
 create mode 100644 db/fixtures/ae_datastore/ManageIQ/System/Event/EmsEvent/OpenStack.class/router.interface.delete.yaml
 create mode 100644 spec/lib/miq_automation_engine/service_methods/miq_ae_service_manageiq-providers-openstack-network_manager-network_router_spec.rb

Comment 9 CFME Bot 2017-01-11 01:00:52 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/8a3e81ec49df70ebf71cb7495420fd2c912b7aad

commit 8a3e81ec49df70ebf71cb7495420fd2c912b7aad
Author:     Scott Seago <sseago>
AuthorDate: Mon Dec 5 19:34:37 2016 -0500
Commit:     Scott Seago <sseago>
CommitDate: Mon Dec 5 19:34:37 2016 -0500

    Added exception clases for router add/remove interfaces for euwe
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1394284
    
    This is a euwe-only commit as this class is moved to the
    manageiq-gems-pending repository on master

 gems/pending/util/miq-exception.rb | 2 ++
 1 file changed, 2 insertions(+)

Comment 10 CFME Bot 2017-01-13 08:22:41 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/b91242eb8fd268bab9ed89d7330506084dbb66ea

commit b91242eb8fd268bab9ed89d7330506084dbb66ea
Author:     Scott Seago <sseago>
AuthorDate: Tue Jan 3 22:28:58 2017 -0500
Commit:     Scott Seago <sseago>
CommitDate: Wed Jan 11 10:07:28 2017 -0500

    UI for add/remove interface on network router
    
    This commit adds UI code for adding and removing router interfaces.
    
    BZ for this is:
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1394284
    
    Euwe backport depends on merging https://github.com/ManageIQ/manageiq/pull/13005

 .../network_router_form_controller.js              |  19 +-
 app/controllers/network_router_controller.rb       | 216 +++++++++++++++++++--
 .../toolbar/network_router_center.rb               |  18 ++
 .../toolbar/network_routers_center.rb              |  16 ++
 .../network_router/add_interface_select.html.haml  |  23 +++
 .../remove_interface_select.html.haml              |  23 +++
 config/routes.rb                                   |   6 +
 spec/controllers/network_router_controller_spec.rb |  92 +++++++++
 8 files changed, 401 insertions(+), 12 deletions(-)
 create mode 100644 app/views/network_router/add_interface_select.html.haml
 create mode 100644 app/views/network_router/remove_interface_select.html.haml

Comment 12 CFME Bot 2017-02-08 18:33:33 UTC
New commit detected on ManageIQ/manageiq-content/master:
https://github.com/ManageIQ/manageiq-content/commit/e8132aa66bd510e3ba535c7bd3ee51b242e3972b

commit e8132aa66bd510e3ba535c7bd3ee51b242e3972b
Author:     Scott Seago <sseago>
AuthorDate: Tue Dec 6 23:26:49 2016 -0500
Commit:     Brandon Dunne <bdunne>
CommitDate: Wed Feb 8 11:28:21 2017 -0500

    model for add/remove interface on network router
    
    This commit adds model code for adding and removing router interfaces.
    
    BZ for this is:
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1394284
    
    Euwe backport depends on merging https://github.com/ManageIQ/manageiq/pull/13005

 .../EmsEvent/OpenStack.class/router.interface.create.yaml    | 12 ++++++++++++
 .../EmsEvent/OpenStack.class/router.interface.delete.yaml    | 12 ++++++++++++
 2 files changed, 24 insertions(+)
 create mode 100644 content/automate/ManageIQ/System/Event/EmsEvent/OpenStack.class/router.interface.create.yaml
 create mode 100644 content/automate/ManageIQ/System/Event/EmsEvent/OpenStack.class/router.interface.delete.yaml

Comment 13 Ronnie Rasouli 2017-03-08 07:19:58 UTC
verified on 5.8.0.3