craton

PUT requests for the user and project resources not implemented

Bug #1655940 reported by git-harry
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
craton
In Progress
Undecided
Syed Ahsan Shamim Zaidi

Bug Description

Support for PUT requests was added to the classes ProjectById and UserById however the methods return 200 without making and changes [1].

Given that these methods are at best likely to cause users confusion, I will submit a patch to remove them. It seems most likely they were added to keep track of functionality that needs implementing, I think it would be better to use this bug report for that purpose.

[1] https://github.com/openstack/craton/commit/02439918

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to craton (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/419437

Revision history for this message
Syed Ahsan Shamim Zaidi (ahsanmohsin04) wrote :

Thanks harry for raising this. Just confirmed.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to craton (master)

Reviewed: https://review.openstack.org/419437
Committed: https://git.openstack.org/cgit/openstack/craton/commit/?id=550bc43baa86498f34d11ee377ef1846609b4738
Submitter: Jenkins
Branch: master

commit 550bc43baa86498f34d11ee377ef1846609b4738
Author: git-harry <email address hidden>
Date: Thu Jan 12 12:03:55 2017 +0000

    Remove methods not properly implemented

    PUT requests for users and projects report success without performing
    any actions. This commit removes them to prevent any confusion caused by
    their use. The related bug is being used to track their absense so that
    the work to implement them is not missed.

    Change-Id: I1bbd3403c1991d0b13509c97d8983d60541b7136
    Related-Bug: 1655940

Syed Ahsan Shamim Zaidi (ahsanmohsin04)
Changed in craton:
assignee: nobody → Syed Ahsan Shamim Zaidi (ahsanmohsin04)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to craton (master)

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

Changed in craton:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on craton (master)

Change abandoned by Syed Ahsan Shamim Zaidi (<email address hidden>) on branch: master
Review: https://review.openstack.org/425463

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

Other bug subscribers

Remote bug watches

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