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

Create extra snapshots for ec2_snapshot_info test - minimum max_results is 5 #786

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Apr 21, 2022

This is a partial backport of PR #356 as merged into main (b7087f4).

SUMMARY

Fixes a bug in the integration test

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_snapshot_info

ADDITIONAL INFORMATION

@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) tests tests labels Apr 21, 2022
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 21, 2022
@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

@tremble
Copy link
Contributor Author

tremble commented Apr 22, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@tremble
Copy link
Contributor Author

tremble commented Apr 22, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@tremble tremble force-pushed the backport/stable-1.5/pull-356-tests branch from dada8f8 to 04284cf Compare April 22, 2022 14:42
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@tremble tremble force-pushed the backport/stable-1.5/pull-356-tests branch from 04284cf to cf04b5a Compare April 23, 2022 09:43
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 19c23db into ansible-collections:stable-1.5 Apr 23, 2022
@tremble tremble deleted the backport/stable-1.5/pull-356-tests branch September 9, 2022 08:51
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…r auto. (ansible-collections#1633)

Addressing Style S3 (ssm_connection) - choose between path, virtual or auto.

SUMMARY
Added the chance of setting up the addressing style for S3 URLs, this fixes ansible-collections#637
ISSUE TYPE


Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION
I came across this issue when I was trying to connect and execute tasks on a fresh AWS set-up (new S3 bucket + EC2), I'm using the latest (5.1.0) release and Ansible 5.10 (as per pip show).
The issue itself is the same as ansible-collections#637 and the last comment is the fix, there's another PR addressing this issue ansible-collections#786 but is quiet since May
Also AWS seems to be deprecating PATH addressing style: https://docs.aws.amazon.com/AmazonS3/latest/userguide/VirtualHosting.html#path-style-access
Thanks to timburnet-systematica and james-masson for their initial work on this.

Reviewed-by: Mark Chappell <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 mergeit Merge the PR (SoftwareFactory) needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants