FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add 'Sink.reload' API wrapper. by tseaver · Pull Request #1598 · googleapis/google-cloud-python · GitHub

Add 'Sink.reload' API wrapper. - #1598

Merged
tseaver merged 1 commit into
googleapis:logging-apifrom
tseaver:logging-sink_reload
Mar 11, 2016
Merged

Add 'Sink.reload' API wrapper.#1598
tseaver merged 1 commit into
googleapis:logging-apifrom
tseaver:logging-sink_reload

Conversation

tseaver commented Mar 10, 2016

Copy link
Copy Markdown
Contributor

Uses #1597 as a base.

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

tseaver commented Mar 11, 2016

Copy link
Copy Markdown
Contributor Author

@dhermes Rebased after merge of #1597 to logging-api branch. PTAL

Comment thread gcloud/logging/sink.py
client = self._require_client(client)
data = client.connection.api_request(method='GET', path=self.path)
self.filter_ = data['filter']
self.destination = data['destination']

This comment was marked as spam.

This comment was marked as spam.

dhermes commented Mar 11, 2016

Copy link
Copy Markdown
Contributor

LGTM

tseaver added a commit that referenced this pull request Mar 11, 2016
tseaver merged commit 427f96d into googleapis:logging-api Mar 11, 2016
tseaver deleted the logging-sink_reload branch March 11, 2016 21:49
parthea pushed a commit that referenced this pull request Apr 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL