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

ec2_instance - raise an error when missing permission to stop instance #756

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented Mar 29, 2022

SUMMARY

Fixes #671
When missing permission to stop an instance, the module quietly exists instead of raising an error.
This PR fixes that.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_instance

@abikouo abikouo force-pushed the ec2_instance_reboot branch from 9ccc5cd to c4e2344 Compare March 29, 2022 15:04
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review module module needs_triage plugins plugin (any type) labels Mar 29, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@abikouo abikouo added the mergeit Merge the PR (SoftwareFactory) label Apr 6, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 4483fde into ansible-collections:main Apr 6, 2022
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…le-healthcheck

Add support for disabling route53 health checks

SUMMARY
Add support for disabling route53 health checks
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
route53_health_check
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
@abikouo abikouo deleted the ec2_instance_reboot branch October 20, 2023 16:47
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…le-healthcheck

Add support for disabling route53 health checks

SUMMARY
Add support for disabling route53 health checks
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
route53_health_check
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No error when missing permission to reboot instance
3 participants