Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread danken 
Dan Kenigsberg has submitted this change and it was merged.

Change subject: migrations: change migration parameters
..


migrations: change migration parameters

Changed the migration params to:
max_outgoing/incoming_migrations=2
migration_max_bandwidth=52
migration_progress_timeout=240
migration_downtime_delay=20
migration_downtime_steps=5

The reason to change migration_max_bandwidth and
max_outgoing/incoming_migrations is to have less migrations running in parallel
faster which helps convergence.

The reason to change the migration_progress_timeout is to leave more time for
big VMs to converge.

The reason to change migration_downtime_delay is to make the downtime thread
start applying the changes faster which is important for VMs which would start
stalling.

The reason to change the migration_downtime_steps is to make the growing of the
downtime more fast.

Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Bug-Url: https://bugzilla.redhat.com/1323952
Signed-off-by: Tomas Jelinek 
Reviewed-on: https://gerrit.ovirt.org/55676
Reviewed-by: Vinzenz Feenstra 
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani 
---
M lib/vdsm/config.py.in
1 file changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Tomas Jelinek: Verified
  Jenkins CI: Passed CI tests
  Vinzenz Feenstra: Looks good to me, but someone else must approve
  Francesco Romani: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread automation 
gerrit-hooks has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 4:

* #1323952::Update tracker: OK
* Set MODIFIED::bug 1323952#1323952OK

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread fromani 
Francesco Romani has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3:

yes, makes sense. The "issue" is just we have a specific workload/vm size in 
mind when do testing/tuning. But this isn't really an issue, rather just a 
fact. We can't really solve this without migration_enhancements patchset.

These defaults have a rationale and some documentation, so it is a step forward.

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik 
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread tjelinek 
Tomas Jelinek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik 
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread michal . skrivanek 
Michal Skrivanek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3:

@francesco: I tried to describe that in previous comment. (and i had a longer 
version which i lost due to stupid gerrit auto-logout, damn!)
for completeness - currently 3*32MB=768Mbps max, we're increasing it to 
832Mbps, which should still be ok. Rest is based on practical 
measurementsthey are never complete of course, but that's the best we got:)

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik 
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread fromani 
Francesco Romani has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3: Code-Review+2

I trust you about it, and the (concise) rationale makes sense to me, so +2 
again. But to improve my knowledge even further,
could you just share the formulas/reasining behind the new settings?

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik 
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread vfeenstr 
Vinzenz Feenstra has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik 
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread michal . skrivanek 
Michal Skrivanek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3:

well, to make it clear - the values are intentionally conservative as I would 
want to do this change in 3.6.z.
4.0 migration enhancements are a different thing

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik 
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: Vinzenz Feenstra 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread michal . skrivanek 
Michal Skrivanek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 2:

we're testing that a bit more with mpolednik. so far the best values seem to be:
concurrency 2
bandwidth 52
stalling 240
downtime_delay 20
downtime steps 5

why? 
1Gb network, with 2 migrations should occupy 832Mbps, practically the line can 
handle ~880Mbps, so we have a 50Mbps buffer even in the worst case
stalling - 150 was very often not enough. rasising from 2.5 minute to 4 might 
significantly help to overcome peaks in memory usage, but still conservative 
enough.
downtime - 10 steps with 75s per GB is too much. theoretically you need less 
than 20s at 52MB/s on 1Gb line to transfer everything. In reality there are 
zero pages costing nothing, and dirty memory on the other hand. So roughly 20s 
to get through one full pass is enough, then it makes sense to tweak downtime. 
The algorithm is not ideal either, so 10->5 means it's going to increase sooner

what do you think?

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread automation 
gerrit-hooks has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 3:

* #1323952::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1323952::OK, public bug
* Check Product::#1323952::OK, Correct classification oVirt
* Check TM::SKIP, not in a monitored branch (ovirt-3.6 ovirt-3.5 ovirt-3.4 
ovirt-3.3 ovirt-3.2)
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-06 Thread fromani 
Francesco Romani has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 2: Code-Review+1

lowering score to not hide Michal's

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread michal . skrivanek 
Michal Skrivanek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 2: Code-Review-1

Didn't we talk about 54?;) it's an increase from 768Mbps to  864Mbps, but that 
should still be ok

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek 
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread tjelinek 
Tomas Jelinek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 2:

(3 comments)

https://gerrit.ovirt.org/#/c/55676/1//COMMIT_MSG
Commit Message:

PS1, Line 10: incoming_migrations=
> incoming
Done


PS1, Line 13: in parall
> in parallel? I'm not a native english speaker, I may be wrong
Done


PS1, Line 14: convergence
> typo: convergence
Done


-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread fromani 
Francesco Romani has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 2: Code-Review+2

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread automation 
gerrit-hooks has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 2:

* #1323952::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1323952::OK, public bug
* Check Product::#1323952::OK, Correct classification oVirt
* Check TM::SKIP, not in a monitored branch (ovirt-3.6 ovirt-3.5 ovirt-3.4 
ovirt-3.3 ovirt-3.2)
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread fromani 
Francesco Romani has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 1: Code-Review+1

(3 comments)

besides typos in the commit message, looks good to me, hence partial ACK

https://gerrit.ovirt.org/#/c/55676/1//COMMIT_MSG
Commit Message:

PS1, Line 10: incomming_migrations
incoming


PS1, Line 13: parallely
in parallel? I'm not a native english speaker, I may be wrong


PS1, Line 14: convergance
typo: convergence


-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread tjelinek 
Tomas Jelinek has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 1: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: Francesco Romani 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread tjelinek 
Tomas Jelinek has uploaded a new change for review.

Change subject: migrations: change migration parameters
..

migrations: change migration parameters

Changed the migration params to:
max_outgoing/incomming_migrations=2
migration_max_bandwidth=45

The reason is to have less migrations running parallely faster which helps
convergance.

Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Bug-Url: https://bugzilla.redhat.com/1323952
Signed-off-by: Tomas Jelinek 
---
M lib/vdsm/config.py.in
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/76/55676/1

diff --git a/lib/vdsm/config.py.in b/lib/vdsm/config.py.in
index 96b0351..7d36745 100644
--- a/lib/vdsm/config.py.in
+++ b/lib/vdsm/config.py.in
@@ -79,7 +79,7 @@
 'Time to wait (in seconds) for migration destination to start '
 'listening before migration begins.'),
 
-('migration_max_bandwidth', '32',
+('migration_max_bandwidth', '45',
 'Maximum bandwidth for migration, in MiBps, 0 means libvirt\'s '
 'default, since 0.10.x default in libvirt is unlimited'),
 
@@ -119,10 +119,10 @@
 ('migration_downtime_steps', '10',
 'Incremental steps used to reach migration_downtime.'),
 
-('max_outgoing_migrations', '3',
+('max_outgoing_migrations', '2',
 'Maximum concurrent outgoing migrations'),
 
-('max_incoming_migrations', '3',
+('max_incoming_migrations', '2',
 'Maximum concurrent incoming migrations'),
 
 ('sys_shutdown_timeout', '120',


-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Change in vdsm[master]: migrations: change migration parameters

2016-04-05 Thread automation 
gerrit-hooks has posted comments on this change.

Change subject: migrations: change migration parameters
..


Patch Set 1:

* #1323952::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1323952::OK, public bug
* Check Product::#1323952::OK, Correct classification oVirt
* Check TM::SKIP, not in a monitored branch (ovirt-3.6 ovirt-3.5 ovirt-3.4 
ovirt-3.3 ovirt-3.2)
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/55676
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ed1b04ee77a7144ae0206e5c5ae1426ede0c75f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches