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

Switch from t3.micro to t3a.micro in some of the integration tests #2188

Merged
merged 1 commit into from
Jul 11, 2024
Merged

Switch from t3.micro to t3a.micro in some of the integration tests #2188

merged 1 commit into from
Jul 11, 2024

Conversation

GomathiselviS
Copy link
Contributor

@GomathiselviS GomathiselviS commented Jul 11, 2024 •
edited by gravesm
Loading

SUMMARY

Refer: https://issues.redhat.com/browse/ACA-1691

Requires mattclay/aws-terminator#304 for permissions.

Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI

ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

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

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/993f1c477d944f3c9d1533d5fa873d59

✔️ ansible-galaxy-importer SUCCESS in 3m 21s
✔️ build-ansible-collection SUCCESS in 12m 19s
✔️ ansible-test-splitter SUCCESS in 5m 06s
❌ integration-amazon.aws-1 FAILURE in 6m 36s
✔️ integration-amazon.aws-2 SUCCESS in 6m 23s
❌ integration-amazon.aws-3 FAILURE in 7m 32s
❌ integration-amazon.aws-4 FAILURE in 6m 56s
❌ integration-amazon.aws-5 FAILURE in 7m 08s
✔️ integration-amazon.aws-6 SUCCESS in 10m 10s
❌ integration-amazon.aws-7 FAILURE in 7m 34s
❌ integration-amazon.aws-8 FAILURE in 6m 39s
❌ integration-amazon.aws-9 FAILURE in 7m 05s
✔️ integration-amazon.aws-10 SUCCESS in 9m 02s
Skipped 34 jobs

@GomathiselviS GomathiselviS marked this pull request as draft July 11, 2024 07:24
@GomathiselviS GomathiselviS changed the title Switch from t3.micro to t3a.micro in some of the integration tests WIP Switch from t3.micro to t3a.micro in some of the integration tests Jul 11, 2024
@morucci
Copy link

morucci commented Jul 11, 2024

Hi could you please remove the depends-on on this change ^ because that repo (mattclay/aws-terminator) is not part of our Zuul tenant config. Perhaps use another keyword to define the dependency ?

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

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

✔️ ansible-galaxy-importer SUCCESS in 4m 13s
✔️ build-ansible-collection SUCCESS in 12m 11s
✔️ ansible-test-splitter SUCCESS in 4m 54s
✔️ integration-amazon.aws-1 SUCCESS in 26m 24s
✔️ integration-amazon.aws-2 SUCCESS in 9m 58s
✔️ integration-amazon.aws-3 SUCCESS in 14m 58s
✔️ integration-amazon.aws-4 SUCCESS in 10m 49s
✔️ integration-amazon.aws-5 SUCCESS in 11m 25s
✔️ integration-amazon.aws-6 SUCCESS in 9m 06s
✔️ integration-amazon.aws-7 SUCCESS in 15m 00s
✔️ integration-amazon.aws-8 SUCCESS in 8m 45s
✔️ integration-amazon.aws-9 SUCCESS in 7m 28s
✔️ integration-amazon.aws-10 SUCCESS in 9m 38s
Skipped 34 jobs

@GomathiselviS GomathiselviS marked this pull request as ready for review July 11, 2024 16:54
@GomathiselviS GomathiselviS changed the title WIP Switch from t3.micro to t3a.micro in some of the integration tests Switch from t3.micro to t3a.micro in some of the integration tests Jul 11, 2024
@alinabuzachis alinabuzachis added backport-7 PR should be backported to the stable-7 branch backport-8 PR should be backported to the stable-8 branch labels Jul 11, 2024
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jul 11, 2024
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 46s
✔️ build-ansible-collection SUCCESS in 12m 12s
✔️ ansible-test-splitter SUCCESS in 4m 50s
✔️ integration-amazon.aws-1 SUCCESS in 18m 06s
✔️ integration-amazon.aws-2 SUCCESS in 8m 17s
✔️ integration-amazon.aws-3 SUCCESS in 13m 23s
✔️ integration-amazon.aws-4 SUCCESS in 13m 25s
✔️ integration-amazon.aws-5 SUCCESS in 7m 55s
✔️ integration-amazon.aws-6 SUCCESS in 10m 29s
✔️ integration-amazon.aws-7 SUCCESS in 14m 33s
✔️ integration-amazon.aws-8 SUCCESS in 10m 42s
✔️ integration-amazon.aws-9 SUCCESS in 8m 16s
✔️ integration-amazon.aws-10 SUCCESS in 9m 03s
Skipped 34 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 2f1fcab into ansible-collections:main Jul 11, 2024
38 checks passed
@patchback Patchback
Copy link

patchback bot commented Jul 11, 2024 •
edited
Loading

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/2f1fcabc2c3a7b008f8d93a7733baaf56509062e/pr-2188

Backported as #2189

🤖 @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 Jul 11, 2024
@GomathiselviS @patchback
Switch from t3.micro to t3a.micro in some of the integratin tests (#2188
9313728 
)

SUMMARY

Refer: https://issues.redhat.com/browse/ACA-1691
Requires mattclay/aws-terminator#304 for permissions.
Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI
ISSUE TYPE

Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
(cherry picked from commit 2f1fcab)
@patchback patchback bot mentioned this pull request Jul 11, 2024
Merged
@patchback Patchback
Copy link

patchback bot commented Jul 11, 2024 •
edited
Loading

Backport to stable-8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-8/2f1fcabc2c3a7b008f8d93a7733baaf56509062e/pr-2188

Backported as #2190

🤖 @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 Jul 11, 2024
@GomathiselviS @patchback
Switch from t3.micro to t3a.micro in some of the integratin tests (#2188
81dff98 
)

SUMMARY

Refer: https://issues.redhat.com/browse/ACA-1691
Requires mattclay/aws-terminator#304 for permissions.
Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI
ISSUE TYPE

Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
(cherry picked from commit 2f1fcab)
@patchback patchback bot mentioned this pull request Jul 11, 2024
Merged
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jul 12, 2024
@patchback
Switch from t3.micro to t3a.micro in some of the integratin tests (#2188
95ec9e8 
) (#2190)

This is a backport of PR #2188 as merged into main (2f1fcab).
SUMMARY


Refer: https://issues.redhat.com/browse/ACA-1691
Requires mattclay/aws-terminator#304 for permissions.
Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI
ISSUE TYPE


Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jul 12, 2024
@patchback
Switch from t3.micro to t3a.micro in some of the integratin tests (#2188
54ddd01 
) (#2189)

This is a backport of PR #2188 as merged into main (2f1fcab).
SUMMARY


Refer: https://issues.redhat.com/browse/ACA-1691
Requires mattclay/aws-terminator#304 for permissions.
Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI
ISSUE TYPE


Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tremble tremble tremble approved these changes

@alinabuzachis alinabuzachis alinabuzachis approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@GomathiselviS @morucci @tremble @alinabuzachis