diff options
author | Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp> | 2015-04-07 23:35:55 +0000 |
---|---|---|
committer | Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp> | 2015-04-13 00:07:06 +0000 |
commit | 976a3294f5932c6e83499599da2e16439eef49e7 (patch) | |
tree | 40f8a2fce3c8145d354fb4d181fdc3691dfeffa8 | |
parent | 7e3ccd5341b165199bea14f65d2e758ee9bfc0eb (diff) | |
download | tempest-976a3294f5932c6e83499599da2e16439eef49e7.tar.gz |
Rename an internal method to _helper_list()
helper_list() is just a helper method for "list_<resoruces>" of
telemetry_client and it is used in telemetry_client only.
telemetry_client will be migrated to tempest-lib and we need to
clarify whether each method is external or internal.
This patch renames the method for clarifying an internal method.
Partially implements blueprint consistent-service-method-names
Change-Id: I12922cf1ca968eb1d9d793879aa22921cc39a56b
-rw-r--r-- | tempest/services/telemetry/json/telemetry_client.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tempest/services/telemetry/json/telemetry_client.py b/tempest/services/telemetry/json/telemetry_client.py index 36c123b32..070309d3d 100644 --- a/tempest/services/telemetry/json/telemetry_client.py +++ b/tempest/services/telemetry/json/telemetry_client.py @@ -50,7 +50,7 @@ class TelemetryClientJSON(service_client.ServiceClient): body = self.deserialize(body) return service_client.ResponseBody(resp, body) - def helper_list(self, uri, query=None, period=None): + def _helper_list(self, uri, query=None, period=None): uri_dict = {} if query: uri_dict = {'q.field': query[0], @@ -67,23 +67,23 @@ class TelemetryClientJSON(service_client.ServiceClient): def list_resources(self, query=None): uri = '%s/resources' % self.uri_prefix - return self.helper_list(uri, query) + return self._helper_list(uri, query) def list_meters(self, query=None): uri = '%s/meters' % self.uri_prefix - return self.helper_list(uri, query) + return self._helper_list(uri, query) def list_alarms(self, query=None): uri = '%s/alarms' % self.uri_prefix - return self.helper_list(uri, query) + return self._helper_list(uri, query) def list_statistics(self, meter, period=None, query=None): uri = "%s/meters/%s/statistics" % (self.uri_prefix, meter) - return self.helper_list(uri, query, period) + return self._helper_list(uri, query, period) def list_samples(self, meter_id, query=None): uri = '%s/meters/%s' % (self.uri_prefix, meter_id) - return self.helper_list(uri, query) + return self._helper_list(uri, query) def get_resource(self, resource_id): uri = '%s/resources/%s' % (self.uri_prefix, resource_id) |