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

core: Add terraform state rm command #8200

Merged
merged 3 commits into from
Aug 16, 2016
Merged

core: Add terraform state rm command #8200

merged 3 commits into from
Aug 16, 2016

Conversation

jen20
Copy link
Contributor

@jen20 jen20 commented Aug 15, 2016

This was previously merged in #5953, however was lost at some point during the Terraform 0.7 feature branch merge. This has resulted in a variety of issues and pull requests, so this pull request fixes #8060 and fixes #8088.

@jen20
Copy link
Contributor Author

jen20 commented Aug 15, 2016

cc @phinze, @stack72.

@jen20
Copy link
Contributor Author

jen20 commented Aug 15, 2016

Also fixes #8066.

@phinze
Copy link
Contributor

phinze commented Aug 15, 2016

@jen20 build failing, maybe a bad merge?

@jen20
Copy link
Contributor Author

jen20 commented Aug 16, 2016

This should be better, fixing up the merge conflicts left us without a trailing ,.

@stack72
Copy link
Contributor

stack72 commented Aug 16, 2016

# github.com/hashicorp/terraform/command
command/state_test.go:11: testStateBackups redeclared in this block
    previous declaration at command/state_rm_test.go:93

This commit adds the `state rm` command for removing an address from
state. It is the result of a rebase from pull-request #5953 which was
lost at some point during the Terraform 0.7 feature branch merges.
@jen20
Copy link
Contributor Author

jen20 commented Aug 16, 2016

8th time should be the charm...

Removing sort as it was imported and not used
@jen20
Copy link
Contributor Author

jen20 commented Aug 16, 2016

Thanks for fixing this up @stack72, I swear it built here but apparently not...

@jen20 jen20 merged commit f933b2c into master Aug 16, 2016
@jen20 jen20 deleted the fix-state-rm branch August 16, 2016 18:10
@ghost
Copy link

ghost commented Apr 23, 2020

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.

@ghost ghost locked and limited conversation to collaborators Apr 23, 2020
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.

docs for state CLI link to rm command, which doesn't exist
4 participants