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

[7.0.0] sort imports #1771

Merged
merged 5 commits into from
Sep 29, 2023
Merged

[7.0.0] sort imports #1771

merged 5 commits into from
Sep 29, 2023

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Sep 27, 2023

SUMMARY

Sort imports and add isort to the tox triggered linting

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

tox.ini

ADDITIONAL INFORMATION

@tremble tremble added this to the 7.0.0 milestone Sep 27, 2023
@tremble tremble linked an issue Sep 27, 2023 that may be closed by this pull request
1 task
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 26s
✔️ build-ansible-collection SUCCESS in 12m 56s
✔️ ansible-test-splitter SUCCESS in 5m 13s
✔️ integration-amazon.aws-1 SUCCESS in 29m 49s
✔️ integration-amazon.aws-2 SUCCESS in 35m 22s
integration-amazon.aws-3 RETRY_LIMIT in 1m 44s
✔️ integration-amazon.aws-4 SUCCESS in 35m 11s
✔️ integration-amazon.aws-5 SUCCESS in 29m 30s
✔️ integration-amazon.aws-6 SUCCESS in 47m 13s
✔️ integration-amazon.aws-7 SUCCESS in 18m 33s
✔️ integration-amazon.aws-8 SUCCESS in 37m 15s
✔️ integration-amazon.aws-9 SUCCESS in 43m 44s
✔️ integration-amazon.aws-10 SUCCESS in 34m 13s
✔️ integration-amazon.aws-11 SUCCESS in 39m 51s
✔️ integration-amazon.aws-12 SUCCESS in 15m 14s
✔️ integration-amazon.aws-13 SUCCESS in 25m 38s
✔️ integration-amazon.aws-14 SUCCESS in 25m 52s
✔️ integration-amazon.aws-15 SUCCESS in 44m 51s
✔️ integration-amazon.aws-16 SUCCESS in 37m 29s
✔️ integration-amazon.aws-17 SUCCESS in 46m 36s
✔️ integration-amazon.aws-18 SUCCESS in 41m 21s
✔️ integration-amazon.aws-19 SUCCESS in 41m 18s
✔️ integration-amazon.aws-20 SUCCESS in 23m 41s
✔️ integration-amazon.aws-21 SUCCESS in 35m 08s
✔️ integration-amazon.aws-22 SUCCESS in 19m 24s
integration-community.aws-1 FAILURE in 58m 01s
✔️ integration-community.aws-2 SUCCESS in 22m 53s
✔️ integration-community.aws-3 SUCCESS in 7m 40s
✔️ integration-community.aws-4 SUCCESS in 23m 59s
✔️ integration-community.aws-5 SUCCESS in 32m 39s
✔️ integration-community.aws-6 SUCCESS in 18m 34s
✔️ integration-community.aws-7 SUCCESS in 24m 14s
✔️ integration-community.aws-8 SUCCESS in 13m 37s
✔️ integration-community.aws-9 SUCCESS in 15m 50s
✔️ integration-community.aws-10 SUCCESS in 28m 28s
✔️ integration-community.aws-11 SUCCESS in 32m 41s
✔️ integration-community.aws-12 SUCCESS in 18m 00s
✔️ integration-community.aws-13 SUCCESS in 34m 58s
✔️ integration-community.aws-14 SUCCESS in 11m 44s
✔️ integration-community.aws-15 SUCCESS in 11m 25s
integration-community.aws-16 FAILURE in 53m 07s
✔️ integration-community.aws-17 SUCCESS in 21m 16s
Skipped 5 jobs

@tremble tremble requested review from alinabuzachis and hakbailey and removed request for alinabuzachis September 27, 2023 14:17
@tremble tremble marked this pull request as ready for review September 27, 2023 14:17
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 50s
✔️ build-ansible-collection SUCCESS in 13m 13s
✔️ ansible-test-splitter SUCCESS in 4m 40s
✔️ integration-amazon.aws-1 SUCCESS in 29m 23s
✔️ integration-amazon.aws-2 SUCCESS in 33m 58s
✔️ integration-amazon.aws-3 SUCCESS in 17m 58s
✔️ integration-amazon.aws-4 SUCCESS in 33m 35s
✔️ integration-amazon.aws-5 SUCCESS in 32m 03s
✔️ integration-amazon.aws-6 SUCCESS in 46m 58s
✔️ integration-amazon.aws-7 SUCCESS in 16m 21s
✔️ integration-amazon.aws-8 SUCCESS in 47m 18s
✔️ integration-amazon.aws-9 SUCCESS in 41m 28s
✔️ integration-amazon.aws-10 SUCCESS in 36m 38s
✔️ integration-amazon.aws-11 SUCCESS in 37m 26s
✔️ integration-amazon.aws-12 SUCCESS in 13m 55s
✔️ integration-amazon.aws-13 SUCCESS in 26m 41s
✔️ integration-amazon.aws-14 SUCCESS in 29m 30s
✔️ integration-amazon.aws-15 SUCCESS in 45m 30s
✔️ integration-amazon.aws-16 SUCCESS in 42m 24s
✔️ integration-amazon.aws-17 SUCCESS in 45m 32s
✔️ integration-amazon.aws-18 SUCCESS in 40m 19s
✔️ integration-amazon.aws-19 SUCCESS in 46m 59s
✔️ integration-amazon.aws-20 SUCCESS in 25m 37s
✔️ integration-amazon.aws-21 SUCCESS in 34m 57s
✔️ integration-amazon.aws-22 SUCCESS in 20m 48s
✔️ integration-community.aws-1 SUCCESS in 35m 42s
✔️ integration-community.aws-2 SUCCESS in 23m 19s
✔️ integration-community.aws-3 SUCCESS in 8m 15s
✔️ integration-community.aws-4 SUCCESS in 21m 41s
✔️ integration-community.aws-5 SUCCESS in 27m 46s
✔️ integration-community.aws-6 SUCCESS in 16m 40s
✔️ integration-community.aws-7 SUCCESS in 24m 28s
✔️ integration-community.aws-8 SUCCESS in 21m 36s
✔️ integration-community.aws-9 SUCCESS in 16m 52s
✔️ integration-community.aws-10 SUCCESS in 26m 11s
✔️ integration-community.aws-11 SUCCESS in 22m 39s
✔️ integration-community.aws-12 SUCCESS in 16m 51s
✔️ integration-community.aws-13 SUCCESS in 34m 04s
integration-community.aws-14 FAILURE in 26m 19s
✔️ integration-community.aws-15 SUCCESS in 12m 10s
✔️ integration-community.aws-16 SUCCESS in 29m 40s
✔️ integration-community.aws-17 SUCCESS in 21m 10s
Skipped 5 jobs

@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 41s
✔️ build-ansible-collection SUCCESS in 13m 14s
✔️ ansible-test-splitter SUCCESS in 4m 52s
✔️ integration-amazon.aws-1 SUCCESS in 36m 15s
integration-amazon.aws-2 TIMED_OUT in 1h 00m 52s
✔️ integration-amazon.aws-3 SUCCESS in 18m 56s
✔️ integration-amazon.aws-4 SUCCESS in 36m 37s
✔️ integration-amazon.aws-5 SUCCESS in 32m 48s
✔️ integration-amazon.aws-6 SUCCESS in 42m 17s
✔️ integration-amazon.aws-7 SUCCESS in 19m 10s
✔️ integration-amazon.aws-8 SUCCESS in 44m 56s
✔️ integration-amazon.aws-9 SUCCESS in 42m 35s
✔️ integration-amazon.aws-10 SUCCESS in 45m 28s
✔️ integration-amazon.aws-11 SUCCESS in 40m 50s
✔️ integration-amazon.aws-12 SUCCESS in 10m 46s
✔️ integration-amazon.aws-13 SUCCESS in 23m 48s
✔️ integration-amazon.aws-14 SUCCESS in 31m 09s
✔️ integration-amazon.aws-15 SUCCESS in 45m 52s
✔️ integration-amazon.aws-16 SUCCESS in 42m 39s
✔️ integration-amazon.aws-17 SUCCESS in 43m 39s
✔️ integration-amazon.aws-18 SUCCESS in 36m 22s
✔️ integration-amazon.aws-19 SUCCESS in 47m 08s
✔️ integration-amazon.aws-20 SUCCESS in 28m 24s
✔️ integration-amazon.aws-21 SUCCESS in 36m 21s
✔️ integration-amazon.aws-22 SUCCESS in 19m 46s
✔️ integration-community.aws-1 SUCCESS in 35m 57s
✔️ integration-community.aws-2 SUCCESS in 24m 31s
✔️ integration-community.aws-3 SUCCESS in 6m 23s
✔️ integration-community.aws-4 SUCCESS in 20m 56s
✔️ integration-community.aws-5 SUCCESS in 32m 18s
✔️ integration-community.aws-6 SUCCESS in 16m 34s
✔️ integration-community.aws-7 SUCCESS in 23m 30s
✔️ integration-community.aws-8 SUCCESS in 19m 03s
✔️ integration-community.aws-9 SUCCESS in 20m 07s
✔️ integration-community.aws-10 SUCCESS in 27m 29s
✔️ integration-community.aws-11 SUCCESS in 16m 23s
✔️ integration-community.aws-12 SUCCESS in 19m 33s
✔️ integration-community.aws-13 SUCCESS in 30m 13s
✔️ integration-community.aws-14 SUCCESS in 14m 27s
✔️ integration-community.aws-15 SUCCESS in 24m 41s
✔️ integration-community.aws-16 SUCCESS in 34m 12s
✔️ integration-community.aws-17 SUCCESS in 8m 47s
Skipped 5 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Sep 29, 2023
@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/bd7fb8c14a0442e6b22b1341339356da

