Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'Client.list_metrics' API wrapper. #1590

Merged
merged 7 commits into from
Mar 14, 2016
Merged

Add 'Client.list_metrics' API wrapper. #1590

merged 7 commits into from
Mar 14, 2016

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Mar 8, 2016

Also, add 'Client.metric' factory.

Uses #1588 as a base.

@tseaver tseaver added the api: logging Issues related to the Cloud Logging API. label Mar 8, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 8, 2016
@tseaver
Copy link
Contributor Author

tseaver commented Mar 14, 2016

@dhermes Rebased after merging #1588 to the logging-api branch. PTAL

@@ -79,7 +80,8 @@ def _entry_from_resource(self, resource, loggers):
raise ValueError('Cannot parse job resource')

def list_entries(self, projects=None, filter_=None, order_by=None,
page_size=None, page_token=None):
page_size=None,
page_token=None):

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@@ -112,6 +114,7 @@ def list_entries(self, projects=None, filter_=None, order_by=None,
more topics can be retrieved with another call (pass that
value as ``page_token``).
"""
# pylint: disable=too-many-branches

This comment was marked as spam.

This comment was marked as spam.

@tseaver
Copy link
Contributor Author

tseaver commented Mar 14, 2016

@dhermes PTAL.

@dhermes
Copy link
Contributor

dhermes commented Mar 14, 2016

LGTM

tseaver added a commit that referenced this pull request Mar 14, 2016
@tseaver tseaver merged commit 5648192 into googleapis:logging-api Mar 14, 2016
@tseaver tseaver deleted the logging-client_list_metrics branch March 14, 2016 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: logging Issues related to the Cloud Logging API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants