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 #1891/dc080593 backport][stable-7] Avoid unsafe conditionals in integration tests #1903

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Dec 5, 2023

This is a backport of PR #1891 as merged into main (dc08059).

SUMMARY

Update a batch of assertions in integration tests to avoid templating

See: ansible/ansible-documentation#866 and ansible/ansible-documentation#866 for more information

This PR covers about half of the impacted test tasks for this repo but is already going to be a lot to get through CI so additional PRs will follow.

Affected tasks identified with:
https://gist.github.com/sivel/7b012ae2ebf4ab4cb6225379a72cc502

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

tests

ADDITIONAL INFORMATION

There are unsupported tests in this change. The cloudtrail tests seem to be broken independent of this change, iam_group tests pass for me locally

Avoid unsafe conditionals in integration tests

SUMMARY
Update a batch of assertions in integration tests to avoid templating
See: ansible/ansible-documentation#866 and ansible/ansible-documentation#866 for more information
This PR covers about half of the impacted test tasks for this repo but is already going to be a lot to get through CI so additional PRs will follow.
Affected tasks identified with:
https://gist.github.com/sivel/7b012ae2ebf4ab4cb6225379a72cc502
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
tests
ADDITIONAL INFORMATION
There are unsupported tests in this change.  The cloudtrail tests seem to be broken independent of this change, iam_group tests pass for me locally

Reviewed-by: Bikouo Aubin
Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
Reviewed-by: Mike Graves <[email protected]>
(cherry picked from commit dc08059)
@tremble tremble added mergeit Merge the PR (SoftwareFactory) skip-changelog labels Dec 5, 2023
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 48s
✔️ build-ansible-collection SUCCESS in 14m 15s
✔️ ansible-test-splitter SUCCESS in 5m 11s
✔️ integration-amazon.aws-1 SUCCESS in 14m 02s
✔️ integration-amazon.aws-2 SUCCESS in 32m 18s
✔️ integration-amazon.aws-3 SUCCESS in 26m 25s
✔️ integration-amazon.aws-4 SUCCESS in 42m 41s
✔️ integration-amazon.aws-5 SUCCESS in 36m 03s
✔️ integration-amazon.aws-6 SUCCESS in 25m 34s
✔️ integration-amazon.aws-7 SUCCESS in 10m 26s
✔️ integration-amazon.aws-8 SUCCESS in 18m 27s
✔️ integration-amazon.aws-9 SUCCESS in 12m 18s
✔️ integration-amazon.aws-10 SUCCESS in 28m 33s
✔️ integration-amazon.aws-11 SUCCESS in 12m 03s
✔️ integration-amazon.aws-12 SUCCESS in 21m 36s
✔️ integration-amazon.aws-13 SUCCESS in 21m 27s
✔️ integration-amazon.aws-14 SUCCESS in 20m 35s
✔️ integration-amazon.aws-15 SUCCESS in 26m 00s
✔️ integration-amazon.aws-16 SUCCESS in 15m 58s
✔️ integration-amazon.aws-17 SUCCESS in 18m 49s
✔️ integration-amazon.aws-18 SUCCESS in 30m 34s
✔️ integration-amazon.aws-19 SUCCESS in 21m 58s
✔️ integration-amazon.aws-20 SUCCESS in 22m 54s
✔️ integration-amazon.aws-21 SUCCESS in 14m 19s
Skipped 23 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit f42f76a into stable-7 Dec 5, 2023
40 of 41 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-7/dc0805937cc748bb26b4618ea7189dca7984a546/pr-1891 branch December 5, 2023 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory) skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants