[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2017-01-26 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1749 LGTM. From the discussion, looks like the test failures are unrelated to this PR. merging this now. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-19 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-396 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-18 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-18 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski thanks, I'll see if we can get some help. @abhinandanprateek @murali-reddy @blueorangutan package --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 The semi-unusual part about this PR, @rhtyd, is that is contains code jointly developed by me and @syed. He has reviewed my contributions and I have reviewed his. We

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-15 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 Hi @rhtyd We have one LTGM (from @syed) and I have run and posted all regression tests for managed storage. If we could have one more person take a look, that would be great. I don't

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Trillian test result (tid-683) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46910 seconds Marvin logs:

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan test --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-395 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-14 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 Ping where are we with this PR, @mike-tutkowski @syed @abhinandanprateek @murali-reddy thanks. @blueorangutan package --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Trillian test result (tid-657) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 40628 seconds Marvin logs:

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Trillian test result (tid-654) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 35269 seconds Marvin logs:

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski I've reviewed the code. LGTM :+1: --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Trillian test result (tid-655) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27881 seconds Marvin logs:

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-384 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-11 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan test --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-377 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan test centos7 xenserver-65sp1 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Trillian test result (tid-643) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36803 seconds Marvin logs:

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan test centos7 vmware-55u3 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-374 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-372 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✖centos6 ✔centos7 ✖debian. JID-368 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski Sure. I can review in the weekend --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-359 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-07 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-346 --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-07 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski thanks, I'll have another look at the PR. I'll kick some tests shortly. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-07 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @syed Would you be able to review this code when you have a little time? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-07 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-07 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 /cc @abhinandanprateek @blueorangutan package --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-06 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd It took a few force pushes, but it worked. :) --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-05 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski can you push -f again? --- 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

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-05 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 @mike-tutkowski yes, the failure was due to an intermittent unit test failure. This will be fixed with #1816 --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd Are we having trouble with Travis? I've amended my SHA a few times and get different Travis failures. Thanks! --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1749: CLOUDSTACK-9619: Updates for SAN-assisted snapshots

2016-12-05 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 I've updated the commit summary, @rhtyd --- 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