You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@deltacloud.apache.org by "Michal Fojtik (JIRA)" <ji...@apache.org> on 2012/07/12 11:53:33 UTC

[jira] [Commented] (DTACLOUD-269) FGCP/mock: support metrics

    [ https://issues.apache.org/jira/browse/DTACLOUD-269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13412648#comment-13412648 ] 

Michal Fojtik commented on DTACLOUD-269:
----------------------------------------

ACK both. 

One through for the 'mock' one:

You don't need to use @@ for the METRIC_NAMES constant. The '@@' sign is used for class-variables, but the METRIC_NAMES is a constant not a variable. This has no implications or does not cause any bug, just a coding style through.

Btw. the metrics for mock looks super awesome, thanks for contributing this. We should also write couple tests for this into 'server/tests/drivers/mock'.
                
> FGCP/mock: support metrics
> --------------------------
>
>                 Key: DTACLOUD-269
>                 URL: https://issues.apache.org/jira/browse/DTACLOUD-269
>             Project: DeltaCloud
>          Issue Type: Improvement
>          Components: Server
>         Environment: FGCP and mock drivers
>            Reporter: Dies Koper
>            Priority: Minor
>         Attachments: 0001-mock-added-metrics-support.patch, 0002-FGCP-added-metrics-support.patch
>
>
> add support for metrics

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira