-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix content
in the google_storage_bucket_object_content
data source which is incorrectly marked as a non-computed attribute
#10778
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Hello! I am a robot. Tests will require approval from a repository maintainer to run. @NickElliot, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Breaking Change(s) DetectedThe following breaking change(s) were detected within your pull request.
If you believe this detection to be incorrect please raise the concern with your reviewer. |
Tests analyticsTotal tests: Click here to see the affected service packages
Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccDataSourceStorageBucketObjectContent_Issue15717 |
|
This PR has been waiting for review for 2 weekdays. Please take a look! Use the label |
Going to verify some of the behavior here locally to ensure this change isn't actually breaking due to the weirdness of how GCS buckets function as handwritten resources. |
@GoogleCloudPlatform/terraform-team This PR has been waiting for review for 1 week. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team This PR has been waiting for review for 2 weeks. Please take a look! Use the label |
Is there anything I should do to move this Pull Request forward? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about that prolonged check on my end.
Can you undo the change removing the "computed" attribute from the resource? I don't believe it should affect the change made in mmv1/third_party/terraform/services/storage/data_source_storage_bucket_object_content.go
that would resolve the null
issue, and would prevent this from being a breaking change.
I don't think this is a breaking change as Nevertheless, I reverted the change to the resource and force-pushed the branch. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: Click here to see the affected service packages
|
@NickElliot This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team @NickElliot This PR has been waiting for review for 1 week. Please take a look! Use the label |
Could you take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
…ata source which is incorrectly marked as a non-computed attribute (GoogleCloudPlatform#10778)" This reverts commit 2d56f59.
@wafrelka this change is going to be reverted because removing the ability to supply to the data source is breaking (despite this not having functional purpose), if you want to reopen the PR against the 6.0.0 release branch (per https://googlecloudplatform.github.io/magic-modules/develop/breaking-changes/make-a-breaking-change/) that would be appreciated! I'll go ahead and handle adding the deprecation notice in the meantime. |
@NickElliot can we mark the |
…ce which is incorrectly marked as a non-computed attribute (GoogleCloudPlatform#10778)
…ce which is incorrectly marked as a non-computed attribute (GoogleCloudPlatform#10778)
Fixes hashicorp/terraform-provider-google#15717
Release Note Template for Downstream PRs (will be copied)