[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1586 Merging as all the required LGTMs are there. --- 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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1586 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-213 --- 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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1586 @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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1586 @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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/1586 @syed @koushik-das can one of you please merge this --- 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 #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/1586 @cloudmonger None of the test failures are related to the code change --- 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

[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1586 ### ACS CI BVT Run **Sumarry:** Build Number 122 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=3 Skipped=5 _Link to logs Folder

[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1586 code change 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

[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1586 agree with @yvsubhash. I don't see an need to set the name-label to `detached` the code LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user chiradeep commented on the issue: https://github.com/apache/cloudstack/pull/1586 I believe that http://dl.openvm.eu/cloudstack/ has modern templates that have the appropriate userdata / ssh key and password change agents built in. I have raised

[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user yvsubhash commented on the issue: https://github.com/apache/cloudstack/pull/1586 @wido Replacing the original name with 'detached is not correct as it is confusing. Anyway data disk being detached is captured in the state of the volume --- If your project is set up for

[GitHub] cloudstack issue #1586: CLOUDSTACK-9410 Data Disk shown as detached in XS

Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1586 Thanks! Is there a issue attached to this PR or any additional information on why 'detached' isn't correct? --- If your project is set up for it, you can reply to this email and have your reply