On Sun, Nov 27, 2016 at 12:31 PM, Eyal Edri <[email protected]> wrote: > Not sure I understand what do you mean by Gerrit Headers. > Can you give examples?
Here the point when the header disappeared: commit 82bebee084cb9841a42cc9131a724362771a2c80 Author: Petr Horáček <[email protected]> Date: Wed Nov 16 15:28:01 2016 +0100 net: enable link.bond to handle options Note that this change also requires change in slaves editation (in _revert_transaction) since we cannot edit options of a bonding with attached slaves. Change-Id: I4399432347dc00f11d6bea28d731dd6e37914c20 Signed-off-by: Petr Horáček <[email protected]> Bug-Url: https://bugzilla.redhat.com/1379115 commit 505f5da01785ece0e238a7fa3125358b67595d69 Author: Maor Lipchuk <[email protected]> Date: Wed Nov 9 16:01:04 2016 +0200 API: Introduce getQemuImageInfo API. The new API suppose to return information fetched using qemuimg info. This should be usable to get the compat version of the volume. This API will not use a job since it only returns information about the volume and do not change it. It is similar to how getVolumeInfo is working. Change-Id: Ic170e2f142e8e70b534083c9468d249fe2478943 Signed-off-by: Maor Lipchuk <[email protected]> Reviewed-on: https://gerrit.ovirt.org/66295 Reviewed-by: Nir Soffer <[email protected]> Continuous-Integration: Jenkins CI Note that the older commit had: - Reviewed-on: - Reviewed-by: - Continuous-Integration: > > On Fri, Nov 25, 2016 at 4:57 PM, Nir Soffer <[email protected]> wrote: >> >> On Fri, Nov 25, 2016 at 4:45 PM, Tomáš Golembiovský <[email protected]> >> wrote: >> > Hi, >> > >> > I've noticed that in vdsm repo the merged commits do not contain the >> > info headers added by Gerrit any more (Reviewed-by/Reviewed-on/etc.). >> > >> > Is that intentional? If yes, what was the motivation behind this? >> > >> > The change seem to have happened about 4 days ago. Sometime between the >> > following two commits: >> > >> > * 505f5da API: Introduce getQemuImageInfo API. [Maor Lipchuk] >> > * 1c4a39c protocoldetector: Avoid unneeded getpeername() [Nir Soffer] >> >> We switched vdsm to fast-forward 4 days ago, maybe this was unintended >> side effect of this change? >> >> The gerrit headers are very useful, please add back. >> >> Nir >> _______________________________________________ >> Infra mailing list >> [email protected] >> http://lists.ovirt.org/mailman/listinfo/infra >> >> > > > > -- > Eyal Edri > Associate Manager > RHV DevOps > EMEA ENG Virtualization R&D > Red Hat Israel > > phone: +972-9-7692018 > irc: eedri (on #tlv #rhev-dev #rhev-integ) _______________________________________________ Devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/devel
