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

Drop compile/import tests for old versions of python in sanity tests #447

Merged
merged 3 commits into from
Aug 11, 2021

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Aug 7, 2021

SUMMARY

We don't support Python < 3.6, so drop the tests at the Zuul job level rather than with a large number of !skip entries

ISSUE TYPE
  • Feature request
COMPONENT NAME

sanity tests

ADDITIONAL INFORMATION

Depends-On: ansible/ansible-zuul-jobs#1028

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request needs_triage tests tests labels Aug 7, 2021
@tremble tremble requested review from alinabuzachis and jillr August 7, 2021 16:50
@tremble
Copy link
Contributor Author

tremble commented Aug 7, 2021

ansible/ansible-zuul-jobs#1028 now limits the compile/import tests to Python 3.6 and 3.8, and drops the Python 3-ism boilerplate tests, which in turn means we can drop the swathes of "!skip" entries in the ignore files

@alinabuzachis
Copy link
Collaborator

@tremble That's a nice thing in my opinion. BTW, It seems to be an issue with the splitter.

@tremble tremble added the gate label Aug 10, 2021
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

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

LGTM!

ansible-zuul bot added a commit to ansible-collections/community.aws that referenced this pull request Aug 10, 2021
 Drop compile/import tests for old versions of python in sanity tests

SUMMARY
We don't support Python < 3.6, so drop the tests at the Zuul job level rather than with a large number of !skip entries
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
tests/sanity
ADDITIONAL INFORMATION
(see also ansible-collections/amazon.aws#447)
Zuul change: ansible/ansible-zuul-jobs#1028

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
@tremble
Copy link
Contributor Author

tremble commented Aug 10, 2021

recheck

@tremble tremble force-pushed the sanity/cleanup-skip branch from 3fea402 to 72e8405 Compare August 10, 2021 14:31
@ansible-zuul ansible-zuul bot removed the gate label Aug 10, 2021
@ansibullbot
Copy link

@ansibullbot ansibullbot added integration tests/integration plugins plugin (any type) labels Aug 10, 2021
@tremble tremble force-pushed the sanity/cleanup-skip branch from 72e8405 to 07d57e3 Compare August 10, 2021 17:34
@tremble tremble added the gate label Aug 11, 2021
@ansible-zuul ansible-zuul bot merged commit 5876c14 into ansible-collections:main Aug 11, 2021
@tremble tremble deleted the sanity/cleanup-skip branch November 26, 2021 12:42
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
* Migrate s3_logging to boto3
* Re-enable s3_logging tests
* Add support for check_mode
* Catch and retry on "InvalidTargetBucketForLogging" - ACL updates occasionally take time to propogate
* changelog
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
* Migrate s3_logging to boto3
* Re-enable s3_logging tests
* Add support for check_mode
* Catch and retry on "InvalidTargetBucketForLogging" - ACL updates occasionally take time to propogate
* changelog
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
* Migrate s3_logging to boto3
* Re-enable s3_logging tests
* Add support for check_mode
* Catch and retry on "InvalidTargetBucketForLogging" - ACL updates occasionally take time to propogate
* changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants