OpenStack Ceilometer API's support (Skeleton & meters/samples API)
open issues I need help with -
1. meters and samples can get query params, problem is they cant be sent as
null/empty string since ceilometer fails. example - {"filter":
"", "orderby": "", "limit":
""}
need a way to avoid sending empty query params
2. for meters API -
currently I've created "MetersApi.listMeters(QueryOptions...
options)" should I follow this notion or simplify it as
"MetersApi.listMeters(String filter)" where filter is a json filter
as in item #1
You can merge this Pull Request by running:
git pull https://github.com/avibh/jclouds-labs-openstack ceilometer-support
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds-labs-openstack/pull/166
-- Commit Summary --
* OS Ceilometer API support
-- File Changes --
M README.md (1)
A openstack-ceilometer/README.md (15)
A openstack-ceilometer/pom.xml (145)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/CeilometerApi.java
(53)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/CeilometerApiMetadata.java
(90)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/config/CeilometerHttpApiModule.java
(44)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/domain/Meter.java
(199)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/domain/Sample.java
(259)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/features/MetersApi.java
(59)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/features/QueryApi.java
(62)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/handlers/CeilometerErrorHandler.java
(52)
A
openstack-ceilometer/src/main/java/org/jclouds/openstack/ceilometer/v2/options/QueryOptions.java
(102)
A
openstack-ceilometer/src/main/resources/META-INF/services/org.jclouds.apis.ApiMetadata
(18)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/CeilometerApiMetadataTest.java
(34)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/features/MetersApiLiveTest.java
(72)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/features/MetersApiMockTest.java
(116)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/features/SamplesApiLiveTest.java
(74)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/features/SamplesApiMockTest.java
(116)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/internal/BaseCeilometerApiLiveTest.java
(44)
A
openstack-ceilometer/src/test/java/org/jclouds/openstack/ceilometer/v2/internal/BaseCeilometerApiMockTest.java
(39)
A openstack-ceilometer/src/test/resources/access.json (242)
A openstack-ceilometer/src/test/resources/logback.xml (56)
A openstack-ceilometer/src/test/resources/meters_list_query_response.json
(6502)
A openstack-ceilometer/src/test/resources/meters_list_response.json (6502)
A openstack-ceilometer/src/test/resources/samples_list_response.json (337)
M pom.xml (1)
-- Patch Links --
https://github.com/jclouds/jclouds-labs-openstack/pull/166.patch
https://github.com/jclouds/jclouds-labs-openstack/pull/166.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/166
