GomathiselviS comments

Results 101 comments of


                                            GomathiselviS

trafficstars

cloudwatch_metric_alarm: fix idempotency for alarm without dimensions

> Hi @GomathiselviS thank you for taking the time to review my PR. I've added the assertion task and merge the latest main commits into the branch. Could you please...

Sanity fixes

/retest-required

Sanity fixes

/close

fixes ec2_transit_gateway bug with empty description.

Hi @paulczar I have pushed the changelog fragment. Once approved, I shall merge this PR.

Refactor modules ec2_ami, ec2_eni, ec2_security_group, ec2_key, ec2_vol, ec2_import_image, ec2_eip*

Hi @abikouo I appreciate the effort you’ve put into this PR. To facilitate a more efficient and quicker review process, could you please break this PR into multiple smaller PRs?

Use configure_ci tool to scaffold gh workflows

> @GomathiselviS Has this PR been abandoned? No, it hasn't been abandoned; rather, it has been moved to the backlog to prioritize high-impact tasks. I will resume work on it...

fix(ec2_vol): output volume informations when volume exists in check mode

Hi @theredcat Thank you for working on this PR. Could you please rebase the branch and add a changelog to describe the changes made?

ec2_vpc_vpn - Add additional VPN tunnel options

Thanks for submitting this PR. Can you please add integration tests to test the feature?

ec2_vol: Add volume initialization rate when creating from snapshot

Hi @dmarav, If you need to run your tests against a specific version of boto, you can do that by setting the botocore_virtualenv_interpreter variable. We have a good example of...

ec2_vol: Add volume initialization rate when creating from snapshot

Hi @dmarav , Please check [this file](https://github.com/ansible-collections/amazon.aws/blob/stable-9/tests/integration/targets/backup_plan/meta/main.yml). Let me know if this helps.