← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1698779] Re: aggregate related notification samples are missing from the notification dev-ref

 

Reviewed:  https://review.openstack.org/475349
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=5a35a2b99e5ae81937a6a0cbf3d4274ff385a5e6
Submitter: Jenkins
Branch:    master

commit 5a35a2b99e5ae81937a6a0cbf3d4274ff385a5e6
Author: Balazs Gibizer <balazs.gibizer@xxxxxxxxxxxx>
Date:   Mon Jun 19 11:42:49 2017 +0200

    add missing notification samples to dev ref
    
    The already transformed aggregate notifications were missing from the
    notification dev-ref due to a missing import in the doc generation code.
    
    This patch improves the doc generation to automatically pick up new modules
    added under nova.notitications.objects.
    
    Change-Id: I688bc59f19814956d73e04b4970add21ce64ba90
    Closes-Bug: #1698779


** 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/1698779

Title:
  aggregate related notification samples are missing from the
  notification dev-ref

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  The list in the devref [1] shall contain the already transformed
  aggregate notifications [2] but it doesn't due to a missing import in
  the doc generation code [3].

  As an improvement the doc generation shall automatically pick up the
  new modules under nova.notifications.objects to avoid similar
  regression in the future.

  [1] https://docs.openstack.org/developer/nova/notifications.html#existing-versioned-notifications
  [2] https://github.com/openstack/nova/blob/master/nova/notifications/objects/aggregate.py#L45
  [3] https://github.com/openstack/nova/blob/master/doc/ext/versioned_notifications.py#L35

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


References