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_vol - support changing from volume without IOPS to one with IOPS #687

Conversation

alinabuzachis
Copy link
Contributor

ec2_vol - support changing from volume without IOPS to one with IOPS

Backport of PR #627 as merged into main (a9b9bd1) to stable-3 as well (it was originally only backported to stable-2).
SUMMARY

Fixes #626
Changing from standard volume type to gp3 failed when the module was trying to read the "original" IOPS value from the existing volume, since it doesn't have one.
See the linked issue for detailed description.

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ec2_vol
ADDITIONAL INFORMATION

N/A

Reviewed-by: Jill R

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

ansible-collections#627)

`ec2_vol` - support changing from volume without IOPS to one with IOPS

SUMMARY

Fixes ansible-collections#626
Changing from standard volume type to gp3 failed when the module was trying to read the "original" IOPS value from the existing volume, since it doesn't have one.
See the linked issue for detailed description.

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ec2_vol
ADDITIONAL INFORMATION



N/A

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Feb 18, 2022
@markuman
Copy link
Member

recheck

@alinabuzachis
Copy link
Contributor Author

cc @goneri

ModuleNotFoundError: No module named 'markupsafe'

@felixfontein
Copy link
Contributor

@alinabuzachis the ansible-core 2.11 sanity tests (more precisely: the Python 3.7 import test) currently fail. This will be fixed beginning of next week.

Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks identical to original PR.

@alinabuzachis
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Feb 22, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 7db19ae into ansible-collections:stable-3 Feb 22, 2022
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 9, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 9, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 9, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 9, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 16, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 16, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE

New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@3661b1c
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@5e7acbd
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE


New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE


New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE


New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE


New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
New modules for RDS clusters management

SUMMARY

Two new modules for RDS clusters management (rds_cluster ans rds_cluster_info)
Took over from: ansible-collections#262
Depends-On: ansible-collections#553
Closes ansible-collections#849
Should also close ansible-collections#191

ISSUE TYPE


New Module Pull Request

Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…nsible-collections#788)

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting

SUMMARY

rds_cluster_snapshot -  new module to handle RDS cluster snapshotting
Requires rds_cluster to be merged first ansible-collections#687
Depends-On: ansible-collections#840
Requires: mattclay/aws-terminator#212
Requires also mattclay/aws-terminator#184

ISSUE TYPE


New Module Pull Request

COMPONENT NAME

rds_cluster_snapshot

Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <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 integration tests/integration mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants