-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
more integration test improvements #41184
more integration test improvements #41184
Conversation
The test
|
The test
|
Could you please fill in the issue template correctly? |
account_type: Standard_LRS | ||
|
||
- name: Create virtual network | ||
azure_rm_virtualnetwork: | ||
resource_group: "{{ resource_group }}" | ||
name: testvm001 | ||
name: "vn{{ rpfx }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set this as variable?
state: absent | ||
vm_size: Standard_A0 | ||
register: output | ||
|
||
- name: Delete public ip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will deletion fail caused test fail?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, already tested
@nitzmahone do you think we could merge this pr? |
merge this purely test fixing. |
storage_name: "st{{ resource_group | hash('md5') | truncate(7, True, '') }}{{ 1000 | random }}" | ||
storage_container_name: "sc{{ resource_group | hash('md5') | truncate(7, True, '') }}{{ 1000 | random }}" | ||
public_ip_name: "pip{{ resource_group | hash('md5') | truncate(7, True, '') }}{{ 1000 | random }}" | ||
security_group_name: "pip{{ resource_group | hash('md5') | truncate(7, True, '') }}{{ 1000 | random }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
another prefix instead of 'pip'?
storage_blob: testvm001.vhd | ||
storage_account: "{{ storage_name }}" | ||
storage_container: "{{ storage_container_name }}" | ||
storage_blob: "blob.vhd" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
randomize the blob name?
SUMMARY
Fixing issues related to Azure integration tests.
This PR is supposed to fix too many azure_rm_publicip resources create and not deleted by tests.
Also assigning random names to resources to avoid conflicts.
ISSUE TYPE
COMPONENT NAME
azure_rm_networkinterface
azure_rm_publicipaddress
azure_rm_image
azure_rm_managed_disk
azure_rm_loadbalancer
ANSIBLE VERSION
2.6
ADDITIONAL INFORMATION