[nailgun] Removed handlers from stable API don't return right http code and message

Bug #1335081 reported by Evgeniy L
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Ivan Kliuk
5.1.x
Won't Fix
Medium
Fuel Python (Deprecated)
6.0.x
Fix Committed
Medium
Ivan Kliuk

Bug Description

We have stable api v1, but we had to remove [1] some unsupported functional, the best what we can do in this case is to return right http code 410 Gone and information in response, that resource was removed in version of fuel (5.1) and commit 703b34cbd6eca81dc7ab18876ca21daa76c4cfe7

[1] https://review.openstack.org/#/c/85045/25/nailgun/nailgun/api/v1/urls.py

Dmitry Ilyin (idv1985)
summary: - Removed handlers from stable API don't return right http code and
- message
+ [nailgun] Removed handlers from stable API don't return right http code
+ and message
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 5.1 → 6.0
Ivan Kliuk (ivankliuk)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Ivan Kliuk (ivankliuk)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/126418
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=385914993c0d27a2a182d772da184224e99b0456
Submitter: Jenkins
Branch: master

commit 385914993c0d27a2a182d772da184224e99b0456
Author: Ivan Kliuk <email address hidden>
Date: Sat Oct 4 23:06:40 2014 +0300

    Add handler for removed services

        BaseRemovedInHandler base handler added for generating correct
        response 410 (Gone) for services which were removed in a certain version
        of Fuel.
        BaseRemovedIn51Handler handler implements the response for Fuel 5.1.

    Change-Id: I67505ec6f8dca33a7ebeabd82849b7c7162815e3
    Closes-Bug: #1335081

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Mike Scherbakov (mihgen) wrote :

If it's Medium priority bug, are you sure we truly want it to be backported into stable/5.1 (5.1.1 upcoming release)?

We are backporting bugfixes for Critical bugs, and for many High priority, not sure if we should backport any Medium priority ones.

Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Set to won't fix. I thought it leads to development documentation build errors but it is all ok with doc.

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.