[GitHub] cloudstack pull request: CLOUDSTACK-9133: Two volume.delete usage ...

2015-12-23 Thread priyankparihar
Github user priyankparihar closed the pull request at: https://github.com/apache/cloudstack/pull/1207 --- 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 fe

[GitHub] cloudstack pull request: CLOUDSTACK-9133: Two volume.delete usage ...

2015-12-12 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1207#issuecomment-164178065 @priyankparihar Could you point me where in VolumeStateListener it is emitted? --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack pull request: CLOUDSTACK-9133: Two volume.delete usage ...

2015-12-12 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1207#issuecomment-164177131 @priyankparihar these are nice results but I have no way of verifying the latter is actually the good result. can you write a test that proves proper behaviou

[GitHub] cloudstack pull request: CLOUDSTACK-9133: Two volume.delete usage ...

2015-12-11 Thread priyankparihar
Github user priyankparihar commented on the pull request: https://github.com/apache/cloudstack/pull/1207#issuecomment-163912073 @DaanHoogland Prior Behaviour | 127 | VOLUME.DELETE | 2 | 2015-12-08 09:30:18 | 1 | 25 | ROOT-14

[GitHub] cloudstack pull request: CLOUDSTACK-9133: Two volume.delete usage ...

2015-12-11 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1207#issuecomment-163903617 @priyankparihar please expand on testing that can prove prior and new behaviour. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] cloudstack pull request: CLOUDSTACK-9133: Two volume.delete usage ...

2015-12-10 Thread priyankparihar
GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1207 CLOUDSTACK-9133: Two volume.delete usage events are getting generated… It was happening because event triggering was happening in UserVmManagerImpl.java and VolumeStateListener.java r