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

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://g

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/tun

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.o

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 du

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 t

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.o

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

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 bandwid

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 Pr

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 T

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 s

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_mi

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.o

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 Pr

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

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.o

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 migratio

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 Pr