Bug 1394281

Summary: Network CRUD actions do not use task queue
Product: Red Hat CloudForms Management Engine Reporter: Tzu-Mainn Chen <tzumainn>
Component: ProvidersAssignee: Gilles Dubreuil <gdubreui>
Status: CLOSED CURRENTRELEASE QA Contact: Omri Hochman <ohochman>
Severity: high Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, dajohnso, gdubreui, jfrey, jhardy, obarenbo, simaishi
Target Milestone: GAKeywords: 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:
: 1405641 (view as bug list) Environment:
Last Closed: 2017-06-12 17:16:31 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:
Bug Depends On:    
Bug Blocks: 1405641    

Description Tzu-Mainn Chen 2016-11-11 14:55:26 UTC
Network CRUD actions do not use task queue; they should to enable task tracking and consistent logs.

Comment 2 Gilles Dubreuil 2016-11-17 09:25:58 UTC
Addressed with https://github.com/ManageIQ/manageiq/pull/12071

Comment 4 Gilles Dubreuil 2016-12-06 23:11:18 UTC
Update: Master's Pull Request has been merged.
The 5.7.1 (euwe branch) backport to come next.

Comment 8 CFME Bot 2017-01-13 13:10:51 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/0b86cdd06cbb6d621af697f38e06a7ce0156c850

commit 0b86cdd06cbb6d621af697f38e06a7ce0156c850
Author:     Martin Povolny <mpovolny>
AuthorDate: Tue Dec 6 14:44:05 2016 +0100
Commit:     Tzu-Mainn Chen <tzumainn>
CommitDate: Thu Jan 12 03:55:56 2017 +0100

    [EUWE] Cloud Network UI: Task queue, validation, buttons and translation
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1394281
    
    This is an Euwe-only PR to resolve conflicts with:
    
    	app/controllers/cloud_network_controller.rb
    	app/models/manageiq/providers/openstack/network_manager.rb
    
    This PR is equivalent to https://github.com/ManageIQ/manageiq/pull/12071
    on master.

 app/controllers/cloud_network_controller.rb        | 197 ++++++++++-----------
 .../toolbar/cloud_networks_center.rb               |  36 ++--
 app/models/cloud_network.rb                        |  42 +----
 app/models/cloud_network/operations.rb             |  49 -----
 .../providers/openstack/network_manager.rb         |  22 +++
 .../openstack/network_manager/cloud_network.rb     |  96 ++++++----
 config/routes.rb                                   |   1 +
 spec/controllers/cloud_network_controller_spec.rb  | 114 ++++++++----
 ...shared_examples_for_cloud_network_controller.rb |   2 -
 9 files changed, 281 insertions(+), 278 deletions(-)
 delete mode 100644 app/models/cloud_network/operations.rb

Comment 9 Ronnie Rasouli 2017-03-08 07:17:18 UTC
verified on 5.8.0.3