Concerns about POST /v2/meters API

Bug #1487297 reported by Can Zhang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Won't Fix
Low
Unassigned

Bug Description

After some discussion with @lianhao-lu about the current implementation of POST /v2/meters, there're some potential problems, listed below:

1. When `direct` is set to false, API server would send the samples to the message queue, which is asynchronous. So the HTTP status code should be 202(Accepted) instead of 201(Created)
2. When notification agent receives those samples, the pipeline rules apply and the outcome might not be what users wanted. Some samples might be lost while others might be changed. This is not the original intention of this API.
3. (Related to bug https://bugs.launchpad.net/ceilometer/+bug/1426426) When multiple samples are posted, the HTTP location header is difficult to determine because only one URI can be included in the header. This can be worked around by using `link` section in response body, but it requires API schema modifications.
4. On asynchronous samples creation, we should provide a way for users to check if samples are created successfully. One possible way, is also providing links to /v2/samples/<sample-id>

Can Zhang (acme-ican)
Changed in ceilometer:
assignee: nobody → Can Zhang (acme-ican)
status: New → In Progress
Can Zhang (acme-ican)
description: updated
Revision history for this message
gordon chung (chungg) wrote :

just an fyi, but regarding item2, that is the original intention of API posting, it was always going through the pipeline.

Lianhao Lu (lianhao-lu)
Changed in ceilometer:
importance: Undecided → Low
gordon chung (chungg)
Changed in ceilometer:
assignee: Can Zhang (acme-ican) → nobody
status: In Progress → Triaged
guolei (guolei-5)
Changed in ceilometer:
assignee: nobody → guolei (guolei-5)
guolei (guolei-5)
Changed in ceilometer:
assignee: guolei (guolei-5) → nobody
Changed in ceilometer:
assignee: nobody → PeterZhang (zhangshengping2012)
Changed in ceilometer:
assignee: PeterZhang (zhangshengping2012) → nobody
Changed in ceilometer:
assignee: nobody → fengchaoyang (fengchaoyang)
Changed in ceilometer:
assignee: fengchaoyang (fengchaoyang) → nobody
Revision history for this message
Mehdi Abaakouk (sileht) wrote :

This API is deprecated in Octava

Changed in ceilometer:
status: Triaged → Won't Fix
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.