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

test: aws_msk_cluster is slow #622

Conversation

goneri
Copy link
Member

@goneri goneri commented Jul 3, 2021

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.
@ansibullbot
Copy link

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

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

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot
Copy link

@ansibullbot ansibullbot added integration tests/integration 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 plugins plugin (any type) tests tests labels Jul 3, 2021
@goneri
Copy link
Member Author

goneri commented Jul 8, 2021

recheck

@goneri goneri added the gate label Jul 8, 2021
@ansible-zuul ansible-zuul bot merged commit 4956f97 into ansible-collections:main Jul 8, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 15, 2021
…-aws_msk_cluster-is-slow_26143

test: aws_msk_cluster is slow

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@4956f97
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
…-aws_msk_cluster-is-slow_26143

test: aws_msk_cluster is slow

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@4956f97
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 17, 2021
…-aws_msk_cluster-is-slow_26143

test: aws_msk_cluster is slow

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@74a5aa7
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…luster-is-slow_26143

test: aws_msk_cluster is slow

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…luster-is-slow_26143

test: aws_msk_cluster is slow

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…luster-is-slow_26143

test: aws_msk_cluster is slow

The aws_msk_cluster test takes a bit of time. Let's be sure this does
not become a problem in the future by isolating it on its own
dedicated job.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
…e-collections#646)

Handle ResourceNotFoundException while iterating certificates

SUMMARY

The module/utils/acm.py was not correctly handling deletion of certificates. While iterating over a list of certificates, the get_certificate function was making API calls to obtain more information about the certificates, but some certificates may be deleted while iterating.
Fixes ansible-collections#622

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

acm.py
ADDITIONAL INFORMATION



Wow, it seems many tests are very flaky. I'm attempting to fix an issue in ACM, but problems occur elsewhere. Not to mention I raised this PR to fix ansible-collections#622, which was discovered while working on ansible-collections#870. And I discovered other issues as well, and so it looks like it's not possible to make any progress without going down a tree of bug fixes.
TypeError: 'NoneType' object is not subscriptable
fatal: [testhost]: FAILED! => {
    "changed": false,
    "module_stderr": "Traceback (most recent call last):\n  File \"<stdin>\", line 121, in <module>\n  File \"<stdin>\", line 113, in _ansiballz_main\n  File \"<stdin>\", line 61, in invoke_module\n  File \"/usr/lib64/python3.8/runpy.py\", line 207, in run_module\n    return _run_module_code(code, init_globals, run_name, mod_spec)\n  File \"/usr/lib64/python3.8/runpy.py\", line 97, in _run_module_code\n    _run_code(code, mod_globals, init_globals,\n  File \"/usr/lib64/python3.8/runpy.py\", line 87, in _run_code\n    exec(code, run_globals)\n  File \"/tmp/ansible_ec2_vpc_igw_payload_g4o1kl_r/ansible_ec2_vpc_igw_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vpc_igw.py\", line 248, in <module>\n  File \"/tmp/ansible_ec2_vpc_igw_payload_g4o1kl_r/ansible_ec2_vpc_igw_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vpc_igw.py\", line 242, in main\n  File \"/tmp/ansible_ec2_vpc_igw_payload_g4o1kl_r/ansible_ec2_vpc_igw_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vpc_igw.py\", line 132, in process\n  File \"/tmp/ansible_ec2_vpc_igw_payload_g4o1kl_r/ansible_ec2_vpc_igw_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vpc_igw.py\", line 220, in ensure_igw_present\n  File \"/tmp/ansible_ec2_vpc_igw_payload_g4o1kl_r/ansible_ec2_vpc_igw_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vpc_igw.py\", line 158, in get_igw_info\nTypeError: 'NoneType' object is not subscriptable\n",
    "module_stdout": "",
    "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error",
    "rc": 1
}

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Jill R <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration tests/integration 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 plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants