Skip to content

Commit

Permalink
# This is a combination of 3 commits.
Browse files Browse the repository at this point in the history
# This is the 1st commit message:

rebase 2

# This is the commit message #2:

througput

# This is the commit message ansible-collections#3:

conditionnal attribute
  • Loading branch information
abikouo committed Jun 21, 2021
1 parent 129665d commit d12b4bd
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
minor_changes:
- ec2_vol - add parameter ``multi_attach`` to support Multi-Attach on volume creation/update (https://github.com/ansible-collections/amazon.aws/pull/362).
- ec2_vol_info - return ``attachment_set`` is now a list of attachments with Multi-Attach support on disk. (https://github.com/ansible-collections/amazon.aws/pull/362)
- ec2_vol_info - return ``attachment_set`` is now a list of attachments with Multi-Attach support on disk. (https://github.com/ansible-collections/amazon.aws/pull/362).
1 change: 0 additions & 1 deletion plugins/modules/ec2_vol.py
Original file line number Diff line number Diff line change
Expand Up @@ -489,7 +489,6 @@ def create_volume(module, ec2_conn, zone):

if throughput:
additional_params['Throughput'] = int(throughput)

if multi_attach is True:
if volume_type not in ("io1", "io2"):
module.warn("multi_attach is supported with io1 and io2 volumes only.")
Expand Down
17 changes: 17 additions & 0 deletions tests/integration/targets/ec2_vol/tasks/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -609,6 +609,23 @@
debug:
var: vol_facts

- name: Read volume information to validate throughput
ec2_vol_info:
filters:
volume-id: "{{ gp3_volume.volume_id }}"
register: verify_throughput

- name: throughput must be equal to 130
assert:
that:
- v.throughput == 130
vars:
v: "{{ verify_throughput.volumes[0] }}"

- name: print out facts
debug:
var: vol_facts

- name: increase throughput
ec2_vol:
volume_size: 7
Expand Down

0 comments on commit d12b4bd

Please sign in to comment.