Fwd merge 4.8->4.9 marvintestfixes to master #1708
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…l networks modified script to handle case where there can be multple physical networks in the zone Signed-off-by: Rohit Yadav <[email protected]>
…rom the Use config file, instead of hardcoded crendentials Signed-off-by: Rohit Yadav <[email protected]>
- Fix macchnina template url - Fix a variable scope issue Signed-off-by: Rohit Yadav <[email protected]>
Signed-off-by: Rohit Yadav <[email protected]>
The quota integration test requires special setup and is moved to plugins directory as in 4.9 and master branch. Signed-off-by: Rohit Yadav <[email protected]>
…json, resulting in internal LB vm not come up parsing cmd_line to create 'ips' data bag, never handled internal lb vm, but still worked due to another bug. support for internal lb vm is added with this fix
…ded username and password Signed-off-by: Rohit Yadav <[email protected]>
- Handle case where physical network instance does not have vlan attribute - Handle case where listIso response may not have status attribute Signed-off-by: Rohit Yadav <[email protected]>
SSH to VR for vmware, goes via the mgmt server and uses ssh keys at /var/cloudstack path. Add suitable checks to tests failing on vmware. Signed-off-by: Rohit Yadav <[email protected]>
- test_01_test_vm_volume_snapshot not supported for Xen, tests keep failing - Skip snapshot tests for centos6/kvm as snapshot is not supported by older qemu-img versions Signed-off-by: Rohit Yadav <[email protected]>
As per previous discussions and ticket, a template deletion may result in failure (exception thrown) for templates that are not properly downloaded. The tearDown method, a template may be tried for deletion but on failure we may ignore it as account deletion/tearDown would retry to cleanup resource owned by the account. Signed-off-by: Rohit Yadav <[email protected]>
… again - test_01_test_vm_volume_snapshot not supported for Xen, tests keep failing Signed-off-by: Rohit Yadav <[email protected]>
- Switches to macchinina as template for VM in the tests - Modifies the ostype of the macchinina template to 'Other Linux (64-bit)' - Check template download status, fixes Nonetype iterable issue Signed-off-by: Rohit Yadav <[email protected]>
…arsing the cmd_line.json to create ips.json databag using 'gateway' in cmd_line for the public ip gateway while creating ips.json databag
- Cleans up volumes, snapshots and VM after running test_snapshots Signed-off-by: Rohit Yadav <[email protected]>
Signed-off-by: Rohit Yadav <[email protected]>
rohityadavcloud
changed the title
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-61 |
|
Trillian test result (tid-146)
|
|
Trillian test result (tid-147)
|
Member
Author
|
Note: the |
|
Trillian test result (tid-148)
|
Member
Author
|
On manual run, all oobm related tests passed. It was a false positive. The issue was environment related with an alert related count where the test Tests out-of-band management ownership expiry across multi-mgmt server ... === TestName: test_oobm_multiple_mgmt_server_ownership | Status : SUCCESS === Ran 16 tests in 202.462s OK |
Member
Author
|
@karuturi can you help run BVT tests on this PR? |
Merged
Contributor
|
If we can confirm that these were false positives, we're good to merge, no? |
Member
Author
|
Thanks all for your review, tests. Fwd merge did not automatically close this PR, so closing this manually. |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@blueorangutan package
FYI /cc @jburwell @karuturi