[GitHub] GabrielBrascher commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
GabrielBrascher commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329778911 Thanks @nathanejohnson. Code LGTM! This is an automated message from the Apache Git

[GitHub] blueorangutan commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
blueorangutan commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329846263 Trillian test result (tid-1512) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 39474 seconds Marvin logs:

[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-15 Thread git
blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-329689764 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
blueorangutan commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329693744 Packaging result: ?centos6 ?centos7 ?debian. JID-1085 This is an automated message from the

[GitHub] blueorangutan commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
blueorangutan commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329693891 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
rhtyd commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329693827 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the

[GitHub] rhtyd commented on a change in pull request #2248: CLOUDSTACK-10056

2017-09-15 Thread git
rhtyd commented on a change in pull request #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#discussion_r139069450 ## File path: server/src/com/cloud/vm/UserVmManagerImpl.java ## @@ -3619,8 +3619,14 @@ public UserVmVO

[GitHub] rhtyd commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-15 Thread git
rhtyd commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-329689641 @blueorangutan test This is an automated

[GitHub] blueorangutan commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
blueorangutan commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329689767 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2248: CLOUDSTACK-10056

2017-09-15 Thread git
rhtyd commented on issue #2248: CLOUDSTACK-10056 URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-329689595 LGTM, I'll kick one final test run and then we can merge based on test results. Thanks @nathanejohnson @blueorangutan package

[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-15 Thread git
blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-329916687 Trillian test result (tid-1511) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server