Skip to content

Conversation

@pritisarap12
Copy link

No description provided.

@asfbot
Copy link

asfbot commented Jun 11, 2015

cloudstack-pull-requests #358 SUCCESS
This pull request looks good

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

@asfbot
Copy link

asfbot commented Jun 19, 2015

cloudstack-pull-requests #507 SUCCESS
This pull request looks good

@asfbot
Copy link

asfbot commented Jun 19, 2015

cloudstack-pull-requests #508 SUCCESS
This pull request looks good

@asfbot
Copy link

asfbot commented Jun 19, 2015

cloudstack-pull-requests #509 FAILURE
Looks like there's a problem with this pull request

@rohityadavcloud
Copy link
Member

Ping, any updates?

@remibergsma
Copy link
Contributor

@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.

remibergsma added a commit to remibergsma/cloudstack that referenced this pull request Aug 17, 2015
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
remibergsma added a commit to remibergsma/cloudstack that referenced this pull request Aug 17, 2015
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
remibergsma added a commit to remibergsma/cloudstack that referenced this pull request Aug 17, 2015
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
borisroman pushed a commit to borisroman/cloudstack that referenced this pull request Aug 19, 2015
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath.

Reference to apache#384
@borisroman
Copy link
Contributor

See #720 -> Resolved comments.

borisroman pushed a commit to borisroman/cloudstack that referenced this pull request Sep 27, 2015
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath.

Reference to apache#384
borisroman pushed a commit to borisroman/cloudstack that referenced this pull request Oct 21, 2015
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath.

Reference to apache#384
borisroman pushed a commit to borisroman/cloudstack that referenced this pull request Oct 23, 2015
CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath.

Reference to apache#384
@rajap9711
Copy link

@remibergsma can you please comment on this PR, if it was merged ? thanks,

@remibergsma
Copy link
Contributor

It was not merged, see also #720.

Copy link
Contributor

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace with assertNotNone

@rohityadavcloud
Copy link
Member

@pritisarap12 please rebase against latest master and share status of your PR

@jburwell
Copy link
Contributor

jburwell commented Aug 4, 2016

@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?

@DaanHoogland
Copy link
Contributor

opening #720 in favour of this one and will assign to me

rohityadavcloud added a commit that referenced this pull request Jan 20, 2021
Fixes #384

Signed-off-by: Rohit Yadav <[email protected]>
Co-authored-by: Rohit Yadav <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants