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.

Sign up for GitHub

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

Jump to bottom

FR: Storage - Please support managing permissions w/ IAM #1199

Closed
lesv opened this issue Aug 25, 2016 · 6 comments · Fixed by #1228
Closed

FR: Storage - Please support managing permissions w/ IAM #1199

lesv opened this issue Aug 25, 2016 · 6 comments · Fixed by #1228
Assignees
@mziccard
Labels
api: storage Issues related to the Cloud Storage API. iam type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@lesv
Copy link
Contributor

lesv commented Aug 25, 2016

I don't seem to see support in the 0.2.7 JavaDoc
@lesv lesv added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. api: storage Issues related to the Cloud Storage API. iam labels Aug 25, 2016
@mziccard
Copy link
Contributor

Storage has no IAM methods that I am aware of. To manage permissions it uses ACLs for both buckets and objects. In gcloud-java we already support setting ACLs for both buckets and blobs.

@lesv
Copy link
Contributor Author

lesv commented Aug 29, 2016

I've asked why it's on the tracking spreadsheet. Looking at https://cloud.google.com/storage/docs/access-control/iam doesn't suggest a separate API.

@mziccard mziccard mentioned this issue Sep 7, 2016
Merged
@omaray
Copy link

omaray commented Sep 12, 2016

Any updates on this issue @lesv ? Do we need IAM support?

@jgeewax
Copy link

jgeewax commented Sep 12, 2016

AFAIK, GCS has access-control lists, so I think that's what we're talking about here.

@mziccard
Copy link
Contributor

I consider this closed with #1228 which adds support for all ACL-related methods (get, create, list, delete) for both blobs and buckets. I believe this is what @lesv meant with IAM. Feel free to reopen (possibly with some pointers) in case there is some IAM-related stuff we should support that I am not aware of.

@lesv
Copy link
Contributor Author

lesv commented Sep 12, 2016

Yep - as I said, not sure why it was listed on the tracking spreadsheet.

github-actions bot pushed a commit to suztomo/google-cloud-java that referenced this issue Jun 29, 2022
@gcf-owl-bot
build(java): update renovate config to mark conformance tests as a te…
b9298a1 
…st commit type (googleapis#1199) (googleapis#615)
github-actions bot pushed a commit that referenced this issue Jul 1, 2022
@gcf-owl-bot
build(java): update renovate config to mark conformance tests as a te…
c799801 
…st commit type (#1199) (#328)
github-actions bot pushed a commit that referenced this issue Aug 9, 2022
@renovate-bot
deps: update dependency com.google.cloud:google-cloud-core to v2.8.7 (#…
7bd891b 
…1199)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-core](https://github.com/googleapis/java-core) | `2.8.6` -> `2.8.7` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.7/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.7/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.7/compatibility-slim/2.8.6)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-core/2.8.7/confidence-slim/2.8.6)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-core</summary>

### [`v2.8.7`](https://github.com/googleapis/java-core/blob/HEAD/CHANGELOG.md#&#8203;287-httpsgithubcomgoogleapisjava-corecomparev286v287-2022-08-03)

[Compare Source](https://github.com/googleapis/java-core/compare/v2.8.6...v2.8.7)

##### Dependencies

-   update dependency com.google.auth:google-auth-library-bom to v1.9.0 ([#&#8203;894](https://github.com/googleapis/java-core/issues/894)) ([82aaa16](https://github.com/googleapis/java-core/commit/82aaa16cd9813984a0c92d944eb1ad956759a8e5))
-   update dependency io.grpc:grpc-bom to v1.48.1 ([#&#8203;891](https://github.com/googleapis/java-core/issues/891)) ([0b0f9b8](https://github.com/googleapis/java-core/commit/0b0f9b89757b801e8334c764644d7b66032d36d1))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

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

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

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

---

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

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-automl).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xNDEuMCIsInVwZGF0ZWRJblZlciI6IjMyLjE0MS4wIn0=-->
github-actions bot pushed a commit that referenced this issue Aug 9, 2022
@release-please
chore(main): release 2.3.3 (#1207)
fcc68cd 
🤖 I have created a release *beep* *boop*
---


## [2.3.3](googleapis/java-automl@v2.3.2...v2.3.3) (2022-08-09)


### Dependencies

* update dependency com.google.cloud:google-cloud-bigquery to v2.14.1 ([#1195](googleapis/java-automl#1195)) ([e676c4c](googleapis/java-automl@e676c4c))
* update dependency com.google.cloud:google-cloud-bigquery to v2.14.2 ([#1203](googleapis/java-automl#1203)) ([b6d75e2](googleapis/java-automl@b6d75e2))
* update dependency com.google.cloud:google-cloud-bigquery to v2.14.3 ([#1205](googleapis/java-automl#1205)) ([18ee6b6](googleapis/java-automl@18ee6b6))
* update dependency com.google.cloud:google-cloud-core to v2.8.3 ([#1189](googleapis/java-automl#1189)) ([90f3232](googleapis/java-automl@90f3232))
* update dependency com.google.cloud:google-cloud-core to v2.8.4 ([#1190](googleapis/java-automl#1190)) ([77b2a7d](googleapis/java-automl@77b2a7d))
* update dependency com.google.cloud:google-cloud-core to v2.8.5 ([#1191](googleapis/java-automl#1191)) ([f6f08ba](googleapis/java-automl@f6f08ba))
* update dependency com.google.cloud:google-cloud-core to v2.8.6 ([#1197](googleapis/java-automl#1197)) ([9c1ff45](googleapis/java-automl@9c1ff45))
* update dependency com.google.cloud:google-cloud-core to v2.8.7 ([#1199](googleapis/java-automl#1199)) ([7bd891b](googleapis/java-automl@7bd891b))
* update dependency com.google.cloud:google-cloud-core to v2.8.8 ([#1201](googleapis/java-automl#1201)) ([026ae49](googleapis/java-automl@026ae49))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v3 ([#1192](googleapis/java-automl#1192)) ([94c1c3c](googleapis/java-automl@94c1c3c))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.1 ([#1198](googleapis/java-automl#1198)) ([1be2ead](googleapis/java-automl@1be2ead))
* update dependency com.google.cloud:google-cloud-storage to v2.10.0 ([#1196](googleapis/java-automl#1196)) ([73871d6](googleapis/java-automl@73871d6))
* update dependency com.google.cloud:google-cloud-storage to v2.11.0 ([#1200](googleapis/java-automl#1200)) ([ea4490e](googleapis/java-automl@ea4490e))
* update dependency com.google.cloud:google-cloud-storage to v2.11.1 ([#1202](googleapis/java-automl#1202)) ([3c8f71e](googleapis/java-automl@3c8f71e))
* update dependency com.google.cloud:google-cloud-storage to v2.11.2 ([#1204](googleapis/java-automl#1204)) ([cc4d715](googleapis/java-automl@cc4d715))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mziccard mziccard

Labels
api: storage Issues related to the Cloud Storage API. iam type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for create/get/update/delete/list ACLs for blob and bucket mziccard/gcloud-java
4 participants
@jgeewax @mziccard @lesv @omaray