Change in vdsm[master]: remove unused fields in vm stats

2016-09-30 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: remove unused fields in vm stats .. Patch Set 1: -Verified indeed, I didn't take MOM into account. Martin, I guess MOM needs vmName only for logging, right? and

Change in vdsm[master]: remove unused fields in vm stats

2016-09-30 Thread msivak
Martin Sivák has posted comments on this change. Change subject: remove unused fields in vm stats .. Patch Set 1: Code-Review-1 MOM uses vmName and pid. -- To view, visit https://gerrit.ovirt.org/64973 To unsubscribe, visit

Change in vdsm[master]: remove unused fields in vm stats

2016-09-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: remove unused fields in vm stats .. Patch Set 1: Code-Review-1 not sure MOM uses this. -1 for visibility Vdsm tries hard to be backward compatible, so let's

Change in vdsm[master]: remove unused fields in vm stats

2016-09-29 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: remove unused fields in vm stats .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/64973 To unsubscribe, visit

Change in vdsm[master]: remove unused fields in vm stats

2016-09-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: remove unused fields in vm stats .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: remove unused fields in vm stats

2016-09-29 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: remove unused fields in vm stats .. remove unused fields in vm stats The values of 'vmName' and 'vmType' are not read by the engine, thus removed. The value of