← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1723123] Re: missing coverage for trying to update a standard resource class in placement api

 

Reviewed:  https://review.openstack.org/511485
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=6506943a03d857f851c7b34e68d488c5e74218dc
Submitter: Zuul
Branch:    master

commit 6506943a03d857f851c7b34e68d488c5e74218dc
Author: Chris Dent <cdent@xxxxxxxxxxxxx>
Date:   Thu Oct 12 15:01:02 2017 +0100

    [placement] add coverage for update of standard resource class
    
    In microversion <= 1.6 it is possible to update a resource class to have
    a new name. This is only possible for custom resource classes, but there
    was no test to show that the proper 400 was returned when attempting to
    change a standard class.
    
    This adds one test to cover it, and in the process changes the name of
    a related test to distinguish it from the new one.
    
    Change-Id: I6c665db68de91c0de91c16f7e622f0c7e2f28bc2
    Closes-Bug: #1723123


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1723123

Title:
  missing coverage for trying to update a standard resource class in
  placement api

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  This is a note to self for sake of bookkeeping.

  Coverage reports show that there's zero gabbi coverage for a request
  that attempts to PUT to update a standard resource class. This is easy
  to fix, so may as well.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1723123/+subscriptions


References