Skip to content

Storage: accomodate new back-end restriction on retention period.#6388

Merged
tseaver merged 2 commits intogoogleapis:masterfrom
tseaver:6368-storage-bucket_retention_property_setter_none
Nov 5, 2018
Merged

Storage: accomodate new back-end restriction on retention period.#6388
tseaver merged 2 commits intogoogleapis:masterfrom
tseaver:6368-storage-bucket_retention_property_setter_none

Conversation

@tseaver
Copy link
Copy Markdown
Contributor

@tseaver tseaver commented Nov 5, 2018

Closes #6368.

@tseaver tseaver added the api: storage Issues related to the Cloud Storage API. label Nov 5, 2018
@tseaver tseaver requested a review from frankyn November 5, 2018 16:50
@tseaver tseaver requested a review from crwilcox as a code owner November 5, 2018 16:50
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 5, 2018
Copy link
Copy Markdown
Contributor

@frankyn frankyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change LGTM. I'll approve after pending tests are complete.

@tseaver
Copy link
Copy Markdown
Contributor Author

tseaver commented Nov 5, 2018

@frankyn The pubsub failure is a Kokoro docker /networking barf (as well as being obviously unrelated to this PR).

Copy link
Copy Markdown
Contributor

@frankyn frankyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tseaver!

@tseaver tseaver merged commit 062235c into googleapis:master Nov 5, 2018
@tseaver tseaver deleted the 6368-storage-bucket_retention_property_setter_none branch November 5, 2018 18:16
parthea pushed a commit that referenced this pull request Mar 9, 2026
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.

3 participants