Squashed and pushed to
[master](http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/47c4e72a)
and
[2.0.x](http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/4183e2f9).
I also changed the metrics mock test so it uses the date service to parse dates
(see
Closed #396.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/396#event-1141860292
@danielestevez pushed 1 commit.
d353bbc Returns numeric data as Double/Long
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
nacx commented on this pull request.
> +/**
+ *
+ */
+@AutoValue
+public abstract class MetricData {
+
+ /**
+* The timestamp for the metric value in ISO 8601 format.
+*/
+ public abstract Date timeStamp();
+
+ /**
+* The average value in the time range
+*/
+ @Nullable
+
danielestevez commented on this pull request.
> +/**
+ *
+ */
+@AutoValue
+public abstract class MetricData {
+
+ /**
+* The timestamp for the metric value in ISO 8601 format.
+*/
+ public abstract Date timeStamp();
+
+ /**
+* The average value in the time range
+*/
+
@danielestevez pushed 1 commit.
9312229 Addresses comments by @nacx
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/396/files/44cd5b8fe62b6bae060b3995c438533cfc249dc2..9312229b35c40debb33e66e324958102348dc751
danielestevez commented on this pull request.
> + * A Metric definition for a resource
+ */
+@AutoValue
+public abstract class MetricDefinition {
+
+ @Nullable
+ public abstract String resourceid();
+
+ public abstract MetricDefinition.MetricName name();
+
+ @Nullable
+ public
danielestevez commented on this pull request.
> +/**
+ *
+ */
+@AutoValue
+public abstract class MetricData {
+
+ /**
+* The timestamp for the metric value in ISO 8601 format.
+*/
+ public abstract Date timeStamp();
+
+ /**
+* The average value in the time range
+*/
+
danielestevez commented on this pull request.
> +* The metrics API includes operations to get insights into entities
> within your
+* subscription.
+*
+* @see https://docs.microsoft.com/en-us/rest/api/monitor/metrics;>docs
+*/
+ @Delegate
+ MetricsApi
nacx requested changes on this pull request.
Thanks @danielestevez!
> +* The metrics API includes operations to get insights into entities
> within your
+* subscription.
+*
+* @see https://docs.microsoft.com/en-us/rest/api/monitor/metrics;>docs
+*/
+ @Delegate
+
- https://docs.microsoft.com/en-us/rest/api/monitor/metrics
- https://docs.microsoft.com/en-us/rest/api/monitor/metricdefinitions
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-labs/pull/396
-- Commit Summary --
* Implements metrics and
11 matches
Mail list logo