DC: manage subcloud cmd fails

Bug #1885141 reported by Nimalini Rasa
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
High
Tao Liu

Bug Description

Brief Description
-----------------
DC subcloud Couldn't manage, cmd is failing:
[sysadmin@controller-0 ~(keystone_admin)]$ dcmanager subcloud list
+----+------------+------------+--------------+---------------+-------------+
| id | name | management | availability | deploy status | sync |
+----+------------+------------+--------------+---------------+-------------+
| 1 | subcloud2 | unmanaged | online | complete | out-of-sync |
| 2 | subcloud6 | unmanaged | online | complete | out-of-sync |
| 3 | subcloud7 | unmanaged | online | complete | out-of-sync |
| 4 | subcloud8 | unmanaged | online | complete | out-of-sync |
| 5 | subcloud1 | unmanaged | online | complete | out-of-sync |
| 6 | subcloud10 | unmanaged | online | complete | out-of-sync |
| 7 | subcloud3 | unmanaged | online | complete | out-of-sync |
| 8 | subcloud4 | unmanaged | online | complete | out-of-sync |
| 9 | subcloud5 | unmanaged | online | complete | out-of-sync |
| 10 | subcloud9 | unmanaged | online | complete | out-of-sync |
+----+------------+------------+--------------+---------------+-------------+
[sysadmin@controller-0 ~(keystone_admin)]$ date;dcmanager subcloud manage subcloud1
Thu Jun 25 14:42:37 UTC 2020
'NoneType' object has no attribute 'items'
ERROR (app) Unable to manage subcloud subcloud1
Severity
--------
Major

Steps to Reproduce
------------------
Install dc system controllers, bootstrap subclouds, and mange it

Expected Behavior
------------------
Expected the manage command to work

Actual Behavior
----------------
Manage subcloud failed (cmd failed)

Reproducibility
---------------
Reproducible - seen on all 10 subclouds

System Configuration
--------------------
DC system controllers: Duplex
Subclouds: One node system (10 subclouds)
IPv6

Branch/Pull Time/Commit
-----------------------
2020-06-24_22-16-59

Last Pass
---------
2020-06-23_20-00-00

Timestamp/Logs
--------------
Thu Jun 25 14:42:37 UTC 2020

Test Activity
-------------
System Test

Workaround
----------
N/A

Yang Liu (yliu12)
description: updated
description: updated
Revision history for this message
Nimalini Rasa (nrasa) wrote :
  • Logs Edit (42.9 MiB, application/x-tar)
Revision history for this message
Ghada Khalil (gkhalil) wrote :

stx.4.0 - issue introduced by recent code changes which merged on 2020-06-24
https://review.opendev.org/#/c/737630/
https://review.opendev.org/#/c/737631/

Changed in starlingx:
assignee: nobody → Tao Liu (tliu88)
importance: Undecided → High
status: New → Triaged
tags: added: stx.4.0 stx.distcloud
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to distcloud-client (master)

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

Changed in starlingx:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to distcloud-client (master)

Reviewed: https://review.opendev.org/738073
Committed: https://git.openstack.org/cgit/starlingx/distcloud-client/commit/?id=e3736b5227ab7e264e5f6e6df277dc3f14997f35
Submitter: Zuul
Branch: master

commit e3736b5227ab7e264e5f6e6df277dc3f14997f35
Author: Tao Liu <email address hidden>
Date: Thu Jun 25 13:32:57 2020 -0400

    Fix subcloud manage/unmanage commands

    The subcloud_update interface was changed to support
    files and data parameters. The manage/unmanage CLI
    that uses the subcloud update interface was not updated.
    As a result, the command failed. This update fixes this
    issue.

    Tests:
    Manage/unmanage subclouds few times.

    Closes-Bug: 1885141

    Change-Id: I729895d76bbc76eac87420215bff04836d7eb678
    Signed-off-by: Tao Liu <email address hidden>

Changed in starlingx:
status: In Progress → Fix Released
Revision history for this message
Yang Liu (yliu12) wrote :

This is not seen in DC regression with 2020-06-27 load.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to distcloud-client (f/centos8)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on distcloud-client (f/centos8)

Change abandoned by "Chuck Short <email address hidden>" on branch: f/centos8
Review: https://review.opendev.org/c/starlingx/distcloud-client/+/792255
Reason: Updated merge coming

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to distcloud-client (f/centos8)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on distcloud-client (f/centos8)

Change abandoned by "Chuck Short <email address hidden>" on branch: f/centos8
Review: https://review.opendev.org/c/starlingx/distcloud-client/+/793407

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to distcloud-client (f/centos8)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on distcloud-client (f/centos8)

Change abandoned by "Chuck Short <email address hidden>" on branch: f/centos8
Review: https://review.opendev.org/c/starlingx/distcloud-client/+/793776

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

Reviewed: https://review.opendev.org/c/starlingx/distcloud-client/+/794794
Committed: https://opendev.org/starlingx/distcloud-client/commit/0d0781278fd07d93b65d0be666dd116f14fa5449
Submitter: "Zuul (22348)"
Branch: f/centos8

commit 8d2b5478c68579c1a9121f71b2085dc397ce9b85
Author: albailey <email address hidden>
Date: Wed May 19 15:47:55 2021 -0500

    Specify the nodeset for zuul jobs

    The py2.7 jobs need to specify xenial
    The py3.6 jobs need to specify bionic
    The focal zuul nodes only have python 3.8 installed in them

    The copyright date was updated in order to trigger
    the zuul jobs, as a no-delta type of change

    Note: pep8 and pylint jobs are not being specified for this repo
    because they specify a generic python3 interpreter, and are
    currently passing in python 3.6 and python 3.8.

    Partial-Bug: 1928978
    Signed-off-by: albailey <email address hidden>
    Change-Id: I4de3a640419ed431619cc4154ab928eebef71280

commit d52a9080082db5fda2e77fb9e342f812ea8c17e1
Author: Rafael Jordão Jardim <email address hidden>
Date: Thu May 6 16:27:06 2021 -0400

    Specify the cacert file in the verify option when in secure mode

    When running in secure mode we want to set the "verify"
    option to the path to the cacert file

    Tests:
    1° I generate a certificate using the documentation and installed on the
    controller (activate https), I got the ca cert and I set up a remote
    cli pointing to a DC and I exported the certificate in a variable
    OS_CACERT to the client get it, I ran some commands from DC.
    2° I passed a flag insecure to execute the dcmanager client
    in a insecure mode
    3° Built an ISO to check if something broke and installed on DC using
    VDM to check if the dcmanager keeps the current behavior

    Note: I think is a good ideia to plan a standardization of the clients
    mainly the keystone (authentication) part

    This change is based on the documentation of the requests documentation
    https://docs.python-requests.org/en/master/user/advanced/#ssl-cert-verification

    Closes-bug: 1927723
    Signed-off-by: Rafael Jordão Jardim <email address hidden>
    Change-Id: I4221657b97592b319b3fbf54b5b8c6d325ec9aa3

commit 859864c21dadf0fc1888f5df94853a3c6d5472ac
Author: Rafael Jordão Jardim <email address hidden>
Date: Wed Apr 7 13:05:53 2021 -0400

    Python 2 to Python 3 compatibility DC

    The code was adding two content-types, and when executing with
    python3 it got an error from the server, cause it was sending
    content-type application json but it was supposed to send
    a form data, so the fix was just add a verification to ensure that is
    not add 2 content-types if it already exists

    Development: When I was trying to find things to modify I followed the
    approach of build the client, get the tar file, I set up 2 environments
    one based on python2 and another python3, I installed the tar client
    in both environments and i exported the env vars that the client expect
    to get to request the controller, and doing that I could switch between
    t...

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

Change abandoned by "Bart Wensley <email address hidden>" on branch: f/centos8
Review: https://review.opendev.org/c/starlingx/distcloud-client/+/767072
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

Bug attachments

Remote bug watches

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