Tomas Jelinek has abandoned this change.
Change subject: migration: enhance legacy downtime algorithm
..
Abandoned
--
To view, visit https://gerrit.ovirt.org/56561
To unsubscribe, visit https://gerrit.ovirt.org/settings
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/61033
To unsubscribe, visit
Hello Francesco Romani,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/61033
to review the following change.
Change subject: migration: wait properly for migration to begin
..
migration: wait
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 9: Verified+1
done
--
To view, visit https://gerrit.ovirt.org/60073
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 7:
(2 comments)
https://gerrit.ovirt.org/#/c/60073/7//COMMIT_MSG
Commit Message:
Line 17: -
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 7: Verified+1
--
To view, visit https://gerrit.ovirt.org/60073
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/60073/3/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/60073/3/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait properly for migration to begin
..
Patch Set 3:
(4 comments)
https://gerrit.ovirt.org/#/c/60073/2//COMMIT_MSG
Commit Message:
PS2, Line 13:
Tomas Jelinek has uploaded a new change for review.
Change subject: migration: wait properly for migration to begin
..
migration: wait properly for migration to begin
If the time it takes the migration to begin is longer than
Tomas Jelinek has posted comments on this change.
Change subject: client: use representation object
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/59827
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: downtime: start thread after 1st iter
..
Patch Set 8: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/59220
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: track qemu iterations
..
Patch Set 8: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/59219
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: downtime: handle steps=1
..
Patch Set 6: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/59218
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: early exit when stopped
..
Patch Set 10: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/57935
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: make MonitorThread own DowntimeThread
..
Patch Set 10: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/57933
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: wait for monitor thread before to exit
..
Patch Set 5: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/59217
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: downtime: update step delay computation
..
Patch Set 4: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/59221
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: log the convergence schedule only if provided
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/57370
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migrations: enhance legacy downtime alg
..
Patch Set 6: Verified+1
--
To view, visit https://gerrit.ovirt.org/56561
To unsubscribe, visit
Tomas Jelinek has uploaded a new change for review.
Change subject: migration: log the convergence schedule only if provided
..
migration: log the convergence schedule only if provided
No need to flood the logs with logs which
Tomas Jelinek has posted comments on this change.
Change subject: migrations: enhance legacy downtime alg
..
Patch Set 5:
(6 comments)
https://gerrit.ovirt.org/#/c/56561/3//COMMIT_MSG
Commit Message:
PS3, Line 12: during the
Tomas Jelinek has posted comments on this change.
Change subject: migrations: change convergence schedule from time to iterations
..
Patch Set 4:
@Milan: About small VMs:
1: we don't really have convergence issues with small
Tomas Jelinek has posted comments on this change.
Change subject: migrations: change convergence schedule from time to iterations
..
Patch Set 4:
(2 comments)
https://gerrit.ovirt.org/#/c/56558/3//COMMIT_MSG
Commit Message:
Tomas Jelinek has posted comments on this change.
Change subject: migrations: change convergence schedule from time to iterations
..
Patch Set 3:
(2 comments)
https://gerrit.ovirt.org/#/c/56558/3//COMMIT_MSG
Commit Message:
Tomas Jelinek has posted comments on this change.
Change subject: migrations: enhance legacy downtime alg
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/56561
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migrations: change convergence schedule from time to iterations
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/56558
To unsubscribe,
Tomas Jelinek has abandoned this change.
Change subject: migration: enhanced the legacy downtime alg
..
Abandoned
experimental code - melded into different patches
--
To view, visit https://gerrit.ovirt.org/56502
To
Tomas Jelinek has abandoned this change.
Change subject: Changed the convergence from num of seconds to num of iterations
..
Abandoned
experimental code - melded into different patches
--
To view, visit
Tomas Jelinek has posted comments on this change.
Change subject: migrations: enhance legacy downtime alg
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/56561
To unsubscribe, visit
Tomas Jelinek has uploaded a new change for review.
Change subject: migrations: enhance legacy downtime alg
..
migrations: enhance legacy downtime alg
This patch contains the following enhancements of the legacy downtime
Tomas Jelinek has posted comments on this change.
Change subject: migrations: change convergence schedule from time to iterations
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/56558
To unsubscribe,
Tomas Jelinek has uploaded a new change for review.
Change subject: migrations: change convergence schedule from time to iterations
..
migrations: change convergence schedule from time to iterations
Currently the convergence
Tomas Jelinek has uploaded a new change for review.
Change subject: Changed the convergence from num of seconds to num of iterations
..
Changed the convergence from num of seconds to num of iterations
Change-Id:
Tomas Jelinek has posted comments on this change.
Change subject: migration: added logging of more detailed stats
..
Patch Set 9: Verified+1
(1 comment)
https://gerrit.ovirt.org/#/c/55976/1/vdsm/virt/migration.py
File
Tomas Jelinek has uploaded a new change for review.
Change subject: migration: added logging of more detailed stats
..
migration: added logging of more detailed stats
The VIR_DOMAIN_JOB_MEMORY_BPS,
Tomas Jelinek has posted comments on this change.
Change subject: spec: bump libguestfs-tools-c requirement again
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55749
To unsubscribe, visit
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/55786
To unsubscribe, visit
Hello Vinzenz Feenstra, Francesco Romani,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/55786
to review the following change.
Change subject: migrations: change migration parameters
..
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
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:
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
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
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 21: Verified+1
--
To view, visit https://gerrit.ovirt.org/46940
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/46940/19/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 19:
(4 comments)
https://gerrit.ovirt.org/#/c/46940/19/lib/api/vdsmapi-schema.json
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 19: Verified+1
--
To view, visit https://gerrit.ovirt.org/46940
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 19:
ahh, sry, fixed
--
To view, visit https://gerrit.ovirt.org/46940
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 18: Verified+1
--
To view, visit https://gerrit.ovirt.org/46940
To unsubscribe, visit
Tomas Jelinek has abandoned this change.
Change subject: migration: added support for setting the convergenceSchedule
..
Abandoned
the code has been merged to the migration: added support for convergance
schedule
--
To
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for setting the convergenceSchedule
..
Patch Set 15:
(1 comment)
https://gerrit.ovirt.org/#/c/47734/15/lib/api/vdsmapi-schema.json
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for setting the convergenceSchedule
..
Patch Set 15:
(6 comments)
https://gerrit.ovirt.org/#/c/47734/15/lib/api/vdsmapi-schema.json
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 14:
(1 comment)
https://gerrit.ovirt.org/#/c/46940/14/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 14: Verified+1
--
To view, visit https://gerrit.ovirt.org/46940
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for setting the convergenceSchedule
..
Patch Set 13:
(1 comment)
https://gerrit.ovirt.org/#/c/47734/13/lib/api/vdsmapi-schema.json
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for setting the convergenceSchedule
..
Patch Set 14: Verified+1
--
To view, visit https://gerrit.ovirt.org/47734
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for setting the convergenceSchedule
..
Patch Set 13:
(1 comment)
https://gerrit.ovirt.org/#/c/47734/13/lib/api/vdsmapi-schema.json
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergence schedule
..
Patch Set 14:
@Francesco: comparing to the previous version you have acked I have done one
small change -
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 13: Verified+1
only rebased and re-verified
--
To view, visit https://gerrit.ovirt.org/46940
To
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 11:
(1 comment)
https://gerrit.ovirt.org/#/c/46940/11/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 11:
hmm, not sure how to split this up into patches which bring value by
themselves. What I can
Tomas Jelinek has posted comments on this change.
Change subject: migration: Added support for setting the convergenceSchedule
..
Patch Set 8: Verified+1
--
To view, visit https://gerrit.ovirt.org/47734
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 9: Verified+1
--
To view, visit https://gerrit.ovirt.org/46940
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 10: Verified+1
--
To view, visit https://gerrit.ovirt.org/46934
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 11: Verified+1
--
To view, visit https://gerrit.ovirt.org/46846
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 8:
(2 comments)
https://gerrit.ovirt.org/#/c/46940/8/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 8:
- added the support for the init part
- fixed some issues - not tested too deeply but seems to
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 7: Verified-1
-1 for now since the "abort" part does not work properly since it is called
twice.
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 6: Code-Review-1
there is a missing part of how to setup the initial downtime. The initial
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/46940/4/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 8:
(2 comments)
https://gerrit.ovirt.org/#/c/46846/8/vdsm/rpc/vdsmapi-schema.json
File
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/7/vdsm/rpc/vdsmapi-schema.json
File
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/7/vdsm/virt/migration.py
File vdsm/virt/migration.py:
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/46940/4/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 4:
(10 comments)
https://gerrit.ovirt.org/#/c/46940/4/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/5/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1557:
Tomas Jelinek has posted comments on this change.
Change subject: migration: added support for convergance schedule
..
Patch Set 1:
(6 comments)
https://gerrit.ovirt.org/#/c/46940/1/vdsm/virt/migration.py
File
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1546:
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/46846/5/vdsm/rpc/vdsmapi-schema.json
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 7: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/46846
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 7: -Code-Review Verified+1
--
To view, visit https://gerrit.ovirt.org/46846
To
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/46934
To unsubscribe, visit
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/46934/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1546:
Tomas Jelinek has posted comments on this change.
Change subject: migration added migrateChangeParams verb
..
Patch Set 2:
(2 comments)
https://gerrit.ovirt.org/#/c/46934/2/vdsm/rpc/vdsmapi-schema.json
File
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 3:
done
--
To view, visit https://gerrit.ovirt.org/46846
To unsubscribe, visit
Tomas Jelinek has uploaded a new change for review.
Change subject: migration added migrateChangeParams verb
..
migration added migrateChangeParams verb
Currently it contains only the bandwidth but it will be used also for
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add support for max migration bandwidth setting
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/46846/2/vdsm/virt/migration.py
File
Tomas Jelinek has uploaded a new change for review.
Change subject: migration: Add support for max migration bandwidth setting
..
migration: Add support for max migration bandwidth setting
Enriched the migrate verb by
Tomas Jelinek has posted comments on this change.
Change subject: migration: Add incoming migration semaphore
..
Patch Set 2:
(3 comments)
please set this patch a topic like "migration" - I guess much more will be
coming so
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 7:
(2 comments)
https://gerrit.ovirt.org/#/c/39796/7/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 8: Verified+1
--
To view, visit https://gerrit.ovirt.org/39796
To unsubscribe, visit https://gerrit.ovirt.org/settings
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 7:
so created a simpler version of the patch which just takes the value from spec
params and passes it to libvirt. If
Tomas Jelinek has uploaded a new change for review.
Change subject: Added support for iothreads
..
Added support for iothreads
Added a new parameter 'numOfIoThreads' to VM and
a new parameter 'pinToIoThread' to Drive's spec
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/39796/4/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 831:
Line
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 2:
(4 comments)
https://gerrit.ovirt.org/#/c/39796/2/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 2:
(4 comments)
https://gerrit.ovirt.org/#/c/39796/2/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:
Tomas Jelinek has posted comments on this change.
Change subject: Added support for iothreads
..
Patch Set 2:
(2 comments)
https://gerrit.ovirt.org/#/c/39796/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 827: # the
Tomas Jelinek has posted comments on this change.
Change subject: vm: fix odd migration progress reporting
..
Patch Set 2: Verified+1
for me this fix works well
--
To view, visit http://gerrit.ovirt.org/27465
To unsubscribe,
Tomas Jelinek has posted comments on this change.
Change subject: Integrate Smartcard support
..
Patch Set 1: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/10547
To unsubscribe,
Tomas Jelinek has posted comments on this change.
Change subject: Integrate Smartcard support
..
Patch Set 21:
thank you Toni
--
To view, visit http://gerrit.ovirt.org/8450
To unsubscribe, visit
1 - 100 of 106 matches
Mail list logo