✔️ ansible-galaxy-importer SUCCESS in 5m 23s
✔️ build-ansible-collection SUCCESS in 13m 26s
✔️ ansible-test-splitter SUCCESS in 4m 45s
✔️ integration-amazon.aws-1 SUCCESS in 31m 18s
integration-amazon.aws-2 FAILURE in 51m 06s
✔️ integration-amazon.aws-3 SUCCESS in 19m 09s
✔️ integration-amazon.aws-4 SUCCESS in 37m 17s
✔️ integration-amazon.aws-5 SUCCESS in 30m 28s
integration-amazon.aws-6 TIMED_OUT in 1h 00m 56s
✔️ integration-amazon.aws-7 SUCCESS in 19m 25s
✔️ integration-amazon.aws-8 SUCCESS in 37m 37s
✔️ integration-amazon.aws-9 SUCCESS in 46m 39s
✔️ integration-amazon.aws-10 SUCCESS in 38m 51s
✔️ integration-amazon.aws-11 SUCCESS in 39m 24s
✔️ integration-amazon.aws-12 SUCCESS in 12m 18s
✔️ integration-amazon.aws-13 SUCCESS in 20m 54s
✔️ integration-amazon.aws-14 SUCCESS in 26m 35s
✔️ integration-amazon.aws-15 SUCCESS in 43m 41s
✔️ integration-amazon.aws-16 SUCCESS in 41m 46s
✔️ integration-amazon.aws-17 SUCCESS in 40m 21s
✔️ integration-amazon.aws-18 SUCCESS in 36m 30s
✔️ integration-amazon.aws-19 SUCCESS in 39m 55s
✔️ integration-amazon.aws-20 SUCCESS in 34m 03s
✔️ integration-amazon.aws-21 SUCCESS in 40m 29s
✔️ integration-amazon.aws-22 SUCCESS in 19m 52s
✔️ integration-community.aws-1 SUCCESS in 41m 03s
✔️ integration-community.aws-2 SUCCESS in 21m 38s
✔️ integration-community.aws-3 SUCCESS in 23m 00s
✔️ integration-community.aws-4 SUCCESS in 19m 48s
✔️ integration-community.aws-5 SUCCESS in 32m 37s
✔️ integration-community.aws-6 SUCCESS in 16m 47s
✔️ integration-community.aws-7 SUCCESS in 25m 22s
✔️ integration-community.aws-8 SUCCESS in 15m 37s
✔️ integration-community.aws-9 SUCCESS in 16m 26s
✔️ integration-community.aws-10 SUCCESS in 31m 29s
✔️ integration-community.aws-11 SUCCESS in 15m 39s
✔️ integration-community.aws-12 SUCCESS in 16m 49s
✔️ integration-community.aws-13 SUCCESS in 30m 24s
✔️ integration-community.aws-14 SUCCESS in 12m 11s
✔️ integration-community.aws-15 SUCCESS in 24m 43s
✔️ integration-community.aws-16 SUCCESS in 30m 33s
✔️ integration-community.aws-17 SUCCESS in 7m 34s
Skipped 5 jobs

@tremble
Copy link
Contributor Author

tremble commented Sep 29, 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/4d556232a15b489ba214f9a7f0f497df

