contrail raises wrong exception when reaching quota limit

Bug #1590930 reported by sajuptpm
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.1
In Progress
Undecided
Unassigned
R2.20
Fix Committed
Undecided
Unassigned
R3.0
Fix Committed
Undecided
Unassigned
R3.1
Fix Committed
Undecided
Unassigned
R3.2
Fix Committed
Undecided
Unassigned
Trunk
Fix Committed
Undecided
Unassigned
OpenContrail
New
Undecided
Unassigned

Bug Description

contrail raises PermissionDenied exception on quota hit.
contrail should raise OverQuota exception on quota hit.

Tags: api config
sajuptpm (sajuptpm)
tags: added: config
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/21057
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.1

Review in progress for https://review.opencontrail.org/21058
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/21059
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/21060
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.1

Review in progress for https://review.opencontrail.org/21062
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/21063
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/21064
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/21065
Submitter: sajuptpm (<email address hidden>)

sajuptpm (sajuptpm)
description: updated
tags: added: api
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/21059
Committed: http://github.org/Juniper/contrail-controller/commit/2641107ce4de05167446d532f76692fed5de8ea3
Submitter: Zuul
Branch: R2.20

commit 2641107ce4de05167446d532f76692fed5de8ea3
Author: Saju Madhavan <email address hidden>
Date: Fri Jun 10 01:27:26 2016 +0530

contrail raises wrong quota exception

I added a new exception named OverQuota to raise on
quota hit.Also added new status code 412 to catch
and raise new quota exception.I also have path in
contrail-neutron-plugin to catch this new OverQuota
exception.

Change-Id: I02fdf80b04999f995ab34d00c4d18605648229d6
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/21057
Committed: http://github.org/Juniper/contrail-controller/commit/7064505f322c95b12704a2daba412ffb4ec008b2
Submitter: Zuul
Branch: master

commit 7064505f322c95b12704a2daba412ffb4ec008b2
Author: Saju Madhavan <email address hidden>
Date: Fri Jun 10 00:45:48 2016 +0530

contrail raises wrong quota exception

I added a new exception named OverQuota to raise on
quota hit.Also added new status code 412 to catch
and raise new quota exception.I also have path in
contrail-neutron-plugin to catch this new OverQuota
exception.

Change-Id: I6f0e6844006c5822c28fd92af73e22fbbfac4d2f
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/21060
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/21293
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/21295
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/21063
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/21295
Committed: http://github.org/Juniper/contrail-controller/commit/803886a1d2de384d5aba6cf34404a07be10295d8
Submitter: Zuul
Branch: R2.20

commit 803886a1d2de384d5aba6cf34404a07be10295d8
Author: Saju Madhavan <email address hidden>
Date: Mon Jun 20 15:44:29 2016 +0530

standardize quota error message

send overs=[resource_type] to match standard neutron
implementation of OverQuota exception

Change-Id: I4da2f1a63891ae37a3aef81db484ea4898487c3c
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/21063
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/223b9ff3e64fe8d6090a01baa5fa7b0ab9b898c6
Submitter: Zuul
Branch: R2.20

commit 223b9ff3e64fe8d6090a01baa5fa7b0ab9b898c6
Author: Saju Madhavan <email address hidden>
Date: Fri Jun 10 01:54:02 2016 +0530

contrail raises wrong quota exception

This patch reraise OverQuota exception which
come from contrail.

In kilo and Liberty, OverQuota exception is defined
in neutron/common/exceptions.py.In Mitaka, it moved
to neutron_lib project neutron_lib/exceptions.py#L212

Change-Id: I92a8f36709f0f44eece2cff7afa5fcde2edc413e
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/21060
Committed: http://github.org/Juniper/contrail-controller/commit/186aeb722b2d83aef9a0a6fc4791121018edcadd
Submitter: Zuul
Branch: R3.0

commit 186aeb722b2d83aef9a0a6fc4791121018edcadd
Author: Saju Madhavan <email address hidden>
Date: Fri Jun 10 00:45:48 2016 +0530

contrail raises wrong quota exception

I added a new exception named OverQuota to raise on
quota hit.Also added new status code 412 to catch
and raise new quota exception.I also have path in
contrail-neutron-plugin to catch this new OverQuota
exception.

Change-Id: I6f0e6844006c5822c28fd92af73e22fbbfac4d2f
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/21293
Committed: http://github.org/Juniper/contrail-controller/commit/2ff4a038cdf34661fe6b629fa3295a6f10991077
Submitter: Zuul
Branch: master

commit 2ff4a038cdf34661fe6b629fa3295a6f10991077
Author: Saju Madhavan <email address hidden>
Date: Mon Jun 20 15:32:20 2016 +0530

standardize quota error message

send overs=[resource_type] to match standard neutron
implementation of OverQuota exception

Change-Id: I4286fd3fc4c8687beb175de97d5d87ea51ea1f0c
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/21414
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/21415
Submitter: sajuptpm (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/21415
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/534e7ddb4e7576f9fdff872307078da163ac302b
Submitter: Zuul
Branch: master

commit 534e7ddb4e7576f9fdff872307078da163ac302b
Author: Saju Madhavan <email address hidden>
Date: Fri Jun 24 16:31:58 2016 +0530

contrail raises wrong quota exception

This patch reraise OverQuota exception which
come from contrail.

In kilo and Liberty, OverQuota exception is defined
in neutron/common/exceptions.py.In Mitaka, it moved
to neutron_lib project neutron_lib/exceptions.py#L212

Change-Id: I1d3615f073fd3352b74a522604601c02f34bd814
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/21414
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/3fc7fee3a44acb0f2c8668003a2f568de57dedf9
Submitter: Zuul
Branch: R3.0

commit 3fc7fee3a44acb0f2c8668003a2f568de57dedf9
Author: Saju Madhavan <email address hidden>
Date: Fri Jun 24 16:31:23 2016 +0530

contrail raises wrong quota exception

This patch reraise OverQuota exception which
come from contrail.

In kilo and Liberty, OverQuota exception is defined
in neutron/common/exceptions.py.In Mitaka, it moved
to neutron_lib project neutron_lib/exceptions.py#L212

Change-Id: I1528ceab8607cad1bd88ea0e3230d5751f32e9b5
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/28619
Submitter: Antoine Eiche (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/28619
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/1051ad5831621c07b39f5b20d5f6376973e9b7b8
Submitter: Zuul (<email address hidden>)
Branch: master

commit 1051ad5831621c07b39f5b20d5f6376973e9b7b8
Author: Antoine Eiche <email address hidden>
Date: Thu Feb 9 16:23:37 2017 +0100

Propagate OverQuota exception

Change-Id: I87479c2b88aff7f91295ae9c7fea32dd28eb5ce7
Closes-Bug: #1590930

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.2

Review in progress for https://review.opencontrail.org/28705
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/28705
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/097b73ea25dbecb13c7021ba4fcbb51e4becd83c
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit 097b73ea25dbecb13c7021ba4fcbb51e4becd83c
Author: Antoine Eiche <email address hidden>
Date: Thu Feb 9 16:23:37 2017 +0100

Propagate OverQuota exception

Change-Id: I87479c2b88aff7f91295ae9c7fea32dd28eb5ce7
Closes-Bug: #1590930
(cherry picked from commit 1051ad5831621c07b39f5b20d5f6376973e9b7b8)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/28760
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/28760
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/0974b09ff8645cff4c297069a2a98f2b1c3b8d16
Submitter: Zuul (<email address hidden>)
Branch: R3.1

commit 0974b09ff8645cff4c297069a2a98f2b1c3b8d16
Author: Antoine Eiche <email address hidden>
Date: Thu Feb 9 16:23:37 2017 +0100

Propagate OverQuota exception

Change-Id: I87479c2b88aff7f91295ae9c7fea32dd28eb5ce7
Closes-Bug: #1590930
(cherry picked from commit 1051ad5831621c07b39f5b20d5f6376973e9b7b8)

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

Other bug subscribers

Remote bug watches

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