Skip to content

CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath - #170

Closed
pritisarap12 wants to merge 3 commits into
apache:masterfrom
pritisarap12:CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath-testpath-snapshot-limits
Closed

CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath#170
pritisarap12 wants to merge 3 commits into
apache:masterfrom
pritisarap12:CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath-testpath-snapshot-limits

Conversation

@pritisarap12

@pritisarap12pritisarap12 commented Apr 15, 2015

Copy link
Copy Markdown

--Adding Snapshot Test cases that are relating secondary
storage limits and snapshot limits

@gauravaradhye

gauravaradhye commented Apr 15, 2015

Copy link
Copy Markdown

Please include test results.

@asfbot

asfbot commented Apr 15, 2015

Copy link
Copy Markdown

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

gauravaradhye
gauravaradhye reviewed Apr 15, 2015
View reviewed changes
Comment threadtest/integration/testpaths/testpath_snapshot_limits.py Outdated

@gauravaradhyegauravaradhyeApr 15, 2015

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Modify comment to specify what we are targeting to test. In this case snapshot limits

@pritisarap12

pritisarap12 commented Apr 15, 2015

Copy link
Copy Markdown
Author

Adding Snapshot Test cases that are relating secondary
storage limits and snapshot limits

Test Result:

Storage and Snapshot Limit ... === TestName: test_01_storage_snapshots_limits | Status : SUCCESS ===
ok

Ran 1 test in 228.596s

OK

@pritisarap12
pritisarap12force-pushed the CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath-testpath-snapshot-limits branch 2 times, most recently from df802fe to b49a02fCompareApril 16, 2015 06:14
@pritisarap12

pritisarap12 commented Apr 16, 2015

Copy link
Copy Markdown
Author

Adding Snapshot Test cases that are relating secondary
storage limits and snapshot limits

Test result after integrating review changes:

Storage and Snapshot Limit ... === TestName: test_01_storage_snapshots_limits | Status : SUCCESS ===
ok

Ran 1 test in 493.037s

OK

@pritisarap12
pritisarap12force-pushed the CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath-testpath-snapshot-limits branch from b49a02f to 5f43885CompareApril 16, 2015 06:25
@asfbot

asfbot commented Apr 16, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Apr 16, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #14 ABORTED

@asfbot

asfbot commented Apr 16, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #13 ABORTED

gauravaradhye
gauravaradhye reviewed Apr 16, 2015
View reviewed changes
Comment threadtest/integration/testpaths/testpath_snapshot_limits.py Outdated

@gauravaradhyegauravaradhyeApr 16, 2015

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

listall not needed

@asfbot

asfbot commented Apr 17, 2015

Copy link
Copy Markdown

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

@asfbot

asfbot commented Apr 30, 2015

Copy link
Copy Markdown

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

pritisarap12 added 3 commits May 5, 2015 14:32
pritisarap12
CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-t…
7536c40
…estpath-testpath-snapshot-limits
--Integrating review changes
pritisarap12
CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-t…
94de692
…estpath-testpath-snapshot-limits
--Adding review changes
--Cleanup data volume created
pritisarap12
CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-t…
ab19494
…estpath-testpath-snapshot-limits
--Removing snapsot states as it is already there in the codes.py
@pritisarap12
pritisarap12force-pushed the CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath-testpath-snapshot-limits branch from 0e05408 to ab19494CompareMay 5, 2015 09:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pritisarap12@gauravaradhye@asfbot