Re: Mouting Dvd/Cd Rom from VM and Vm Networking

2015-08-07 Thread Prapul
Hi Giri, For Cloudstack deployed VM you have option  to attache iso,for that you make that dvd as iso and upload it through Cloudstack. Regards, Prapul sriram, Cloud Architect, PSI Cloud Solutions, Psiclouds.com. On Aug 7, 2015 11:26, Giri Prasad wrote: > > Hi All, > > I have installed Cloudst

Re: Off topic about fork on git hub and pull requests

2015-08-07 Thread Rafael Weingärtner
Many thanks mike that worked out. On Fri, Aug 7, 2015 at 1:17 AM, Mike Tutkowski wrote: > If the end goal is to just have the master branch on your fork equal to the > master branch on origin, how's about fetching master on origin to your > local system, then switching to your local master branc

[GitHub] cloudstack pull request: CLOUDSTACK-8638: install update_host_pass...

2015-08-07 Thread kishankavala
Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/663#issuecomment-128666248 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

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread kevindierkx
GitHub user kevindierkx opened a pull request: https://github.com/apache/cloudstack/pull/664 Removed leading tabs and trailing spaces from ui/scripts/*.js You can merge this pull request into a Git repository by running: $ git pull https://github.com/kevindierkx/cloudstack mas

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/664#issuecomment-128682659 @karuturi @DaanHoogland @remibergsma As discussed on Slack August 3rd. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack pull request: CLOUDSTACK-8716: Verify creation of snaps...

2015-08-07 Thread pritisarap12
GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/665 CLOUDSTACK-8716: Verify creation of snapshot from volume when the task is performed repeatedly in zone wide primary Storage On VMWare with a Zone wide primary storage and more than two clus

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/666 CLOUDSTACK-8709 No out of band migrate alert for non-routers This prevents the out-of-band migrate alerts being generated for non-routers. You can merge this pull request into a Git reposit

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/666#issuecomment-128685143 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

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/664#issuecomment-128685238 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

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

2015-08-07 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128685407 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

[GitHub] cloudstack pull request: Implemented condition that only admin or ...

2015-08-07 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/658#issuecomment-128686068 Code LGTM, but is it the case right now that anybody can change a template? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-8714 Restore VM (Re-install VM...

2015-08-07 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/659#issuecomment-128686244 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

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128686334 @kishankavala Made the change to use VM type instead of name to distinguish between user and system VMs --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: Implemented condition that only admin or ...

2015-08-07 Thread manuiiit
Github user manuiiit commented on the pull request: https://github.com/apache/cloudstack/pull/658#issuecomment-128686845 @wido Ya I checked it.Any user was able to change permissions of the template.Updated the description with a case i tried. --- If your project is set up for it, yo

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36513703 --- Diff: ui/scripts/instances.js --- @@ -1959,16 +1959,16 @@ }, converter:

[GitHub] cloudstack pull request: CLOUDSTACK-8717: Failed to start instance...

2015-08-07 Thread pritisarap12
GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/667 CLOUDSTACK-8717: Failed to start instance after restoring the running instance On setup with two cluster wide primary storage verify restoring a running instance.(As while restoring insta

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36513758 --- Diff: ui/scripts/regions.js --- @@ -197,7 +197,7 @@ var region = args.context.regions[0];

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36513912 --- Diff: ui/scripts/storage.js --- @@ -55,13 +55,13 @@ } /* -

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/666#issuecomment-128689198 @borisroman Thx, removed the redundant parentheses --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/664#issuecomment-128689329 We've used the following scripts to remove the trailing white spaces and leading tabs: Tabs to 4 white spaces: find . -iname '*.js' -type f -exec

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514016 --- Diff: ui/scripts/templates.js --- @@ -519,16 +519,16 @@ }); }

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514301 --- Diff: ui/scripts/templates.js --- @@ -2689,14 +2689,14 @@ //do nothing } else { allowedActions.push

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514359 --- Diff: ui/scripts/templates.js --- @@ -2735,8 +2735,8 @@ /* if(g_regionsecondaryenabled != true) {

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514540 --- Diff: ui/scripts/ui-custom/regions.js --- @@ -37,9 +37,9 @@ $li.data('region-data', region);

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514600 --- Diff: ui/scripts/ui/widgets/detailView.js --- @@ -1062,19 +1062,19 @@ //??? /* -

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514699 --- Diff: ui/scripts/ui/widgets/listView.js --- @@ -275,7 +275,7 @@ });

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514806 --- Diff: ui/scripts/vpc.js --- @@ -155,41 +155,41 @@ }; */ //virtualmachineids param

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36514941 --- Diff: ui/scripts/zoneWizard.js --- @@ -963,16 +963,16 @@ if ($(this).val() == "VMware") {

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515422 --- Diff: ui/scripts/zoneWizard.js --- @@ -983,12 +983,12 @@ } else { -

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515489 --- Diff: ui/scripts/zoneWizard.js --- @@ -1221,14 +1221,14 @@ }, */ --- End diff --

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515504 --- Diff: ui/scripts/zoneWizard.js --- @@ -1221,14 +1221,14 @@ }, */ -

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515479 --- Diff: ui/scripts/zoneWizard.js --- @@ -1139,15 +1139,15 @@ dependsOn: 'overridepublictraffic'

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515461 --- Diff: ui/scripts/zoneWizard.js --- @@ -1139,15 +1139,15 @@ dependsOn: 'overridepublictraffic'

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515531 --- Diff: ui/scripts/zoneWizard.js --- @@ -1972,45 +1972,45 @@ } }, secondaryStorage: { -

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515568 --- Diff: ui/scripts/zoneWizard.js --- @@ -2108,17 +2108,17 @@ $fields.filter('[rel=username]').hide

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515666 --- Diff: ui/scripts/zoneWizard.js --- @@ -4363,12 +4363,12 @@ array1.push("&password=" + todb(args.data.cluster.vCenter

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515683 --- Diff: ui/scripts/zoneWizard.js --- @@ -4377,23 +4377,23 @@ if (args.data.cluster.vSwitchGuestType != "")

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36515654 --- Diff: ui/scripts/zoneWizard.js --- @@ -4363,12 +4363,12 @@ array1.push("&password=" + todb(args.data.cluster.vCenter

[GitHub] cloudstack pull request: CLOUDSTACK-8301: Enable configuring local...

2015-08-07 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/661#issuecomment-128693767 Travis is green and 2LGTM, merging now. --- 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 proj

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36516165 --- Diff: ui/scripts/templates.js --- @@ -2746,7 +2722,7 @@ if (((isAdmin() == false && !(jsonObj.domainid == g_domainid && jsonObj.account

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/664#discussion_r36516193 --- Diff: ui/scripts/ui-custom/accountsWizard.js --- @@ -130,89 +130,89 @@ dataType: "json",

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread kevindierkx
Github user kevindierkx commented on the pull request: https://github.com/apache/cloudstack/pull/664#issuecomment-128695172 I've removed the above mentions of commented code. Should be all set. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] cloudstack pull request: CLOUDSTACK-8638: install update_host_pass...

2015-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/663 --- 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 feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8714 Restore VM (Re-install VM...

2015-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/659 --- 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 feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/666#issuecomment-128717347 LGTM from my super phone ;) --- 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 doe

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/666 --- 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 feature is

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128719973 @koushik-das I' testing this tonight and will merge if it's OK. Before I proceed, could you please squash the 3 commits into 1. I cannot merge it like this. Than

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/666#discussion_r36526622 --- Diff: server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java --- @@ -2627,7 +2627,7 @@ public boolean postStateTransitionEven

[GitHub] cloudstack pull request: CLOUDSTACK-8709 No out of band migrate al...

2015-08-07 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/666#discussion_r36527589 --- Diff: server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java --- @@ -2627,7 +2627,7 @@ public boolean postStateTransitionEven

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128727494 @koushik-das you can solve it like this: - start with a new branch from your first commit - then cherry-pick the second one in - consider if you want t

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128745820 @remibergsma Will the following work? - do git pull on master - then do a git pull --rebase, this will make the commits appear on top - then do the squ

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128748395 Hi @koushik-das Sorry to step in like this, but I talked to @remibergsma and I think you can fix your branch by doing this: ``` git checkout

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128748550 Thanks @miguelaferreira :-) --- 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

Re: Off topic about fork on git hub and pull requests

2015-08-07 Thread Mike Tutkowski
Glad to hear it worked. :) On Friday, August 7, 2015, Rafael Weingärtner wrote: > Many thanks mike that worked out. > > On Fri, Aug 7, 2015 at 1:17 AM, Mike Tutkowski < > mike.tutkow...@solidfire.com > > wrote: > > > If the end goal is to just have the master branch on your fork equal to > the

[GitHub] cloudstack pull request: [DISCUSS] CLOUDSTACK-8652: remove old upg...

2015-08-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/608#issuecomment-128792204 @DaanHoogland Shall we close this PR and discuss on list what we want to do? --- If your project is set up for it, you can reply to this email and have your repl

[4.6] Issues deploying VM on VMware

2015-08-07 Thread Mike Tutkowski
Hi, I was wondering if anyone was aware of issues in 4.6 around deploying VMs on VMware. I have a Basic Zone running system VMs on XenServer and am trying to deploy a VM to local storage on ESXi. CloudStack is claiming insufficient space to deploy the VM. I plan to look into this in more depth,

[GitHub] cloudstack pull request: CLOUDSTACK-8704: Schedule restart of rout...

2015-08-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/656#issuecomment-128828795 Hi @koushik-das I was able to test your PR and it seems to do what you said: start the system VMs before the other instances. However, it doesn't solve the probl

Cloudstack collab Dublin 8-9 October

2015-08-07 Thread Giles Sirett
All We've got the schedule published for CCCEU15 http://events.linuxfoundation.org/events/cloudstack-collaboration-conference-europe/program/schedule Thanks to everybody who is participating Could folks on this list please help by promoting the schedule on twitter,etc Kind Regards Giles Gile

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/664#issuecomment-128833636 LGTM, i will have a go at merging when I have Miguels new version of the merge script and the origin/apache triangle ready on my new (post schuberg philis) work

[GitHub] cloudstack pull request: Removed leading tabs and trailing spaces ...

2015-08-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/664 --- 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 feature is

[GitHub] cloudstack pull request: [DISCUSS] CLOUDSTACK-8652: remove old upg...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/608#issuecomment-128839114 No momentum or consensus for this kind of work. discussion on dev@ is fine but I expect no technical solution and political/marketing impasse. --- If your proj

[GitHub] cloudstack pull request: [DISCUSS] CLOUDSTACK-8652: remove old upg...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland closed the pull request at: https://github.com/apache/cloudstack/pull/608 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-128843113 @devdeep You have written a very fine test and as it passes and tests what its target should do you would get a definate lgtm on tat one from me. However it doe

Problem in configuring autoscaling feature in CloudStack

2015-08-07 Thread cloud stack
Hi All, We are trying to use autoscaling feature in Cloudstack either with NetScaler or without NetScaler. We have tried with both Cloudstack versions 4.4 and 4.5. *Our configuration:* -- management server : centos 6.4 ip:192.168.1.10 netmask:255.255.255.0 gateway:192.168.1.1 -- hos

Re: Mouting Dvd/Cd Rom from VM and Vm Networking

2015-08-07 Thread Giri Prasad
Hi Prapul, Thanks for the comment. I will try that. Vmware users are saying that they can mount a dvd/cd directly from within the vm itself. Is there any options like that for cloudstack. Also, Windows7 vm networking is not working. In the host containing Ubuntu 14.04, lspci says : "Realtek