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 #1052/558a0252 backport][stable-3] aws_kms - stabilize and add integration tests #1099

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Apr 26, 2022

This is a backport of PR #1052 as merged into main (558a025).

SUMMARY
  • update/add integration tests for various actions
  • return list of policies as a list of jsons for clarity
  • sleep on updates (no kms waiter, attempted manual waiters but still had test failures)
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_kms

ADDITIONAL INFORMATION

I tried adding manual waiters for different actions like waiting for tags to be correct, policy to be updated, etc, but would still fail ~half of the time on idempotency tests. seems like after updating the key's status is a bit buggy.

aws_kms - stabilize and add integration tests

SUMMARY

update/add integration tests for various actions
return list of policies as a list of jsons for clarity
sleep on updates (no kms waiter, attempted manual waiters but still had test failures)

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_kms
ADDITIONAL INFORMATION
I tried adding manual waiters for different actions like waiting for tags to be correct, policy to be updated, etc, but would still fail ~half of the time on idempotency tests. seems like after updating the key's status is a bit buggy.

Reviewed-by: Jill R <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 558a025)
@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Apr 26, 2022
@jillr
Copy link
Collaborator

jillr commented Apr 26, 2022

recheck

@markuman
Copy link
Member

recheck

@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label May 5, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

ansible-galaxy-importer FAILURE in 6m 04s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 59s
ansible-test-sanity-docker-devel FAILURE in 10m 18s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 57s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 46s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 58s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 45s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 8m 11s
✔️ ansible-test-splitter SUCCESS in 3m 34s
✔️ integration-community.aws-1 SUCCESS in 10m 24s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@jatorcasso
Copy link
Contributor

regate

@jatorcasso
Copy link
Contributor

Depends-On: #1123

@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

ansible-galaxy-importer FAILURE in 5m 06s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 51s
ansible-test-sanity-docker-devel FAILURE in 10m 29s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 14m 15s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 26s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 12m 06s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 37s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 17s
✔️ ansible-test-splitter SUCCESS in 2m 55s
✔️ integration-community.aws-1 SUCCESS in 12m 35s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@jatorcasso
Copy link
Contributor

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 5m 27s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 28s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 11s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 04s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 15m 08s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 02s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 34s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 07s
✔️ ansible-test-splitter SUCCESS in 4m 54s
✔️ integration-community.aws-1 SUCCESS in 12m 58s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 0f85221 into stable-3 May 6, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/558a0252a3d57657b8c6be50e65b3663d37958fa/pr-1052 branch May 6, 2022 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants