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

Added missing features on parameter store resource #1

Merged
merged 2 commits into from
Jul 31, 2017

Conversation

jocgir
Copy link

@jocgir jocgir commented Jul 29, 2017

Add support for description, allowed_pattern and tags for ssm parameters
Allow detection of key_id change

… error if the parameter no longer exist. If a user manually delete a parameter, Terraform is no longer able to recreate it or destroy it.

The defautt value for overwrite should be true. Otherwise, this is causing a breaking change with previously written code where overwrite is not set and the behaviour was to overwrite the parameter by default. Moreover, terraform already provides a mechanism to handle the lifecycle of the values.
@jocgir jocgir merged commit d3d59dc into master Jul 31, 2017
jocgir pushed a commit that referenced this pull request Oct 13, 2017
…pdate

Update lightsail_instance.html.markdown
jocgir pushed a commit that referenced this pull request Oct 13, 2017
jocgir pushed a commit that referenced this pull request May 1, 2018
hi @nicksantamaria , 
thanks for the contribution and the helpful feedback on this PR.
jocgir pushed a commit that referenced this pull request Jul 3, 2018
make glue crawler creation wait for role ready
julienduchesne pushed a commit that referenced this pull request Feb 27, 2019
jocgir pushed a commit that referenced this pull request Apr 25, 2019
* Update sagemaker_endpoint_configuration not found message when reading resources

* Update sagemaker_endpoint error when resource is NotFound

* Fix ResourceNotFound to ValidationException
julienduchesne pushed a commit that referenced this pull request Jul 8, 2019
Update ecr_repository_policy.html.markdown
jocgir pushed a commit that referenced this pull request Jul 22, 2019
julienduchesne pushed a commit that referenced this pull request Nov 5, 2019
jocgir pushed a commit that referenced this pull request Mar 20, 2020
jocgir pushed a commit that referenced this pull request Jun 10, 2020
jocgir pushed a commit that referenced this pull request Jun 26, 2020
julienduchesne pushed a commit that referenced this pull request Sep 10, 2020
* Implementation of resource new resource type for GuardDuty S3 Export (hashicorp#10920)

* Added tests and documentation

* Fixed test namings

* Fixed linter issues and removed explicit import test case

* Fixed HCL formatting in documentation

* Fixed some namings and sidebar link

* Update/refactor publishing destination (#1)

* Merged latest master changes and resolved conflicts

* Merged from Upstream master and squashed commits

* Delete defaults.go

* Removed changes from vendor subdirectory and synched with master.

* Refactor GuardDuty to waiter pattern

* Remove unused constant

* Small refactor based on PR review

* Add disappears test and refactor based on PR review

* Refactor based on PR review

* Refactor according to PR review

* Additional fix wrt tf 0.12

Co-authored-by: Sebastian Häpe <Sebastian.Haepe@nordcloud.com>
Co-authored-by: shaepe <44882151+shaepe@users.noreply.github.com>
jocgir pushed a commit that referenced this pull request Jun 22, 2021
@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 issues.
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 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant