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

Minor linting fixups - 2022-08-04 #953

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Aug 4, 2022

SUMMARY

Minor linting fixups

  • docs
  • unused variables
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/acm.py
plugins/module_utils/botocore.py
plugins/module_utils/cloud.py
plugins/module_utils/iam.py
plugins/module_utils/modules.py
plugins/module_utils/rds.py

ADDITIONAL INFORMATION

@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble force-pushed the linting/2022-08-04 branch from 018ce83 to b1512c9 Compare August 4, 2022 20:03
@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request inventory inventory plugin lookup lookup plugin module_utils module_utils needs_triage plugins plugin (any type) tests tests labels Aug 4, 2022
@softwarefactory-project-zuul

This comment was marked as outdated.

@softwarefactory-project-zuul

This comment was marked as outdated.

@softwarefactory-project-zuul

This comment was marked as outdated.

@softwarefactory-project-zuul

This comment was marked as outdated.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 5m 13s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 13s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 11m 01s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 32s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 13s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 14s
✔️ ansible-test-splitter SUCCESS in 3m 33s
✔️ integration-amazon.aws-1 SUCCESS in 16m 56s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 7m 22s
✔️ integration-community.aws-2 SUCCESS in 23m 32s
✔️ integration-community.aws-3 SUCCESS in 48m 43s
⚠️ 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 mergeit Merge the PR (SoftwareFactory) backport-4 PR should be backported to the stable-4 branch labels Aug 9, 2022
@softwarefactory-project-zuul

This comment was marked as outdated.

softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 9, 2022
Minor linting fixups - 2022-08-08

SUMMARY
Minor linting fixups

docs
unused variables

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/rds.py
tests/unit/plugins/inventory/test_aws_ec2.py
ADDITIONAL INFORMATION
Split from #953 due to integration test timeouts in community.aws

Reviewed-by: Gonéri Le Bouder <[email protected]>
patchback bot pushed a commit that referenced this pull request Aug 9, 2022
Minor linting fixups - 2022-08-08

SUMMARY
Minor linting fixups

docs
unused variables

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/rds.py
tests/unit/plugins/inventory/test_aws_ec2.py
ADDITIONAL INFORMATION
Split from #953 due to integration test timeouts in community.aws

Reviewed-by: Gonéri Le Bouder <[email protected]>
(cherry picked from commit 4780226)
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble
Copy link
Contributor Author

tremble commented Aug 10, 2022

regate

softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 10, 2022
[PR #958/47802269 backport][stable-4] Minor linting fixups - 2022-08-08

This is a backport of PR #958 as merged into main (4780226).
SUMMARY
Minor linting fixups

docs
unused variables

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/rds.py
tests/unit/plugins/inventory/test_aws_ec2.py
ADDITIONAL INFORMATION
Split from #953 due to integration test timeouts in community.aws

Reviewed-by: Mark Chappell <None>
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble tremble force-pushed the linting/2022-08-04 branch from bc85d19 to a8c50b2 Compare August 10, 2022 14:44
@tremble tremble force-pushed the linting/2022-08-04 branch from a8c50b2 to b9b7a1e Compare August 10, 2022 14:54
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 5m 18s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 7m 56s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 20s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 00s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 47s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 19s
✔️ ansible-test-splitter SUCCESS in 2m 33s
✔️ integration-amazon.aws-1 SUCCESS in 13m 22s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 6m 32s
✔️ integration-community.aws-2 SUCCESS in 42m 07s
⚠️ 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
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

✔️ ansible-galaxy-importer SUCCESS in 4m 16s
✔️ build-ansible-collection SUCCESS in 4m 49s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 19s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 13s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 35s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 10s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 06s
✔️ ansible-test-splitter SUCCESS in 2m 37s
✔️ integration-amazon.aws-1 SUCCESS in 11m 36s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 5m 24s
integration-community.aws-2 TIMED_OUT in 1h 00m 39s
⚠️ 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
Copy link
Contributor Author

tremble commented Aug 10, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 52s
✔️ build-ansible-collection SUCCESS in 4m 59s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 22s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 8m 32s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 56s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 54s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 14s
✔️ ansible-test-splitter SUCCESS in 2m 26s
✔️ integration-amazon.aws-1 SUCCESS in 11m 41s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 5m 11s
✔️ integration-community.aws-2 SUCCESS in 53m 36s
⚠️ 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 ffd8c1a into ansible-collections:main Aug 10, 2022
@patchback
Copy link

patchback bot commented Aug 10, 2022

Backport to stable-4: 💔 cherry-picking failed — conflicts found

❌ Failed to cleanly apply ffd8c1a on top of patchback/backports/stable-4/ffd8c1a328f13df2385239a0826c8cb84e519c27/pr-953

Backporting merged PR #953 into main

  1. Ensure you have a local repo clone of your fork. Unless you cloned it
    from the upstream, this would be your origin remote.
  2. Make sure you have an upstream repo added as a remote too. In these
    instructions you'll refer to it by the name upstream. If you don't
    have it, here's how you can add it:
    $ git remote add upstream https://github.com/ansible-collections/amazon.aws.git
  3. Ensure you have the latest copy of upstream and prepare a branch
    that will hold the backported code:
    $ git fetch upstream
    $ git checkout -b patchback/backports/stable-4/ffd8c1a328f13df2385239a0826c8cb84e519c27/pr-953 upstream/stable-4
  4. Now, cherry-pick PR Minor linting fixups - 2022-08-04 #953 contents into that branch:
    $ git cherry-pick -x ffd8c1a328f13df2385239a0826c8cb84e519c27
    If it'll yell at you with something like fatal: Commit ffd8c1a328f13df2385239a0826c8cb84e519c27 is a merge but no -m option was given., add -m 1 as follows intead:
    $ git cherry-pick -m1 -x ffd8c1a328f13df2385239a0826c8cb84e519c27
  5. At this point, you'll probably encounter some merge conflicts. You must
    resolve them in to preserve the patch from PR Minor linting fixups - 2022-08-04 #953 as close to the
    original as possible.
  6. Push this branch to your fork on GitHub:
    $ git push origin patchback/backports/stable-4/ffd8c1a328f13df2385239a0826c8cb84e519c27/pr-953
  7. Create a PR, ensure that the CI is green. If it's not — update it so that
    the tests and any other checks pass. This is it!
    Now relax and wait for the maintainers to process your pull request
    when they have some cycles to do reviews. Don't worry — they'll tell you if
    any improvements are necessary when the time comes!

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

tremble added a commit to tremble/amazon.aws that referenced this pull request Aug 11, 2022
##### SUMMARY

Minor linting fixups

- docs
- unused variables

##### ISSUE TYPE

Feature Pull Request

##### COMPONENT NAME

plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/acm.py
plugins/module_utils/botocore.py
plugins/module_utils/cloud.py
plugins/module_utils/iam.py
plugins/module_utils/modules.py
plugins/module_utils/rds.py

##### ADDITIONAL INFORMATION

Original PR
Reviewed-by: Jill R
Reviewed-by: Gonéri Le Bouder
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 17, 2022
[PR #953 backport][stable-4] Minor linting fixups - 2022-08-04

SUMMARY
Minor linting fixups

docs
unused variables

ISSUE TYPE
Feature Pull Request
COMPONENT NAME
plugins/inventory/aws_ec2.py
plugins/inventory/aws_rds.py
plugins/lookup/aws_account_attribute.py
plugins/lookup/aws_secret.py
plugins/lookup/aws_ssm.py
plugins/module_utils/acm.py
plugins/module_utils/botocore.py
plugins/module_utils/cloud.py
plugins/module_utils/iam.py
plugins/module_utils/modules.py
plugins/module_utils/rds.py
ADDITIONAL INFORMATION
Original PR
Reviewed-by: Jill R
Reviewed-by: Gonéri Le Bouder

Reviewed-by: Alina Buzachis <None>
@tremble tremble deleted the linting/2022-08-04 branch September 9, 2022 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4 PR should be backported to the stable-4 branch community_review feature This issue/PR relates to a feature request inventory inventory plugin lookup lookup plugin mergeit Merge the PR (SoftwareFactory) module_utils module_utils plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants