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

Various lint fixups #1707

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Feb 4, 2023

SUMMARY

minor linting fixups

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_ssm
batch_job_definition
inspector_target

ADDITIONAL INFORMATION

See Also: ansible-collections/news-for-maintainers#34

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review connection connection plugin feature This issue/PR relates to a feature request module module needs_maintainer needs_triage plugins plugin (any type) tests tests labels Feb 4, 2023
@tremble tremble marked this pull request as ready for review February 4, 2023 11:51
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 6m 20s
ansible-test-sanity-docker-devel RETRY_LIMIT in 1m 34s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 30s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 29s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 08s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 14s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 12s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 08s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 04s
ansible-test-units-amazon-aws-python310 POST_FAILURE in 6m 17s
✔️ ansible-test-changelog SUCCESS in 2m 22s
✔️ ansible-test-splitter SUCCESS in 2m 46s
✔️ integration-community.aws-1 SUCCESS in 11m 01s
✔️ integration-community.aws-2 SUCCESS in 9m 38s
✔️ integration-community.aws-3 SUCCESS in 10m 16s
✔️ integration-community.aws-4 SUCCESS in 10m 31s
✔️ integration-community.aws-5 SUCCESS in 9m 04s
✔️ integration-community.aws-6 SUCCESS in 9m 57s
✔️ integration-community.aws-7 SUCCESS in 8m 18s
✔️ integration-community.aws-8 SUCCESS in 9m 14s
✔️ integration-community.aws-9 SUCCESS in 9m 59s
✔️ integration-community.aws-10 SUCCESS in 6m 53s
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Feb 4, 2023
@tremble
Copy link
Contributor Author

tremble commented Feb 4, 2023

recheck

@tremble tremble added the backport-5 PR should be backported to the stable-5 branch label Feb 4, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 00s
✔️ build-ansible-collection SUCCESS in 5m 50s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 05s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 36s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 43s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 05s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 33s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 49s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 13s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 02s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 5m 56s
✔️ ansible-test-changelog SUCCESS in 2m 22s
✔️ ansible-test-splitter SUCCESS in 2m 54s
✔️ integration-community.aws-1 SUCCESS in 9m 29s
✔️ integration-community.aws-2 SUCCESS in 8m 40s
✔️ integration-community.aws-3 SUCCESS in 9m 17s
✔️ integration-community.aws-4 SUCCESS in 10m 25s
✔️ integration-community.aws-5 SUCCESS in 8m 53s
✔️ integration-community.aws-6 SUCCESS in 10m 38s
✔️ integration-community.aws-7 SUCCESS in 8m 31s
✔️ integration-community.aws-8 SUCCESS in 8m 11s
✔️ integration-community.aws-9 SUCCESS in 9m 34s
✔️ integration-community.aws-10 SUCCESS in 5m 50s
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 549ca71 into ansible-collections:main Feb 4, 2023
@patchback
Copy link

patchback bot commented Feb 4, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/549ca71c638fafe6eac30fac16f4e7eb4f8ef7b5/pr-1707

Backported as #1708

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Feb 4, 2023
Various lint fixups

SUMMARY
minor linting fixups
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
batch_job_definition
inspector_target
ADDITIONAL INFORMATION
See Also: ansible-collections/news-for-maintainers#34

Reviewed-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit 549ca71)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Feb 4, 2023
[PR #1707/549ca71c backport][stable-5] Various lint fixups

This is a backport of PR #1707 as merged into main (549ca71).
SUMMARY
minor linting fixups
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
batch_job_definition
inspector_target
ADDITIONAL INFORMATION
See Also: ansible-collections/news-for-maintainers#34

Reviewed-by: Mark Chappell <None>
@tremble tremble deleted the sanity/20230204 branch February 15, 2023 09:24
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
CI: Force PyYAML version that can be installed

SUMMARY
Sibling to ansible-collections#1917.
ISSUE TYPE

Test Pull Request

COMPONENT NAME
integration tests

Reviewed-by: Mike Graves <mgraves@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-5 PR should be backported to the stable-5 branch community_review connection connection plugin feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module module needs_maintainer needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants