Horizon: DC: managing/unmanaging a subcloud fails

Bug #1888301 reported by Kristine Bujold
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Medium
Kristine Bujold

Bug Description

Brief Description
-----------------
In Horizon in the System Controller Cloud Overview clicking on Manage Subcloud or Unmanage Subcloud fails with error "'NoneType' object is not iterable"

It appears to be caused by API changes in distcloud-client that were done with this commit https://opendev.org/starlingx/distcloud-client/commit/dd05eccb0826ad4047201d361abea20b7a788bf5

Severity
--------
Major

Steps to Reproduce
------------------
See Description

Expected Behavior
-----------------
Horizon should not report an error

Actual Behavior
----------------
Horizon reports an error

Reproducibility
---------------
reproducible

System Configuration
--------------------
DC vbox lab with one subcloud

Branch/Pull Time/Commit
-----------------------
master

Last Pass
---------
Unknown

Timestamp/Logs
--------------
2020-07-20 10:33:01,597 [ERROR] openstack_dashboard.api.rest.utils: error invoking apiclient
Traceback (most recent call last):
  File "/usr/share/openstack-dashboard/openstack_dashboard/api/rest/utils.py", line 128, in _wrapped
    data = function(self, request, *args, **kw)
  File "/usr/lib/python2.7/site-packages/starlingx_dashboard/api/rest/dc_manager.py", line 39, in patch
    dc_manager.subcloud_update(request, subcloud_id, request.DATA)
  File "/usr/lib/python2.7/site-packages/starlingx_dashboard/api/dc_manager.py", line 71, in subcloud_update
    subcloud_id, **changes.get('updated'))
  File "/usr/lib/python2.7/site-packages/dcmanagerclient/api/v1/subcloud_manager.py", line 220, in update_subcloud
    return self.subcloud_update(url, files, data)
  File "/usr/lib/python2.7/site-packages/dcmanagerclient/api/v1/subcloud_manager.py", line 105, in subcloud_update
    fields.update(data)
TypeError: 'NoneType' object is not iterable

Test Activity
-------------
Developer Testing

Workaround
----------
Use the CLI

tags: added: stx.gui
Changed in starlingx:
assignee: nobody → Kristine Bujold (kbujold)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to gui (master)

Fix proposed to branch: master
Review: https://review.opendev.org/742021

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gui (master)

Reviewed: https://review.opendev.org/742021
Committed: https://git.openstack.org/cgit/starlingx/gui/commit/?id=959427c8bd1c8d074f46ba5be310850e2ee817ae
Submitter: Zuul
Branch: master

commit 959427c8bd1c8d074f46ba5be310850e2ee817ae
Author: Kristine Bujold <email address hidden>
Date: Mon Jul 20 15:38:38 2020 -0400

    Fix managing/unmanaging subclouds

    Change that were done in https://opendev.org/starlingx/distcloud-client
    /commit/dd05eccb0826ad4047201d361abea20b7a788bf5 changed the
    dcmanagerclient update_subcloud API and caused the call to the API
    from Horizon to break.

    Closes-Bug: 1888301

    Change-Id: Ia009f625a402c5480e669f61d1630f7dc3ba589f
    Signed-off-by: Kristine Bujold <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
tags: added: stx.distcloud
Revision history for this message
Ghada Khalil (gkhalil) wrote :

stx.4.0 / medium priority - given this has broken horizon for stx.4.0

tags: added: stx.4.0
Changed in starlingx:
importance: Undecided → Medium
Revision history for this message
Ghada Khalil (gkhalil) wrote :

@Kristine, please cherry-pick this to the r/stx.4.0 branch.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to gui (r/stx.4.0)

Fix proposed to branch: r/stx.4.0
Review: https://review.opendev.org/742442

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gui (r/stx.4.0)

Reviewed: https://review.opendev.org/742442
Committed: https://git.openstack.org/cgit/starlingx/gui/commit/?id=309913c07f17c8971532032f5ffc14238f462f9e
Submitter: Zuul
Branch: r/stx.4.0

commit 309913c07f17c8971532032f5ffc14238f462f9e
Author: Kristine Bujold <email address hidden>
Date: Mon Jul 20 15:38:38 2020 -0400

    Fix managing/unmanaging subclouds

    Change that were done in https://opendev.org/starlingx/distcloud-client
    /commit/dd05eccb0826ad4047201d361abea20b7a788bf5 changed the
    dcmanagerclient update_subcloud API and caused the call to the API
    from Horizon to break.

    Closes-Bug: 1888301

    Change-Id: Ia009f625a402c5480e669f61d1630f7dc3ba589f
    Signed-off-by: Kristine Bujold <email address hidden>
    (cherry picked from commit 959427c8bd1c8d074f46ba5be310850e2ee817ae)

Ghada Khalil (gkhalil)
tags: added: in-r-stx40
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to gui (f/centos8)

Fix proposed to branch: f/centos8
Review: https://review.opendev.org/c/starlingx/gui/+/792252

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gui (f/centos8)
Download full text (16.7 KiB)

Reviewed: https://review.opendev.org/c/starlingx/gui/+/792252
Committed: https://opendev.org/starlingx/gui/commit/63d6de4701a7f21779ad9ea4060fce9ed85bc71f
Submitter: "Zuul (22348)"
Branch: f/centos8

commit e05e1a43531499d94cfb1e538683ee36eea92b43
Author: Teresa Ho <email address hidden>
Date: Thu May 13 23:04:04 2021 -0400

    Do not display primary_reselect if not specified

    If the attribute 'primary_reselect' is not specified, the sysinv API
    will leave it as null and GUI will not display the attribute.

    Closes-Bug: 1928461

    Change-Id: I5b8ef8b29fb7775dde8607bb14cd733015269f82
    Signed-off-by: Teresa Ho <email address hidden>

commit f1a4d30eca91c7a239ebd7479a56fef7870a4b2e
Author: Pablo Bovina <email address hidden>
Date: Fri May 7 16:59:50 2021 -0300

    Display DataNetworks list

    DataNetworks are listed for pci-sriov
    under Create/Edit Interface forms.

    Closes-bug: 1927782
    Signed-off-by: Pablo Bovina <email address hidden>
    Change-Id: If927bb0facdec9e587a13354bef56eca5df08785

commit 7973677a3d7d518c31757b36037373d2c4ac769c
Author: Andre Fernando Zanella Kantek <email address hidden>
Date: Thu May 6 07:32:59 2021 -0400

    In AIO-SX, interface edit rejected with Host administrative unlocked

    It was detected the edit rejection when the user, on an unlocked
    host, tries to convert an ethernet non-SRIOV to an SRIOV-PF
    interface, with the server responding "Host 'controller-0' is
    administrative 'unlocked'".

    This is caused because UpdateInterface.handle() executes first the
    datanetwork assignment and then modifies the interface. Since the
    assignment, on an unlocked host, is only possible for SRIOV
    interfaces, the order of execution matters, we need to have the
    interface modified and then assigned. The correction consists of
    altering the order (first modify then assign) to do the described.

    Tests:
    To ensure the continuous operation of the other types of conversion
    the following combinations were tested (all were done adding the
    interface to a network or datanetwork, depending on the class):

    Unlocked state:
    ethernet/[none,data,pci-pt,platform] to pci-sriov: accepted
    modify parameters of a pci-sriov: rejected
    conversion to other than pci-sriov: rejected

    Locked state:
    all conversions (with network/datanetwork assignment) are accepted

    Closes-Bug: 1925183

    Signed-off-by: Andre Fernando Zanella Kantek <email address hidden>
    Change-Id: Ib124bf7222e07966becbb81198f65f5bc55715ce

commit ddcc4fd3ccb4c02580c71414345993252b089761
Author: Enzo Candotti <email address hidden>
Date: Tue May 4 11:08:57 2021 -0300

    Enable add/edit Worker personality on DC AIO-DX's GUI

    This update is to allow the option to add a new host with Worker
    personality on Distributed Cloud mode.

    Closes-Bug: 1927107

    Signed-off-by: Enzo Candotti <email address hidden>
    Change-Id: Idfed9352c7c6467014a2ed2cf10b70f6b470c28c

commit de43c019c0b7f038d0184d10aab2bf61b6c5e147
Author: Andre Fer...

tags: added: in-f-centos8
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on gui (f/centos8)

Change abandoned by "Bart Wensley <email address hidden>" on branch: f/centos8
Review: https://review.opendev.org/c/starlingx/gui/+/765064
Reason: This patch has been idle for more than six months. I am abandoning it to keep the review queue sane. If you are still interested in working on this patch, please unabandon it and upload a new patchset.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.