-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
r/aws_instance - add disable_api_stop
argument
#25185
Conversation
disable_api_stop
argument
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀.
% make testacc TESTARGS='-run=TestAccEC2Instance_disableAPIStop\|TestAccEC2Instance_basic\|TestAccEC2InstanceDataSource_disableApiStopTermination\|TestAccEC2InstanceDataSource_basic\|TestAccEC2LaunchTemplateDataSource_name\|TestAccEC2LaunchTemplate_basic\|TestAccEC2LaunchTemplate_data' PKG=ec2 ACCTEST_PARALLELISM=3
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/ec2/... -v -count 1 -parallel 3 -run=TestAccEC2Instance_disableAPIStop\|TestAccEC2Instance_basic\|TestAccEC2InstanceDataSource_disableApiStopTermination\|TestAccEC2InstanceDataSource_basic\|TestAccEC2LaunchTemplateDataSource_name\|TestAccEC2LaunchTemplate_basic\|TestAccEC2LaunchTemplate_data -timeout 180m
=== RUN TestAccEC2InstanceDataSource_basic
=== PAUSE TestAccEC2InstanceDataSource_basic
=== RUN TestAccEC2InstanceDataSource_disableApiStopTermination
=== PAUSE TestAccEC2InstanceDataSource_disableApiStopTermination
=== RUN TestAccEC2Instance_basic
=== PAUSE TestAccEC2Instance_basic
=== RUN TestAccEC2Instance_disableAPIStop
=== PAUSE TestAccEC2Instance_disableAPIStop
=== RUN TestAccEC2LaunchTemplateDataSource_name
=== PAUSE TestAccEC2LaunchTemplateDataSource_name
=== RUN TestAccEC2LaunchTemplate_basic
=== PAUSE TestAccEC2LaunchTemplate_basic
=== RUN TestAccEC2LaunchTemplate_data
=== PAUSE TestAccEC2LaunchTemplate_data
=== CONT TestAccEC2InstanceDataSource_basic
=== CONT TestAccEC2LaunchTemplateDataSource_name
=== CONT TestAccEC2LaunchTemplate_data
--- PASS: TestAccEC2LaunchTemplate_data (18.00s)
=== CONT TestAccEC2LaunchTemplate_basic
--- PASS: TestAccEC2LaunchTemplate_basic (14.36s)
=== CONT TestAccEC2Instance_disableAPIStop
--- PASS: TestAccEC2LaunchTemplateDataSource_name (45.59s)
=== CONT TestAccEC2Instance_basic
--- PASS: TestAccEC2InstanceDataSource_basic (121.58s)
=== CONT TestAccEC2InstanceDataSource_disableApiStopTermination
--- PASS: TestAccEC2Instance_basic (126.82s)
--- PASS: TestAccEC2Instance_disableAPIStop (161.58s)
--- PASS: TestAccEC2InstanceDataSource_disableApiStopTermination (192.25s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/ec2 318.838s
% make semall providerlint golangci-lint
==> Running Semgrep checks locally (must have semgrep installed)...
Scanning 3955 files with 37 go rules.
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████|3955/3955 tasks
Some files were skipped or only partially analyzed.
Scan was limited to files tracked by git.
Scan skipped: 1613 files matching .semgrepignore patterns
For a full list of skipped files, run semgrep with the --verbose flag.
Ran 37 rules on 3955 files: 0 findings.
If Semgrep missed a finding, please send us feedback to let us know!
$ semgrep shouldafound --help
Scanning 3955 files with 107 go rules.
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████|3955/3955 tasks
Some files were skipped or only partially analyzed.
Scan was limited to files tracked by git.
Scan skipped: 1613 files matching .semgrepignore patterns
For a full list of skipped files, run semgrep with the --verbose flag.
Ran 107 rules on 3955 files: 0 findings.
If Semgrep missed a finding, please send us feedback to let us know!
$ semgrep shouldafound --help
Scanning 891 files with 179 go rules.
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████|891/891 tasks
Some files were skipped or only partially analyzed.
Scan was limited to files tracked by git.
Scan skipped: 1613 files matching .semgrepignore patterns
For a full list of skipped files, run semgrep with the --verbose flag.
Ran 180 rules on 891 files: 0 findings.
If Semgrep missed a finding, please send us feedback to let us know!
$ semgrep shouldafound --help
Scanning 883 files with 197 go rules.
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████|883/883 tasks
Some files were skipped or only partially analyzed.
Scan was limited to files tracked by git.
Scan skipped: 1613 files matching .semgrepignore patterns
For a full list of skipped files, run semgrep with the --verbose flag.
Ran 201 rules on 883 files: 0 findings.
If Semgrep missed a finding, please send us feedback to let us know!
$ semgrep shouldafound --help
Scanning 1122 files with 198 go rules.
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████|1122/1122 tasks
Some files were skipped or only partially analyzed.
Scan was limited to files tracked by git.
Scan skipped: 1613 files matching .semgrepignore patterns
For a full list of skipped files, run semgrep with the --verbose flag.
Ran 198 rules on 1122 files: 0 findings.
If Semgrep missed a finding, please send us feedback to let us know!
$ semgrep shouldafound --help
==> Checking source code with providerlint...
==> Checking source code with golangci-lint... |
@wedge-jarrad Thanks for the contribution 🎉 👏. |
This functionality has been released in v4.20.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
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. |
Implements stop protection for EC2 instances https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/Stop_Start.html#Using_StopProtection.
Community Note
Closes #25018
Output from acceptance testing: