Re: Review Request 24646: CLOUDSTACK-7337: Volume state remains in allocated after volume creation faliure from snapshot

2014-08-20 Thread Koushik Das
096 - Koushik Das On Aug. 20, 2014, 11:38 a.m., Harikrishna Patnala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apa

RE: Unable to connect to management server on recent builds

2014-09-02 Thread Koushik Das
This is the erring commit commit c730bc3491f8b684c5ae51e0bff54cf7577cc246 Author: Rohit Yadav Date: Mon Sep 1 21:14:13 2014 +0200 server: Add event bus bean as commented xml in META-INF core This adds a spring bean xml to have EventBus for ACS, but the bean is commented so th

RE: simulator run

2014-09-04 Thread Koushik Das
Are the tests present under test/integration/smoke/misc also getting executed? If so please exclude them for now. The tests under /misc folder is based on the new simulator changes to test failures from agent layer. These tests can cause interference in other tests. There are 2 options to make t

RE: simulator run

2014-09-04 Thread Koushik Das
t: Re: simulator run Hey, This is what i'm executing to run the tests: /usr/local/bin/nosetests-2.7 -v --with-marvin --marvin-config=setup/dev/advanced.cfg --with-xunit --xunit-file=xunit.xml -a tags=advanced,required_hardware=false --zone=Sandbox-simulator --hypervisor=simulator -w test

RE: simulator run

2014-09-04 Thread Koushik Das
chance of getting a list of tests that are valid for execution against the simulator for 4.3, 4.4 and master? Thanks, Ian On 4 September 2014 13:03, Koushik Das wrote: > Ok, in that case the tests under /misc won't get executed. Looks like > there are some other issues with the fai

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

2014-09-08 Thread Koushik Das
tps://reviews.apache.org/r/25430/#comment91419> In the prvious line, service offering is getting set, why set it again? Since there is a way to pass offering in ctor, is an explicit setter really required? - Koushik Das On Sept. 8, 2014, 9:16 a.m., bharat kumar

Re: Review Request 26168: There are new VM State Machine changes introduced which were missed to capture the usage events

2014-10-09 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26168/#review55964 --- - Koushik Das On Oct. 9, 2014, 6:26 a.m., Damodar Reddy Talakanti

RE: Review Request 26168: There are new VM State Machine changes introduced which were missed to capture the usage events

2014-10-09 Thread Koushik Das
Somehow the review comments are not showing? 1. Would it make sense to make the pre/post method signature identical? 2. In StateEntry.addTransition(), do you need E and S as separate parameters? From: Koushik Das [mailto:nore...@reviews.apache.org] On Behalf Of Koushik Das Sent

Re: [ANNOUNCE] New PMC Member: Rajani Karuturi

2014-10-17 Thread Koushik Das
Congrats Rajani! On 17-Oct-2014, at 3:51 PM, Daan Hoogland wrote: > The Project Management Committee (PMC) for Apache CloudStack are pleased to > announce that Rajani Karuturi has accepted our invitation to join the PMC. > > Please join me in congratulating her. > > On behalf of the Apache Clo

RE: long running async job

2015-02-05 Thread Koushik Das
There is another config 'job.cancel.threshold.minutes' (defaulted to 60 mins) based on which async job is forcefully cancelled. Can you check if this is cancelling the job? Also if you want to debug - checkout the GC code present in AsyncJobManagerImpl.java -Koushik -Original Message-

[DISCUSS] Allow configuring system VMs to use local storage at zone level

2015-02-27 Thread Koushik Das
Currently it is not possible to configure system VMs to use local storage on a per zone basis. Since it is a global config (system.vm.use.local.storage), either it is enabled for all zones or none. The idea is to make it a zone level config. The current zone level setting of 'enabling local stor

Re: [DISCUSS] Allow configuring system VMs to use local storage at zone level

2015-03-01 Thread Koushik Das
> So, it is already implemented. > > Please give me 12 hours to send a code snippet... I'm replying from my phone > now. > > Cheers, > Wilder > > Sent from my iPhone > >> On 27 Feb 2015, at 14:21, Rohit Yadav wrote: >> >> Sounds good, g

Re: [ANNOUNCE] New committer: Wilder Rodrigues

2015-03-01 Thread Koushik Das
Congrats Wilder! On 01-Mar-2015, at 3:44 PM, Daan Hoogland wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Wilder Rodrigues to become a committer and we are pleased to > announce that he has accepted. > > Keep up the good work and your contagious good spirit Wi

Re: [DISCUSS] Allow configuring system VMs to use local storage at zone level

2015-03-02 Thread Koushik Das
to the message we get when we select the local storage > option, but it works. > > If when creating a zone you select local storage but on the global settings > local storage is false, it will respect the selection per zone. > > So, it is already implemented. > > Please give

[PROPOSAL] Allow configuring system VMs to use local storage at zone level

2015-03-05 Thread Koushik Das
Based on the discussion [1], I have created a FS [2] and tracking bug [3]. Please review and provide your comments. -Koushik [1] http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201502.mbox/%3cad8a3382-cfc6-4fd3-92de-df61f5a8e...@citrix.com%3e [2] https://cwiki.apache.org/confluence/dis

RE: [ANNOUNCE] New committer: Harikrishna Patnala

2015-03-09 Thread Koushik Das
Congrats Hari! -Original Message- From: Rajani Karuturi [mailto:raj...@apache.org] Sent: Tuesday, 10 March 2015 11:00 To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Harikrishna Patnala The Project Management Committee (PMC) for Apache CloudStack has asked Harikrishna P

RE: [DISCUSS] Support Docker as a hypervisor in CloudStack ( CloudStack / CLOUDSTACK-8205)

2015-03-15 Thread Koushik Das
This is another alternate https://coreos.com/using-coreos/containers/. CS needs to support CoreOS. -Original Message- From: Diwas Joshi [mailto:dj.dij...@gmail.com] Sent: Monday, 16 March 2015 2:53 To: dev@cloudstack.apache.org Subject: [DISCUSS] Support Docker as a hypervisor in CloudSt

Re: [ANNOUNCE] New committer: Karen Vuong

2015-03-19 Thread Koushik Das
Congrats Karen! On 20-Mar-2015, at 3:37 AM, Pierre-Luc Dion wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Karen Vuong to become a committer and we are pleased to > announce that she have accepted. > > Karen is doing a huge effort on promoting CloudStack and o

RE: [Announce] - New VP Apache Cloudstack

2015-03-23 Thread Koushik Das
Congrats Sebastien and thanks Hugo! -Original Message- From: Giles Sirett [mailto:giles.sir...@shapeblue.com] Sent: Monday, 23 March 2015 17:11 To: dev@cloudstack.apache.org; us...@cloudstack.apache.org; market...@cloudstack.apache.org Subject: [Announce] - New VP Apache Cloudstack All

RE: [ANNOUNCE] Rohit Yadav as new PMC member of CloudStack

2015-03-27 Thread Koushik Das
Congrats Rohit! -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Friday, 27 March 2015 13:38 To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Rohit Yadav as new PMC member of CloudStack The Project Management Committee (PMC) for Apache CloudStack are pleased t

RE: [ANNOUNCE] New committer Pierre-Yves Ritschard

2015-04-02 Thread Koushik Das
Congrats Pierre-Yves! -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Wednesday, 1 April 2015 2:41 To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer Pierre-Yves Ritschard The Project Management Committee (PMC) for Apache CloudStack has asked Pierre-Yves Ris

RE: [ANNOUNCE] New committer: Rene Moser

2015-04-02 Thread Koushik Das
Congrats Rene! -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Wednesday, 1 April 2015 2:43 To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Rene Moser The Project Management Committee (PMC) for Apache CloudStack has asked Rene Moser to become a committer

Re: [VOTE] Accept the donation of ec2stack to apache cloudstack

2015-04-07 Thread Koushik Das
+1 -Koushik On 05-Apr-2015, at 2:44 PM, Ian Duffy wrote: > Hi All, > > As stated in a previous thread [1], myself, Sebastien and Darren are > proposing the donation of ec2stack [2]. > > I would like to call a vote, so that we have formal consensus on > accepting the code into the project. I s

RE: Simulator.

2015-04-07 Thread Koushik Das
After you have run deployDataCenter.py, it is possible to do VM/volume/network related operations from UI. From: Jeronimo Garcia [mailto:garciaj...@gmail.com] Sent: Tuesday, 7 April 2015 17:25 To: dev@cloudstack.apache.org Cc: cloudstack-...@incubator.apache.org Subject: Re: Simulator. One more:

RE: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye

2015-04-14 Thread Koushik Das
Congrats Gaurav! -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Tuesday, 14 April 2015 1:14 To: dev Subject: [ANNOUNCE] New committer: Gaurav Nandkumar Aradhye The Project Management Committee (PMC) for Apache CloudStack has asked Gaurav Aradhya to become

RE: [ANNOUNCE] New committer: Funs Kessen

2015-04-14 Thread Koushik Das
Congrats Funs! -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Tuesday, 14 April 2015 1:12 To: dev Subject: [ANNOUNCE] New committer: Funs Kessen The Project Management Committee (PMC) for Apache CloudStack has asked Funs Kessen to become a committer and we

RE: listUsageRecord only for Admins? Why?

2015-04-15 Thread Koushik Das
Agree. I don't think it's possible currently. The listUsageRecords needs to be modified. -Original Message- From: Ramzi Youssefi [mailto:ryouss...@gmail.com] Sent: Wednesday, 15 April 2015 13:28 To: dev Subject: listUsageRecord only for Admins? Why? Hello developers! is there any reas

Re: Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27868/#review60941 --- - Koushik Das On Nov. 11, 2014, 2:19 p.m., bharat kumar wrote

Re: Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread Koushik Das
tps://reviews.apache.org/r/27868/#comment102368> What operation, which hypervisor and VM? Also the info log is not required as the exception message will be present in logs. - Koushik Das On Nov. 11, 2014, 2:19 p.m., bharat kumar

Re: Review Request 27904: cloudstackoperations like deployvm, deleteNW are failing if CS fail to contact rabbit mq server

2014-11-11 Thread Koushik Das
4.5 -> f1361796bf1fc73a65c47583dd0a5d6ef2dc3c27 - Koushik Das On Nov. 12, 2014, 4:13 a.m., Damodar Reddy Talakanti wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 28700: Attach volume events are not correctly shown in events.

2014-12-04 Thread Koushik Das
4.5 -> 7ef0f5507d3b0debe4cfc7fb3bcada39c069e33f - Koushik Das On Dec. 4, 2014, 10:02 a.m., Damodar Reddy Talakanti wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: merge from 4.5 to master

2014-12-08 Thread Koushik Das
I tried a merge just now and it failed for tools/appliance/build.sh On 09-Dec-2014, at 11:23 AM, Mike Tutkowski wrote: > Sounds good - thanks > > On Mon, Dec 8, 2014 at 10:52 PM, Rajani Karuturi wrote: > >> yes >> >> ~Rajani >> >> On Tue, Dec 9, 2014 at 11:09 AM, Mike Tutkowski < >> mike.

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2015-12-16 Thread koushik-das
GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/1251 CLOUDSTACK-9180: Optimize concurrent VM deployment operation on same … …network Check if VR needs to be allocated for a given network and only acquire lock if required

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2015-12-16 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-165092578 Simulator test results for basic zone. Failure is for NiciraNvp which can be ignored

[GitHub] cloudstack pull request: CLOUDSTACK-9195: Cancelled/failed async j...

2015-12-22 Thread koushik-das
GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/1272 CLOUDSTACK-9195: Cancelled/failed async jobs not getting cleaned up f… …rom DB Setting 'job_complete_msid' field in async_job table when marking job as cancelled

[GitHub] cloudstack pull request: CLOUDSTACK-9195: Cancelled/failed async j...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1272#issuecomment-166552880 Steps to test: - Prior to fix check for jobs in async_job table where job_status = 2 and job_complete_msid is NULL. - These entries won'

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-166820236 Github is not showing the diff for the below file as there are too many changes. Is there any way to display it? engine/orchestration/src/org/apache

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1273#issuecomment-166823449 @nitin-maharana HypervisorUtilsTest test have failed, please do a force push again. Code changes LGTM --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-9196: Fixing null pointer exce...

2015-12-22 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1274#issuecomment-166824866 Code changes 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

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2015-12-23 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-166884395 @remibergsma What you are suggesting is to pull the commit locally and then view it using git show. But then we are loosing the benefit of viewing and in-place

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2015-12-23 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1285#issuecomment-167044637 @miguelaferreira Looks like this PR is going to impact the feature https://cwiki.apache.org/confluence/display/CLOUDSTACK/LDAP%3A+Trust+AD+and+Auto+Import that

[GitHub] cloudstack pull request: CLOUDSTACK-4572: findHostsForMigration AP...

2015-12-23 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1283#issuecomment-167045587 @anshul1886 If the fix was added in 4.2, why it got missed out? Can you check the history and update? --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Removed unused variables from "NetworkSta...

2015-12-23 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1261#issuecomment-167047897 Code changes 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

[GitHub] cloudstack pull request: Removed unused methods from EventBus inte...

2015-12-24 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1260#discussion_r48411330 --- Diff: plugins/event-bus/inmemory/src/org/apache/cloudstack/mom/inmemory/InMemoryEventBus.java --- @@ -58,34 +58,6 @@ public void setName(String

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2016-01-04 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1285#issuecomment-168905195 @remibergsma In the FS there is a requirement "Cloud admin should be able to to map AD OU / group to a Domain in CloudStack". So this means that LDAP

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-08 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-169979032 As a general cleanup, all configs defined in Config.java should be gradually moved to the appropriate java classes from where they are used. We should do away

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-08 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170011469 I have clearly mentioned that the refactoring needs to be done gradually. But in the changes you have reverted to using the config defined in Config.java when

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-08 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170202865 On latest master (without any fixes) I see that the config values (network.gc.interval and network.gc.wait) are read correctly. Did the following: - Started

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-09 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170237613 @wilderrodrigues I simply ran the steps mentioned against latest master in a simulator setup and attached a debugger. The network GC thread was running every

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-09 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170238553 @wilderrodrigues The GC thread execution frequency shouldn't depend on real hardware or simulator. It will always execute at gc.interval seconds as so

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-09 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170240228 @wilderrodrigues - Do you agree that gc thread should run every gc.interval seconds whether it has to cleanup any network or not? - Do you agree that

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-11 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170483420 @wilderrodrigues My questions and comments are very valid. If you don't see any value in them too bad. Looks like you don't want to understand the p

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-01-11 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-170488605 @pedro-martins By java doc are you referring to adding some comments for isRouterDeployed()? Note that this is a private method. If you look at the unit

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-11 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170520102 @wilderrodrigues My questions were not in the context of refactor at all. The reason for asking the questions was to highlight the point that for testing

[GitHub] cloudstack pull request: CLOUDSTACK-9195: Cancelled/failed async j...

2016-01-17 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1272#issuecomment-172421544 @DaanHoogland Thanks for reviewing the code. On MS shutdown/restart the incomplete job entry will always get marked as cancelled in DB. You are right that

[GitHub] cloudstack pull request: CLOUDSTACK-9130: Make RebootCommand simil...

2016-01-20 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1200#issuecomment-173474463 @remibergsma RebootCoomand is used by rebootVirtualMachine API. Refer to test_03_reboot_vm in smoke/test_vm_life_cycle.py. It is also used in rebootSystemVm

RE: git commit: updated refs/heads/master to 07c3089

2015-04-29 Thread Koushik Das
Agree. The commit message should provide a summary of the changes. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Wednesday, 29 April 2015 17:07 To: dev Subject: Re: git commit: updated refs/heads/master to 07c3089 On Wed, Apr 29, 2015 at 5:45 AM, wrote:

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-04-29 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-97452036 Reading the code, if testIpAddress() returns anything other than Up, Down or null then returned status is ignored. --- If your project is set up for it, you

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-04-29 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-97454863 @DaanHoogland If you look at the code from where testIpAddress() is called (for e.g. UserVmDomRInvestigator.isAgentAlive()), it only deals with return of Up

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-04-29 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-97503915 @remibergsma My comments are based on the latest master code. Based on my reading of the code, the changes you made shouldn't have any impact on the outco

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-04-29 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-97512774 Also about the scenario that is being addressed - all hosts in a XS cluster are shutdown by pulling the plug. Instead of immediately putting the hosts in Alert

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-05-01 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-98100363 I looked at the code and saw that if investigate() is not able to determine the state of host then by default Alert is returned, earlier it was null. This may

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-05-01 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-98100569 One possible solution I can think of the original problem for this PR is to wait for sometime (say based on the config parameter alert.wait) and if even after

RE: Jenkins build is back to normal : build-master-slowbuild #1429

2015-05-01 Thread Koushik Das
Hi Daan, In the slow build report mails, is it possible to add the findbug failures at the top along with the commit ids? -Koushik -Original Message- From: Daan Hoogland [mailto:dhoogl...@schubergphilis.com] Sent: Friday, 1 May 2015 14:31 To: dev@cloudstack.apache.org Subject: FW: Jenki

Re: [ANNOUNCE] New committer: Remi Bergsma

2015-05-01 Thread Koushik Das
Congrats Remi! On 01-May-2015, at 12:15 PM, Daan Hoogland wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Remi Bergsma to become a committer and we are pleased to > announce that he has accepted. > > Remi has reported and helped in investigating and solving man

[GitHub] cloudstack pull request: return a state instead of null in Abstrac...

2015-05-01 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/211#issuecomment-98120238 @remibergsma Yes XS (6.5) pool HA was enabled. @DaanHoogland Agree on the new state UNKNOWN --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: server: Introduce Unknown Status to be us...

2015-05-01 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/222#issuecomment-98181315 @bhaisaab The 'null' semantics of all investigators needs to change to 'Unknown'. Just look at all classes that implements Investigator

[GitHub] cloudstack pull request: server: Introduce Unknown Status to be us...

2015-05-01 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/222#issuecomment-98183123 Also callers for investigate()/isAgentAlive() methods needs to be fixed to handle Unknown instead of null. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: server: Introduce Unknown Status to be us...

2015-05-02 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/222#issuecomment-98372263 I thought the intent of introducing 'Unkonwn' host status was to replace the entire null semantics that was thought to be confusing in the investi

[GitHub] cloudstack pull request: server: Introduce Unknown Status to be us...

2015-05-02 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/222#issuecomment-98374391 @DaanHoogland It is not only to prevent NPE but also to go to the next item in the list when the status cannot be determined. Think of it like this - status

[GitHub] cloudstack pull request: server: Introduce Unknown Status to be us...

2015-05-03 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/222#issuecomment-98589348 I think there is agreement that this is a partial fix (with the right intent). And that this PR doesn't add any value from the end-users perspe

Re: [DISCUSS] XenServer and HA: the way forward

2015-05-04 Thread Koushik Das
The below is the proposal for switching to XenServer HA. http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201403.mbox/%3c83f77ff1bd50194ab65afa5479d082a71e4...@sjcpex01cl02.citrite.net%3E On 04-May-2015, at 9:03 PM, Tim Mackey wrote: > Thanks for starting this thread Remi. > > From my

[GitHub] cloudstack pull request: CLOUDSTACK-8471:Automation for feature En...

2015-05-14 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/253#issuecomment-102248563 Added to CLOUDSTACK-8301 branch. --- 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 pull request: Cloudstack-8301

2015-05-19 Thread koushik-das
GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/263 Cloudstack-8301 PR for configuring local storage for system VMs at zone level. FS @ https://cwiki.apache.org/confluence/display/CLOUDSTACK/Enable+configuring+local+storage+use+for

RE: SystemVM (kvm) : 3 different versions for the same CloudStack version (4.5.1)

2015-05-27 Thread Koushik Das
I feel that the ACS docs needs to include clear/detailed steps for building system VM templates. That way anyone can build templates from the source commit a particular release is voted against. -Koushik -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Thur

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-03 Thread Koushik Das
In case the VR is moved out of band (say as part of Vmware DRS), all network rules are lost. Rebooting VR from CS re-applies all the rules. Either the reboot is done manually from UI/API or automatically as was done as part of CLOUDSTACK-7994. I haven't looked at the aggregate command. If it ca

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-03 Thread Koushik Das
also limit the fix to this hypervisor only. Regards, Remi On 03 Jun 2015, at 14:12, Koushik Das mailto:koushik@citrix.com>> wrote: In case the VR is moved out of band (say as part of Vmware DRS), all network rules are lost. Rebooting VR from CS re-applies all the rules. Either the reb

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-03 Thread Koushik Das
regression in xen, i'm afraid Op wo 3 jun. 2015 om 14:38 schreef Koushik Das : > I think as a design principle we shouldn't introduce HV specific > checks in the orchestration/API layers. > I am not sure if the problem is specific to Vmware. Any out of band VR > movement can l

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-03 Thread Koushik Das
at). Great it is available in 4.4 as well so we can fix it in both 4.4 and 4.5 in the same way. On 03 Jun 2015, at 14:38, Koushik Das wrote: > > I think as a design principle we shouldn't introduce HV specific checks in > the orchestration/API layers. > I am not sure if the pr

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-03 Thread Koushik Das
not on the dev ML so I could not reply I reported this current behavior to be an issue on the user ML and wanted to ask Koushik Das about his experiences. I would not agree, in an Vmware environment live migrations, e.g. Vmware DRS breaks IPtables normally. In my opinion, this would make DRS

Re: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-03 Thread Koushik Das
On 04-Jun-2015, at 12:04 AM, Remi Bergsma wrote: > Hi all, > > I just had a look at this more closely and had a chat with Ian about it. The > only way for the original problem to happen (losing iptables rules) is if the > live migrate would fail and the hypervisor rebooted the vm. The cause i

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-04 Thread Koushik Das
Ian, With the persistent config change, the VR has become 'stateful'. Is there a possibility for VR config and DB config to go out of sync? If so how is the config in the VR and DB kept in sync? -Original Message- From: Ian Southam [mailto:isout...@schubergphilis.com] Sent: Thursday, 4 J

Re: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-04 Thread Koushik Das
? On 04-Jun-2015, at 5:28 PM, Daan Hoogland wrote: > On Thu, Jun 4, 2015 at 1:27 PM, Koushik Das wrote: >> Is there a possibility for VR config and DB config to go out of sync? If so >> how is the config in the VR and DB kept in sync? > > > Koushik, the db is leading a

Re: Strange bug? "spam" in management log files...

2015-06-04 Thread Koushik Das
This is expected in a clustered MS setup. What is the distribution of HV hosts across these MS (check host table in db for MS id)? MS owning the HV host processes all commands for that host. Grep for the sequence numbers (for e.g. 73-7374644389819187201) in both MS logs to correlate. On 04-Ju

RE: [DISCUSS] Out of Band VR migration, should we reboot VR or not?

2015-06-05 Thread Koushik Das
From the persistent config changes, it appears that aggregate command approach is not required in 4.6 and onwards. If someone wants to implement the aggregate approach only for 4.4 and 4.5 irrespective of the effort involved then that should be fine. Otherwise we can just put some config to dec

RE: Third party VR / L2 support

2015-06-11 Thread Koushik Das
Agree to what Funs mentioned. The current network service model is flexible, there is option to select a provider for a given service by means of network offering. About using 3rd party VR, there are 2 possibilities: - Fully replace the existing VR with 3rd party VR - Both co-exist and complement

RE: [PROPOSAL] Quota management service for cloudstack

2015-07-01 Thread Koushik Das
I like the concept of quota. But I am thinking whether this should be a core part of CS or developed as a plugin. Cloud coins example given below may be a plugin, based on which decision to allow/disallow operation is made. Another plugin could be to allow/disallow operations based on VM count o

Re: Blameless post mortem

2015-09-28 Thread Koushik Das
I had asked for the documentation on persistent VR (PR # 118) changes in the context of another discussion and this is what I got at that time. http://dev.cloudstack.apache.narkive.com/MH47etbS/discuss-out-of-band-vr-migration-should-we-reboot-vr-or-not#post39 Right now as I see from the discus

Re: Blameless post mortem

2015-09-28 Thread Koushik Das
inline On 28-Sep-2015, at 9:15 PM, Sebastien Goasguen wrote: > Let me try to reply, > >> On Sep 28, 2015, at 5:17 PM, Koushik Das wrote: >> >> I had asked for the documentation on persistent VR (PR # 118) changes in the >> context of another discussion and t

[GitHub] cloudstack pull request: CLOUDSTACK-8911: VM start job got stuck i...

2015-09-29 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/895#discussion_r40657381 --- Diff: server/src/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java --- @@ -297,6 +297,7 @@ s_logger.debug

[GitHub] cloudstack pull request: CLOUDSTACK-8911: VM start job got stuck i...

2015-09-29 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/895#discussion_r40657679 --- Diff: server/src/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java --- @@ -297,6 +297,7 @@ s_logger.debug

[GitHub] cloudstack pull request: CLOUDSTACK-8901: PrepareTemplate job thre...

2015-09-29 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/880#discussion_r40658483 --- Diff: server/src/com/cloud/template/TemplateManagerImpl.java --- @@ -278,6 +278,8 @@ @Inject private EndPointSelector selector

[GitHub] cloudstack pull request: CLOUDSTACK-8901: PrepareTemplate job thre...

2015-09-29 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/880#discussion_r40658531 --- Diff: setup/db/db/schema-452to460.sql --- @@ -413,3 +413,6 @@ CREATE TABLE `cloud`.`ldap_trust_map` ( UNIQUE KEY

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-29 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144030748 @resmo The changes LGTM. About testing on XS, the following should work: - Deploy a VM - Destroy the VM outside of CS (for e.g. from XenCenter

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Koushik Das
Raja, The earlier scale vm test couldn't run on simulator because of the validation steps (IIRC it tries to connect to the real VM created during test). Since it wasn't possible to run with simulator, the new test was added to run just on the simulator. Check the required_hardware tag. -Ori

[GitHub] cloudstack pull request: CLOUDSTACK-8911: VM start job got stuck i...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/895#issuecomment-144343011 LGTM. Have verified the scenario for max. guest limit. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-8906: /var/log/cloud/ doesn't ...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/883#issuecomment-144347154 @SudharmaJain Why can't the XenServer600Resource be used to handle XS 6.0.2 hosts? The only change I see is the patch script (scripts/vm/hypervisor/xens

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/861#issuecomment-144347634 I have updated #883 with my comments, looking at the changes it shouldn't be required. Lets wait for @SudharmaJain response --- If your project is set u

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144348817 LGTM. Since now test_01_scale_vm can be run on simulator, no need for the other one. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-8656: tests ignoring exception...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/850#issuecomment-144384074 @DaanHoogland @borisroman The test needs to invoke a protected method from a class and so is done using reflection. The test case already asserts for the method

<    1   2   3   4   5   6   7   8   9   10   >