openstack/cloudkitty
Code Issues Proposed changes

Deprecate /v1/report/total endpoint

Browse Source 
Change-Id: I559304c6da7b78e83821a7d164cc12faf2e7194c
Task: 6586
Story: 2001612
This commit is contained in:
Luka Peschke 2018-03-07 11:44:37 +01:00
parent acf1978421
commit ae1a473dc7
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cloudkitty/api/v1/controllers/report.py
View File

@ -18,6 +18,7 @@
import datetime import datetime
import decimal import decimal
from oslo_log import log as logging
import pecan import pecan
from pecan import rest from pecan import rest
from wsme import types as wtypes from wsme import types as wtypes
@ -28,6 +29,9 @@ from cloudkitty.common import policy
from cloudkitty import utils as ck_utils from cloudkitty import utils as ck_utils
LOG = logging.getLogger(__name__)
class ReportController(rest.RestController): class ReportController(rest.RestController):
"""REST Controller managing the reporting. """REST Controller managing the reporting.
@ -68,6 +72,8 @@ class ReportController(rest.RestController):
"""Return the amount to pay for a given period. """Return the amount to pay for a given period.
""" """
LOG.warning('/v1/report/total is deprecated, please use '
'/v1/report/summary instead.')
if not begin: if not begin:
begin = ck_utils.get_month_start() begin = ck_utils.get_month_start()
if not end: if not end:

5
releasenotes/notes/deprecate-report-total-62544dce42bb19a6.yaml Normal file
View File

@ -0,0 +1,5 @@
---
deprecations:
- |
The /v1/report/total route has been deprecated. /v1/report/summary should
be used instead.