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

Ensure ec2_instance tests filter correctly #1505

Merged
merged 2 commits into from
May 3, 2023

Conversation

jillr
Copy link
Contributor

@jillr jillr commented May 2, 2023

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.

@jillr
Copy link
Contributor Author

jillr commented May 2, 2023

WIP includes a change to the module to ensure all ec2_instance related targets are run at least once. This will be reverted assuming it passes.

@jillr jillr force-pushed the ec2_instance_test_collisions branch from b0d982d to bb7a847 Compare May 2, 2023 23:00
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/54174d7b05e5493eb09e561052140856

✔️ ansible-galaxy-importer SUCCESS in 4m 05s
✔️ build-ansible-collection SUCCESS in 12m 30s
✔️ ansible-test-splitter SUCCESS in 4m 59s
✔️ integration-amazon.aws-1 SUCCESS in 20m 07s
✔️ integration-amazon.aws-2 SUCCESS in 9m 21s
✔️ integration-amazon.aws-3 SUCCESS in 8m 00s
✔️ integration-amazon.aws-4 SUCCESS in 14m 18s
✔️ integration-amazon.aws-5 SUCCESS in 10m 28s
✔️ integration-amazon.aws-6 SUCCESS in 10m 40s
✔️ integration-amazon.aws-7 SUCCESS in 9m 17s
✔️ integration-amazon.aws-8 SUCCESS in 10m 08s
✔️ integration-amazon.aws-9 SUCCESS in 7m 48s
✔️ integration-amazon.aws-10 SUCCESS in 12m 38s
✔️ integration-amazon.aws-11 SUCCESS in 7m 46s
✔️ integration-amazon.aws-12 SUCCESS in 12m 07s
✔️ integration-amazon.aws-13 SUCCESS in 8m 35s
✔️ integration-amazon.aws-14 SUCCESS in 10m 06s
✔️ integration-amazon.aws-15 SUCCESS in 8m 01s
✔️ integration-amazon.aws-16 SUCCESS in 6m 51s
✔️ integration-amazon.aws-17 SUCCESS in 8m 44s
Skipped 27 jobs

Multiple instances test picks up instances from other tests when run
concurrently. Use tag filters to prevent this.
@jillr jillr force-pushed the ec2_instance_test_collisions branch from bb7a847 to 29d942b Compare May 2, 2023 23:54
@jillr
Copy link
Contributor Author

jillr commented May 2, 2023

Tests passing with all ec2_instance targets: https://ansible.softwarefactory-project.io/zuul/buildset/54174d7b05e5493eb09e561052140856

@jillr jillr changed the title [WIP] Ensure ec2_instance tests filter correctly Ensure ec2_instance tests filter correctly May 2, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/a5b4299c8d444ae3a0ee262a73466af7

✔️ ansible-galaxy-importer SUCCESS in 3m 47s
✔️ build-ansible-collection SUCCESS in 14m 18s
✔️ ansible-test-splitter SUCCESS in 5m 59s
✔️ integration-amazon.aws-1 SUCCESS in 23m 40s
Skipped 43 jobs

@alinabuzachis alinabuzachis added mergeit Merge the PR (SoftwareFactory) backport-5 PR should be backported to the stable-5 branch labels May 3, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 02s
✔️ build-ansible-collection SUCCESS in 12m 59s
✔️ ansible-test-splitter SUCCESS in 4m 38s
✔️ integration-amazon.aws-1 SUCCESS in 26m 38s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 116f882 into main May 3, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the ec2_instance_test_collisions branch May 3, 2023 10:18
@patchback
Copy link

patchback bot commented May 3, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/116f882fd2ee1e1694dd9ac92d79c6a5fc0a5f59/pr-1505

Backported as #1506

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request May 3, 2023
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)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 3, 2023
[PR #1505/116f882f backport][stable-5] Ensure ec2_instance tests filter correctly

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.

Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-5 PR should be backported to the stable-5 branch mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants