Bug #1498773 “Huawei driver add check before use a available QoS...” : Bugs : Cinder

Wilson Liu (295988511-t) on 2015-09-23

Changed in cinder:
assignee:	nobody → Liu Xinguo (295988511-t)

Wilson Liu (295988511-t) on 2015-09-23

summary:

- Huawei driverAdd check before use a available QoS
+ Huawei driver add check before use a available QoS

Wilson Liu (295988511-t) on 2015-09-23

description:

updated

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-09-23: Fix proposed to cinder (master)

#1

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-09-24:

#2

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-09-24: Change abandoned on cinder (master)

#3

Change abandoned by Wilson Liu (<email address hidden>) on branch: master
Review: https://review.openstack.org/227129
Reason: Commit by mistaken.

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-09-28: Fix merged to cinder (master)

#4

Reviewed: https://review.openstack.org/226637
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=32099d547a2bb9c5f21de4592c5bf08272fcb14c
Submitter: Jenkins
Branch: master

commit 32099d547a2bb9c5f21de4592c5bf08272fcb14c
Author: Wilson Liu <email address hidden>
Date: Wed Sep 23 15:04:33 2015 +0800

Huawei driver add check before use a QoS

    One QoS in Huawei storage array can contain at most 64 LUNs.
    Do a check before adding LUN to QoS, if the number of the
    LUNs in the QoS is already equal or larger than 64, then
    this QoS should not be used.

Closed-Bug: #1498773
Change-Id: I9dd78cc1378051ded135a885cfc9347d42977311

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-09-29: Fix proposed to cinder (stable/liberty)

#5

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/228754

Sean McGinnis (sean-mcginnis) on 2015-10-02

Changed in cinder:
milestone:	none → liberty-rc2

Thierry Carrez (ttx) on 2015-10-02

Changed in cinder:
status:	New → Fix Committed
importance:	Undecided → Medium

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-02: Fix merged to cinder (stable/liberty)

#6

Reviewed: https://review.openstack.org/228754
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=4d42d496abf25630d1e6bd3ccf8c43ea1b754afb
Submitter: Jenkins
Branch: stable/liberty

commit 4d42d496abf25630d1e6bd3ccf8c43ea1b754afb
Author: Wilson Liu <email address hidden>
Date: Wed Sep 23 15:04:33 2015 +0800

Huawei driver add check before use a QoS

    One QoS in Huawei storage array can contain at most 64 LUNs.
    Do a check before adding LUN to QoS, if the number of the
    LUNs in the QoS is already equal or larger than 64, then
    this QoS should not be used.

    Closed-Bug: #1498773
    Change-Id: I9dd78cc1378051ded135a885cfc9347d42977311
    (cherry picked from commit 32099d547a2bb9c5f21de4592c5bf08272fcb14c)

tags:

added: in-stable-liberty

Thierry Carrez (ttx) on 2015-10-03

Changed in cinder:
status:	Fix Committed → Fix Released

Thierry Carrez (ttx) on 2015-10-15

Changed in cinder:
milestone:	liberty-rc2 → 7.0.0

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-15: Fix proposed to cinder (master)

#7

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-15: Fix merged to cinder (master)

#8

Download full text (7.3 KiB)

Reviewed: https://review.openstack.org/235282
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=0f7d289a2899082802499a498cba34a3abca321d
Submitter: Jenkins
Branch: master

commit 61026d4e4f2a58dd84ffb2e4e40ab99860b9316a
Author: Davanum Srinivas <email address hidden>
Date: Tue Oct 6 18:45:12 2015 -0700

Squashed commit of WebOb 1.5 and oslo.db fixes

Add testresources and testscenarios used by oslo.db fixture

If we use oslo.db fixtures, we'll need these 2 packages or
the next version of oslo.db release will break us.

    Closes-Bug: #1503501
    Change-Id: I8facdaf69c79b1b1ae4f9f64e9856e12f14440ed
    (cherry picked from commit ec40c3b6ddbfa4912ca6e612558efaef6043f3ae)

Fix test_misc for WebOb 1.5

    WebOb 1.5 was released at 2015-10-11. With this new version,
    webob.exc.WSGIHTTPException() constructor now fails with a KeyError
    when the HTTP status code is 0.

    test_exceptions_raise() of test_misc tries to instanciate all
    exceptions of cinder.exception. The problem is that
    ConvertedException uses a default HTTP status code of 0.

    Modify the default HTTP status code of ConvertedException to 400 to
    fix the unit test. The bug is only in the test,
    cinder/api/openstack/wsgi.py copies an existing HTTP code:

Fault(exception.ConvertedException(code=ex_value.code, ...)

    Closes-Bug: #1505153
    Change-Id: I1aec8038774828d48da4b0e831b390e33243809a
    (cherry picked from commit 867fccf833ffc597aa986cb6ff1b3b5c1101b9ba)

Change default Exception code to 500

    As a part of the fix for LP#1505153, the invalid
    response of 0 was changed to 400 to fix a running
    issue with the new version of WebOb.

It was pointed out after the fact however that a
500 might be more appropriate here.

    Additionally, other projects have implemented a 500
    as the default so for the sake of consistency we should
    consider doing the same.

This patch just changes the 400 to a 500 as the default
code.

Conflicts:
cinder/tests/unit/test_exception.py

NOTE(mriedem): The conflict is due to 17802086f not being
in stable/liberty.

    Change-Id: Ie486dc49c927f9b50f07c1fc562e89c090924a40
    Closes-Bug: #1505488
    (cherry picked from commit 9a7cbe540c94d54194194f4747a52b8211f6372e)

commit 5268a8d61beee1b18d14fd3738e763a0a61b00f1
Author: Xing Yang <email address hidden>
Date: Wed Sep 23 11:22:43 2015 -0400

Fix VMAX live migration problem

    Live migration fails for VMAX because the host is not taken into
    account when determining whether a volume is masked or not. For
    live migration it is necessary to know which host a volume is masked
    to. This patch fixes the problem by checking which host the volume
    is masked to.

    Change-Id: I693e29f7b26145bd1fd357b7d98377e26bfdfed8
    Closes-Bug: #1498964
    (cherry picked from commit 794e27c78255c94084b54821d44a36fd8f2096d7)

commit 2c09897f569a786998fd1ab50e42d81e9a2093a0
Author: OpenStack Proposal Bot <email address hidden>
Date: Sat Oct 3 06:22:46 2015 +0000

Imported Tr...

Reviewed:  https://review.openstack.org/235282
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=0f7d289a2899082802499a498cba34a3abca321d
Submitter: Jenkins
Branch:    master

commit 61026d4e4f2a58dd84ffb2e4e40ab99860b9316a
Author: Davanum Srinivas <davanum@gmail.com>
Date:   Tue Oct 6 18:45:12 2015 -0700

Squashed commit of WebOb 1.5 and oslo.db fixes
    
    Add testresources and testscenarios used by oslo.db fixture
    
    If we use oslo.db fixtures, we'll need these 2 packages or
    the next version of oslo.db release will break us.
    
    Closes-Bug: #1503501
    Change-Id: I8facdaf69c79b1b1ae4f9f64e9856e12f14440ed
    (cherry picked from commit ec40c3b6ddbfa4912ca6e612558efaef6043f3ae)
    
    Fix test_misc for WebOb 1.5
    
    WebOb 1.5 was released at 2015-10-11. With this new version,
    webob.exc.WSGIHTTPException() constructor now fails with a KeyError
    when the HTTP status code is 0.
    
    test_exceptions_raise() of test_misc tries to instanciate all
    exceptions of cinder.exception. The problem is that
    ConvertedException uses a default HTTP status code of 0.
    
    Modify the default HTTP status code of ConvertedException to 400 to
    fix the unit test. The bug is only in the test,
    cinder/api/openstack/wsgi.py copies an existing HTTP code:
    
        Fault(exception.ConvertedException(code=ex_value.code,  ...)
    
    Closes-Bug: #1505153
    Change-Id: I1aec8038774828d48da4b0e831b390e33243809a
    (cherry picked from commit 867fccf833ffc597aa986cb6ff1b3b5c1101b9ba)
    
    Change default Exception code to 500
    
    As a part of the fix for LP#1505153, the invalid
    response of 0 was changed to 400 to fix a running
    issue with the new version of WebOb.
    
    It was pointed out after the fact  however that a
    500 might be more appropriate here.
    
    Additionally, other projects have implemented a 500
    as the default so for the sake of consistency we should
    consider doing the same.
    
    This patch just changes the 400 to a 500 as the default
    code.
    
    Conflicts:
            cinder/tests/unit/test_exception.py
    
    NOTE(mriedem): The conflict is due to 17802086f not being
    in stable/liberty.
    
    Change-Id: Ie486dc49c927f9b50f07c1fc562e89c090924a40
    Closes-Bug: #1505488
    (cherry picked from commit 9a7cbe540c94d54194194f4747a52b8211f6372e)

commit 5268a8d61beee1b18d14fd3738e763a0a61b00f1
Author: Xing Yang <xing.yang@emc.com>
Date:   Wed Sep 23 11:22:43 2015 -0400

Fix VMAX live migration problem
    
    Live migration fails for VMAX because the host is not taken into
    account when determining whether a volume is masked or not. For
    live migration it is necessary to know which host a volume is masked
    to. This patch fixes the problem by checking which host the volume
    is masked to.
    
    Change-Id: I693e29f7b26145bd1fd357b7d98377e26bfdfed8
    Closes-Bug: #1498964
    (cherry picked from commit 794e27c78255c94084b54821d44a36fd8f2096d7)

commit 2c09897f569a786998fd1ab50e42d81e9a2093a0
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Sat Oct 3 06:22:46 2015 +0000

Imported Translations from Zanata
    
    For more information about this automatic import see:
    https://wiki.openstack.org/wiki/Translations/Infrastructure
    
    Change-Id: Ie155bc342c6f7af507c567577c97652a16d533c8

commit f9e6c5f88c96bcbd181e991219d4ecbf9b8cf833
Author: Tom Swanson <tom_swanson@dell.com>
Date:   Tue Sep 29 10:26:48 2015 -0500

Dell SC: cgsnapshot-delete doesn't actually delete
    
    dell_storagecenter_api.py was looking for the cgsnapshot for
    cgsnapshot-delete via the wrong REST API call.  It was also only
    looking for the first volume's snapshot rather than all snapshots
    associated with a cgsnapshot.  It now locates the snapshots properly
    and expires the associated replays on cgsnapshot-delete.
    
    Tests updated appropriately.
    
    Change-Id: I1e635db8b06d3d7a78573e6e8a2a4c25b7fb458d
    Closes-Bug: #1500583
    (cherry picked from commit e529e4d6cc71003c53126612dd869fa6fb23493a)

commit 4d42d496abf25630d1e6bd3ccf8c43ea1b754afb
Author: Wilson Liu <liuxinguo@huawei.com>
Date:   Wed Sep 23 15:04:33 2015 +0800

Huawei driver add check before use a QoS
    
    One QoS in Huawei storage array can contain at most 64 LUNs.
    Do a check before adding LUN to QoS, if the number of the
    LUNs in the QoS is already equal or larger than 64, then
    this QoS should not be used.
    
    Closed-Bug: #1498773
    Change-Id: I9dd78cc1378051ded135a885cfc9347d42977311
    (cherry picked from commit 32099d547a2bb9c5f21de4592c5bf08272fcb14c)

commit 6b7794a0d7c7a9eb76424eb2f7bd259d4f6ab0aa
Author: Wilson Liu <liuxinguo@huawei.com>
Date:   Thu Sep 24 16:52:31 2015 +0800

Report *real* free capacity in Huawei driver
    
    When we query capacity of a storage pool on Huawei storage,
    there are two items in the result: one is "USERFREECAPACITY"
    and the another one is "DATASPACE". In fact the "DATASPACE"
    is the *real* space we can use to create LUN, the
    "USERFREECAPACITY" contains some metadata that we can not use
    to store user data.
    
    Closes-Bug: #1499227
    Change-Id: I34a8c67dea2a7b7f5a2a693a65c9303b2cc3d972
    (cherry picked from commit e28e87258351c4c0d261d7c04ce495ef73e1520c)

commit bdfe5fa04e8e452c7262f265255b272f8fcfaa1d
Author: Goutham Pacha Ravi <gouthamr@netapp.com>
Date:   Fri Sep 25 01:52:43 2015 -0400

Revert use of netapp_lib from NetApp Drivers
    
    This patch cleanly reverts the changes made via the
    commit: e681ba2a99995dcd999e6539bb1222f8a1ac8adc
    cleanly and mitigates the conflicts that would
    occur with git-revert on the said commit.
    
    The revert is solely for changes pertaining to the use
    of the external library, netapp_lib. Minor code refactors
    from the prior change are retained.
    
    Unit test coverage has been increased for ZAPI and REST
    interface code in netapp/dataontap/client/api.py and
    netapp/eseries/client.py.
    
    Closes-Bug: #1499334
    Change-Id: Icead7e168e1c7187840de87c69365d26aedd5924
    (cherry picked from commit ff81307ca4c796a66f4bd584b4633db07dcf2a16)

commit a20f797a3e00a7e9d9a8a1a3f5659b3b0266394e
Author: John Griffith <john.griffith8@gmail.com>
Date:   Wed Sep 2 20:54:08 2015 -0600

Add placholder for migration backports in Liberty
    
    Adding 5 placeholders for migration backports.
    
    Change-Id: Id3c4efbeee8bb1d800d71576d9de66ae9deb315a
    (cherry picked from commit 3c1d38882eca336bb7735ee8097d0236b2d94b1a)

commit cc0e6118f60964c162d6482dc96454ffec30e061
Author: Andreas Jaeger <aj@suse.de>
Date:   Thu Sep 24 21:14:13 2015 +0200

Cleanup of Translations
    
    In preparation for the release, do some cleanups for translations.
    
    Removes all po files that are partially translated. The translation
    team has decided to exclude files with less than 66 % of translated
    content. There is no content lost, all data is in the translation
    server, we just remove it from the repository.
    
    This updates also recreates pot (translation source files) to
    reflect the state of the repository in case there was no recent
    import for them.
    
    This change needs to be done manually since the automatic import does
    not handle some of these cases.
    
    Change-Id: I7131dbf556627c5ad6fc237bab1e6b867f55c7f8

Cinder

Huawei driver add check before use a available QoS

Bug Description

Other bug subscribers

Remote bug watches