feat: add support of public access prevention

[athakor]

Fixes #616

[codecov]

Codecov Report

Merging #636 (37e19e3) into master (1dfcc02) will increase coverage by 0.11%.
The diff coverage is 88.00%.

❗ Current head 37e19e3 differs from pull request most recent head ec75bb2. Consider uploading reports for the commit ec75bb2 to get more accurate results

@@             Coverage Diff              @@
##             master     #636      +/-   ##
============================================
+ Coverage     64.62%   64.73%   +0.11%     
  Complexity      634      634              
============================================
  Files            32       32              
  Lines          5314     5337      +23     
  Branches        519      522       +3     
============================================
+ Hits           3434     3455      +21     
- Misses         1718     1719       +1     
- Partials        162      163       +1     

Impacted Files	Coverage Δ
...main/java/com/google/cloud/storage/BucketInfo.java	84.24% <88.00%> (+0.24%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fad4a31...ec75bb2. Read the comment docs.

[athakor]

@shaffeeullah @tritone

[elharo]

Dec 03, 2020 7:29:41 AM com.google.cloud.tools.dependencies.linkagemonitor.LinkageMonitor run
SEVERE: Newly introduced problems:
(com.google.apis:google-api-services-storage:v1-rev20200927-1.30.10) com.google.api.services.storage.model.Bucket$IamConfiguration's method setPublicAccessPrevention(String) is not found
  referenced from com.google.cloud.storage.BucketInfo (com.google.cloud:google-cloud-storage:1.113.5-SNAPSHOT)
(com.google.apis:google-api-services-storage:v1-rev20200927-1.30.10) com.google.api.services.storage.model.Bucket$IamConfiguration's method getPublicAccessPrevention() is not found
  referenced from com.google.cloud.storage.BucketInfo (com.google.cloud:google-cloud-storage:1.113.5-SNAPSHOT)

com.google.apis:google-api-services-storage:v1-rev20200927-1.30.10 is at:
  com.google.cloud:google-cloud-nio:0.122.1 (compile) / com.google.apis:google-api-services-storage:v1-rev20200927-1.30.10 (compile)
  and 2 other dependency paths.
com.google.cloud:google-cloud-storage:1.113.5-SNAPSHOT is at:
  com.google.cloud:google-cloud-storage:1.113.5-SNAPSHOT (compile)

[frankyn]

For the linker issues, we need to cut a new release of storage with the updated Apiary client (#613) and a new release of libraries-bom as well.

[tritone]

For the linker issues, we need to cut a new release of storage with the updated Apiary client (#613) and a new release of libraries-bom as well.

Looks like this has been completed. @athakor can you merge in master?

[athakor]

For the linker issues, we need to cut a new release of storage with the updated Apiary client (#613) and a new release of libraries-bom as well.

Looks like this has been completed. @athakor can you merge in master?

done.. Still linker issue

I'm now running point as co-author

[frankyn]

@BenWhitehead can you take a look when you have a moment?

[BenWhitehead]

A couple new nits, and the outstanding valueOf call

[tritone]

Thanks so much for fixing this up Ben!

[tritone]

Looks like there is a lint failure; otherwise LGTM

[shaffeeullah]

@BenWhitehead & @frankyn ,

We would like to merge and release this feature next Monday (06/14/21). Does that work for you? Do one of you have bandwidth this week to resolve the conflicts? Sorry, I know this PR has been open for a while.

[snippet-bot]

No region tags are edited in this PR.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

• Refresh this comment

[shaffeeullah]

Public access prevention rollout has been delayed due to a bug surfaced during Googler preview. I will keep this PR updated as I learn new release timeline details.

[shaffeeullah]

@BenWhitehead This feature can now be merged and released.