-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
PubSub Schema evolution - when revision limit is reached, resource can't be updated #15228
PubSub Schema evolution - when revision limit is reached, resource can't be updated #15228
Comments
@uthark sounds like this is the restriction of the feature (api). Not sure how much the provider can do. Are you able to create more than 20 revisions via other means? |
old revisions could be removed via API.
O.
…On Wed, Jul 19, 2023 at 12:45 PM Edward Sun ***@***.***> wrote:
@uthark <https://github.com/uthark> sounds like this is the restriction
of the feature (api). Not sure how much the provider can do. Are you able
to create more than 20 revisions via other means?
—
Reply to this email directly, view it on GitHub
<#15228 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABK75WHKQYRCGDQTO7L4KTXRA2L7ANCNFSM6AAAAAA2QLXQ5U>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey @melinath, I assume that you may have some context on this issue so I assign it to you. Apologize if I'm wrong and feel free to unassign yourself. (not forwarding this ticket to service team at this moment) |
This is a constraint of the API - when pubsub schema updates were implemented, we had a lot of trouble deciding how Terraform should behave and ended up settling on this for now. We're definitely interested in hearing how folks would like to see this behave. The best workaround in the meantime is to delete old revisions manually using gcloud or cloud console. attn @kamalaboulhosn |
Note: We may just want to describe how this works in the resource docs- the 20 revision limit, and the fact that users will need to manually clear revisions if they reach it. |
wondering if it could be an additional flag to remove old revisions
automatically.
In the meantime we decided to implement additional pre-flight checks to
verify if the limit is reached.
O.
…On Mon, Jul 31, 2023 at 9:50 AM Riley Karson ***@***.***> wrote:
Note: We may just want to describe how this works in the resource docs-
the 20 revision limit, and the fact that users will need to manually clear
revisions if they reach it.
—
Reply to this email directly, view it on GitHub
<#15228 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABK75TEXIRKTGBMO4HYQ5LXS7O3ZANCNFSM6AAAAAA2QLXQ5U>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Community Note
modular-magician
user, it is either in the process of being autogenerated, or is planned to be autogenerated soon. If an issue is assigned to a user, that user is claiming responsibility for the issue. If an issue is assigned tohashibot
, a community member has claimed the issue already.Terraform Version
Affected Resource(s)
Terraform Configuration Files
Debug Output
Panic Output
Expected Behavior
Actual Behavior
Steps to Reproduce
terraform apply
Important Factoids
References
b/294553291
The text was updated successfully, but these errors were encountered: