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

Document that blob-level IAM APIs are deprecated. #4200

Closed
wants to merge 1 commit into from
Closed

Document that blob-level IAM APIs are deprecated. #4200

wants to merge 1 commit into from

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Oct 16, 2017

Closes #4087.

[ci skip].

@tseaver tseaver added api: storage Issues related to the Cloud Storage API. docs labels Oct 16, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 16, 2017
@dhermes
Copy link
Contributor

dhermes commented Oct 16, 2017

@lukesneeringer @jonparrott Can we get confirmation of this fact from an API owner?

@tseaver
Copy link
Contributor Author

tseaver commented Oct 16, 2017

@dhermes It was confirmed by @frankyn in conversation which led to #4087.

@frankyn
Copy link
Member

frankyn commented Oct 16, 2017

I haven't synced with the Storage team on how they would like to move forward with this case. To be clear: the issue here is Object-level IAM shouldn't have been introduced into the client and not that it's deprecated.

I'll ping with an update after I sync with them.

@tseaver
Copy link
Contributor Author

tseaver commented Oct 16, 2017

@frankyn FWIW, those methods were part of the storage API documentation: in fact, the URLs we have in our docstrings are still there, although they are no longer linked into the navigation (they were linked back when we added IAM support to storage):

The fact that "the secretary has disavowed all knowledge" of them means that the endpoints are now deprecated. We very carefully researched adding them back in the sprint for issue #1679 (PRs #3309, #3311).

@frankyn
Copy link
Member

frankyn commented Oct 18, 2017

These methods are actually not going to be deprecated based on an email exchange we had with the storage team. I'm closing this PR as it isn't what the Storage team wants to do.

@frankyn frankyn closed this Oct 18, 2017
@tseaver tseaver deleted the 4087-storage-blob-iam-docstrings branch January 4, 2018 18:37
parthea pushed a commit that referenced this pull request Oct 21, 2023
…dPlatform/python-docs-samples#4200)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafeas](https://github.com/googleapis/python-grafeas) | patch | `==0.4.0` -> `==0.4.1` |

---

### Release Notes

<details>
<summary>googleapis/python-grafeas</summary>

### [`v0.4.1`](https://github.com/googleapis/python-grafeas/blob/master/CHANGELOG.md#&#8203;041-httpswwwgithubcomgoogleapispython-grafeascomparev040v041-2020-06-25)

[Compare Source](https://github.com/googleapis/python-grafeas/compare/v0.4.0...v0.4.1)

</details>

---

### Renovate configuration

:date: **Schedule**: At any time (no schedule defined).

:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

:recycle: **Rebasing**: Never, or you tick the rebase/retry checkbox.

:no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#GoogleCloudPlatform/python-docs-samples).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants