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

provider/openstack: Add 'value_specs' option to 'openstack_fw_firewall_v1' resource #9836

Merged

Conversation

fatmcgav
Copy link
Contributor

@fatmcgav fatmcgav commented Nov 3, 2016

Refactor to use common 'types.go' and 'MapValueSpecs' function.
Website docs updated to reflect changes.

Will require rebase after merging #9830, #9834 and #9835

@fatmcgav
Copy link
Contributor Author

fatmcgav commented Nov 3, 2016

@jtopjian 3rd and final one for these resources :)

@fatmcgav fatmcgav force-pushed the openstack_fw_firewall_v1_add_value_specs branch 3 times, most recently from 5117bec to 7e15b2e Compare November 5, 2016 10:07
@fatmcgav
Copy link
Contributor Author

fatmcgav commented Nov 5, 2016

@jtopjian That one took a bit more work :( But rebased and ready to merge :)

…l_v1' resource

Refactor to use common 'types.go' and 'MapValueSpecs' function.
Website docs updated to reflect changes.
@fatmcgav fatmcgav force-pushed the openstack_fw_firewall_v1_add_value_specs branch from 7e15b2e to b098156 Compare November 5, 2016 10:17
@jtopjian jtopjian merged commit 6f67b69 into hashicorp:master Nov 5, 2016
@jtopjian
Copy link
Contributor

jtopjian commented Nov 5, 2016

@fatmcgav Nice work!

@fatmcgav
Copy link
Contributor Author

fatmcgav commented Nov 5, 2016

Cheers again @jtopjian :)

@fatmcgav fatmcgav deleted the openstack_fw_firewall_v1_add_value_specs branch November 5, 2016 16:22
@ghost
Copy link

ghost commented Apr 20, 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 20, 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.

2 participants