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

Removing leftover storage account from test #1449

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

thekad
Copy link
Contributor

@thekad thekad commented Feb 15, 2024

SUMMARY

Last account is not cleaned up properly after test runs. This probably is masked if you re-create the resource group every time.

ISSUE TYPE
  • Bugfix Pull Request

(really a tech debt fix)

COMPONENT NAME

tests/integration/targets/azure_rm_storageaccount

ADDITIONAL INFORMATION
  1. Configure your environment for integration tests
  2. Run SA integration test on an empty resource group
  3. Resource group should be empty afterwards, but still shows one lingering SA

Last account is not cleaned up properly after test runs. This probably
is masked if you re-create the resource group every time.

Signed-off-by: Jorge Gallegos <jgallego@redhat.com>
@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged medium_priority Medium priority labels Feb 17, 2024
@xuzhang3 xuzhang3 merged commit 1a4a7a8 into ansible-collections:dev Feb 29, 2024
Justwmz pushed a commit to Justwmz/azure that referenced this pull request Nov 4, 2024
Last account is not cleaned up properly after test runs. This probably
is masked if you re-create the resource group every time.

Signed-off-by: Jorge Gallegos <jgallego@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants