Unstable integration tests ec2_eip #159

tremble · 2020-07-25T11:47:09Z

Test is susceptible to race conditions when running concurrently in the same Region

https://app.shippable.com/github/ansible-collections/community.aws/runs/438/27/tests
https://app.shippable.com/github/ansible-collections/community.aws/runs/438/23/tests
https://app.shippable.com/github/ansible-collections/community.aws/runs/438/24/tests
https://app.shippable.com/github/ansible-collections/community.aws/runs/439/27/tests
https://app.shippable.com/github/ansible-collections/community.aws/runs/440/23/tests
https://app.shippable.com/github/ansible-collections/community.aws/runs/440/26/tests

…rates over the last week ansible-collections#158 ansible-collections#157 ansible-collections#159 ansible-collections#153 ansible-collections#154 ansible-collections#155 ansible-collections#156

…rates over the last week (#160) #158 #157 #159 #153 #154 #155 #156

ansibullbot · 2020-08-19T23:25:28Z

Files identified in the description:
None

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

ansibullbot · 2020-08-19T23:25:29Z

@tremble: Greetings! Thanks for taking the time to open this issue. In order for the community to handle your issue effectively, we need a bit more information.

Here are the items we could not find in your description:

issue type
ansible version
component name

Please set the description of this issue with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/ISSUE_TEMPLATE/bug_report.md

click here for bot help

…rates over the last week (ansible-collections#160) ansible-collections#158 ansible-collections#157 ansible-collections#159 ansible-collections#153 ansible-collections#154 ansible-collections#155 ansible-collections#156

jatorcasso · 2022-02-15T16:48:43Z

@tremble @jillr @alinabuzachis what's a good way to validate that this is fixed? I updated the integration tests and am not running into this issue, but unsure if it's actually fixed or not

alinabuzachis · 2022-02-15T18:05:21Z

@tremble @jillr @alinabuzachis what's a good way to validate that this is fixed? I updated the integration tests and am not running into this issue, but unsure if it's actually fixed or not

I would probably go through the CI logs tremble listed there and check if that issues are solved or still happen with your changes.

jatorcasso · 2022-02-15T18:08:56Z

Thanks! I didn't run into any of those failures so I'll go ahead and link it

Stabilize ec2_eip module SUMMARY fixed check_mode issues added integration tests for check_mode / idempotency updated json returned when state = absent for clarity removed json_query references fixes #159 Depends-On: ansible-collections/amazon.aws#672 ISSUE TYPE Feature Pull Request COMPONENT NAME ec2_eip Reviewed-by: Mark Woolley <[email protected]> Reviewed-by: Mark Chappell <None> Reviewed-by: Alina Buzachis <None> Reviewed-by: Joseph Torcasso <None> Reviewed-by: Jill R <None>

backport-3: Stabilize ec2_eip module (#936) Backport into stable-3 of #936 with commit id: d0596e3 SUMMARY fixed check_mode issues added integration tests for check_mode / idempotency updated json returned when state = absent for clarity removed json_query references fixes #159 Depends-On: ansible-collections/amazon.aws#672 ISSUE TYPE Feature Pull Request COMPONENT NAME ec2_eip Reviewed-by: Mark Woolley Reviewed-by: Mark Chappell Reviewed-by: Alina Buzachis Reviewed-by: Joseph Torcasso Reviewed-by: Jill R SUMMARY ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION

…ble-collections#159) * Add is_boto3_errror_message partner to is_boto3_errror_code Amazon has some examples where the exceptions they return are very broad. Where we want to special case behaviour for specific failure modes it_boto3_error_message can help avoid the need to re-raise the exception and mess with the tracebacks. * Add specific examples for is_boto3_error_message * Add initial unit tests for is_boto3_error_message()

…rates over the last week (ansible-collections#160) ansible-collections/community.aws#158 ansible-collections/community.aws#157 ansible-collections/community.aws#159 ansible-collections/community.aws#153 ansible-collections/community.aws#154 ansible-collections/community.aws#155 ansible-collections/community.aws#156 This commit was initially merged in https://github.com/ansible-collections/community.aws See: ansible-collections/community.aws@61783c7

Stabilize ec2_eip module SUMMARY fixed check_mode issues added integration tests for check_mode / idempotency updated json returned when state = absent for clarity removed json_query references fixes ansible-collections#159 Depends-On: ansible-collections/amazon.aws#672 ISSUE TYPE Feature Pull Request COMPONENT NAME ec2_eip Reviewed-by: Mark Woolley <[email protected]> Reviewed-by: Mark Chappell <None> Reviewed-by: Alina Buzachis <None> Reviewed-by: Joseph Torcasso <None> Reviewed-by: Jill R <None> This commit was initially merged in https://github.com/ansible-collections/community.aws See: ansible-collections@d0596e3

…rates over the last week (ansible-collections#160) ansible-collections/community.aws#158 ansible-collections/community.aws#157 ansible-collections/community.aws#159 ansible-collections/community.aws#153 ansible-collections/community.aws#154 ansible-collections/community.aws#155 ansible-collections/community.aws#156 This commit was initially merged in https://github.com/ansible-collections/community.aws See: ansible-collections/community.aws@61783c7

tremble mentioned this issue Jul 25, 2020

Mark a number of integration tests as unstable based on high failure rates over the last week #160

Merged

jillr pushed a commit that referenced this issue Jul 27, 2020

Mark a number of integration tests as unstable based on high failure …

61783c7

…rates over the last week (#160) #158 #157 #159 #153 #154 #155 #156

ansibullbot added affects_2.10 has_pr needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage labels Aug 19, 2020

goneri added CI and removed needs_triage labels Feb 10, 2021

markuman mentioned this issue Feb 1, 2022

4.0.0 release plan #895

Closed

1 task

alinabuzachis mentioned this issue Feb 15, 2022

Stabilize ec2_eip module #936

Merged

softwarefactory-project-zuul bot closed this as completed in #936 Feb 16, 2022

markuman mentioned this issue Feb 22, 2022

backport-3: Stabilize ec2_eip module (#936) #946

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Unstable integration tests ec2_eip #159

Unstable integration tests ec2_eip #159

tremble commented Jul 25, 2020

ansibullbot commented Aug 19, 2020

ansibullbot commented Aug 19, 2020

jatorcasso commented Feb 15, 2022

alinabuzachis commented Feb 15, 2022

jatorcasso commented Feb 15, 2022

Unstable integration tests ec2_eip #159

Unstable integration tests ec2_eip #159

Comments

tremble commented Jul 25, 2020

ansibullbot commented Aug 19, 2020

ansibullbot commented Aug 19, 2020

jatorcasso commented Feb 15, 2022

alinabuzachis commented Feb 15, 2022

jatorcasso commented Feb 15, 2022