volume usage notifications should include billing period identification

Bug #1259147 reported by Ollie Leahy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Ollie Leahy

Bug Description

Currently volume exists, volume create and volume delete notification messages do not include information identifying the audit period with which they should be associated.

This means that the systems consuming these records must assume that they refer to the audit period in which they were generated. The utility function notify_about_volume_usage() provides a mechanism to define the audit period start and end times, the cinder functions that generate these notifications should use this mechanism so that records can be applied to the correct audit period.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
assignee: nobody → Ollie Leahy (oliver-leahy-l)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/61843
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=d09b7c78b3c905badb1112434eb9c09c2c8ffb5a
Submitter: Jenkins
Branch: master

commit d09b7c78b3c905badb1112434eb9c09c2c8ffb5a
Author: Ollie Leahy <email address hidden>
Date: Thu Dec 12 11:01:34 2013 +0000

    Allow user to specify audit period.

    The script 'cinder-volume-usage-audit' is generally run by cron to
    generate volume and snapshot exists audit notifications that can be
    consumed by ceilometer or other auditing systems. In this mode it sends
    notifications for volumes that existed in the 'last completed audit
    period' as calculated by cinder.utils.last_completed_audit_period()

    However, it may sometimes be necessary to be able to send notifications
    for audit periods other than than the last completed audit period. For
    example if the host running the audit script was down for a period.

    This patch provides three optional command line parameters to the script
    cinder-volume-usage-audit. If no parameters are provided then the
    operation of the script is unchanged. If parameters are provided then
    they have the following effects:

    --start_time specifies a date and time that is used in place of the
    start of the last completed audit period.

    --end_time specifies a date and time that is used in place of the end of
    the last completed audit period.

    --send_actions As well as sending exits notifications the script will
    send any create or delete notifications that would have been sent during
    the specified audit period.

    Closes-Bug: #1259147

    Change-Id: I0cfe963ea59d811ba4bac0bd3de31382a088b8c8

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → icehouse-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-2 → 2014.1
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.