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

ecs_service - Minor fixups #1328

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jul 8, 2022

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

- bad rebase resulted in a duplicated parameter
- Fixup "changed" after adding initial support for tags
@tremble tremble requested a review from markuman July 8, 2022 17:59
@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Jul 8, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 07s
✔️ build-ansible-collection SUCCESS in 5m 09s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 19s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 08s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 22s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 47s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 06s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 11s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-community.aws-1 SUCCESS in 5m 37s
⚠️ 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

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

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 08s
✔️ build-ansible-collection SUCCESS in 6m 16s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 22s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 40s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 26s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 10s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 8m 26s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 7m 40s
✔️ ansible-test-splitter SUCCESS in 2m 51s
✔️ integration-community.aws-1 SUCCESS in 6m 27s
⚠️ 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 f3f7a29 into ansible-collections:main Jul 9, 2022
@github-actions
Copy link

github-actions bot commented Jul 9, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

markuman pushed a commit to markuman/community.aws that referenced this pull request Aug 4, 2022
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>
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 4, 2022
backport-4: Minor fixups (#1328)

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

Reviewed-by: Mark Chappell <None>
@tremble tremble deleted the ecs_service/create branch September 9, 2022 09:02
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
ec2_vpc_route_table - fix integration test after 1308

SUMMARY
ansible-collections#1308 means query is no longer needed (and no longer supported)
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_vpc_route_table
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants