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

backport-4: Minor fixups (#1328) #1393

Conversation

markuman
Copy link
Member

@markuman markuman commented Aug 4, 2022

forgotten backport-4 breaks ecs_service in 4.1.0
after it's merged, 4.1.1 must be released asap

Traceback (most recent call last):
  File "/root/.ansible/tmp/ansible-tmp-1659604767.311191-146-86085088410654/AnsiballZ_ecs_service.py", line 102, in <module>
    _ansiballz_main()
  File "/root/.ansible/tmp/ansible-tmp-1659604767.311191-146-86085088410654/AnsiballZ_ecs_service.py", line 94, in _ansiballz_main
    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)
  File "/root/.ansible/tmp/ansible-tmp-1659604767.311191-146-86085088410654/AnsiballZ_ecs_service.py", line 40, in invoke_module
    runpy.run_module(mod_name='ansible_collections.community.aws.plugins.modules.ecs_service', init_globals=None, run_name='__main__', alter_sys=True)
  File "/usr/lib/python3.8/runpy.py", line 207, in run_module
    return _run_module_code(code, init_globals, run_name, mod_spec)
  File "/usr/lib/python3.8/runpy.py", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/tmp/ansible_ecs_service_payload_jubeafm6/ansible_ecs_service_payload.zip/ansible_collections/community/aws/plugins/modules/ecs_service.py", line 1134, in <module>
  File "/tmp/ansible_ecs_service_payload_jubeafm6/ansible_ecs_service_payload.zip/ansible_collections/community/aws/plugins/modules/ecs_service.py", line 969, in main
  File "/tmp/ansible_ecs_service_payload_jubeafm6/ansible_ecs_service_payload.zip/ansible_collections/community/aws/plugins/modules/ecs_service.py", line 733, in is_matching_service
KeyError: 'deploymentController'

ecs_service - Minor fixups

SUMMARY

Bad rebase resulted in a duplicated parameter
Fixup "changed" after adding initial support for tags

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_service
ADDITIONAL INFORMATION
Bugs aren't in a released version yet, no need for changelog

Reviewed-by: Joseph Torcasso

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

ecs_service - Minor fixups

SUMMARY

Bad rebase resulted in a duplicated parameter
Fixup "changed" after adding initial support for tags

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ecs_service
ADDITIONAL INFORMATION
Bugs aren't in a released version yet, no need for changelog

Reviewed-by: Joseph Torcasso <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 traceback labels Aug 4, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 06s
✔️ build-ansible-collection SUCCESS in 5m 07s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 10s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 01s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 24s
ansible-test-sanity-docker-stable-2.13 RETRY_LIMIT in 1m 34s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 17s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 59s
✔️ ansible-test-splitter SUCCESS in 2m 26s
✔️ integration-community.aws-1 SUCCESS in 6m 28s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@markuman
Copy link
Member Author

markuman commented Aug 4, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 07s
✔️ build-ansible-collection SUCCESS in 5m 01s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 56s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 18s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 19s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 19s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 7m 02s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 52s
✔️ ansible-test-splitter SUCCESS in 2m 24s
✔️ integration-community.aws-1 SUCCESS in 5m 17s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

Copy link
Contributor

@tremble tremble left a comment

Choose a reason for hiding this comment

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

Needs a changelog

@ansibullbot ansibullbot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR and removed community_review labels Aug 4, 2022
@ansibullbot ansibullbot added community_review and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Aug 4, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 57s
✔️ build-ansible-collection SUCCESS in 4m 59s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 05s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 34s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 15s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 18s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 58s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 47s
✔️ ansible-test-splitter SUCCESS in 2m 24s
✔️ integration-community.aws-1 SUCCESS in 6m 18s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@markuman markuman requested a review from tremble August 4, 2022 11:41
@tremble
Copy link
Contributor

tremble commented Aug 4, 2022

Tests passing locally (fail without the patch)

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Aug 4, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 49s
✔️ build-ansible-collection SUCCESS in 5m 07s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 50s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 10s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 45s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 45s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 08s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 19s
✔️ ansible-test-splitter SUCCESS in 2m 38s
✔️ integration-community.aws-1 SUCCESS in 5m 22s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit b772c39 into ansible-collections:stable-4 Aug 4, 2022
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…lections#1394)

Cleanup and move tasks (vs playbooks) in inventory tests

SUMMARY
Inventory plugin tests currently mix "tasks" content in with playbooks, this triggers ansible-lint warnings.
ISSUE TYPE

Tests Pull Request

COMPONENT NAME
ec2_vpc_net
plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
ADDITIONAL INFORMATION
Spotted by work on ansible-collections#1393

Reviewed-by: Alina Buzachis
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Minor Sanity test fixes

SUMMARY
Steal sanity fixes from ansible-collections#1393
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/backup_tag_info.py
plugins/modules/backup_vault.py
plugins/modules/route53_info.py
test-requirements.txt
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
SUMMARY

Add github action workflow.

ISSUE TYPE

    Feature Pull Request
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 traceback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants