[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/613 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-27 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/613#issuecomment-125133196 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-26 Thread Sanjeev N
LGTM!! On Thu, Jul 23, 2015 at 11:34 AM, pritisarap12 wrote: > Github user pritisarap12 commented on the pull request: > > https://github.com/apache/cloudstack/pull/613#issuecomment-123986839 > > Updated the testcase as per review comments. > > > --- > If your project is set up for it,

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-22 Thread pritisarap12
Github user pritisarap12 commented on the pull request: https://github.com/apache/cloudstack/pull/613#issuecomment-123986839 Updated the testcase as per review comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-22 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/613#discussion_r35288977 --- Diff: test/integration/testpaths/testpath_uuid_event.py --- @@ -0,0 +1,198 @@ +# Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-22 Thread sanju1010
Github user sanju1010 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/613#discussion_r35288952 --- Diff: test/integration/testpaths/testpath_uuid_event.py --- @@ -0,0 +1,198 @@ +# Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-22 Thread nitt10prashant
Github user nitt10prashant commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/613#discussion_r35197771 --- Diff: test/integration/testpaths/testpath_uuid_event.py --- @@ -0,0 +1,198 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-8659: Verify presentation of v...

2015-07-22 Thread pritisarap12
GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/613 CLOUDSTACK-8659: Verify presentation of volume ID in events table Verify if events table records UUID of the volume instead of volume ID in description from which snapshot is created for 'S