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

Replace usage of deprecated fields in schema.Resource #230

Closed
1 task done
bendbennett opened this issue Apr 12, 2022 · 1 comment · Fixed by #231
Closed
1 task done

Replace usage of deprecated fields in schema.Resource #230

bendbennett opened this issue Apr 12, 2022 · 1 comment · Fixed by #231
Labels
enhancement

Comments

@bendbennett
Copy link
Contributor

bendbennett commented Apr 12, 2022 •
edited
Loading

Terraform CLI and Provider Versions

Terraform v1.1.7
random provider v3.1.2

Use Cases or Problem Statement

The random provider is using deprecated fields within schema.Resource.
For example resource_id uses the Create field which has been deprecated in favour of CreateContext or CreateWithoutTimeout.

Proposal

Replace usage of deprecated fields in schema.Resource (e.g., replace Create with CreateContext or CreateWithoutTimeout).

How much impact is this issue causing?

Low

Additional Information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@bendbennett bendbennett changed the title Usage of deprecated fields in schema.Resource Replace usage of deprecated fields in schema.Resource Apr 12, 2022
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Running gofmt (#230)
44d7e05
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create field with CreateContext in resource_id (#230)
26911a7
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usgae of Read field with ReadContext in resource_id (#230)
5720b72
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Using blank identifier for unused context passed to RepopulateEncodin…
d262d66 
…gs func in resource_id (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Delete field with DeleteContext in resource_id (#230)
c63329b
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of State field with StateContext in resource_id (#230)
3bb503a
@bendbennett bendbennett mentioned this issue Apr 12, 2022
Merged
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create field with CreateContext in resource_integer (#…
ebf4911 
…230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create field with CreateContext in resource_password…
9f8012a 
… and resource_string (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Read field with ReadContext in resource_password and…
2ea5683 
… resource_string (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Delete field with DeleteContext in resource_password…
9d51c60 
… and resource_string (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create field with CreateContext in resource_pet (#230)
708d4ef
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create field with CreateContext in resource_pet (#230)
726acdc
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Delete field with DeleteContext in resource_pet (#230)
b47c746
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create, Read and Delete fields with CreateContext, R…
1d6b7b1 
…eadContext and DeleteContext in resource_shuffle (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Add comment to justify not replacing deprecated MigrateState field in…
d47e381 
… resource_string (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Replace usage of Create, Read, Delete and State fields with CreateCon…
082dce6 
…text, ReadContext, DeleteContext and StateContext in resource_uuid (#230)
bendbennett added a commit that referenced this issue Apr 12, 2022
@bendbennett
Sorting imports (#230)
1ec934b
bendbennett added a commit that referenced this issue Apr 13, 2022
@bendbennett
Consolidating DeleteContext to a shared func (#230)
445ab1d
bendbennett added a commit that referenced this issue Apr 13, 2022
@bendbennett
Moving DeleteContext func to provider (#230)
c83c891
bendbennett added a commit that referenced this issue Apr 13, 2022
@bendbennett
Renaming DeleteContext func to RemoveResourceFromState (#230)
5ed4718
@vfoucault vfoucault mentioned this issue May 10, 2022
Closed
@github-actions GitHub Actions
Copy link

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.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace usage of deprecated fields in schema.Resource hashicorp/terraform-provider-random
1 participant
@bendbennett