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-05 #954

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Aug 5, 2022

SUMMARY

Minor linting fixups

  • docs linting and cleanup
  • unused variables
  • unused import
  • "mutually_exclusive" which was accidentally dropped on the floor.
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/cloudformation.py
plugins/modules/ec2_eni.py
plugins/modules/ec2_eni_info.py
plugins/modules/ec2_instance.py
plugins/modules/ec2_security_group.py
plugins/modules/ec2_vol.py
plugins/modules/ec2_vol_info.py
plugins/modules/ec2_vpc_dhcp_option.py
plugins/modules/ec2_vpc_net.py
plugins/modules/ec2_vpc_subnet.py
plugins/modules/elb_classic_lb.py
plugins/modules/s3_bucket.py
plugins/modules/s3_object.py

ADDITIONAL INFORMATION

Removes some documentation references to old "versions", these were Ansible versions which are long out of support.

CloudFormations module previously talked about keeping boto3 and botocore up to date. Now that we have an official policy around supported versions this is no longer as relevant and the module itself is no longer changing as fast.

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) labels Aug 5, 2022
@github-actions
Copy link

github-actions bot commented Aug 5, 2022

Docs Build 📝

Thank you for contribution!✨

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

@tremble tremble changed the title Minor Linting fixups Minor Linting fixups - 2022-08-05 Aug 5, 2022
@softwarefactory-project-zuul

This comment was marked as resolved.

@softwarefactory-project-zuul

This comment was marked as resolved.

@ansibullbot ansibullbot added integration tests/integration tests tests and removed owner_pr PR created by owner/maintainer labels Aug 8, 2022
@softwarefactory-project-zuul

This comment was marked as outdated.

@tremble
Copy link
Contributor Author

tremble commented Aug 8, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 30s
✔️ build-ansible-collection SUCCESS in 4m 54s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 7m 59s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 05s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 7m 55s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 51s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 15s
✔️ ansible-test-splitter SUCCESS in 2m 37s
✔️ integration-amazon.aws-1 SUCCESS in 36m 25s
✔️ integration-amazon.aws-2 SUCCESS in 14m 42s
✔️ integration-amazon.aws-3 SUCCESS in 29m 01s
✔️ integration-amazon.aws-4 SUCCESS in 29m 26s
⚠️ 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 29m 02s
✔️ integration-community.aws-2 SUCCESS in 25m 48s
⚠️ 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

@@ -0,0 +1,2 @@
breaking_changes:
Copy link
Member

Choose a reason for hiding this comment

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

It may be nice to isolate this specific change in a different PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll strip it from the backports

@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
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 54s
✔️ build-ansible-collection SUCCESS in 5m 43s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 56s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 8m 35s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 10m 11s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 47s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 48s
✔️ ansible-test-splitter SUCCESS in 2m 42s
✔️ integration-amazon.aws-1 SUCCESS in 36m 41s
✔️ integration-amazon.aws-2 SUCCESS in 14m 54s
✔️ integration-amazon.aws-3 SUCCESS in 26m 38s
✔️ integration-amazon.aws-4 SUCCESS in 34m 05s
⚠️ 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 SKIPPED
⚠️ 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 dd0bdcc into ansible-collections:main Aug 9, 2022
@patchback
Copy link

patchback bot commented Aug 9, 2022

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

❌ Failed to cleanly apply dd0bdcc on top of patchback/backports/stable-4/dd0bdcc099416a64bc9fb7d34ffc968274c46637/pr-954

Backporting merged PR #954 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/dd0bdcc099416a64bc9fb7d34ffc968274c46637/pr-954 upstream/stable-4
  4. Now, cherry-pick PR Minor Linting fixups - 2022-08-05 #954 contents into that branch:
    $ git cherry-pick -x dd0bdcc099416a64bc9fb7d34ffc968274c46637
    If it'll yell at you with something like fatal: Commit dd0bdcc099416a64bc9fb7d34ffc968274c46637 is a merge but no -m option was given., add -m 1 as follows intead:
    $ git cherry-pick -m1 -x dd0bdcc099416a64bc9fb7d34ffc968274c46637
  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-05 #954 as close to the
    original as possible.
  6. Push this branch to your fork on GitHub:
    $ git push origin patchback/backports/stable-4/dd0bdcc099416a64bc9fb7d34ffc968274c46637/pr-954
  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 10, 2022
…tions#954)

Manual backport of ansible-collections#954

Minor linting fixups

- docs linting and cleanup
- unused variables
- unused import
- "mutually_exclusive" which was accidentally dropped on the floor.

Feature Pull Request

plugins/modules/cloudformation.py
plugins/modules/ec2_eni.py
plugins/modules/ec2_eni_info.py
plugins/modules/ec2_instance.py
plugins/modules/ec2_security_group.py
plugins/modules/ec2_vol.py
plugins/modules/ec2_vol_info.py
plugins/modules/ec2_vpc_dhcp_option.py
plugins/modules/ec2_vpc_net.py
plugins/modules/ec2_vpc_subnet.py
plugins/modules/elb_classic_lb.py
plugins/modules/s3_bucket.py
plugins/modules/s3_object.py

Removes some documentation references to old "versions", these were Ansible versions which are long out of support.
CloudFormations module previously talked about keeping boto3 and botocore up to date.  Now that we have an official policy around supported versions this is no longer as relevant and the module itself is no longer changing as fast.

Reviewed-by: Gonéri Le Bouder
tremble added a commit to tremble/amazon.aws that referenced this pull request Aug 10, 2022
…tions#954)

Manual backport of ansible-collections#954

Minor linting fixups

- docs linting and cleanup
- unused variables
- unused import
- "mutually_exclusive" which was accidentally dropped on the floor.

Feature Pull Request

plugins/modules/cloudformation.py
plugins/modules/ec2_eni.py
plugins/modules/ec2_eni_info.py
plugins/modules/ec2_instance.py
plugins/modules/ec2_security_group.py
plugins/modules/ec2_vol.py
plugins/modules/ec2_vol_info.py
plugins/modules/ec2_vpc_dhcp_option.py
plugins/modules/ec2_vpc_net.py
plugins/modules/ec2_vpc_subnet.py
plugins/modules/elb_classic_lb.py
plugins/modules/s3_bucket.py
plugins/modules/s3_object.py

Removes some documentation references to old "versions", these were Ansible versions which are long out of support.
CloudFormations module previously talked about keeping boto3 and botocore up to date.  Now that we have an official policy around supported versions this is no longer as relevant and the module itself is no longer changing as fast.

Reviewed-by: Gonéri Le Bouder
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 25, 2022
[PR #954 backport][stable-4] Minor Linting fixups - 2022-08-05

Manual backport of #954
SUMMARY
Minor linting fixups

docs linting and cleanup
unused variables
unused import
"mutually_exclusive" which was accidentally dropped on the floor.

ISSUE TYPE
Feature Pull Request
COMPONENT NAME
plugins/modules/cloudformation.py
plugins/modules/ec2_eni.py
plugins/modules/ec2_eni_info.py
plugins/modules/ec2_instance.py
plugins/modules/ec2_security_group.py
plugins/modules/ec2_vol.py
plugins/modules/ec2_vol_info.py
plugins/modules/ec2_vpc_dhcp_option.py
plugins/modules/ec2_vpc_net.py
plugins/modules/ec2_vpc_subnet.py
plugins/modules/elb_classic_lb.py
plugins/modules/s3_bucket.py
plugins/modules/s3_object.py
ADDITIONAL INFORMATION
Removes some documentation references to old "versions", these were Ansible versions which are long out of support.
CloudFormations module previously talked about keeping boto3 and botocore up to date.  Now that we have an official policy around supported versions this is no longer as relevant and the module itself is no longer changing as fast.
Reviewed-by: Gonéri Le Bouder

Reviewed-by: Alina Buzachis <None>
@tremble tremble deleted the linting/2022-08-05 branch September 9, 2022 08:50
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 integration tests/integration mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants