---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18768/#review40542
---
Ship it!
Ship It!
- Koushik Das
On April 16, 2014, 10:14 a.m
In 4.3 you need to set global config router.version.check to false to
workaround this issue in simulator. This is fixed in 4.4/master by commit
38b8677e01fe1ec89df43abe9fac26295597080f.
-Koushik
-Original Message-
From: David Grizzanti [mailto:david.grizza...@sungard.com]
Sent: Wednesd
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20250/#review40378
---
Ship it!
Ship It!
- Koushik Das
On April 11, 2014, 2:37 a.m
On 12-Apr-2014, at 2:14 AM, Alex Hitchins wrote:
> Daan,
>
> Are you referring to keeping line lengths up to 80 characters? Sorry - tired
> eyes.
>
> My thoughts were more that in a function there should only be one "return"
> statement rather than many, all nested in layers of if/else stateme
+1 to the feature.
On 11-Apr-2014, at 6:05 AM, Michael Phillips wrote:
> I can create a JIRA request for this, but how many +1's do we need before it
> gets added to the "To Do" list?
>
>> From: alex.hu...@citrix.com
>> To: dev@cloudstack.apache.org
>> Subject: RE: Remove Mgmt Server
>> Date:
Thanks for updating the wiki Santhosh.
Also for any new feature adding agent commands, the simulator needs to be fixed
as well. The integration tests added for new features should be run against
simulator as well to ensure it is not broken.
-Koushik
On 03-Apr-2014, at 2:16 AM, Santhosh Edukulla
ontrol the agent state as well? Say I want to test VM
> HA, Can I be able to set the host status to be disconnected and then
> verify that VM's on the host gets migrated?
>
> On 01/04/14 5:47 PM, "Koushik Das" wrote:
>
>> The current simulator provides a way
shik] Yes.
> 5. Is there an ability to simulate fault for a specific storage/host? For
> example, I have 5 storages in my cluster but I want to simulate failure for 3
> of the storage pools. Is that possible?
>
[Koushik] Good point. I will add storage pool also into the scope.
>
>&g
The current simulator provides a way to test various Cloudstack features
without using actual resources. But its current capabilities are limited since
it only allows success response for all agent commands and some ability to
introduce delays. I am proposing some enhancements to the simulator [
patch?
- Koushik Das
On March 5, 2014, 11:17 a.m., Saksham Srivastava wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.a
4.4 -> fc8cb754f0c6645bac5f35d8ad3d16e5e17a2d50
- Koushik Das
On March 17, 2014, 5:33 a.m., Rajani Karuturi wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
60c,
f293c94bc0198f162c80ff807ca81459150354a5
- Koushik Das
On March 14, 2014, 9:36 a.m., Harikrishna Patnala wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apa
ommand=listVirtualMachines&listAll=true&ids=e
>>>> ddac053-9b12-4d2e-acb7-233de2e98112,009966fc-4d7b-4f84-8609-
>>>> 254979ba0134
>>>> The new parameter will be mutually exclusive with the existing 'id'
>>>> parameter.
>>>&g
> On March 13, 2014, 12:19 p.m., Koushik Das wrote:
> > Ship It!
Commit 5779292e93fe52a7fb5ec0abffe1e3f6b9c10121 in cloudstack's branch
refs/heads/master from [~koushikd]
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5779292 ]
CLOUDSTACK-6052: List VM enhancem
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19022/#review37045
---
Ship it!
Ship It!
- Koushik Das
On March 11, 2014, 6:29 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18902/#review37031
---
Ship it!
Ship It!
- Koushik Das
On March 7, 2014, 11:04 a.m
On 11-Mar-2014, at 7:24 PM, Daan Hoogland wrote:
> You are saying it must be null and not empty?
[Koushik] Yes.
>
>
> On Tue, Mar 11, 2014 at 1:41 PM, Koushik Das wrote:
>
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.ap
ids?
> > You are being strict on something that isn't hurtful but an empty ids
> > when no id is given is not checked!
>
> Koushik Das wrote:
> I haven't removed 'id' due to back-compat. As I have mentioned 'id' and
> 'ids
lly exclusive. Also it is a valid scenario to not pass any of id or
ids.
- Koushik
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19022/#review36764
------------
little 'English' rules implied in using a programming language, like the
> use of adjectives. They matter in Westerns languages and unless we define a
> system wide set of different rules let's stick with English.
>
> kind regards,
>
> Daan Hoogland
>
>
>
tion/smoke/test_deploy_vm.py 425aeb7
Diff: https://reviews.apache.org/r/19022/diff/
Testing
---
Added integration test, also verified manually.
Thanks,
Koushik Das
This is a new install using master. So this is not about db changes but mostly
a bug.
On 10-Mar-2014, at 9:58 PM, Marcus wrote:
> Mike has sent out several emails about the DB changes he made.
>
> On Mon, Mar 10, 2014 at 5:28 AM, Koushik Das wrote:
>> I am seeing the foll
I am seeing the following exception
ERROR [c.c.a.ApiServer] (38419149@qtp-1917357200-11:ctx-116b8207 ctx-9a54f764)
unhandled exception executing api command: [Ljava.lang.String;@36d30b1d
java.lang.NullPointerException
at
org.apache.cloudstack.engine.orchestration.CloudOrchestrator.creat
Before doing a git pull, I generally check the sql schema changes and run the
delta manually on my existing setup. In most of the cases that works for me
without having to redeploy the db.
-Koushik
On 06-Mar-2014, at 11:43 AM, Mike Tutkowski
wrote:
> Yeah, I definitely just meant a "heads up
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18352/#review36215
---
Ship it!
Ship It!
- Koushik Das
On March 5, 2014, 8:50 a.m
7f9
- Koushik Das
On March 4, 2014, 5:56 a.m., Rajani Karuturi wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apa
26d
- Koushik Das
On March 4, 2014, 5:55 a.m., Rajani Karuturi wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apa
I am facing a build failure but for a different reason on 4.3-forward. Looks
like due to
commit 065ac54bd7755cb44667c71311293c8f2110b3f3
Author: tuna
Date: Sat Mar 1 11:36:27 2014 +0700
CLOUDSTACK-6162: add UI for OVS plugin
[INFO] --
The primary storage needs to be put in maintenance before doing any
upgrade/reboot as mentioned in the previous mails.
-Koushik
On 03-Mar-2014, at 6:07 AM, Marcus wrote:
> Also, please note that in the bug you referenced it doesn't have a
> problem with the reboot being triggered, but with the
va
<https://reviews.apache.org/r/17889/#comment66515>
Same comments as in orchestrateStart()
- Koushik Das
On Feb. 27, 2014, 12:35 p.m., Sanjay Tripathi wrote:
>
> ---
> This is an automatically gene
t;
What is the use case for null host id? Make it long instead of Long.
- Koushik Das
On Feb. 21, 2014, 11:36 a.m., Saksham Srivastava wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
>
For XS this works as expected.
For system VMs (SSVM, CPVM) there is a monitoring thread to check if they are
up and running. If not then it tries to bring them up. Now if there is no PS
available then the deployment fails and it is tried again. This is expected.
-Koushik
On 15-Feb-2014, at 3:34
Please add more details for upgrade scenario. Also can you give some examples
as to how the capacity computation is done for operations where the volume gets
migrated from one PS to another having different over provisioning factors.
Is there any value add if you give this also at cluster scope?
f course, we can never delete the ³ID² parameter just to satisfy
>> the old convention, as this is the most used parameter :)
>>
>> I can see that several existing commands - archive/deleteAlerts are
>> using ApiConstants.IDs parameter. We can mark IDs as deprecated, so
>>
i, you
will notice that they are also using the similar convention, id parameter can
be one or more.
Thanks
-min
On 2/6/14 3:24 AM, "Koushik Das"
mailto:koushik@citrix.com>> wrote:
Yes it will be like a findByIds() and the one id case is just a
special case for this.
On
ne. If you look at Amazon EC2 api, you will
> notice that they are also using the similar convention, id parameter can
> be one or more.
>
> Thanks
> -min
>
> On 2/6/14 3:24 AM, "Koushik Das" wrote:
>
>> Yes it will be like a findByIds() and the one
ndan Prateek
> wrote:
>> +1, The listVM call is one of the most resource intensive call. Any step
>> to optimise it are welcome.
>>
>> On 06/02/14 2:01 pm, "Koushik Das" wrote:
>>
>>> Currently list VM can only be called using a single VM I
Currently list VM can only be called using a single VM ID. So if there is a
need to query a set of VMs using ID then either multiple list VM calls need to
be made or all VMs needs to be fetched and then do a client side filtering.
Both approaches are sub-optimal - the former results in multiple
As I understand this is a best-effort approach. Even if the proposed API
returns YES, deployment may still fail. I personally feel that it is better to
improve the deployVM API itself to return accurate error message rather than
having multiple API calls. If the deployvm call call fails, it will
Animesh,
Please cherry-pick the following commit to 4.3
Commit 3fae917ebefb56ba280c2e6c5b6ff0f1720d46ed in branch
refs/heads/4.3-forward from Koushik
Das<https://issues.apache.org/jira/secure/ViewProfile.jspa?name=koushikd>
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3
The only UI related commit I see is 81a83b5b31f0a237ae56449c60350160915f28f8.
List VM API is working fine.
On 27-Jan-2014, at 2:26 PM, Koushik Das wrote:
> I noticed that VM instance view is broken in UI. It works fine in 4.3 branch.
> It worked till
I noticed that VM instance view is broken in UI. It works fine in 4.3 branch.
It worked till commit d31fa09c7e5bb04fcc4594c0352d9b91171e9d1a.
-Koushik
d1a
master -> f999a01837e60f0c51ef0eb4ad19c29e43ca3037
- Koushik Das
On Jan. 23, 2014, 10:27 a.m., Santhosh Edukulla wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
Some questions:
- Is there a concept of generic permission (any action, any resource etc.)?
There shouldn't be a need to define hundreds of explicit permissions for admin
account.
- I think it would be good to have a notion of parent policy. This will avoid
duplication of permissions.
- Can you
eorganize as below
if (currentServiceOffering.isDynamic()) {
if (newServiceOffering.isDynamic()) {
saveCustomOfferingDetails(vmId, newServiceOffering);
} else {
removeCustomOfferingDetails(vmId);
}
}
- Koushik Das
On Jan. 10, 2014, 6:33 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16732/#review31367
---
Ship it!
Ship It!
- Koushik Das
On Jan. 8, 2014, 9:43 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16602/#review31120
---
Ship it!
Ship It!
- Koushik Das
On Jan. 3, 2014, 10:06 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16566/#review31030
---
Patch doesn't apply on master.
- Koushik Das
On Jan. 2, 201
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16566/#review31029
---
Ship it!
Ship It!
- Koushik Das
On Jan. 2, 2014, 10:20 a.m
Yes that is the current behaviour. All hosts in a cluster should be owned by a
single MS. In case when agent load balancing is enabled (agent.lb is set to
true), this constraint no longer holds and the hosts in a cluster may get split
across multiple MS.
If you look at the 'host' table there is
What is the account creation logic on Solidfire SAN if you try to create the
same simultaneously? I am assuming there should be some unique constraint like
account name/id etc. Try to see if this can be utilised so that one of the
account creation call fails with appropriate error/exception.
-K
pache.org/r/16350/
> ---
>
> (Updated Dec. 19, 2013, 10:43 a.m.)
>
>
> Review request for cloudstack, Abhinandan Prateek and Koushik Das.
>
>
> Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-5457
>
> https://issues.apache.org/jira/browse/https://issues.apache.
323
- Koushik Das
On Dec. 19, 2013, 10:43 a.m., Damodar Reddy Talakanti wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apa
For now I have removed cpu, ram and cpu speed fields from vm_instance table.
On 18-Dec-2013, at 6:22 AM, Prachi Damle wrote:
> Koushik,
>
> My comments inline.
>
> Prachi
> -Original Message-----
> From: Koushik Das
> Sent: Tuesday, December 17, 2013 4:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16315/#review30529
---
Ship it!
Ship It!
- Koushik Das
On Dec. 17, 2013, 11:58 a.m
tities for which this has been done so far, Alex has
> put everything under cloud-engine-schema recently where you can find some
> more of such examples.
>
> Prachi
>
> -Original Message-
> From: Koushik Das
> Sent: Sunday, December 15, 2013 9:50 PM
> To:
>
Manager.java
- Koushik Das
On Dec. 17, 2013, 9:38 a.m., bharat kumar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
Prachi, From the history I see that you added VMEntityVO. Any specific reason
for having both?
On 12-Dec-2013, at 2:36 PM, Koushik Das wrote:
> I see that both the VOs uses the same vm_instance table. What is the need for
> having 2 VOs for the same table? Should one be removed?
I see that both the VOs uses the same vm_instance table. What is the need for
having 2 VOs for the same table? Should one be removed?
Also VMEntityVO I see a field called "speed" which is not there in
VMInstanceVO. Why this discrepancy?
Thanks,
Koushik
b54
- Koushik Das
On Dec. 10, 2013, 10:55 a.m., Harikrishna Patnala wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apa
g/r/16129/#comment57658>
In an existing 4.2/master setup I see that the component of both
vmsnapshot.create.wait and vmsnapshot.max is set to none.
Handle this scenario as well.
- Koushik Das
On Dec. 9, 2013, 1:17 p.m., Harikrishna Patnala
I see that for XS the location of script files in the host has changed from
/opt/xensource/bin to /opt/cloud/bin. On latest master branch I see that the
scripts are still getting placed in the old location while adding a host. Is
there any commit missed out on master?
-Koushik
On 10-Dec-2013,
> On Dec. 6, 2013, 4:19 a.m., Koushik Das wrote:
> > Ship It!
master -> 88f64b2b93a3898e7868e9e55c1feca57b2ed48e
- Koushik
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15763/
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15763/#review29870
---
Ship it!
Ship It!
- Koushik Das
On Dec. 5, 2013, 3:28 p.m
evious incorrect one, but I couldn't
find how.
So let me know if it is better to discard this review and create a new one.
Thanks
Alex Ough
On Wed, Dec 4, 2013 at 12:04 AM, Koushik Das
mailto:koushik@citrix.com>> wrote:
-
showing up. Also
there are 2 commits in the patch file. Can you resubmit the patch with only a
single commit?
- Koushik Das
On Dec. 3, 2013, 5:07 p.m., Alex Ough wrote:
>
> ---
> This is an automatically generated e-mail. To rep
> On Dec. 3, 2013, 8:44 a.m., Koushik Das wrote:
> > core/src/com/cloud/agent/api/MigrateCommand.java, line 32
> > <https://reviews.apache.org/r/15763/diff/1/?file=389559#file389559line32>
> >
> > I see that you have tested for XS. In the mail thread people h
I have posted my review comments. Except for a few minor comments changes look
good to me.
-Koushik
On 02-Dec-2013, at 9:15 PM, Alex Ough wrote:
> All,
>
> It's been a while since this review was requested, so can anyone review
> this to move on?
>
> Thanks in advance.
> Alex Ough
>
>
> On
s to be updated in VirtualMachineManager.java
where ExecuteInSequence is defined.
- Koushik Das
On Nov. 21, 2013, 6:39 p.m., Alex Ough wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> http
For using system VMs on local storage there is a global setting
system.vm.use.local.storage which needs to be set to true.
-Koushik
On 29-Nov-2013, at 7:47 PM, Antonio Petrocelli wrote:
> Hello,
>
> we want use cloudstack 4.2 with local storage + VMware 5.1.
>
> https://cloudstack.apache.org
3f8
- Koushik Das
On Nov. 29, 2013, 12:35 p.m., bharat kumar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15832/
> ---
514
- Koushik Das
On Nov. 29, 2013, 7:46 a.m., bharat kumar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15832/
> ---
egarding this. I generally don't think VM
HA can be done with hypervisor HA because of this.
--Alex
-Original Message-
From: Koushik Das [mailto:koushik@citrix.com<http://citrix.com>]
Sent: Tuesday, November 26, 2013 10:51 PM
To: mailto:dev@cloudstack.apache.org>>
Sub
ap cannot be null
server/src/com/cloud/vm/UserVmManagerImpl.java
<https://reviews.apache.org/r/15832/#comment56781>
Put some comment to explain the logic
server/src/com/cloud/vm/UserVmManagerImpl.java
<https://reviews.apache.org/r/15832/#comment56784>
I see the usage event publishin
ew followup comments inline. I look forward
>> to seeing this work.
>>
>> Other folks: please read the entire thread and the links from Koushik;
>> there's a planned deprecation here.
>>
>> --David
>>
>> On Mon, Nov 25, 2013 at 2:38 AM, Koush
> On Nov. 26, 2013, 11:58 a.m., Koushik Das wrote:
> > master -> b2f0a0bce26e342f18da42a51ae19d51c238a762
4.3 -> 8970c9515441c50a0249b9624b120d4e5348b5af
- Koushik
---
This is an automatically generated e-mail. To repl
> On Nov. 26, 2013, 9:12 a.m., Koushik Das wrote:
> > master -> ef40e156ed376968fa9fd15078034d54b97deecc
4.3 -> 6af8021ef7188b8f2d92bc891b5108f0e3503284
- Koushik
---
This is an automatically generated e-mail. To repl
762
- Koushik Das
On Nov. 26, 2013, 11:28 a.m., Harikrishna Patnala wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apa
ecc
- Koushik Das
On Nov. 26, 2013, 8:42 a.m., bharat kumar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15505/
> ---
c and not let it fall into disrepair.
[Koushik] That's the plan going forward. For the next release both options will
be there. Maybe post that the custom HA solution can be removed for XS 6.2 and
above.
>
> --David
>
> On Fri, Nov 22, 2013 at 7:27 AM, Koushik Das wrote
Initial draft of the FS
https://cwiki.apache.org/confluence/display/CLOUDSTACK/User+VM+HA+using+native+XS+HA+capabilities
-Koushik
On 21-Nov-2013, at 9:59 AM, Koushik Das wrote:
> Cloudstack relies on custom HA logic for user VMs running on Xenserver. The
> reason for doing it like th
Cloudstack relies on custom HA logic for user VMs running on Xenserver. The
reason for doing it like this may be due the fact that native HA capabilities
in XS was not mature enough during the initial days. Also in the custom HA
logic, Cloudstack has to correctly determine the state of a VM from
I have reverted the same in 4.3 as well
commit f84b729eb01ffc7cae1e210b58f80ff2033ad785
Author: Koushik Das
Date: Fri Nov 15 12:59:48 2013 +0530
Revert "CLOUDSTACK-5176:"
This reverts commit f29c7188ba03f1997238d50f8a11857e94fe4b55.
On 15-Nov-2013, at 10:35 AM
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15507/#review28947
---
Ship it!
Ship It!
- Koushik Das
On Nov. 14, 2013, 9:25 a.m
tps://reviews.apache.org/r/15507/#comment55923>
Is there any other changes in this file? If not remove it.
framework/config/src/org/apache/cloudstack/framework/config/impl/ConfigDepotImpl.java
<https://reviews.apache.org/r/15507/#comment55924>
this is not required
- Koushik Das
On Nov.
Congrats Sanjeev
On 14-Nov-2013, at 11:29 AM, Prasanna Santhanam wrote:
> The Project Management Committee (PMC) for Apache CloudStack has asked Sanjeev
> Neelarapu to become a committer and we are pleased to announce that they have
> accepted.
>
> Being a committer allows many contributors to
();
On 12-Nov-2013, at 11:35 AM, Koushik Das wrote:
> I see the JCE extensions in jdk 1.7 as well. They are present under
> /jre/lib/security. But still I see a test failure. Is there any
> other configuration that is required?
>
> Running org.apache.cloudstack.network.lb.CertServ
I see the JCE extensions in jdk 1.7 as well. They are present under
/jre/lib/security. But still I see a test failure. Is there any
other configuration that is required?
Running org.apache.cloudstack.network.lb.CertServiceTest
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.456
4.3 -> 6a8ec786ad2a843768c26099e17f89782cd77f75
- Koushik Das
On Nov. 8, 2013, 12:24 p.m., Harikrishna Patnala wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15348/#review28538
---
Ship it!
Ship It!
- Koushik Das
On Nov. 8, 2013, 12:02 p.m
>From what I understand, now there is a new interface Configurable that exposes
>the config parameters. In this case if you look at the
>CapacityManagerImpl.java, there is a method getCongigKeys() that return the
>config parameters. During MS startup all these are automatically populated in
>th
Is your data volume and VM on the same host? If not then the volume cannot be
attached to the VM in case of a local volume.
-Koushik
On 07-Nov-2013, at 4:03 PM, Antonio Petrocelli
wrote:
> Hello,
>
> i've this error when i try to attach a volume to vm:
>
> "Failed to attach local data volu
> On Nov. 7, 2013, 7:17 a.m., Koushik Das wrote:
> > Ship It!
commit 7095ea2b5e497620c8c879517565046e3b46e493
- Koushik
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15173/#re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15173/#review28355
---
Ship it!
Ship It!
- Koushik Das
On Nov. 7, 2013, 6:24 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15018/#review28183
---
Ship it!
Ship It!
- Koushik Das
On Nov. 5, 2013, 10:34 a.m
setup/db/db/schema-421to430.sql
<https://reviews.apache.org/r/15018/#comment54681>
Simply drop and recreate the view
- Koushik Das
On Oct. 31, 2013, 10:14 a.m., Harikrishna Patnala wrote:
>
> ---
> This is an aut
chema/src/com/cloud/service/ServiceOfferingVO.java
<https://reviews.apache.org/r/15173/#comment54675>
Again this is not required
engine/schema/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java
<https://reviews.apache.org/r/15173/#comment54674>
ordering of parameter is n
7d3f765
Diff: https://reviews.apache.org/r/15080/diff/
Testing
---
Verified by tweaking the per agent upper cap to a value of 1 and checked that
the requests are getting scheduled but the executor thread simply bails out.
Thanks,
Koushik Das
ink when commands originate from the
> management server, it doesn't go through this code. This is if the command
> came from the agent which doesn't matter if there's multiple management
> servers.
>
> --Alex
>
>> -----Original Message-
>>
Created https://issues.apache.org/jira/browse/CLOUDSTACK-4944 to track this
issue.
Alex, Any reason for adding requests based on sequence and not doing FIFO? Do
you see any issues if request always gets added to the end of the queue?
On 23-Oct-2013, at 6:26 PM, Koushik Das wrote:
> I
701 - 800 of 926 matches
Mail list logo