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.

Sign up for GitHub

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

Jump to bottom

Work on ec2_instance integration test flakes #1845

Merged
merged 3 commits into from
Nov 21, 2023
Merged

Work on ec2_instance integration test flakes #1845

merged 3 commits into from
Nov 21, 2023

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Nov 10, 2023

SUMMARY

As seen in #1828 the integration tests for ec2_instance are still a little flaky (ignoring Zuul also being flaky). This attempts to tweak two of the worst offenders to improve things

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_instance

ADDITIONAL INFORMATION

@tremble tremble added backport-6 PR should be backported to the stable-6 branch skip-changelog backport-7 PR should be backported to the stable-7 branch labels Nov 10, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul

This comment was marked as resolved.

Sign in to view
@tremble tremble force-pushed the ec2_instance/flakes branch from 666bacd to 285ab62 Compare November 10, 2023 11:09
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 37s
✔️ build-ansible-collection SUCCESS in 14m 05s
✔️ ansible-test-splitter SUCCESS in 5m 29s
✔️ integration-amazon.aws-1 SUCCESS in 18m 38s
✔️ integration-amazon.aws-2 SUCCESS in 24m 46s
✔️ integration-community.aws-1 SUCCESS in 31m 11s
Skipped 41 jobs

@tremble tremble marked this pull request as ready for review November 13, 2023 10:34
@tremble tremble force-pushed the ec2_instance/flakes branch from 285ab62 to a218fe8 Compare November 15, 2023 06:03
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 51s
✔️ build-ansible-collection SUCCESS in 14m 45s
✔️ ansible-test-splitter SUCCESS in 5m 21s
✔️ integration-amazon.aws-1 SUCCESS in 18m 53s
✔️ integration-amazon.aws-2 SUCCESS in 13m 07s
Skipped 42 jobs

@yugesk yugesk mentioned this pull request Nov 16, 2023
Merged
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Nov 21, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 9m 11s
✔️ build-ansible-collection SUCCESS in 15m 33s
✔️ ansible-test-splitter SUCCESS in 6m 03s
✔️ integration-amazon.aws-1 SUCCESS in 18m 19s
✔️ integration-amazon.aws-2 SUCCESS in 17m 22s
Skipped 42 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Required status check "ansible/gate" is expected.

@tremble tremble force-pushed the ec2_instance/flakes branch from a218fe8 to 8932b4c Compare November 21, 2023 15:28
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 26s
✔️ build-ansible-collection SUCCESS in 15m 56s
✔️ ansible-test-splitter SUCCESS in 5m 41s
✔️ integration-amazon.aws-1 SUCCESS in 16m 19s
✔️ integration-amazon.aws-2 SUCCESS in 19m 31s
Skipped 42 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 43s
✔️ build-ansible-collection SUCCESS in 14m 16s
✔️ ansible-test-splitter SUCCESS in 5m 34s
✔️ integration-amazon.aws-1 SUCCESS in 19m 17s
✔️ integration-amazon.aws-2 SUCCESS in 20m 26s
Skipped 42 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 02b1511 into ansible-collections:main Nov 21, 2023
39 checks passed
@patchback Patchback
Copy link

patchback bot commented Nov 21, 2023 •
edited
Loading

Backport to stable-6: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-6/02b151113cf46141a4708897336ec9df047be156/pr-1845

Backported as #1868

🤖 @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 Nov 21, 2023
@tremble @patchback
Work on ec2_instance integration test flakes (#1845)
64a76f2 
Work on ec2_instance integration test flakes

SUMMARY
As seen in #1828 the integration tests for ec2_instance are still a little flaky (ignoring Zuul also being flaky).  This attempts to tweak two of the worst offenders to improve things
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_instance
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 02b1511)
@patchback patchback bot mentioned this pull request Nov 21, 2023
Merged
@patchback Patchback
Copy link

patchback bot commented Nov 21, 2023 •
edited
Loading

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/02b151113cf46141a4708897336ec9df047be156/pr-1845

Backported as #1869

🤖 @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 Nov 21, 2023
@tremble @patchback
Work on ec2_instance integration test flakes (#1845)
bfddee5 
Work on ec2_instance integration test flakes

SUMMARY
As seen in #1828 the integration tests for ec2_instance are still a little flaky (ignoring Zuul also being flaky).  This attempts to tweak two of the worst offenders to improve things
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_instance
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 02b1511)
@patchback patchback bot mentioned this pull request Nov 21, 2023
Merged
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Nov 21, 2023
@patchback
Work on ec2_instance integration test flakes (#1845) (#1869)
13f9882 
[PR #1845/02b15111 backport][stable-7] Work on ec2_instance integration test flakes

This is a backport of PR #1845 as merged into main (02b1511).
SUMMARY
As seen in #1828 the integration tests for ec2_instance are still a little flaky (ignoring Zuul also being flaky).  This attempts to tweak two of the worst offenders to improve things
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_instance
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Nov 21, 2023
@patchback
Work on ec2_instance integration test flakes (#1845) (#1868)
b1aae69 
[PR #1845/02b15111 backport][stable-6] Work on ec2_instance integration test flakes

This is a backport of PR #1845 as merged into main (02b1511).
SUMMARY
As seen in #1828 the integration tests for ec2_instance are still a little flaky (ignoring Zuul also being flaky).  This attempts to tweak two of the worst offenders to improve things
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_instance
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
@tremble tremble deleted the ec2_instance/flakes branch January 12, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alinabuzachis alinabuzachis alinabuzachis approved these changes

Assignees
No one assigned
Labels
backport-6 PR should be backported to the stable-6 branch backport-7 PR should be backported to the stable-7 branch mergeit Merge the PR (SoftwareFactory) skip-changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tremble @alinabuzachis