Skip to content

CLOUDSTACK-8308: Adding automation testcases for Delta Snapshot.#720

Closed
borisroman wants to merge 1 commit into
apache:masterfrom
borisroman:pr/384
Closed

CLOUDSTACK-8308: Adding automation testcases for Delta Snapshot.#720
borisroman wants to merge 1 commit into
apache:masterfrom
borisroman:pr/384

Conversation

@borisroman

Copy link
Copy Markdown
Contributor

CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath.

Reference to #384

This closes #384

@borisroman

Copy link
Copy Markdown
Contributor Author

This closes #384. Squashed to commits into 1 as requested per comment.

@borisroman

Copy link
Copy Markdown
Contributor Author

@remibergsma @bhaisaab Any more comments?

@remibergsma

Copy link
Copy Markdown
Contributor

@borisroman Can you add the test output please?

@asfbot

asfbot commented Aug 19, 2015

Copy link
Copy Markdown

cloudstack-pull-rats #356 ABORTED

@asfbot

asfbot commented Aug 19, 2015

Copy link
Copy Markdown

cloudstack-pull-analysis #288 SUCCESS
This pull request looks good

@yadvr

yadvr commented Aug 26, 2015

Copy link
Copy Markdown
Member

LGTM (not tested on a real infra), can you share output ?

@asfbot

asfbot commented Sep 27, 2015

Copy link
Copy Markdown

cloudstack-pull-rats #749 SUCCESS
This pull request looks good

@asfbot

asfbot commented Sep 27, 2015

Copy link
Copy Markdown

cloudstack-pull-analysis #698 SUCCESS
This pull request looks good

CLOUDSTACK-8308: Updating automation testcases for Delta Snapshot testpath.

Reference to apache#384
@borisroman

Copy link
Copy Markdown
Contributor Author

@karuturi Could you run this on your Xen-setup?

@borisroman

Copy link
Copy Markdown
Contributor Author

Been open for 3 months, closing.

@borisroman borisroman closed this Jan 20, 2016
@borisroman borisroman deleted the pr/384 branch January 20, 2016 12:17
@DaanHoogland DaanHoogland self-assigned this Apr 24, 2017
@DaanHoogland

Copy link
Copy Markdown
Contributor

can't reopen will copy and steal the code for a new PR, to see if it will be accepted ;)

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.

5 participants