HTTPBadRequest returns to the client when creating a existing default rule

Bug #1374320 reported by Haiwei Xu
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Abhishek Talwar

Bug Description

$ nova secgroup-add-default-rule icmp -1 -1 0.0.0.0/0

ERROR (BadRequest): This default rule already exists. (HTTP 400) (Request-ID: req-11ee577a-03f2-49c1-b033-4dd1df57ff1a)

while HTTP 409 should be returned.

Haiwei Xu (xu-haiwei)
Changed in nova:
assignee: nobody → Haiwei Xu (xu-haiwei)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Changed in nova:
importance: Undecided → Low
Changed in nova:
assignee: Haiwei Xu (xu-haiwei) → Abhishek Talwar (abhishek-talwar)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

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

commit 71efc3dc56f3f5cb201a0c903a706f1d4288d1e2
Author: abhishek.talwar <email address hidden>
Date: Tue Dec 2 17:03:33 2014 +0530

    Error code for creating secgroup default rule

    Trying to create secgroup group default rules
    that already exists gives HttpBadRequest error,
    but it should raise HttpConflict.

    So updated the code to HttpConflict error.
    Closes-Bug: #1374320

    Change-Id: I93c6eb7fb96e7109404206367ed81495a4724296

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Changed in nova:
assignee: Abhishek Talwar (abhishek-talwar) → tcs_openstack_group (tcs-openstack-group)
assignee: tcs_openstack_group (tcs-openstack-group) → Abhishek Talwar (abhishek-talwar)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/124326
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Thierry Carrez (ttx)
Changed in nova:
milestone: kilo-1 → 2015.1.0
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.