-
Notifications
You must be signed in to change notification settings - Fork 1.3k
CLOUDSTACK-8308:Adding automation testcases for Delta Snapshot testpath #384
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
Conversation
|
cloudstack-pull-requests #358 SUCCESS |
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.
Add docString for the function
…oudstack into delta-snapshot
5999c47 to
c1bfe51
Compare
c1bfe51 to
015c78c
Compare
|
cloudstack-pull-requests #507 SUCCESS |
|
cloudstack-pull-requests #508 SUCCESS |
|
cloudstack-pull-requests #509 FAILURE |
|
Ping, any updates? |
|
@pritisarap12 Any update on this PR? Please make just one commit, and get rid of the merge commit. You can force push to this PR. Ping us if you need help. |
This closes apache#577 This closes apache#566 This closes apache#562 This closes apache#561 This closes apache#556 This closes apache#555 This closes apache#554 This closes apache#548 This closes apache#544 This closes apache#540 This closes apache#508 This closes apache#384 This closes apache#372
This closes apache#577 This closes apache#566 This closes apache#562 This closes apache#561 This closes apache#556 This closes apache#555 This closes apache#554 This closes apache#548 This closes apache#544 This closes apache#540 This closes apache#384 This closes apache#372
This closes apache#577 This closes apache#566 This closes apache#562 This closes apache#561 This closes apache#556 This closes apache#555 This closes apache#554 This closes apache#548 This closes apache#544 This closes apache#540 This closes apache#384 This closes apache#372
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath. Reference to apache#384
|
See #720 -> Resolved comments. |
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath. Reference to apache#384
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath. Reference to apache#384
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath. Reference to apache#384
|
@remibergsma can you please comment on this PR, if it was merged ? thanks, |
|
It was not merged, see also #720. |
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.
This can be replaced with assertNotNone
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.
Replace with assertNotNone
|
@pritisarap12 please rebase against latest master and share status of your PR |
|
@pritisarap12 @remibergsma referenced PR #720 which appears to implement a similar set of tests. How do the tests in this PR differ? it also appears that the Travis build is failing. Could you please fix the failure and trigger Travis to rebuild? Finally, is there a JIRA ticket associated with this change? |
|
opening #720 in favour of this one and will assign to me |
Fixes #384 Signed-off-by: Rohit Yadav <[email protected]> Co-authored-by: Rohit Yadav <[email protected]>
No description provided.