Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 8932810883 for ; Tue, 11 Mar 2014 16:06:04 +0000 (UTC) Received: (qmail 59550 invoked by uid 500); 11 Mar 2014 16:06:01 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 59518 invoked by uid 500); 11 Mar 2014 16:06:00 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 59471 invoked by uid 99); 11 Mar 2014 16:05:59 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 11 Mar 2014 16:05:59 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id B80201D4DEE; Tue, 11 Mar 2014 16:05:57 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============7815347915255606675==" MIME-Version: 1.0 Subject: Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS From: "Damodar Reddy Talakanti" To: "Abhinandan Prateek" , "Chiradeep Vittal" Cc: "Donal Lafferty" , "Damodar Reddy Talakanti" , "cloudstack" Date: Tue, 11 Mar 2014 16:05:57 -0000 Message-ID: <20140311160557.27073.71230@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Damodar Reddy Talakanti" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/18964/ X-Sender: "Damodar Reddy Talakanti" References: <20140311122839.27073.48292@reviews.apache.org> In-Reply-To: <20140311122839.27073.48292@reviews.apache.org> Reply-To: "Damodar Reddy Talakanti" X-ReviewRequest-Repository: cloudstack-git --===============7815347915255606675== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit > On March 11, 2014, 12:28 p.m., Donal Lafferty wrote: > > 1. How do you guarantee that 'sudo' is never added to scripts created by the Script class? See (utils/src/com/cloud/utils/script/Script.java) > > > > 2. How do you version control the .exe? > > > > 3. WiX can be run on Linux in a WINE environment. Could the Windowsfication team check whether it would be reasonable to call WiX from a Maven POM that does uses WINE if in a Linux environment? This might avoid having to checkin a .exe > > Damodar Reddy Talakanti wrote: > 1. The final thing we wanted to do is to convert all shell scripts which are getting called inside management server(This does not include Agents as always they run on Linux Environments) to python scripts. For now I identified the shell scripts those are getting called in side management server and written python script(s) for them (Still need to write for seed system template) and calling for only windows environemnt. In the next phase I will remove OS > checks inside the code and will call python scripts only. > 2. start.exe is generated out of start.bat so version control is on start.bar actually. > 3. We will check on this. > > Donal Lafferty wrote: > Hi Damodar, > > WRT #2, I don't mean source version control. I was thinking of the version reported by the .exe E.g windows .exe's often use the pattern Major.Minor.Build.Revision I want to understand where the version information on start.exe is coming from. > > Could you point me to the source for start.exe? > Hi Donal, I was also looking into the specific format of .exe but I could not get much on that. If you can point me to some link that is good for me. Actually we have used Quick Batch Compiler to convert to .exe format. - Damodar Reddy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review36773 ----------------------------------------------------------- On March 11, 2014, 12:45 p.m., Damodar Reddy Talakanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18964/ > ----------------------------------------------------------- > > (Updated March 11, 2014, 12:45 p.m.) > > > Review request for cloudstack, Abhinandan Prateek and Chiradeep Vittal. > > > Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6105 > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-6105 > > > Repository: cloudstack-git > > > Description > ------- > > Windowsfication of CloudStack Management Server > > > Diffs > ----- > > scripts/installer/windows/acs.wxs PRE-CREATION > scripts/installer/windows/client.wxs PRE-CREATION > scripts/installer/windows/start.bat PRE-CREATION > scripts/installer/windows/start.exe PRE-CREATION > scripts/vm/systemvm/injectkeys.py PRE-CREATION > server/src/com/cloud/server/ConfigurationServerImpl.java b8da4c8 > > Diff: https://reviews.apache.org/r/18964/diff/ > > > Testing > ------- > > Tested in Linux environment after changes > Also tested in Windows environment(For now tested on windows-8) to make sure it is getting installed and management service is running. > > Able to add zones, able to register templates, able to launch a VM when it is running on windows. > > Currently though it is getting added as a windows service, not able to start the service through windows service control manager which I am looking into currently. When run .exe file which is installed then server is getting up and able to access cloud stack UI. > > For now The following assumptions are made: > 1. SSH keys are already installed > 2. JAVA is already installed > 3. tomcat is already installed > > This patch contains the following new files related to WiX tool (To compile and run we need this tool) > 1. acs.wxs > The following command will be used to compile > "\bin\candle.exe" acs.wxs > 2. client.wxs > The following command will be used to generate the above file > "\bin\heat" dir client -gg -cg Test -ke -sfrag -template fragment -out client.wxs -var wix.SourceClient -dr WEBAPPS > The following command will be used to compile the above generated file > "\bin\candle.exe" client.wxs > 3. The following command will be used to generate .msi file > "\bin\light.exe" acs.wixobj client.wixobj -out acs.msi -ext "C:\Program Files (x86)\WiX Toolset v3.8\bin\WixUIExtension.dll" -dSourceClient=SourceDir\client > > > Thanks, > > Damodar Reddy Talakanti > > --===============7815347915255606675==--