Bug #1503501 “oslo.db no longer requires testresources and tests...” : Bugs : Ironic

Revision history for this message

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

#1

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

Changed in cinder:
assignee:	nobody → Davanum Srinivas (DIMS) (dims-v)
status:	New → In Progress
Changed in neutron:
assignee:	nobody → Davanum Srinivas (DIMS) (dims-v)
status:	New → In Progress

Revision history for this message

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

#2

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

Armando Migliaccio (armando-migliaccio) on 2015-10-07

Changed in neutron:
importance:	Undecided → High

Sergey Kraynev (skraynev) on 2015-10-07

Changed in heat:
status:	New → Confirmed
importance:	Undecided → High
assignee:	nobody → Sergey Kraynev (skraynev)
milestone:	none → mitaka-1

Revision history for this message

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

#3

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

Changed in heat:
status:	Confirmed → In Progress

OpenStack Infra (hudson-openstack) on 2015-10-07

Changed in neutron:
assignee:	Davanum Srinivas (DIMS) (dims-v) → Ihar Hrachyshka (ihar-hrachyshka)

Revision history for this message

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

#4

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

Changed in nova:
assignee:	nobody → Davanum Srinivas (DIMS) (dims-v)
status:	New → In Progress

Revision history for this message

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

#5

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

Changed in heat:
assignee:	Sergey Kraynev (skraynev) → Davanum Srinivas (DIMS) (dims-v)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-07: Change abandoned on heat (master)

#6

Change abandoned by Davanum Srinivas (dims) (<email address hidden>) on branch: master
Review: https://review.openstack.org/232071
Reason: OOOOOPppppssss! sorry.

Revision history for this message

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

#7

Reviewed: https://review.openstack.org/231794
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=86ad967e40c2c6752ec0fb46cfd3098ede0c7178
Submitter: Jenkins
Branch: master

commit 86ad967e40c2c6752ec0fb46cfd3098ede0c7178
Author: Davanum Srinivas <email address hidden>
Date: Tue Oct 6 18:47:00 2015 -0700

Add testresources used by oslo.db fixture

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

Closes-Bug: #1503501
Change-Id: I7dfbf240333095d91a414ba15a439bdc4804eb25

Changed in neutron:
status:	In Progress → Fix Committed

Revision history for this message

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

#8

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

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

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

Changed in cinder:
status:	In Progress → Fix Committed

Ihar Hrachyshka (ihar-hrachyshka) on 2015-10-08

Changed in neutron:
assignee:	Ihar Hrachyshka (ihar-hrachyshka) → Davanum Srinivas (DIMS) (dims-v)

Revision history for this message

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

#9

Reviewed: https://review.openstack.org/232055
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=4bcc26487837b7ece7797f88622dea1b6d09bd94
Submitter: Jenkins
Branch: master

commit 4bcc26487837b7ece7797f88622dea1b6d09bd94
Author: Davanum Srinivas <email address hidden>
Date: Wed Oct 7 08:11:35 2015 -0700

Add testresources used by oslo.db fixture

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

Closes-Bug: #1503501
Change-Id: I176c85551007d3d9e7de8896ad12e9b87ff278d8

Changed in nova:
status:	In Progress → Fix Committed

OpenStack Infra (hudson-openstack) on 2015-10-12

Changed in heat:
assignee:	Davanum Srinivas (DIMS) (dims-v) → Sergey Kraynev (skraynev)

Revision history for this message

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

#10

Reviewed: https://review.openstack.org/231860
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=a7368fb68e8c1cccbb601aa1aad444248c645c25
Submitter: Jenkins
Branch: master

commit a7368fb68e8c1cccbb601aa1aad444248c645c25
Author: Sergey Kraynev <email address hidden>
Date: Wed Oct 7 02:59:01 2015 -0400

Add testresources 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.

Change-Id: I71f001c8b75ad52e68ccab8f07cd8946772fd03b
Closes-Bug: #1503501

Changed in heat:
status:	In Progress → Fix Committed

Lucas Alvares Gomes (lucasagomes) on 2015-10-12

Changed in ironic:
assignee:	nobody → Lucas Alvares Gomes (lucasagomes)
importance:	Undecided → High

Revision history for this message

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

#11

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

Changed in ironic:
status:	New → In Progress

Mike Fedosin (mfedosin) on 2015-10-12

Changed in glance:
assignee:	nobody → Mike Fedosin (mfedosin)
status:	New → In Progress

Revision history for this message

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

#12

Reviewed: https://review.openstack.org/233719
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=e7e0a056b4e2060ab53f2402c837a55e440edaaf
Submitter: Jenkins
Branch: master

commit e7e0a056b4e2060ab53f2402c837a55e440edaaf
Author: Lucas Alvares Gomes <email address hidden>
Date: Mon Oct 12 17:46:37 2015 +0100

Add testresources and testscenarios used by oslo.db fixture

oslo.db fixtures need these 2 packages and is breaking us.

Closes-Bug: #1503501
Change-Id: I75757b5bd7d9e75ca37d1b9a1efff810ba84b5df

Changed in ironic:
status:	In Progress → Fix Committed

Revision history for this message

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

#13

Reviewed: https://review.openstack.org/233841
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=693b50b2e79c9f3a00aa9311651972d9e67e1bcf
Submitter: Jenkins
Branch: master

commit 693b50b2e79c9f3a00aa9311651972d9e67e1bcf
Author: Mike Fedosin <email address hidden>
Date: Tue Oct 13 00:33:27 2015 +0300

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.

Change-Id: I7c0d2f6dabc20bd4ff0d29d3b47b948aa24ea56b
Closes-Bug: #1503501

Changed in glance:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Fix proposed to neutron (stable/liberty)

#14

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Fix proposed to neutron-vpnaas (master)

#15

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Change abandoned on neutron (stable/liberty)

#16

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/233950
Reason: to be squashed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13:

#17

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/233931
Reason: superseded by:

https://review.openstack.org/#/c/232270/

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Fix proposed to nova (stable/liberty)

#18

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

Revision history for this message

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

#19

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Fix proposed to nova (stable/liberty)

#20

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Fix merged to neutron-vpnaas (master)

#21

Reviewed: https://review.openstack.org/233953
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=6626a8529c256758f464eef8361913523a360961
Submitter: Jenkins
Branch: master

commit 6626a8529c256758f464eef8361913523a360961
Author: armando-migliaccio <email address hidden>
Date: Mon Oct 12 23:00:06 2015 -0700

Add testresources used by oslo.db fixture

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

Change-Id: I0075968c5d6ee41fa737658b683fa1d68aa44fca
Closes-Bug: #1503501

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Change abandoned on nova (stable/liberty)

#22

Change abandoned by Thierry Carrez (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/234115
Reason: Now part of the omnibus fix at https://review.openstack.org/#/c/234166/

Revision history for this message

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

#23

Reviewed: https://review.openstack.org/233842
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=c5b6901527b8b4a1250bdc179405c8af66fbae7e
Submitter: Jenkins
Branch: stable/liberty

commit c5b6901527b8b4a1250bdc179405c8af66fbae7e
Author: Mike Fedosin <email address hidden>
Date: Tue Oct 13 00:33:27 2015 +0300

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.

Change-Id: I7c0d2f6dabc20bd4ff0d29d3b47b948aa24ea56b
Closes-Bug: #1503501

tags:

added: in-stable-liberty

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Change abandoned on cinder (stable/liberty)

#24

Change abandoned by John Griffith (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/234118
Reason: Squashed this with the other needed fix:
https://review.openstack.org/#/c/233668/

Thierry Carrez (ttx) on 2015-10-13

Changed in glance:
status:	Fix Committed → Fix Released
milestone:	none → liberty-rc3

Thierry Carrez (ttx) on 2015-10-13

Changed in nova:
milestone:	none → liberty-rc3

Revision history for this message

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

#25

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

Michael McCune (mimccune) on 2015-10-13

Changed in sahara:
assignee:	nobody → Michael McCune (mimccune)

OpenStack Infra (hudson-openstack) on 2015-10-13

Changed in sahara:
status:	New → In Progress

Michael McCune (mimccune) on 2015-10-13

Changed in sahara:
importance:	Undecided → High

Sergey Lukjanov (slukjanov) on 2015-10-13

Changed in sahara:
milestone:	none → mitaka-1

Revision history for this message

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

#26

Reviewed: https://review.openstack.org/234166
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=6df6ad3ff32f2b1fe2978df1032002548ad8eb66
Submitter: Jenkins
Branch: stable/liberty

commit 6df6ad3ff32f2b1fe2978df1032002548ad8eb66
Author: Davanum Srinivas <email address hidden>
Date: Wed Oct 7 08:11:35 2015 -0700

Omnibus stable/liberty fix

    There are currently 3 different blocking issues in stable/liberty due
    to library releases: webob 1.5, oslo.db 3.0.0, and
    oslo.versionedobjects 0.11.0. This is a squashed fix for all of them
    as none can land without the others.

Issue #1 - oslo.db

Add testresources used by oslo.db fixture

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

(Cherry-picked from 4bcc26487837b7ece7797f88622dea1b6d09bd94)

Closes-Bug: #1503501

Issue #2 - oslo.versionedobjects

Drop unused obj_to_primitive() override

    This was a band-aid override until o.vo gained the obj_relationships fix
    that this method overrides. That has been in place since o.vo 0.8.0, which
    means this is long since no longer necessary (and is actually blocking our
    ability to absorb bug fixes to this code in o.vo). Further, we no longer
    use this directly because we're doing backports based on version manifests,
    which means we no longer consult child_versions _or_ obj_relationships.

(cherry picked from commit 142f1d9cc4ace90956c665c40b1f78795f9f7e29)

Issue #3 - webob

Default ConvertedException code to 500

    webob 1.5.0 released on 10/11 has change f6c749011 which
    strictly enforces status codes in exceptions, and 0 is not
    a valid status code so tests fail.

Change the default to 500 to match the default in the parent
class in webob.

Closes-Bug: #1505153
(cherry picked from commit 10438c0fc34bd088e018e1a5e8ec57b396528792)

Change-Id: I1e06e77308a7dd23209124f0807d61fb52470188

Reviewed:  https://review.openstack.org/234166
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=6df6ad3ff32f2b1fe2978df1032002548ad8eb66
Submitter: Jenkins
Branch:    stable/liberty

commit 6df6ad3ff32f2b1fe2978df1032002548ad8eb66
Author: Davanum Srinivas <davanum@gmail.com>
Date:   Wed Oct 7 08:11:35 2015 -0700

Omnibus stable/liberty fix
    
    There are currently 3 different blocking issues in stable/liberty due
    to library releases: webob 1.5, oslo.db 3.0.0, and
    oslo.versionedobjects 0.11.0. This is a squashed fix for all of them
    as none can land without the others.
    
    Issue #1 - oslo.db
    
    Add testresources used by oslo.db fixture
    
    If we use oslo.db fixtures, we'll need the package or
    the next version of oslo.db release will break us.
    
    (Cherry-picked from 4bcc26487837b7ece7797f88622dea1b6d09bd94)
    
    Closes-Bug: #1503501
    
    Issue #2 - oslo.versionedobjects
    
    Drop unused obj_to_primitive() override
    
    This was a band-aid override until o.vo gained the obj_relationships fix
    that this method overrides. That has been in place since o.vo 0.8.0, which
    means this is long since no longer necessary (and is actually blocking our
    ability to absorb bug fixes to this code in o.vo). Further, we no longer
    use this directly because we're doing backports based on version manifests,
    which means we no longer consult child_versions _or_ obj_relationships.
    
    (cherry picked from commit 142f1d9cc4ace90956c665c40b1f78795f9f7e29)
    
    Issue #3 - webob
    
    Default ConvertedException code to 500
    
    webob 1.5.0 released on 10/11 has change f6c749011 which
    strictly enforces status codes in exceptions, and 0 is not
    a valid status code so tests fail.
    
    Change the default to 500 to match the default in the parent
    class in webob.
    
    Closes-Bug: #1505153
    (cherry picked from commit 10438c0fc34bd088e018e1a5e8ec57b396528792)
    
    Change-Id: I1e06e77308a7dd23209124f0807d61fb52470188

Revision history for this message

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

#27

Reviewed: https://review.openstack.org/234329
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=61026d4e4f2a58dd84ffb2e4e40ab99860b9316a
Submitter: Jenkins
Branch: stable/liberty

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)

Reviewed:  https://review.openstack.org/234329
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=61026d4e4f2a58dd84ffb2e4e40ab99860b9316a
Submitter: Jenkins
Branch:    stable/liberty

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)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Change abandoned on cinder (stable/liberty)

#28

Change abandoned by Ivan Kolodyazhny (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/233668

Thierry Carrez (ttx) on 2015-10-13

Changed in cinder:
milestone:	none → liberty-rc3
status:	Fix Committed → Fix Released
Changed in nova:
status:	Fix Committed → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-13: Fix proposed to neutron-vpnaas (stable/liberty)

#29

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-14: Fix proposed to ironic (stable/liberty)

#30

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

Revision history for this message

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

#31

Reviewed: https://review.openstack.org/234326
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=a7ecd00a99fc26dce1e210d7e67e03efe752b5d3
Submitter: Jenkins
Branch: master

commit a7ecd00a99fc26dce1e210d7e67e03efe752b5d3
Author: Michael McCune <email address hidden>
Date: Tue Oct 13 12:24:38 2015 -0400

Add testresources used by oslo.db fixture

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

manually picked from: I176c85551007d3d9e7de8896ad12e9b87ff278d8
for more information see: I1e06e77308a7dd23209124f0807d61fb52470188

Change-Id: Ib511ec705206b2168fe18f5870ca81d59b7ba89f
Closes-Bug: 1503501

Changed in sahara:
status:	In Progress → Fix Committed

Thierry Carrez (ttx) on 2015-10-14

Changed in neutron:
milestone:	none → liberty-rc3

John Garbutt (johngarbutt) on 2015-10-14

Changed in nova:
importance:	Undecided → High

Revision history for this message

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

#32

Download full text (4.2 KiB)

Reviewed: https://review.openstack.org/232270
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=6dcfe3a9362ae5fcf18e5cfb59663e43446cd59c
Submitter: Jenkins
Branch: stable/liberty

commit 6dcfe3a9362ae5fcf18e5cfb59663e43446cd59c
Author: Kevin Benton <email address hidden>
Date: Tue Oct 6 19:28:47 2015 -0700

Mock oslo policy HTTPCheck instead of urllib

    We were mocking internal behavior of oslo policy by
    patching urllib. This will break with the upcoming oslo
    release that switches to requests.

This patch changes the mock to the HTTPCheck level and we
can leave implementation details testing up to oslo_policy.

    Change-Id: I07957f01307e25f1547197c720eea6e3e7f0ef5a
    Closes-Bug: #1503890
    (cherry picked from commit a0f1d9d6de1560be91d3001c8ac9f880a7a5a7e0)

Add testresources used by oslo.db fixture

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

    Closes-Bug: #1503501
    Change-Id: I7dfbf240333095d91a414ba15a439bdc4804eb25
    (cherry picked from commit 86ad967e40c2c6752ec0fb46cfd3098ede0c7178)

Fix functional test_server tests

Now oslo.service 0.10.0 no longer sends SIGHUP to parent and
children services.

This was a chance introduced by 286a6ea, and since it invalidated
the very logic under test, this must be revised.

(cherry picked from commit 090fe713592c2b6398d999bfa03b80cbb2054609)

    Change-Id: I18a11283925369bc918002477774f196010a1bc3
    Closes-bug: #1505438
    (cherry picked from commit 090fe713592c2b6398d999bfa03b80cbb2054609)

Make test_server work with older versions of oslo.service

    Change I18a11283925369bc918002477774f196010a1bc3 fixed the test for
    oslo.service >= 0.10.0, but it also broke it for older versions of
    oslo.service. Since the library has minimal version of >= 0.7.0 in
    requirements.txt, test should pass for those versions too.

    Now, instead of validating that either reset() or restart() of workers
    are triggered on SIGHUP, just validate that .start() is triggered the
    expected number of times (either way, no matter how oslo.service decide
    to clean up the children, they exit and then are respawned).

    Change-Id: I41f9d3af780b3178b075bc1e7084f417a2bd1378
    Closes-Bug: #1505645
    (cherry picked from commit 7bb40921660cf29beb68e338e205499efd6ffa36)

Fixed multiple py34 gate issues

1. Scope mock of 'open' to module

By mocking 'open' at the module level, we can avoid affecting
'open' calls from other modules.

2. Stop using LOG.exception in contexts with no sys.exc_info set

    Python 3.4 logger fills in record.exc_info with sys.exc_info() result
    [1], and then it uses it to determine the current exception [2] to
    append to the log message. Since there is no exception, exc_info[1] is
    None, and we get AttributeError inside traceback module.

    It's actually a bug in Python interpreter that it attempt to access the
    attribute when there is no exception. It turns out that it's fixed in
    latest master of cPyth...

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

Revision history for this message

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

#42

Download full text (16.6 KiB)

Reviewed: https://review.openstack.org/235300
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=140ccc36d172bead2605968b3d61b36cca8a0040
Submitter: Jenkins
Branch: master

commit 6dcfe3a9362ae5fcf18e5cfb59663e43446cd59c
Author: Kevin Benton <email address hidden>
Date: Tue Oct 6 19:28:47 2015 -0700

Mock oslo policy HTTPCheck instead of urllib

    We were mocking internal behavior of oslo policy by
    patching urllib. This will break with the upcoming oslo
    release that switches to requests.

This patch changes the mock to the HTTPCheck level and we
can leave implementation details testing up to oslo_policy.

    Change-Id: I07957f01307e25f1547197c720eea6e3e7f0ef5a
    Closes-Bug: #1503890
    (cherry picked from commit a0f1d9d6de1560be91d3001c8ac9f880a7a5a7e0)

Add testresources used by oslo.db fixture

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

    Closes-Bug: #1503501
    Change-Id: I7dfbf240333095d91a414ba15a439bdc4804eb25
    (cherry picked from commit 86ad967e40c2c6752ec0fb46cfd3098ede0c7178)

Fix functional test_server tests

Now oslo.service 0.10.0 no longer sends SIGHUP to parent and
children services.

This was a chance introduced by 286a6ea, and since it invalidated
the very logic under test, this must be revised.

(cherry picked from commit 090fe713592c2b6398d999bfa03b80cbb2054609)

    Change-Id: I18a11283925369bc918002477774f196010a1bc3
    Closes-bug: #1505438
    (cherry picked from commit 090fe713592c2b6398d999bfa03b80cbb2054609)

Make test_server work with older versions of oslo.service

    Change I18a11283925369bc918002477774f196010a1bc3 fixed the test for
    oslo.service >= 0.10.0, but it also broke it for older versions of
    oslo.service. Since the library has minimal version of >= 0.7.0 in
    requirements.txt, test should pass for those versions too.

    Now, instead of validating that either reset() or restart() of workers
    are triggered on SIGHUP, just validate that .start() is triggered the
    expected number of times (either way, no matter how oslo.service decide
    to clean up the children, they exit and then are respawned).

    Change-Id: I41f9d3af780b3178b075bc1e7084f417a2bd1378
    Closes-Bug: #1505645
    (cherry picked from commit 7bb40921660cf29beb68e338e205499efd6ffa36)

Fixed multiple py34 gate issues

1. Scope mock of 'open' to module

By mocking 'open' at the module level, we can avoid affecting
'open' calls from other modules.

2. Stop using LOG.exception in contexts with no sys.exc_info set

    Python 3.4 logger fills in record.exc_info with sys.exc_info() result
    [1], and then it uses it to determine the current exception [2] to
    append to the log message. Since there is no exception, exc_info[1] is
    None, and we get AttributeError inside traceback module.

    It's actually a bug in Python interpreter that it attempt to access the
    attribute when there is no exception. It turns out that it's fixed in
    latest master of cPython [3] (...

Reviewed:  https://review.openstack.org/235300
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=140ccc36d172bead2605968b3d61b36cca8a0040
Submitter: Jenkins
Branch:    master

commit 6dcfe3a9362ae5fcf18e5cfb59663e43446cd59c
Author: Kevin Benton <blak111@gmail.com>
Date:   Tue Oct 6 19:28:47 2015 -0700

Mock oslo policy HTTPCheck instead of urllib
    
    We were mocking internal behavior of oslo policy by
    patching urllib. This will break with the upcoming oslo
    release that switches to requests.
    
    This patch changes the mock to the HTTPCheck level and we
    can leave implementation details testing up to oslo_policy.
    
    Change-Id: I07957f01307e25f1547197c720eea6e3e7f0ef5a
    Closes-Bug: #1503890
    (cherry picked from commit a0f1d9d6de1560be91d3001c8ac9f880a7a5a7e0)
    
    Add testresources used by oslo.db fixture
    
    If we use oslo.db fixtures, we'll need the package or
    the next version of oslo.db release will break us.
    
    Closes-Bug: #1503501
    Change-Id: I7dfbf240333095d91a414ba15a439bdc4804eb25
    (cherry picked from commit 86ad967e40c2c6752ec0fb46cfd3098ede0c7178)
    
    Fix functional test_server tests
    
    Now oslo.service 0.10.0 no longer sends SIGHUP to parent and
    children services.
    
    This was a chance introduced by 286a6ea, and since it invalidated
    the very logic under test, this must be revised.
    
    (cherry picked from commit 090fe713592c2b6398d999bfa03b80cbb2054609)
    
    Change-Id: I18a11283925369bc918002477774f196010a1bc3
    Closes-bug: #1505438
    (cherry picked from commit 090fe713592c2b6398d999bfa03b80cbb2054609)
    
    Make test_server work with older versions of oslo.service
    
    Change I18a11283925369bc918002477774f196010a1bc3 fixed the test for
    oslo.service >= 0.10.0, but it also broke it for older versions of
    oslo.service. Since the library has minimal version of >= 0.7.0 in
    requirements.txt, test should pass for those versions too.
    
    Now, instead of validating that either reset() or restart() of workers
    are triggered on SIGHUP, just validate that .start() is triggered the
    expected number of times (either way, no matter how oslo.service decide
    to clean up the children, they exit and then are respawned).
    
    Change-Id: I41f9d3af780b3178b075bc1e7084f417a2bd1378
    Closes-Bug: #1505645
    (cherry picked from commit 7bb40921660cf29beb68e338e205499efd6ffa36)
    
    Fixed multiple py34 gate issues
    
    1. Scope mock of 'open' to module
    
    By mocking 'open' at the module level, we can avoid affecting
    'open' calls from other modules.
    
    2. Stop using LOG.exception in contexts with no sys.exc_info set
    
    Python 3.4 logger fills in record.exc_info with sys.exc_info() result
    [1], and then it uses it to determine the current exception [2] to
    append to the log message. Since there is no exception, exc_info[1] is
    None, and we get AttributeError inside traceback module.
    
    It's actually a bug in Python interpreter that it attempt to access the
    attribute when there is no exception. It turns out that it's fixed in
    latest master of cPython [3] (the intent of the patch does not seem
    relevant, but it removes the offending code while reshuffling the code).
    Note that now cPython correctly checks the exception value before
    accessing its attributes [4].
    
    The patch in cPython that resulted in the failure is [5] and is present
    since initial Python 3k releases.
    
    The patch in fixtures that broke us is [6].
    
    [1]: https://hg.python.org/cpython/file/tip/Lib/logging/__init__.py#l1412
    [2]: https://hg.python.org/cpython/file/tip/Lib/logging/__init__.py#l575
    [3]: https://hg.python.org/cpython/rev/73afda5a4e4c
    [4]: https://hg.python.org/cpython/rev/73afda5a4e4c#l6.484
    [5]: https://hg.python.org/cpython/rev/2ee09afee126
    [6]: https://github.com/testing-cabal/fixtures/commit/67dd2956943261e845a866dab155208c51da937e
    
    Closes-Bug: #1503847
    Closes-Bug: #1504053
    Co-Authored-By: Ihar Hrachyshka <ihrachys@redhat.com>
    Change-Id: I456b7846b8a53e4d3f8c91583685e0e1eaa84719
    (cherry picked from commit 8f58bbf38f5984e70d6e0be7427deb99d7782d1d)

commit 0b07910f33ed26fbdd13530eafbdefd74104424d
Author: Gary Kotton <gkotton@vmware.com>
Date:   Sun Sep 27 00:24:31 2015 -0700

DHCP: protect against case when device name is None
    
    There are edge cases when the agent attempts to unplug an interface and
    the device does not exist.
    
    Change-Id: I6917ec94f685f3dd3bff6aa1d43dc56aab76274a
    Closes-bug: #1498370
    (cherry picked from commit caebc8fb8e8d9782746c3cc3ddc86f786342c819)

commit fb40dbbce9ac17ee43d8921aa62c28f1ea723fba
Author: shihanzhang <shihanzhang@huawei.com>
Date:   Fri Sep 18 10:38:43 2015 +0800

Execute ipset command using check_exit_code
    
    When l2 agent execute ipset command, we should pass
    parameter 'check_exit_code' to self.execute acording to
    action.
    The intention is to safely fail if we try to destroy a
    non existing set, or delete a non existing member.
    Otherwise the agent gets stuck in a loop if such situation
    happens. Such kind of event would be logged as errors.
    
    Change-Id: If67330523d114d6da13d0280851e7138a51d08f7
    Closes-bug: #1497074

commit a7fa6f9a37c976d971b1be5896ebb6241c815475
Author: Nir Magnezi <nmagnezi@redhat.com>
Date:   Thu Sep 24 17:49:15 2015 +0300

The option force_metadata=True breaks the dhcp agent
    
    Patch I5f6ee9788717c3d4f1f2e2a4b9734fdd8dd92b40 has an issue with
    force_metadata = True.
    
    Using the option force_metadata=True while
    enable_isolated_metadata=False (which is the default), will break the
    dhcp agent because the variable subnet_to_interface_ip is being
    referenced before assignment.
    
    Co-Authored-By: Jakub Libosvar <jlibosva@redhat.com>
    Change-Id: I4e1d918e3a24dd483ee134021f587ae4520bf431
    Closes-Bug: #1499406
    (cherry picked from commit 473c338ff8c5526157d297b7e90d5e4f5e94cbb9)

commit 08971b0fb61170e86319c6801d6b1021d5944e3c
Author: Matt Riedemann <mriedem@us.ibm.com>
Date:   Thu Oct 1 06:55:14 2015 -0700

Do not log an error when deleting a linuxbridge does not exist
    
    When you're racing to teardown and relying on RPC, it should not be
    considered an error that a bridge does not exist when you're trying to
    delete it anyway. This is better served as debug level information.
    
    For context, this shows up over 1000 times in successful
    gate-tempest-dsvm-neutron-linuxbridge job runs in a week so it's
    probably safe to say it's not contributing to test failures.
    
    Closes-Bug: #1501779
    
    Change-Id: I8e9b27e010d033291497f4e660a97bbae92e75b1
    (cherry picked from commit a9dc48f5a4616a505824c3e8520c3dce7436f1b1)

commit f3bb778dcb213d81aa62e504afb10b3bac614bb4
Author: Henry Gessau <gessau@cisco.com>
Date:   Sun Sep 27 15:36:59 2015 -0400

Tag the alembic migration revisions for Liberty
    
    Previously when we had one repo with one alembic branch we would
    create a milestone revision on that single branch. Now we have
    multiple repos and expand/contract branches for each repo.
    
    So from now on we tag the final revision on every branch when we make
    a milestone release. Update the cli to support the command:
      neutron-db-manage upgrade <milestone>
    where <milestone> becomes an alias for all the revisions for a
    milestone.
    
    Closes-Bug: #1499033
    
    Change-Id: I38623986dd574bec01fe147f9c6a747f3f512bb7
    (cherry picked from commit 52236764a3a7c23ec24a86ccb84bc6737e2a4791)
    Conflicts:
            neutron/db/migration/cli.py
    	neutron/tests/unit/db/test_migration.py

commit 677687a3aad20e61f47b3ecbc9b0f4686e817653
Author: Bogdan Tabor <bogdan.tabor@ericpol.com>
Date:   Wed Sep 23 10:52:25 2015 +0200

/common/utils.py py34 incompatibility
    
    Fixes bug found during unit test of get_random_string() method.
    For Python3 it is required to encode string as bytes object
    before hashing.
    
    Related-blueprint: neutron-python3
    Closes-bug: #1502148
    Change-Id: I93061a831b45fc8b3ede0211af665ce02f36f38a
    (cherry picked from commit 3d54459c2971f01581354c8a6b9c58abea7743d8)

commit c66fa826ab2f25a21f8c2fa9a386aff1e54c7427
Author: Terry Wilson <twilson@redhat.com>
Date:   Mon Sep 28 21:01:30 2015 -0500

Just call set-manager if connecting fails
    
    Instead of calling set-manager every init, only call it when
    getting the schema fails inside of Connection.start() which is
    only called once.
    
    Closes-Bug: #1499784
    Change-Id: Ib28cd912e1f2eba318a6b0a82269919a1c95312d
    (cherry picked from commit c561c7c20acef7227e2eab2be30a2626e16d2354)

commit cf1689438079de264dd4a61cccaf2ad8b0bd05b9
Author: Adelina Tuvenie <atuvenie@cloudbasesolutions.com>
Date:   Mon Sep 21 23:51:42 2015 -0700

Fixes 'ovs-agent cannot start on Windows because root_helper opt is not found'
    
    Change Ie1224f8a1c17268cd7d1c474ed82fdfb8852eaa8 removed
    config.register_root_helper() from ovs_neutron_agent:main, thus the agent
    cannot start on Windows since root_helper config option is never
    registered.
    
    The fix consists in adding config.register_root_helper in the Windows side
    utils.
    
    Closes-Bug: #1497930
    
    Change-Id: I6adc3b869c9564cac3ab99174b8fe47584213e59
    (cherry picked from commit 074f82d3ec18fd591e9d1805676dfe13bd508510)

commit 18344140bd0aae61dd6f50667d719cd9cfe149cb
Author: James Page <james.page@ubuntu.com>
Date:   Thu Oct 1 11:54:28 2015 +0100

Use format to convert ints to strings
    
    This ensures that on 32bit architectures, where
    sometimes an implicit long is created, the resulting
    string does not contain a 'L' suffix as generated
    by the hex function.
    
    Change-Id: I264f90d68009963fe7a217c2170e1cf2f46ae2bb
    Closes-Bug: 1501703
    (cherry picked from commit 49883f1b2bfc9a4f06b1d336553d7f0ab4289a0d)

commit 155746c171c24d11e77d9624e9af31b45dad87f3
Author: Adelina Tuvenie <atuvenie@cloudbasesolutions.com>
Date:   Thu Sep 24 06:17:06 2015 -0700

Fixes 'ovs-agent fails to start on Windows beacause of SIGHUP'
    
    Change Ic0cf8a9ca7f3a16b556a6825e2979471ae136c33 has added the handling
    of SIGHUP in neutron_ovs_agent. This does not work on Windows as there is
    no SIGHUP signal on windows.
    
    The fix consists in checking if signal has SIGHUP attr before setting the handler
    for SIGHUP.
    
    Change-Id: I08258580252d68db222cbb9db7812ff1dfb37a56
    Closes-Bug: #1499345
    (cherry picked from commit 584bb1aad8647872cb8722beeb84e9ec08abc208)

commit dd197db312f4bc5a2a661c86125b06d71c45e742
Author: Toni Ylenius <toni.ylenius@cybercom.com>
Date:   Wed Aug 5 15:06:00 2015 +0300

usage_audit: Fix usage_audit to work with ML2
    
    The script didn't work with the ML2 plugin as router related
    functionality is moved to the L3 router service plugin. The fix assumes
    that the L3 router service plugin is in use, and this seems to apply to
    most of the current core plugins.
    
    Change-Id: If3eb7bde4a5b8b9ca2ac0bd7327325f6c0ad620a
    Closes-Bug: 1481692
    (cherry picked from commit 2bded62d6a6a141e610a06e5f40bb28c0b26da5c)

commit d1d5f2a3bc11fbaa61dcbcce7de2be03b88dd606
Author: Salvatore Orlando <salv.orlando@gmail.com>
Date:   Thu Oct 1 16:07:59 2015 -0700

Pecan: Fix quota enforcement
    
    Ensure that core resources, and not only extension resources,
    are registered with the quota engine. Otherwise no enforcement
    will happen for them.
    
    Further, do not pass anymore the 'resource name' to the resource's
    count method as it is not necessary anymore since Liberty.
    
    Change-Id: I895b4e69e50dbf1aac39e07eba07c3e3ff30808a
    Closes-bug: #1501948
    (cherry picked from commit 809ac2161577ef354b75713c0d5210e03fa16d6a)

commit 13b6d76da361178ea0b84baf4831a4c7f53e29c7
Author: Ihar Hrachyshka <ihrachys@redhat.com>
Date:   Thu Oct 1 17:13:25 2015 +0200

metadata: don't crash proxy on non-unicode user data
    
    We attempt to log every successful metadata response with LOG.debug. But
    as per oslo.log docs [1], we should make sure that what we pass into the
    library is unicode.
    
    Http.request returns a tuple of Response object and a string, which is
    bytes in Python 2.x [2].
    
    That's why we need to convert the response content to unicode before
    passing it into oslo.log.
    
    To achieve it, we utilize encodeutils.safe_decode with 'replace' errors
    handling strategy, so that we don't get exceptions on input that does
    not conform unicode.
    
    For the unit test case, we pass a string that is not expected to convert
    to unicode with errors='strict' strategy or similar, and check that we
    still don't crash.
    
    While at it, we remove a check for the number of log calls being
    triggered, because it's something that we should avoid validating in
    test cases, and it cannot trigger a real bug. The mock that was used to
    count the number would also hide the bug that we try to reproduce.
    
    Note that the bug does not require debug to be set because the crash
    occurs before oslo.log machinery decides it should not log the message.
    
    [1]: http://docs.openstack.org/developer/oslo.log/usage.html#no-more-implicit-conversion-to-unicode-str
    [2]: http://bitworking.org/projects/httplib2/doc/html/libhttplib2.html#httplib2.Http.request
    
    Closes-Bug: #1501772
    Change-Id: I6a32c40ff117fae43913386134c8981539697ce8
    (cherry picked from commit 80e3d9be4923ecad17377b1d15c8392b8a43dac6)

commit 48a9cd433c09bbf4b80a4665f625cd3754db0f00
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Fri Oct 2 06:33:59 2015 +0000

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

commit 2ed841f58d931bc9a2d29172d26c03d7cb6862e6
Author: Terry Wilson <twilson@redhat.com>
Date:   Mon Sep 28 17:42:25 2015 -0500

Check idl.run() return value before blocking
    
    The IDL change_seqno can change as a result of running idl.run(), and
    it returns True if there has been a change. The existing code would
    block without checking the return value, and therefor could block
    until another change occurred, or the timeout was reached.
    
    Closes-Bug: #1501090
    Change-Id: I9b7aac2755e6e6b69165023e01583337f9d30f15
    (cherry picked from commit 5dba2ac71c16cfd11937cf4becbaabd5b807678d)

commit 69662eb3cad925a3933f2ba3e5ef03dbde52ffc9
Author: Kevin Benton <blak111@gmail.com>
Date:   Fri Sep 18 07:15:45 2015 -0700

Don't write DHCP opts for SLAAC entries
    
    Change I81b4669eadaa9119e08c6a5e1d2a7b5959babdcc
    caused DHCP options to be written for SLAAC entries
    when they previously were not. This restores the previous
    behavior.
    
    Closes-Bug: #1499054
    Change-Id: I81400305f166d62aa4612aab54602abb8178b64c
    (cherry picked from commit 22f5fef5251cff9dbce35ca9a0ec8ea3b42f359c)

commit 892a325d7e66a164a74c5bafdf7e0d460af275d9
Author: Andreas Jaeger <aj@suse.de>
Date:   Thu Sep 24 20:53:09 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: I597258484d388091366bf9baed6e87dafb3f0cc3

commit 0782a0b38172d37267b9400246b2d5b54c3cf67c
Author: Salvatore Orlando <salv.orlando@gmail.com>
Date:   Thu Sep 24 05:45:40 2015 -0700

Fix quota usage tracker for security group rules
    
    This simple patch ensures usage for security group rules is
    marked as dirty when a security group rule is deleted.
    To this aim, the security group rule is deleted using ORM
    in order to ensure the sqlalchemy even if fired.
    
    Closes-Bug: #1499339
    
    Change-Id: I1e81fe03fed14ec438cea5d7675f66caeb91afd8
    (cherry picked from commit 851b1b4bbbaa965f190bfacab03fd6ece80ad0ec)

commit 3b0934e9f11b1bc9bf339681f2b5abad77a995e7
Author: Kyle Mestery <mestery@mestery.com>
Date:   Thu Sep 24 13:09:48 2015 +0000

Update default branch in .gitreview to stable/liberty
    
    Change-Id: I5463e1ffb9e7fc772ba27dd7b7d7d2414a526b5a
    Signed-off-by: Kyle Mestery <mestery@mestery.com>

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-15: Fix proposed to sahara (stable/liberty)

#43

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-15: Change abandoned on sahara (stable/liberty)

#44

Change abandoned by Sergey Lukjanov (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/235540

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-16: Fix merged to neutron-vpnaas (master)

#45

Reviewed: https://review.openstack.org/235318
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=9a1860cc8d463c4d9e85d1098c183e43f6d21f15
Submitter: Jenkins
Branch: master

commit 1ab399faa6a907bb67e981b728dd06369f0a66d8
Author: armando-migliaccio <email address hidden>
Date: Mon Oct 12 23:00:06 2015 -0700

Add testresources used by oslo.db fixture

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

    Change-Id: I0075968c5d6ee41fa737658b683fa1d68aa44fca
    Closes-Bug: #1503501
    (cherry picked from commit 6626a8529c256758f464eef8361913523a360961)

commit cc8703abe4d24d403d1eec6269ecb1c74a8c747c
Author: Henry Gessau <email address hidden>
Date: Sun Sep 27 16:04:20 2015 -0400

Tag the alembic migration revisions for Liberty

    Previously when we had one repo with one alembic branch we would
    create a milestone revision on that single branch. Now we have
    multiple repos and expand/contract branches for each repo.

So from now on we tag the final revision on every branch when we make
a milestone release.

Partial-Bug: #1499033

Depends-On: I38623986dd574bec01fe147f9c6a747f3f512bb7

Change-Id: I237cd984273212da930794414c761f7748e6bd0c
(cherry picked from commit eac6ea6ef23d2734fd664e8c70a7a599a9f8bcff)

commit 0c4e3bd61f4a55fcc74542761ae64ba8e510969b
Author: Ihar Hrachyshka <email address hidden>
Date: Fri Sep 25 13:03:07 2015 +0200

Use stable/liberty branch for neutron dep

Change-Id: I04cb52ab5e918059c8f19882d6a95106344b52e6

commit ffe66482371c99bb3c4d1b253c3b5f1015629863
Author: Ihar Hrachyshka <email address hidden>
Date: Thu Sep 24 17:12:16 2015 +0200

Update defaultbranch in .gitreview to stable/liberty

Change-Id: Ic83d51a5740b56679f46985ed4b4a824082d4149

Revision history for this message

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

#46

Reviewed: https://review.openstack.org/235133
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=7ab41f1384e2638020a02e7ae875cd46297b5b04
Submitter: Jenkins
Branch: stable/liberty

commit 7ab41f1384e2638020a02e7ae875cd46297b5b04
Author: Thierry Carrez <email address hidden>
Date: Thu Oct 15 09:35:41 2015 +0200

Switch to post-versioning

No longer mention next version on setup.cfg on stable branch.

    Add testresources used by oslo.db fixture
    If we use oslo.db fixtures, we'll need the package or
    the next version of oslo.db release will break us.
    for more information see: I1e06e77308a7dd23209124f0807d61fb52470188

Closes-Bug: 1503501

Change-Id: Iea3c737bc51a599d756514b82b1f4102b389a2c2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-19: Change abandoned on heat (stable/liberty)

#47

Change abandoned by Roman Podoliaka (<email address hidden>) on branch: stable/liberty
Review: https://review.openstack.org/234709

Revision history for this message

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

#48

Reviewed: https://review.openstack.org/235246
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=6d611b7f9758b04cfe67bd9533d59044a7844bc7
Submitter: Jenkins
Branch: stable/liberty

commit 6d611b7f9758b04cfe67bd9533d59044a7844bc7
Author: Sirushti Murugesan <email address hidden>
Date: Thu Oct 8 09:13:04 2015 +0530

Merge of three backported commits for fixing liberty gate

First patch description:

Use LOG.error when no exception handler is used

    There's no need to use LOG.exception here since sys.exc_info()
    results in (None, None, None) and thus would effectively just
    be LOG.error.

    Also, the failure of invoking LOG.exception outside of an
    exception handler is due to a bug in Python3[1]. The fix for
    that is only available in Python3.5.

[1] http://bugs.python.org/issue17911

Closes-Bug: #1482261
(cherry picked from commit f5c415accfff2ad399194b30b9408a903042f009)

Second patch description:

Add testresources 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
(cherry picked from commit a7368fb68e8c1cccbb601aa1aad444248c645c25)

Third patch description:

Switch to post-versioning

No longer mention next version on setup.cfg on stable branch.

Change-Id: Ib40c664a4f0e01cd27e1f24cfdbc9de278aece42

Doug Hellmann (doug-hellmann) on 2015-10-20

Changed in ironic:
status:	Fix Committed → Fix Released
status:	Fix Released → Fix Committed

Revision history for this message

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

#49

Download full text (4.1 KiB)

Reviewed: https://review.openstack.org/235346
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=431fa95ecf92a9ebb7082f829c0a99f958363cc3
Submitter: Jenkins
Branch: master

commit 69516fad5f651a085a047a337a05c58b39023c1b
Author: Mike Fedosin <email address hidden>
Date: Mon Oct 12 15:34:54 2015 +0300

Add 'deactivated' status to image schema.

New 'deactivated' status was introduced in Kilo release,
but it doesn't listed in available image statuses in the schema.

It leads to issues on the client side, when it can't validate
the image with this status against the schema and returns the error.

    Change-Id: I5ec264614ae7ecf54b846ad0600442a18c61d24c
    Closes-bug: #1505218
    Related-bug: #1505134

commit c5b6901527b8b4a1250bdc179405c8af66fbae7e
Author: Mike Fedosin <email address hidden>
Date: Tue Oct 13 00:33:27 2015 +0300

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.

Change-Id: I7c0d2f6dabc20bd4ff0d29d3b47b948aa24ea56b
Closes-Bug: #1503501

commit fc32f0554de0ba7773d98e6828da157ca7c66002
Author: Mike Fedosin <email address hidden>
Date: Sun Sep 20 17:01:22 2015 +0300

Cleanup chunks for deleted image if token expired

    In patch I47229b366c25367ec1bd48aec684e0880f3dfe60 it was
    introduced the logic that if image was deleted during file
    upload when we want to update image status from 'saving'
    to 'active' it's expected to get Duplicate error and delete
    stale chunks after that. But if user's token is expired
    there will be Unathorized exception and chunks will stay
    in store and clog it.
    And when, the upload operation for such an image is
    completed the operator configured quota can be exceeded.

    This patch fixes the issue of left over chunks for an image
    which was deleted from saving status, by correctly handle
    auth exceptions from registry server.

Partial-bug: #1498163

Change-Id: I17a66eca55bfb83107046910e69c4da01415deec

commit ca8d909a61ba335805d8d17070230ce9478a000d
Author: Stuart McLaren <email address hidden>
Date: Wed Sep 30 16:54:12 2015 +0000

Download forbidden when get_image_location is set.

When using v2 an attempt to download an image would return a 403 if the
get_image_location policy was set.

    Note: We had been returning both 404 and 204 when no data was
    available. There was no way to detect the 404 case without trying to
    access the image locations so I've standardized on 204.

    Change-Id: I658b08a35d3a8cb8a7096baf716ccb3d6e7d9abf
    Closes-bug: 1501672
    (cherry picked from commit b47f625443c3b46483506926f31fee42478705d4)

commit ebdf076cc9bd5d9239cdc96c6e7cecc72f852bbb
Author: Mike Fedosin <email address hidden>
Date: Thu Oct 1 18:28:48 2015 +0300

Catch NotAuthenticated exception in import task

    If glance uses registry as data_api then it's possible
    that token may expire during image import task and Glance
    will have NotUauthenticated exception.

This code adds...

Reviewed:  https://review.openstack.org/235346
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=431fa95ecf92a9ebb7082f829c0a99f958363cc3
Submitter: Jenkins
Branch:    master

commit 69516fad5f651a085a047a337a05c58b39023c1b
Author: Mike Fedosin <mfedosin@mirantis.com>
Date:   Mon Oct 12 15:34:54 2015 +0300

Add 'deactivated' status to image schema.
    
    New 'deactivated' status was introduced in Kilo release,
    but it doesn't listed in available image statuses in the schema.
    
    It leads to issues  on the client side, when it can't validate
    the image with this status against the schema and returns the error.
    
    Change-Id: I5ec264614ae7ecf54b846ad0600442a18c61d24c
    Closes-bug: #1505218
    Related-bug: #1505134

commit c5b6901527b8b4a1250bdc179405c8af66fbae7e
Author: Mike Fedosin <mfedosin@mirantis.com>
Date:   Tue Oct 13 00:33:27 2015 +0300

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.
    
    Change-Id: I7c0d2f6dabc20bd4ff0d29d3b47b948aa24ea56b
    Closes-Bug: #1503501

commit fc32f0554de0ba7773d98e6828da157ca7c66002
Author: Mike Fedosin <mfedosin@mirantis.com>
Date:   Sun Sep 20 17:01:22 2015 +0300

Cleanup chunks for deleted image if token expired
    
    In patch I47229b366c25367ec1bd48aec684e0880f3dfe60 it was
    introduced the logic that if image was deleted during file
    upload when we want to update image status from 'saving'
    to 'active' it's expected to get Duplicate error and delete
    stale chunks after that. But if user's token is expired
    there will be Unathorized exception and chunks will stay
    in store and clog it.
    And when, the upload operation for such an image is
    completed the operator configured quota can be exceeded.
    
    This patch fixes the issue of left over chunks for an image
    which was deleted from saving status, by correctly handle
    auth exceptions from registry server.
    
    Partial-bug: #1498163
    
    Change-Id: I17a66eca55bfb83107046910e69c4da01415deec

commit ca8d909a61ba335805d8d17070230ce9478a000d
Author: Stuart McLaren <stuart.mclaren@hp.com>
Date:   Wed Sep 30 16:54:12 2015 +0000

Download forbidden when get_image_location is set.
    
    When using v2 an attempt to download an image would return a 403 if the
    get_image_location policy was set.
    
    Note: We had been returning both 404 and 204 when no data was
    available. There was no way to detect the 404 case without trying to
    access the image locations so I've standardized on 204.
    
    Change-Id: I658b08a35d3a8cb8a7096baf716ccb3d6e7d9abf
    Closes-bug: 1501672
    (cherry picked from commit b47f625443c3b46483506926f31fee42478705d4)

commit ebdf076cc9bd5d9239cdc96c6e7cecc72f852bbb
Author: Mike Fedosin <mfedosin@mirantis.com>
Date:   Thu Oct 1 18:28:48 2015 +0300

Catch NotAuthenticated exception in import task
    
    If glance uses registry as data_api then it's possible
    that token may expire during image import task and Glance
    will have NotUauthenticated exception.
    
    This code adds a correct handling of this exception and
    allows Glance to remove stale chunks from store.
    
    Partial-Bug: #1498163
    
    Change-Id: Ia6e1fe0d27b13b920ee7e728feb5305dec77e066

commit 0cc46f4456449186d140b7e5d9c46e08acf181a5
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Thu Oct 1 06:10:45 2015 +0000

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

commit b7fb5bf0f89f657ead98024ee0168f2c2fa7a776
Author: Erno Kuvaja <jokke@usr.fi>
Date:   Mon Sep 28 10:44:38 2015 +0000

Return missing authtoken options
    
    Example configs were missing keystone_authtoken section after moving
    to generated config files. This change returns that to generation.
    
    Closes-Bug: #1500361
    
    Change-Id: I6ee82c38061d483cea7254d155d9a72436880e84
    (cherry picked from commit b1d2d938d282ccd51986e57a638f8ea5bec56b0f)

Revision history for this message

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

#50

Download full text (10.6 KiB)

Reviewed: https://review.openstack.org/235181
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=d8a3b9d6408627ce9a293d1e62d003757af655a1
Submitter: Jenkins
Branch: master

commit 6df6ad3ff32f2b1fe2978df1032002548ad8eb66
Author: Davanum Srinivas <email address hidden>
Date: Wed Oct 7 08:11:35 2015 -0700

Omnibus stable/liberty fix

    There are currently 3 different blocking issues in stable/liberty due
    to library releases: webob 1.5, oslo.db 3.0.0, and
    oslo.versionedobjects 0.11.0. This is a squashed fix for all of them
    as none can land without the others.

Issue #1 - oslo.db

Add testresources used by oslo.db fixture

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

(Cherry-picked from 4bcc26487837b7ece7797f88622dea1b6d09bd94)

Closes-Bug: #1503501

Issue #2 - oslo.versionedobjects

Drop unused obj_to_primitive() override

    This was a band-aid override until o.vo gained the obj_relationships fix
    that this method overrides. That has been in place since o.vo 0.8.0, which
    means this is long since no longer necessary (and is actually blocking our
    ability to absorb bug fixes to this code in o.vo). Further, we no longer
    use this directly because we're doing backports based on version manifests,
    which means we no longer consult child_versions _or_ obj_relationships.

(cherry picked from commit 142f1d9cc4ace90956c665c40b1f78795f9f7e29)

Issue #3 - webob

Default ConvertedException code to 500

    webob 1.5.0 released on 10/11 has change f6c749011 which
    strictly enforces status codes in exceptions, and 0 is not
    a valid status code so tests fail.

Change the default to 500 to match the default in the parent
class in webob.

Closes-Bug: #1505153
(cherry picked from commit 10438c0fc34bd088e018e1a5e8ec57b396528792)

Change-Id: I1e06e77308a7dd23209124f0807d61fb52470188

commit 606204354b5ed96852240020769c81acda9f9fc8
Author: Matt Riedemann <email address hidden>
Date: Mon Oct 5 20:32:58 2015 +0000

Revert "[libvirt] Move cleanup of imported files to imagebackend"

This reverts commit 9ba70756de326ffaa8be43acfde12cad04ed0af2

    The change introduced an UnboundLocalError if we fail to
    create the config_drive_image variable. Also, the original
    change didn't have any unit tests and came late in the
    liberty release so I don't really want to mess with fixing
    this given we need the fix in liberty-rc2.

Change-Id: Ia7b70aa139b67cf58b5c0f9fbcd2a4deb465914e
Closes-Bug: #1502961

commit ef655379445693443146f8a3ed31cabb011d9937
Author: OpenStack Proposal Bot <email address hidden>
Date: Thu Oct 8 06:41:06 2015 +0000

Imported Translations from Zanata

For more information about this automatic import see:
https://wiki.openstack.org/wiki/Translations/Infrastructure

Change-Id: Idcac653033ab9808e06451a0dd690db4736834b2

commit eda3029aa74932f421d2992ac24f5ac3c92f347c
Author: Dan Smith <email address hidden>
Date: Tue Oct 6 10:58:18 2...

Reviewed:  https://review.openstack.org/235181
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=d8a3b9d6408627ce9a293d1e62d003757af655a1
Submitter: Jenkins
Branch:    master

commit 6df6ad3ff32f2b1fe2978df1032002548ad8eb66
Author: Davanum Srinivas <davanum@gmail.com>
Date:   Wed Oct 7 08:11:35 2015 -0700

Omnibus stable/liberty fix
    
    There are currently 3 different blocking issues in stable/liberty due
    to library releases: webob 1.5, oslo.db 3.0.0, and
    oslo.versionedobjects 0.11.0. This is a squashed fix for all of them
    as none can land without the others.
    
    Issue #1 - oslo.db
    
    Add testresources used by oslo.db fixture
    
    If we use oslo.db fixtures, we'll need the package or
    the next version of oslo.db release will break us.
    
    (Cherry-picked from 4bcc26487837b7ece7797f88622dea1b6d09bd94)
    
    Closes-Bug: #1503501
    
    Issue #2 - oslo.versionedobjects
    
    Drop unused obj_to_primitive() override
    
    This was a band-aid override until o.vo gained the obj_relationships fix
    that this method overrides. That has been in place since o.vo 0.8.0, which
    means this is long since no longer necessary (and is actually blocking our
    ability to absorb bug fixes to this code in o.vo). Further, we no longer
    use this directly because we're doing backports based on version manifests,
    which means we no longer consult child_versions _or_ obj_relationships.
    
    (cherry picked from commit 142f1d9cc4ace90956c665c40b1f78795f9f7e29)
    
    Issue #3 - webob
    
    Default ConvertedException code to 500
    
    webob 1.5.0 released on 10/11 has change f6c749011 which
    strictly enforces status codes in exceptions, and 0 is not
    a valid status code so tests fail.
    
    Change the default to 500 to match the default in the parent
    class in webob.
    
    Closes-Bug: #1505153
    (cherry picked from commit 10438c0fc34bd088e018e1a5e8ec57b396528792)
    
    Change-Id: I1e06e77308a7dd23209124f0807d61fb52470188

commit 606204354b5ed96852240020769c81acda9f9fc8
Author: Matt Riedemann <mriedem@us.ibm.com>
Date:   Mon Oct 5 20:32:58 2015 +0000

Revert "[libvirt] Move cleanup of imported files to imagebackend"
    
    This reverts commit 9ba70756de326ffaa8be43acfde12cad04ed0af2
    
    The change introduced an UnboundLocalError if we fail to
    create the config_drive_image variable. Also, the original
    change didn't have any unit tests and came late in the
    liberty release so I don't really want to mess with fixing
    this given we need the fix in liberty-rc2.
    
    Change-Id: Ia7b70aa139b67cf58b5c0f9fbcd2a4deb465914e
    Closes-Bug: #1502961

commit ef655379445693443146f8a3ed31cabb011d9937
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Thu Oct 8 06:41:06 2015 +0000

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

commit eda3029aa74932f421d2992ac24f5ac3c92f347c
Author: Dan Smith <dansmith@redhat.com>
Date:   Tue Oct 6 10:58:18 2015 -0700

Fix InstanceV1 backports to use context
    
    This fixes the InstanceV1 and InstanceListV1 backport code to properly
    initialize the temporary objects with a context. Without them, the
    objects are orphaned and can't do certain things.
    
    This should be backported to liberty
    
    Closes-Bug: #1503708
    Change-Id: I537734c0f10110ac64799cc1232edbc18a16c794
    (cherry picked from commit e571d88604353a0443ae4ca24e3ce72fb4c615f2)

commit 6cc3f4f6043e83d5d180a51bac5b9254cee8245e
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Tue Oct 6 06:39:20 2015 +0000

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

commit 8af97531e3fbd2f24c091f14c3a20cacf538cff2
Author: Jim Rollenhagen <jim@jimrollenhagen.com>
Date:   Thu Oct 1 21:12:53 2015 -0700

Ironic: Fix bad capacity reporting if instance_info is unset
    
    If node.instance_info is unset for a node that has an instance on it
    (for example, when upgrading Nova from before the patch that set these
    values on spawn), the Ironic driver reports 0 for both $resource and
    $resource_used. The resource tracker will correct $resource_used, and
    resources available will be reported as a negative number.
    
    Correct this by reporting the original value if instance_info is unset.
    
    Closes-Bug: #1502177
    Change-Id: I13c5e5430fd305cd0ee2f24cd95304660ccf11eb
    (cherry picked from commit 047da6498dbb3af71bcb9e6d0e2c38aa23b06615)

commit a79e09548844fdb02b96a7f033efc0b23e7882e3
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Mon Oct 5 06:33:17 2015 +0000

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

commit 83089e7e67ca0149f9637d73448edfcba1edbf84
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Sat Oct 3 06:34:44 2015 +0000

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

commit c24bf9e469dbc2df8e8b8900015197dc7c388e8c
Author: Matt Riedemann <mriedem@us.ibm.com>
Date:   Fri Oct 2 13:00:43 2015 -0700

libvirt: check if ImageMeta.disk_format is set before accessing it
    
    Commit 4f43bbafa4bfc2ab149d5db4eb55320c1602658b changed
    blockinfo.get_root_info to access the disk_format attribute on the
    passed in image_meta (ImageMeta object) like an object, but if that
    attribute is not set you'll get an error.
    
    image_meta can be an empty object in the case of evacuate where
    image_ref is passed as None to the compute manager's rebuild_instance
    method in the case of evacuate, so image_meta defaults to {}. So when
    the ImageMeta object is created in LibvirtDriver.spawn, there is nothing
    in it.
    
    We have to check if the attribute is set before accessing it using
    obj_attr_is_set because the ImageMeta class does not include the
    NovaObjectDictCompat mixin, so we can't use
    image_meta.get('disk_format').
    
    Closes-Bug: #1501831
    
    Change-Id: I95e543415b881603bb997ae0e159017c03e58a53
    (cherry picked from commit 2b1a2bc034e371a9f6d95f48ed8fa77e0854d675)

commit 579b02163e66502d22812f91d82e65f4019bedd4
Author: Tony Breeds <tony@bakeyournoodle.com>
Date:   Fri Oct 2 12:28:42 2015 +1000

Add Pillow to test-requirements.txt
    
    Pillow isn't directly used by nova but it is used by
    sphinxcontrib-seqdiag.
    
    Add it to test-requirements as a work around to:
    http://lists.openstack.org/pipermail/openstack-dev/2015-October/076030.html
    
    Change-Id: I4face1fdb56c30bfeffca8808f422565eb21de83
    (cherry picked from commit 03cae1c43124d03e747ea51cfcbc0737bbf4287a)

commit 2dbd0512ed4e51edab2a2f2d23d759ae8da9dc0c
Author: Andreas Jaeger <aj@suse.de>
Date:   Thu Sep 24 20:39:16 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 change needs to be done manually since the automatic import does
    not handle some of these cases.
    
    Change-Id: I662f6af336947188c74ea439ebba308bd4055fb8

commit fb23e596542d6bd052f5d3c5e193ffea7bb26028
Author: Nikola Dipanov <ndipanov@redhat.com>
Date:   Wed Sep 23 19:21:16 2015 +0100

rebuild: Apply migration context before calling the driver
    
    In I0233f964d8f294f0ffd9edcb16b1aaf93486177f we made sure that we are
    properly using the data stashed in the migration context for resource
    tracking, however - we missed that we need to also apply it when passing
    data to the driver to do the spawn (or rebuild if implemented).
    
    This patch adds a context manager that helps us temporarily apply the
    migration context to an instance object, as we want to make sure that we
    don't save the new data before it's ready to be used by the resource
    tracker.
    
    Change-Id: I9a4b86e176f408c1495d5308d3e7041292c2625a
    Closes-bug: #1499028
    (cherry picked from commit 66e0744471ce491371a67ffdd8ba3c105919e255)

commit 3be67646c46f7ca58b570770074c1f269c187a30
Author: He Jie Xu <hejie.xu@intel.com>
Date:   Mon Sep 28 13:24:06 2015 +0800

Correct Instance type check to work with InstanceV1
    
    When Liberty controller running with Kilo compute node, the Kilo
    compute node will send the InstanceV1 to the Liberty controller.
    Then all the checks like "isinstance(instance, objects.Instance)"
    on the Liberty controller won't works. Because objects.Instance
    pointed to objects.InstanceV2 now.
    
    This patch corrects all the Instance type check to make sure the
    upgrade works.
    
    Change-Id: Ib431c08dc6934631a71ef55c564fbbf4bde22642
    Closes-bug: #1500289
    (cherry picked from commit 093272489ceb13d4df26916c7ed8b66e39f7c1a4)

commit 51d74bdf18df39695ad5e500db6db830f21da36a
Author: Yingxin Cheng <yingxin.cheng@intel.com>
Date:   Wed Sep 9 11:20:19 2015 -0400

Fix MetricWeigher to use MonitorMetricList
    
    The commit I4dfbea27ce6c3eecc1a8658b1f9dc0feb2298705 changes
    'HostState.metrics' to 'objects.MonitorMetricList'. But MetricWeigher
    still use the old dictionary of 'host_manager.MetricItem'. This patch
    corrects the behavior of MetricWeigher with related tests, and delete
    unused MetricItem.
    
    Co-Authored-By: Sylvain Bauza <sbauza@redhat.com>
    
    Closes-Bug: #1493680
    (cherry picked from commit fd2b868d41a632748939b82ed1b8927adcdfb113)
    
    Fix NoneType error when calling MetricsWeigher
    
    (this is a squashed commit since commit fd2b868 was buggy)
    Given that dict comprehensions don't allow to have None as a list, we had
    a problem with the Ironic gate-ironic-inspector-dsvm job because
    host_state.metrics was set to None by default.
    
    Adding a unittest to make sure we don't regress.
    
    Closes-Bug: #1499271
    (cherry picked from commit ccea5d6b0ace535b375d3e63bd572885cb5dbc91)
    
    Change-Id: I9690bc4f6f1dca8f2bf8c5a83f16af6200015506

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-02: Fix included in openstack/glance 12.0.0.0b1

#51

This issue was fixed in the openstack/glance 12.0.0.0b1 development milestone.

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-02: Fix included in openstack/nova 13.0.0.0b1

#52

This issue was fixed in the openstack/nova 13.0.0.0b1 development milestone.

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-03: Fix included in openstack/heat 6.0.0.0b1

#53

This issue was fixed in the openstack/heat 6.0.0.0b1 development milestone.

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-03: Fix included in openstack/cinder 8.0.0.0b1

#54

This issue was fixed in the openstack/cinder 8.0.0.0b1 development milestone.

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-03: Fix included in openstack/neutron 8.0.0.0b1

#55

This issue was fixed in the openstack/neutron 8.0.0.0b1 development milestone.

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-03: Fix included in openstack/neutron-vpnaas 8.0.0.0b1

#56

This issue was fixed in the openstack/neutron-vpnaas 8.0.0.0b1 development milestone.

Revision history for this message

Doug Hellmann (doug-hellmann) wrote on 2015-12-03: Fix included in openstack/sahara 4.0.0.0b1

#57

This issue was fixed in the openstack/sahara 4.0.0.0b1 development milestone.

Changed in sahara:
status:	Fix Committed → Fix Released

Doug Hellmann (doug-hellmann) on 2015-12-03

Changed in heat:
status:	Fix Committed → Fix Released

Doug Hellmann (doug-hellmann) on 2015-12-04

Changed in ironic:
status:	Fix Committed → Fix Released

Revision history for this message

Doug Hellmann (doug-hellmann) wrote on 2015-12-07: Fix included in openstack/ironic 4.3.0

#58

This issue was fixed in the openstack/ironic 4.3.0 release.

Revision history for this message

Doug Hellmann (doug-hellmann) wrote on 2016-01-20: Fix included in openstack/heat 5.0.1

#59

This issue was fixed in the openstack/heat 5.0.1 release.

Revision history for this message

Doug Hellmann (doug-hellmann) wrote on 2016-01-25: Fix included in openstack/sahara 3.0.1

#60

This issue was fixed in the openstack/sahara 3.0.1 release.

Ironic

oslo.db no longer requires testresources and testscenarios packages

Bug Description

Duplicates of this bug

Other bug subscribers

Remote bug watches

	Status	Importance	Assigned to	Milestone
Cinder	Fix Released	Undecided	Davanum Srinivas (DIMS)	Cinder 7.0.0 "liberty"
Glance	Fix Released	High	Mike Fedosin	Glance 11.0.0 "liberty"
Ironic	Fix Released	High	Lucas Alvares Gomes
OpenStack Compute (nova)	Fix Released	High	Davanum Srinivas (DIMS)	OpenStack Compute (nova) 12.0.0 "liberty"
OpenStack Heat	Fix Released	High	Sergey Kraynev	OpenStack Heat mitaka-1 "m1"
Liberty	Fix Released	Undecided	Unassigned
Sahara	Fix Released	High	Michael McCune	Sahara mitaka-1 "m1"
Liberty	Fix Released	High	Sergey Lukjanov
Mitaka	Fix Released	High	Michael McCune	Sahara mitaka-1 "m1"
neutron	Fix Released	High	Davanum Srinivas (DIMS)	neutron 7.0.0 "liberty"