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

[PR #1505/116f882f backport][stable-5] Ensure ec2_instance tests filter correctly #1506

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented May 3, 2023

This is a backport of PR #1505 as merged into main (116f882).

SUMMARY

Multiple instances test picks up instances from other tests when run concurrently. Use tag filters to prevent this.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

tests/integration/targets/ec2_instance_instance_multiple/tasks/main.yml

ADDITIONAL INFORMATION

in the example here: https://39d585ceb0884fd58f3d-34626b4834f21056500b5be2783f089d.ssl.cf5.rackcdn.com/1500/eb91ab4ac10d4af904d629a7296502aff59d9d06/check/integration-amazon.aws-1/86d0d4c/job-output.txt
the Create multiple instance (check_mode) task erroneously picks up instances from the ec2_instance_instance_minimal test target.

Ensure ec2_instance tests filter correctly

SUMMARY
Multiple instances test picks up instances from other tests when run concurrently. Use tag filters to prevent this.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
tests/integration/targets/ec2_instance_instance_multiple/tasks/main.yml
ADDITIONAL INFORMATION
in the example here: https://39d585ceb0884fd58f3d-34626b4834f21056500b5be2783f089d.ssl.cf5.rackcdn.com/1500/eb91ab4ac10d4af904d629a7296502aff59d9d06/check/integration-amazon.aws-1/86d0d4c/job-output.txt
the Create multiple instance (check_mode) task erroneously picks up instances from the ec2_instance_instance_minimal test target.

Reviewed-by: Alina Buzachis
(cherry picked from commit 116f882)
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label May 3, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/7dc93799edfa4c378a09b31c56c8b022

✔️ ansible-galaxy-importer SUCCESS in 3m 55s
✔️ build-ansible-collection SUCCESS in 13m 06s
✔️ ansible-test-splitter SUCCESS in 5m 00s
✔️ integration-amazon.aws-1 SUCCESS in 30m 03s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit e71a396 into stable-5 May 3, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/116f882fd2ee1e1694dd9ac92d79c6a5fc0a5f59/pr-1505 branch May 3, 2023 11:24
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…1506)

Migrate  elb_application_lb module to amazon.aws

SUMMARY

Remove elb_application_lb* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…1506)

Migrate  elb_application_lb module to amazon.aws

SUMMARY

Remove elb_application_lb* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…1506)

Migrate  elb_application_lb module to amazon.aws

SUMMARY

Remove elb_application_lb* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Alina Buzachis <None>
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.

3 participants