Skip to content

Add 'Topic.set_iam_policy' API wrapper. - #1641

Merged
tseaver merged 3 commits into
googleapis:masterfrom
tseaver:pubsub-topic-set_iam_policy
Mar 22, 2016
Merged

Add 'Topic.set_iam_policy' API wrapper.#1641
tseaver merged 3 commits into
googleapis:masterfrom
tseaver:pubsub-topic-set_iam_policy

Conversation

@tseaver

@tseavertseaver commented Mar 21, 2016

Copy link
Copy Markdown
Contributor

Uses #1640 as a base.

@tseavertseaver added the api: pubsub Issues related to the Pub/Sub API. label Mar 21, 2016
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 21, 2016
@tseaver

tseaver commented Mar 21, 2016

Copy link
Copy Markdown
ContributorAuthor

Hmm, odd docs build failure:

Warning, treated as error:
WARNING: intersphinx inventory 'http://oauth2client.readthedocs.org/en/latest/objects.inv' not fetchable due to <class 'urllib2.HTTPError'>: HTTP Error 500: Internal Server Error
ERROR: InvocationError: '/home/travis/build/GoogleCloudPlatform/gcloud-python/.tox/docs/bin/sphinx-build -W -b html -d docs/_build/doctrees docs docs/_build/html'

I've restarted the job.

@dhermes

dhermes commented Mar 21, 2016

Copy link
Copy Markdown
Contributor

Not weird. I've seen it from time to time. The objects.inv file is an inventory of all the objects in the library so that Sphinx can link across docs. The 500 just means the request failed.

@tseavertseaver mentioned this pull request Mar 21, 2016
Merged
@tseaver

tseaver commented Mar 22, 2016

Copy link
Copy Markdown
ContributorAuthor

@dhermes Rebased after merging #1640 to master. PTAL

dhermes
dhermes reviewed Mar 22, 2016
View reviewed changes
Comment threadgcloud/pubsub/topic.py
return Policy.from_api_repr(resp)

def set_iam_policy(self, policy, client=None):
"""Update the IAM policy for the topic.

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

dhermes
dhermes reviewed Mar 22, 2016
View reviewed changes
Comment threaddocs/pubsub-usage.rst Outdated
>>> topic.delete() # API request

Fetch the IAM policy for a topic:
Update the IAM policy for a topic:

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes

dhermes commented Mar 22, 2016

Copy link
Copy Markdown
Contributor

LGTM other than the docs questions

tseaver added a commit that referenced this pull request Mar 22, 2016
@tseaver
Merge pull request#1641from tseaver/pubsub-topic-set_iam_policy
aea1e6d
Add 'Topic.set_iam_policy' API wrapper.
@tseaver
tseaver merged commit aea1e6d into googleapis:masterMar 22, 2016
@tseaver
tseaver deleted the pubsub-topic-set_iam_policy branch March 22, 2016 16:15
@dhermesdhermes mentioned this pull request Apr 1, 2016
Merged
parthea pushed a commit that referenced this pull request Nov 26, 2025
@harkamaljot@gcf-owl-bot
feat: allow users to use jwk keys for verifying ID token (#1641)
d34602f
* add support for jwk format
* update formatting
* formatting changes
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* remove pyjwt
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Mar 6, 2026
@shollyman
feat: widen retry predicate to include ServiceUnavailable (#1641)
7cb6bcb
Expands retry. It's possible in the normal lifecycle of an API frontend
for the intermediate response to indicate the API service is not ready.
related: internal issue 294103068
parthea pushed a commit that referenced this pull request Apr 1, 2026
@arwas11
feat: support passing list of values to bigframes.core.sql.simple_lit…
0d29a1c
…eral (#1641)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

api: pubsubIssues related to the Pub/Sub API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tseaver@dhermes@googlebot