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.

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 small typo in tf plan error #33149

Merged
merged 2 commits into from May 5, 2023
Merged

Conversation

reespozzi
Copy link
Contributor

When running terraform plan with -target but also using terraform move blocks, an error of

│ Error: Moved resource instances excluded by targeting
│ 
│ Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource
│ instances while planning in order to ensure a correct result, but your -target=... options to not fully cover all of those resource
│ instances.

Pops up, which includes a minor typo.

This PR updates to fix the typo

Fixes minor typo in error message in terraform cli tool

Target Release

1.4.x

Draft CHANGELOG entry

Minor typo fix in Error: Moved resource instances excluded by targeting error message.

BUG FIXES

…ing`

When running terraform plan with `-target` but also using terraform move blocks, an error of 

```
│ Error: Moved resource instances excluded by targeting
│ 
│ Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource
│ instances while planning in order to ensure a correct result, but your -target=... options to not fully cover all of those resource
│ instances.
```
Pops up, which includes a minor typo.

This PR updates to fix the typo
@hashicorp-cla
Copy link

hashicorp-cla commented May 4, 2023

CLA assistant check
All committers have signed the CLA.

@crw crw added the enhancement label May 4, 2023
@crw
Copy link
Collaborator

crw commented May 4, 2023

Thanks for this, will raise in triage.

@crw crw added this to the v1.5 milestone May 5, 2023
Copy link
Member

@apparentlymart apparentlymart left a comment

Choose a reason for hiding this comment

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

Thanks, @reespozzi!

@apparentlymart apparentlymart merged commit fed6538 into hashicorp:main May 5, 2023
5 of 6 checks passed
@github-actions
Copy link

github-actions bot commented May 5, 2023

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@reespozzi reespozzi deleted the patch-1 branch May 10, 2023 08:25
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, 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 Jun 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants