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

Big Black PR #1784

Merged
merged 7 commits into from Apr 26, 2023
Merged

Big Black PR #1784

merged 7 commits into from Apr 26, 2023

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Apr 24, 2023

SUMMARY

In preparation for 6.0.0 we agreed to mass apply the black formatter to our code base.

Hopefully we're only a couple of weeks out now, and we're much more on top of our PRs than we used to be. This is the big PR...

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/
tests/

ADDITIONAL INFORMATION

There are two commits, the first "pre-tunes" a few things so that they're not exploded into lots of extra lines, or squashed into one line. The second is just the result of black plugins/ tests/

See also: #1798

@tremble tremble marked this pull request as ready for review April 24, 2023 09:27
Copy link
Contributor

@alinabuzachis alinabuzachis left a comment

Choose a reason for hiding this comment

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

@tremble Please add a changelog fragment.

@tremble
Copy link
Contributor Author

tremble commented Apr 24, 2023

@tremble Please add a changelog fragment.

Done

@tremble tremble added mergeit Merge the PR (SoftwareFactory) and removed mergeit Merge the PR (SoftwareFactory) labels Apr 24, 2023
@softwarefactory-project-zuul

This comment was marked as resolved.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/cc78999844d24b6ca10293ba79cfa50f

ansible-galaxy-importer FAILURE in 3m 59s
✔️ build-ansible-collection SUCCESS in 13m 00s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 20s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 58s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 22s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 18m 39s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 53s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 39s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 01s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 49s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 11s
✔️ ansible-test-changelog SUCCESS in 4m 25s
✔️ ansible-test-splitter SUCCESS in 4m 57s
✔️ integration-community.aws-1 SUCCESS in 33m 59s
✔️ integration-community.aws-2 SUCCESS in 22m 13s
✔️ integration-community.aws-3 SUCCESS in 7m 47s
✔️ integration-community.aws-4 SUCCESS in 38m 24s
✔️ integration-community.aws-5 SUCCESS in 38m 21s
✔️ integration-community.aws-6 SUCCESS in 6m 24s
✔️ integration-community.aws-7 SUCCESS in 23m 14s
✔️ integration-community.aws-8 SUCCESS in 28m 03s
✔️ integration-community.aws-9 SUCCESS in 35m 19s
✔️ integration-community.aws-10 SUCCESS in 22m 44s
✔️ integration-community.aws-11 SUCCESS in 29m 32s
✔️ integration-community.aws-12 SUCCESS in 34m 51s
✔️ integration-community.aws-13 SUCCESS in 27m 59s
✔️ integration-community.aws-14 SUCCESS in 22m 08s
✔️ integration-community.aws-15 SUCCESS in 41m 10s
✔️ integration-community.aws-16 SUCCESS in 32m 06s
✔️ integration-community.aws-17 SUCCESS in 21m 41s
✔️ integration-community.aws-18 SUCCESS in 21m 08s
✔️ integration-community.aws-19 SUCCESS in 6m 08s
✔️ integration-community.aws-20 SUCCESS in 11m 22s
Skipped 2 jobs

@tremble
Copy link
Contributor Author

tremble commented Apr 26, 2023

regate

@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

https://ansible.softwarefactory-project.io/zuul/buildset/cb64136489f54cbab98f4cad2c944b0f

✔️ ansible-galaxy-importer SUCCESS in 3m 54s
✔️ build-ansible-collection SUCCESS in 13m 20s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 20s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 08s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 16s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 12m 09s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 12m 18s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 05s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 56s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 8m 11s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 9m 13s
✔️ ansible-test-changelog SUCCESS in 4m 21s
✔️ ansible-test-splitter SUCCESS in 4m 48s
✔️ integration-community.aws-1 SUCCESS in 28m 53s
✔️ integration-community.aws-2 SUCCESS in 21m 40s
✔️ integration-community.aws-3 SUCCESS in 5m 06s
integration-community.aws-4 POST_FAILURE in 43m 57s
✔️ integration-community.aws-5 SUCCESS in 53m 18s
✔️ integration-community.aws-6 SUCCESS in 7m 43s
✔️ integration-community.aws-7 SUCCESS in 27m 07s
✔️ integration-community.aws-8 SUCCESS in 21m 18s
✔️ integration-community.aws-9 SUCCESS in 34m 08s
✔️ integration-community.aws-10 SUCCESS in 23m 10s
✔️ integration-community.aws-11 SUCCESS in 30m 29s
✔️ integration-community.aws-12 SUCCESS in 31m 41s
✔️ integration-community.aws-13 SUCCESS in 21m 58s
✔️ integration-community.aws-14 SUCCESS in 43m 38s
✔️ integration-community.aws-15 SUCCESS in 15m 32s
integration-community.aws-16 TIMED_OUT in 1h 00m 32s
✔️ integration-community.aws-17 SUCCESS in 8m 32s
✔️ integration-community.aws-18 SUCCESS in 10m 26s
✔️ integration-community.aws-19 SUCCESS in 18m 42s
✔️ integration-community.aws-20 SUCCESS in 11m 56s
Skipped 2 jobs

@tremble
Copy link
Contributor Author

tremble commented Apr 26, 2023

regate

@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

https://ansible.softwarefactory-project.io/zuul/buildset/a50ef1c0b9464b54a5e45d87f0dcedf0

ansible-galaxy-importer FAILURE in 4m 09s
✔️ build-ansible-collection SUCCESS in 13m 37s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 27s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 11s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 14m 39s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 29s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 12m 13s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 21s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 38s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 47s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 9m 46s
✔️ ansible-test-changelog SUCCESS in 5m 36s
✔️ ansible-test-splitter SUCCESS in 4m 50s
✔️ integration-community.aws-1 SUCCESS in 29m 13s
✔️ integration-community.aws-2 SUCCESS in 24m 31s
✔️ integration-community.aws-3 SUCCESS in 6m 22s
✔️ integration-community.aws-4 SUCCESS in 39m 58s
✔️ integration-community.aws-5 SUCCESS in 47m 03s
✔️ integration-community.aws-6 SUCCESS in 22m 05s
✔️ integration-community.aws-7 SUCCESS in 25m 11s
✔️ integration-community.aws-8 SUCCESS in 31m 42s
✔️ integration-community.aws-9 SUCCESS in 35m 45s
✔️ integration-community.aws-10 SUCCESS in 24m 40s
✔️ integration-community.aws-11 SUCCESS in 31m 16s
✔️ integration-community.aws-12 SUCCESS in 22m 12s
✔️ integration-community.aws-13 SUCCESS in 24m 13s
✔️ integration-community.aws-14 SUCCESS in 41m 46s
✔️ integration-community.aws-15 SUCCESS in 15m 47s
✔️ integration-community.aws-16 SUCCESS in 45m 23s
✔️ integration-community.aws-17 SUCCESS in 8m 47s
✔️ integration-community.aws-18 SUCCESS in 10m 10s
✔️ integration-community.aws-19 SUCCESS in 18m 26s
✔️ integration-community.aws-20 SUCCESS in 10m 37s
Skipped 2 jobs

@tremble
Copy link
Contributor Author

tremble commented Apr 26, 2023

regate

@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

https://ansible.softwarefactory-project.io/zuul/buildset/1d0904938bc84c7d85552b969c3ade46

ansible-galaxy-importer FAILURE in 5m 33s
✔️ build-ansible-collection SUCCESS in 12m 41s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 24s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 20s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 02s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 55s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 07s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 33s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 38s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 34s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 57s
✔️ ansible-test-changelog SUCCESS in 4m 23s
✔️ ansible-test-splitter SUCCESS in 4m 56s
✔️ integration-community.aws-1 SUCCESS in 56m 16s
✔️ integration-community.aws-2 SUCCESS in 21m 51s
✔️ integration-community.aws-3 SUCCESS in 5m 15s
✔️ integration-community.aws-4 SUCCESS in 38m 04s
✔️ integration-community.aws-5 SUCCESS in 42m 15s
✔️ integration-community.aws-6 SUCCESS in 5m 51s
✔️ integration-community.aws-7 SUCCESS in 30m 38s
✔️ integration-community.aws-8 SUCCESS in 25m 24s
✔️ integration-community.aws-9 SUCCESS in 31m 10s
✔️ integration-community.aws-10 SUCCESS in 23m 41s
✔️ integration-community.aws-11 SUCCESS in 34m 20s
✔️ integration-community.aws-12 SUCCESS in 30m 25s
✔️ integration-community.aws-13 SUCCESS in 23m 29s
✔️ integration-community.aws-14 SUCCESS in 43m 35s
✔️ integration-community.aws-15 SUCCESS in 14m 37s
✔️ integration-community.aws-16 SUCCESS in 43m 53s
✔️ integration-community.aws-17 SUCCESS in 8m 49s
✔️ integration-community.aws-18 SUCCESS in 10m 17s
✔️ integration-community.aws-19 SUCCESS in 20m 57s
✔️ integration-community.aws-20 SUCCESS in 10m 56s
Skipped 2 jobs

@tremble
Copy link
Contributor Author

tremble commented Apr 26, 2023

regate

@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

https://ansible.softwarefactory-project.io/zuul/buildset/583b81753e0b45058a974b51d3e03dc9

ansible-galaxy-importer FAILURE in 3m 49s
✔️ build-ansible-collection SUCCESS in 13m 27s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 32s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 34s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 51s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 18s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 12m 42s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 12s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 46s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 06s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 9m 04s
✔️ ansible-test-changelog SUCCESS in 4m 21s
✔️ ansible-test-splitter SUCCESS in 4m 59s
✔️ integration-community.aws-1 SUCCESS in 34m 50s
✔️ integration-community.aws-2 SUCCESS in 23m 08s
✔️ integration-community.aws-3 SUCCESS in 5m 54s
✔️ integration-community.aws-4 SUCCESS in 38m 33s
✔️ integration-community.aws-5 SUCCESS in 41m 48s
✔️ integration-community.aws-6 SUCCESS in 24m 19s
✔️ integration-community.aws-7 SUCCESS in 27m 57s
✔️ integration-community.aws-8 SUCCESS in 23m 00s
✔️ integration-community.aws-9 SUCCESS in 35m 26s
✔️ integration-community.aws-10 SUCCESS in 24m 18s
✔️ integration-community.aws-11 SUCCESS in 32m 02s
✔️ integration-community.aws-12 SUCCESS in 22m 48s
✔️ integration-community.aws-13 SUCCESS in 22m 18s
✔️ integration-community.aws-14 SUCCESS in 43m 36s
✔️ integration-community.aws-15 SUCCESS in 16m 04s
✔️ integration-community.aws-16 SUCCESS in 46m 10s
✔️ integration-community.aws-17 SUCCESS in 8m 48s
✔️ integration-community.aws-18 SUCCESS in 11m 59s
✔️ integration-community.aws-19 SUCCESS in 20m 05s
✔️ integration-community.aws-20 SUCCESS in 10m 06s
Skipped 2 jobs

Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

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

Manually merging

@jillr jillr merged commit 2c4575c into ansible-collections:main Apr 26, 2023
6 of 9 checks passed
@github-actions
Copy link

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 mentioned this pull request Apr 27, 2023
softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 3, 2023
black - aws_ssm

SUMMARY
The aws_ssm integration tests are currently broken (#1793), apply the black changes
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION
changes were approved as a part of #1784 but split because the tests were broken

Reviewed-by: Alina Buzachis
rmahroua pushed a commit to rmahroua/community.aws that referenced this pull request May 4, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests
rmahroua pushed a commit to rmahroua/community.aws that referenced this pull request May 4, 2023
black - aws_ssm

SUMMARY
The aws_ssm integration tests are currently broken (ansible-collections#1793), apply the black changes
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION
changes were approved as a part of ansible-collections#1784 but split because the tests were broken

Reviewed-by: Alina Buzachis
@tremble tremble deleted the 6.0.0/black branch September 29, 2023 08:28
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Black prep

* Black

* changelog

* Fix pylint unused-import in tests

* Split SSM connection plugin changes

* disable glue tests - bucket's missing

* Disable s3_logging and s3_sync tests

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@2c4575c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants