[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-08 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-356195834 Tests LGTM (template related errors not seen in last run), merging this based on code reviews and test results. -

[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-07 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-355814222 @blueorangutan test This is an automated message from the

[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-07 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-355812483 @blueorangutan package This is an automated message from

[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-06 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-355763383 Thanks @DaanHoogland I'll check again tomorrow when the results are back, may tackle outstanding issues myself. -

[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-05 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-355495080 @nvazquez can you also please fix the conflicts? This is

[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-04 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-355492347 @blueorangutan package @rafaelweingartner @marcaurele can you re-review the changes/lgtm is okay? -

[GitHub] rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates

2018-01-03 Thread GitBox
rhtyd commented on issue #2379: CLOUDSTACK-10146: Bypass Secondary Storage for KVM templates URL: https://github.com/apache/cloudstack/pull/2379#issuecomment-355203101 Tests LGTM. I'll wait for lgtm from @borisstoyanov @DaanHoogland @rafaelweingartner