✔️ ansible-galaxy-importer SUCCESS in 3m 50s
✔️ build-ansible-collection SUCCESS in 13m 11s
✔️ ansible-test-splitter SUCCESS in 4m 51s
✔️ integration-amazon.aws-1 SUCCESS in 31m 05s
integration-amazon.aws-2 FAILURE in 57m 11s
✔️ integration-amazon.aws-3 SUCCESS in 19m 30s
✔️ integration-amazon.aws-4 SUCCESS in 36m 49s
✔️ integration-amazon.aws-5 SUCCESS in 26m 01s
✔️ integration-amazon.aws-6 SUCCESS in 50m 20s
✔️ integration-amazon.aws-7 SUCCESS in 16m 10s
✔️ integration-amazon.aws-8 SUCCESS in 40m 24s
✔️ integration-amazon.aws-9 SUCCESS in 47m 09s
✔️ integration-amazon.aws-10 SUCCESS in 39m 52s
✔️ integration-amazon.aws-11 SUCCESS in 40m 24s
✔️ integration-amazon.aws-12 SUCCESS in 14m 39s
✔️ integration-amazon.aws-13 SUCCESS in 24m 19s
✔️ integration-amazon.aws-14 SUCCESS in 28m 18s
✔️ integration-amazon.aws-15 SUCCESS in 41m 07s
integration-amazon.aws-16 RETRY_LIMIT in 6m 53s
✔️ integration-amazon.aws-17 SUCCESS in 48m 20s
✔️ integration-amazon.aws-18 SUCCESS in 38m 01s
✔️ integration-amazon.aws-19 SUCCESS in 45m 43s
integration-amazon.aws-20 FAILURE in 20m 21s
✔️ integration-amazon.aws-21 SUCCESS in 36m 52s
✔️ integration-amazon.aws-22 SUCCESS in 25m 19s
✔️ integration-community.aws-1 SUCCESS in 39m 50s
✔️ integration-community.aws-2 SUCCESS in 24m 07s
✔️ integration-community.aws-3 SUCCESS in 22m 14s
✔️ integration-community.aws-4 SUCCESS in 20m 32s
✔️ integration-community.aws-5 SUCCESS in 35m 26s
✔️ integration-community.aws-6 SUCCESS in 17m 55s
integration-community.aws-7 RETRY_LIMIT in 1m 45s
integration-community.aws-8 RETRY_LIMIT in 7m 01s
✔️ integration-community.aws-9 SUCCESS in 20m 49s
✔️ integration-community.aws-10 SUCCESS in 27m 17s
✔️ integration-community.aws-11 SUCCESS in 20m 36s
✔️ integration-community.aws-12 SUCCESS in 18m 44s
✔️ integration-community.aws-13 SUCCESS in 34m 50s
integration-community.aws-14 RETRY_LIMIT in 1m 43s
✔️ integration-community.aws-15 SUCCESS in 38m 52s
integration-community.aws-16 FAILURE in 37m 23s
✔️ integration-community.aws-17 SUCCESS in 9m 02s
Skipped 5 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, there is no functional change here and too many files have been touched for CI to possibly pass.

@jillr jillr merged commit e515e05 into ansible-collections:main Sep 29, 2023
63 of 65 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.

alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 2, 2023
* Add tox labels for lint, format and unit tests

* [7.0.0] Apply isort to imports

* Add isort to tox

* Update workflow to run isort tests

* changelog
tremble added a commit to ansible-collections/community.aws that referenced this pull request Oct 17, 2023
SUMMARY

Apply isort - see also ansible-collections/amazon.aws#1771
Apply flynt - see also ansible-collections/amazon.aws#1802

ISSUE TYPE

    Feature Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 20, 2023
SUMMARY

Apply isort - see also ansible-collections#1771
Apply flynt - see also ansible-collections#1802

ISSUE TYPE

    Feature Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@c0117b5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…nsible-collections#1773) (ansible-collections#1781)

eks_nodegroup - fixing remote access and added to integration tests

SUMMARY
This was incorrectly merged directly into stable-5 rather than main.
Fixes ansible-collections#1771
Handling remote_access configuration the right way that boto understands it. Also included it to integration tests.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
eks_nodegroup
ADDITIONAL INFORMATION
This is pulling ansible-collections#1773 from stable-5 into main
Reviewed-by: Markus Bergholz
Reviewed-by: Thomas Bruckmann
Reviewed-by: Mark Chappell

Reviewed-by: Markus Bergholz <git@osuv.de>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 26, 2023
SUMMARY

Apply isort - see also ansible-collections#1771
Apply flynt - see also ansible-collections#1802

ISSUE TYPE

    Feature Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@c0117b5
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 27, 2023
SUMMARY

Apply isort - see also ansible-collections#1771
Apply flynt - see also ansible-collections#1802

ISSUE TYPE

    Feature Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@c0117b5
@tremble tremble deleted the 7.0.0/isort branch January 12, 2024 09:40
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.

Mass apply isort prior to stable-7 branching
3 participants