Uh oh!
There was an error while loading. Please reload this page.
Fwd merge 4.8->4.9 marvintestfixes to master - #1708
Closed
yadvr wants to merge 18 commits into
Closed
Conversation
…l networks modified script to handle case where there can be multple physical networks in the zone Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
…rom the Use config file, instead of hardcoded crendentials Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
- Fix macchnina template url - Fix a variable scope issue Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
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 <rohit.yadav@shapeblue.com>
…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 <rohit.yadav@shapeblue.com>
- 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 <rohit.yadav@shapeblue.com>
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 <rohit.yadav@shapeblue.com>
- 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 <rohit.yadav@shapeblue.com>
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 <rohit.yadav@shapeblue.com>
… again - test_01_test_vm_volume_snapshot not supported for Xen, tests keep failing Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
- 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 <rohit.yadav@shapeblue.com>
…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 <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
blueorangutan
commented
Oct 13, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Oct 13, 2016
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-61 |
blueorangutan
commented
Oct 14, 2016
Trillian test result (tid-146)
|
blueorangutan
commented
Oct 14, 2016
Trillian test result (tid-147)
|
yadvr
commented
Oct 14, 2016
MemberAuthor
Note: the |
blueorangutan
commented
Oct 14, 2016
Trillian test result (tid-148)
|
yadvr
commented
Oct 14, 2016
MemberAuthor
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 |
yadvr
commented
Oct 18, 2016
MemberAuthor
@karuturi can you help run BVT tests on this PR? |
Merged
mlsorensen
commented
Oct 19, 2016
Contributor
If we can confirm that these were false positives, we're good to merge, no? |
yadvr
commented
Oct 21, 2016
MemberAuthor
Thanks all for your review, tests. Fwd merge did not automatically close this PR, so closing this manually. |
blueorangutan
commented
Oct 21, 2016
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
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
Sign up for freeto 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