[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1747 merging --- 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 t

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @karuturi Test passed. Two LGTM received. Looks like it is ready for merging --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 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 and wishes so, or if the fe

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Trillian test result (tid-808) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32193 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-476 --- 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 d

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1747 @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 wi

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 @DaanHoogland 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 r

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 @DaanHoogland 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 rep

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-09 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1747 @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 an

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-08 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner Do we get you LGTM? @DaanHoogland @rhtyd Can we kick off tests for this? --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1747 @serg38 got it. Thanks for the explanation --- 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

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner This PR is two fold. 1. This is something that was done a long ago for hosts and was never implemented for storage. It transitions location of of storage tags over to ded

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1747 @nvazquez, @serg38, before checking code any deeper a have a more high-level question. This PR will change the response of an API method, right? It will remove some data that

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner Can you review this PR? --- 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 fea

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rhtyd @karuturi Can we run tests on this PR and merge on success ? --- 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 #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-455 --- 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 d

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 @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 appe

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-10 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1747 @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 wishes

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-10 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rhtyd Done, thanks! --- 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 wishe

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-10 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1747 @nvazquez can you squash your change and see why Travis failed, in case of intermittent issue, push -f to rekick Travis. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-01-10 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rhtyd Can you kick off tests fro this PR? --- 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 featu

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

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

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

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

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

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