Unity: failed to clean zone when the host initiators are not logged in

Bug #1773305 reported by Ryan Liang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Ryan Liang

Bug Description

Unity driver failed to clean the FC zone when the host initiators are not logged in.

Although the host initiators are not logged in, the FC zone on switch is still active. And the targets information should be returned by Unity driver. Then it can be cleaned by the zone manager.

2018-05-21 10:52:16.129 19022 DEBUG storops.connection.client [req-0fde409f-6333-42a3-bba3-8d3cadace3f9 0688b01e6439ca32d698d20789d52169126fb41fb1a4ddafcebb97d854e836c9 148f31e7a5124274a4449da149f3c565 - default default] REQ URL: [GET] https://xx.xx.xx.xx:443/api/instances/host/Host_68?compact=True&fields=autoManageType,datastores,description,fcHostInitiators,fcHostInitiators.initiatorId,fcHostInitiators.paths.fcPort.wwn,fcHostInitiators.paths.initiator.type,fcHostInitiators.paths.isLoggedIn,health,hostContainer,hostIPPorts,hostIPPorts.address,hostLUNs,hostLUNs.hlu,hostLUNs.lun.id,hostLUNs.lun.name,hostLUNs.snap.id,hostPolledUUID,hostPushedUUID,hostUUID,hostVVolDatastore,id,instanceId,isInitializationComplete,isNfsv4Supported,iscsiHostInitiators,iscsiHostInitiators.initiatorId,lastPollTime,name,nfsShareAccesses,nfsUsername,operationalStatus,osType,registrationType,storageResources,tenant,type,vms, TIME: 0.350433111191, RESP CODE: 200 log_response /usr/lib/python2.7/site-packages/storops/connection/client.py:185
2018-05-21 10:52:16.130 19022 DEBUG storops.connection.client [req-0fde409f-6333-42a3-bba3-8d3cadace3f9 0688b01e6439ca32d698d20789d52169126fb41fb1a4ddafcebb97d854e836c9 148f31e7a5124274a4449da149f3c565 - default default] RESP BODY:
{
    "content": {
        "fcHostInitiators": [
            {
                "id": "HostInitiator_394",
                "initiatorId": "C0:50:76:0A:01:B6:01:80:C0:50:76:0A:01:B6:01:80"
            },
            {
                "id": "HostInitiator_395",
                "initiatorId": "C0:50:76:0A:01:B6:01:81:C0:50:76:0A:01:B6:01:81"
            },
            {
                "id": "HostInitiator_396",
                "initiatorId": "C0:50:76:0A:01:B6:01:84:C0:50:76:0A:01:B6:01:84"
            },
            {
                "id": "HostInitiator_390",
                "initiatorId": "C0:50:76:0A:01:B6:01:7E:C0:50:76:0A:01:B6:01:7E"
            },
            {
                "id": "HostInitiator_397",
                "initiatorId": "C0:50:76:0A:01:B6:01:85:C0:50:76:0A:01:B6:01:85"
            },
            {
                "id": "HostInitiator_391",
                "initiatorId": "C0:50:76:0A:01:B6:01:7F:C0:50:76:0A:01:B6:01:7F"
            },
            {
                "id": "HostInitiator_392",
                "initiatorId": "C0:50:76:0A:01:B6:01:82:C0:50:76:0A:01:B6:01:82"
            },
            {
                "id": "HostInitiator_393",
                "initiatorId": "C0:50:76:0A:01:B6:01:83:C0:50:76:0A:01:B6:01:83"
            }
        ],
        "name": "cindertest3",
        "instanceId": "root/emc:EMC_UEM_HostLeaf%InstanceID=Host_68",
        "operationalStatus": [
            32769
        ],
        "health": {
            "resolutionIds": [
                "host_connect_initiators"
            ],
            "descriptionIds": [
                "ALRT_HOST_NO_LOGGED_IN_INITIATORS"
            ],
            "descriptions": [
                "The host does not have any initiators logged into the storage system. Register one or more initiators on the host to the storage system. This may also require zoning changes on the switches."
            ],
            "value": 10
        },
        "osType": "",
        "type": 1,
        "id": "Host_68",
        "autoManageType": 0,
        "description": ""
    }
} _debug_print_json /usr/lib/python2.7/site-packages/storops/connection/client.py:178

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

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

Changed in cinder:
assignee: nobody → Ryan Liang (ryan-liang)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/570519
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=1ef06d4a31c3010f170da20fcc823dd81e77c1a7
Submitter: Zuul
Branch: master

commit 1ef06d4a31c3010f170da20fcc823dd81e77c1a7
Author: Ryan Liang <email address hidden>
Date: Fri May 25 13:47:52 2018 +0800

    [Unity] Return logged-out initiators

    In current implementation, Unity driver doesn't return the targets which
    connect to the logged-out initiators, which causes the zone manager
    fails to clean up the FC zone.

    The fix returns all the targets of specified initiators no matter they
    are logged in or not.

    Change-Id: I1438d0bd4a43e1813dfb0a6e9b3c4a177c2fee8f
    Closes-bug: #1773305

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/576294

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/queens)

Reviewed: https://review.openstack.org/576294
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=e8c223f1f9041a2204138ea587a25728c4dba6fd
Submitter: Zuul
Branch: stable/queens

commit e8c223f1f9041a2204138ea587a25728c4dba6fd
Author: Ryan Liang <email address hidden>
Date: Fri May 25 13:47:52 2018 +0800

    [Unity] Return logged-out initiators

    In current implementation, Unity driver doesn't return the targets which
    connect to the logged-out initiators, which causes the zone manager
    fails to clean up the FC zone.

    The fix returns all the targets of specified initiators no matter they
    are logged in or not.

    Change-Id: I1438d0bd4a43e1813dfb0a6e9b3c4a177c2fee8f
    Closes-bug: #1773305
    (cherry picked from commit 1ef06d4a31c3010f170da20fcc823dd81e77c1a7)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/580111

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/pike)

Reviewed: https://review.openstack.org/580111
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=f85193b5be6f4b69cf91137c40a34c64af676f52
Submitter: Zuul
Branch: stable/pike

commit f85193b5be6f4b69cf91137c40a34c64af676f52
Author: Ryan Liang <email address hidden>
Date: Fri May 25 13:47:52 2018 +0800

    [Unity] Return logged-out initiators

    In current implementation, Unity driver doesn't return the targets which
    connect to the logged-out initiators, which causes the zone manager
    fails to clean up the FC zone.

    The fix returns all the targets of specified initiators no matter they
    are logged in or not.

    Change-Id: I1438d0bd4a43e1813dfb0a6e9b3c4a177c2fee8f
    Closes-bug: #1773305
    (cherry picked from commit 1ef06d4a31c3010f170da20fcc823dd81e77c1a7)
    (cherry picked from commit e8c223f1f9041a2204138ea587a25728c4dba6fd)

tags: added: in-stable-pike
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 13.0.0.0b3

This issue was fixed in the openstack/cinder 13.0.0.0b3 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 12.0.4

This issue was fixed in the openstack/cinder 12.0.4 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 11.2.0

This issue was fixed in the openstack/cinder 11.2.0 release.

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.