RE: [DISCUSS]CLOUDSTACK-6191

2014-07-21 Thread Santhosh Edukulla
: Re: [DISCUSS]CLOUDSTACK-6191 Seems like it could have been fixed with a smaller code replace but I guess you are right. @Santosh, Can you split your commit and reapply. At least chipping off the BridgeVifDriver bit, but preferably all chopped up as other issues will probably come up. thanks

Re: [DISCUSS]CLOUDSTACK-6191

2014-07-17 Thread Daan Hoogland
27;d better refactor the code in > BridgeVifDriver->Plug > >> -Original Message- >> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] >> Sent: Tuesday, July 15, 2014 4:43 AM >> To: dev; Edison Su; Santhosh Edukulla >> Subject: Re: [DISCUSS]CLOUDSTACK-6191 >&

RE: [DISCUSS]CLOUDSTACK-6191

2014-07-16 Thread Edison Su
t;Plug > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Tuesday, July 15, 2014 4:43 AM > To: dev; Edison Su; Santhosh Edukulla > Subject: Re: [DISCUSS]CLOUDSTACK-6191 > > Edison, > > You reverted a600d8408ea86782318139c17

Re: [DISCUSS]CLOUDSTACK-6191

2014-07-15 Thread Daan Hoogland
ert or change CLOUDSTACK-6191. > >> -Original Message- >> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] >> Sent: Saturday, July 12, 2014 11:02 AM >> To: dev >> Subject: Re: [DISCUSS]CLOUDSTACK-6191 >> >> -0 What does it fix and is the solution

RE: [DISCUSS]CLOUDSTACK-6191

2014-07-14 Thread Edison Su
CLOUDSTACK-7051. So I think, both CLOUDSTACK-6191 and CLOUDSTACK-7051 should be fixed now, no need to revert or change CLOUDSTACK-6191. > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Saturday, July 12, 2014 11:02 AM > To: dev > Subjec

Re: [DISCUSS]CLOUDSTACK-6191

2014-07-12 Thread Daan Hoogland
-0 What does it fix and is the solution bonifide. We should fix the test suite if it is. The test suite not working is not enough reason to revert a commit, it should block the test-suite because the system is broken, not because of the way the test suite works. Disclaimer: I do not know enough of

RE: [DISCUSS]CLOUDSTACK-6191

2014-07-11 Thread Rayees Namathponnan
+1 Revert now and enable after complete full test in KVM KVM automation blocked more than 7 days due to this defect https://issues.apache.org/jira/browse/CLOUDSTACK-7051 Regards, Rayees -Original Message- From: Edison Su [mailto:edison...@citrix.com] Sent: Friday, July 11, 2014 2:49