yahoo-eng-team team mailing list archive

[Ed Leafe <ed@xxxxxxxxx>]

Public bug reported:

The recent change to validators.py to allow for varying scheduler hints
added the parameter 'additional_properties_value', which strongly
affects the behavior of the method, but this behavior was not explained
in the method's docstring. It was felt that we should merge the change
and fix this later, so this bug report is to ensure that we don't
forget.

https://github.com/openstack/nova/blob/master/nova/api/validation/validators.py#L78-L98

** Affects: nova
     Importance: Low
     Assignee: Ed Leafe (ed-leafe)
         Status: New

** Changed in: nova
   Importance: Undecided => Low

** Changed in: nova
     Assignee: (unassigned) => Ed Leafe (ed-leafe)

-- 
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/1494289

Title:
  Docstring for validators._soft_validate_additional_properties is
  incomplete

Status in OpenStack Compute (nova):
  New

Bug description:
  The recent change to validators.py to allow for varying scheduler
  hints added the parameter 'additional_properties_value', which
  strongly affects the behavior of the method, but this behavior was not
  explained in the method's docstring. It was felt that we should merge
  the change and fix this later, so this bug report is to ensure that we
  don't forget.

  https://github.com/openstack/nova/blob/master/nova/api/validation/validators.py#L78-L98

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