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

tests: don't test ancient boto3 versions #636

Conversation

goneri
Copy link
Member

@goneri goneri commented Jul 13, 2021

SUMMARY

These tests were a source of problems and are pointless now we only
support boto3 >= 1.13.0 and botocore >= 1.16.0

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

tests

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration needs_triage plugins plugin (any type) python3 tests tests labels Jul 13, 2021
@goneri goneri force-pushed the elb_target-test-use-2nd-venv_25725 branch from 4c3ba25 to 0446072 Compare July 13, 2021 14:05
@goneri goneri changed the title elb_target/test: use 2nd venv test: use 2nd venv for alt boto version tests Jul 13, 2021
@goneri goneri force-pushed the elb_target-test-use-2nd-venv_25725 branch from 0446072 to 067f6f8 Compare July 13, 2021 14:06
tests/integration/targets/efs/runme.sh Outdated Show resolved Hide resolved
tests/integration/targets/elb_target/runme.sh Outdated Show resolved Hide resolved
@ansibullbot
Copy link

@goneri
Copy link
Member Author

goneri commented Jul 13, 2021

recheck

@goneri goneri closed this Jul 13, 2021
@goneri goneri reopened this Jul 13, 2021
@goneri goneri closed this Jul 13, 2021
@goneri goneri reopened this Jul 13, 2021
@goneri goneri closed this Jul 13, 2021
@goneri goneri reopened this Jul 13, 2021
@goneri
Copy link
Member Author

goneri commented Jul 13, 2021

recheck

5 similar comments
@goneri
Copy link
Member Author

goneri commented Jul 13, 2021

recheck

@goneri
Copy link
Member Author

goneri commented Jul 13, 2021

recheck

@goneri
Copy link
Member Author

goneri commented Jul 14, 2021

recheck

@goneri
Copy link
Member Author

goneri commented Jul 14, 2021

recheck

@goneri
Copy link
Member Author

goneri commented Jul 14, 2021

recheck

@goneri goneri force-pushed the elb_target-test-use-2nd-venv_25725 branch from 067f6f8 to 4a98e59 Compare July 14, 2021 13:45
@goneri goneri changed the title test: use 2nd venv for alt boto version tests tests: don't test ancient boto3 versions Jul 14, 2021
These tests were a source of problems and are pointless now we only
support `boto3 >= 1.13.0` and `botocore >= 1.16.0`
@goneri goneri force-pushed the elb_target-test-use-2nd-venv_25725 branch from 4a98e59 to 512755c Compare July 14, 2021 15:39
# Run full test suite
source virtualenv.sh
pip install 'botocore' 'boto3>=1.16.57'
Copy link
Contributor

Choose a reason for hiding this comment

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

boto3 >= 1.13.0

boto3 1.16.57 is only around 6 months old...

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, I went to fast on this one.

@tremble
Copy link
Contributor

tremble commented Aug 25, 2021

Made moot by #675

@tremble tremble closed this Aug 25, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
…tly (ansible-collections#636)

Fix typo

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Jill R <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review integration tests/integration needs_triage plugins plugin (any type) python3 tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants