From xiao-lei.shi at hp.com Wed May 7 22:31:57 2014 Content-Type: multipart/mixed; boundary="===============4563490757436413309==" MIME-Version: 1.0 From: Shi, Xiao-Lei (Bruce, HP Servers-PSC-CQ) To: devel at ovirt.org Subject: [ovirt-devel] Vdsm patches need review and merge Date: Thu, 08 May 2014 02:30:55 +0000 Message-ID: <08AA403C8399104A89AF710307FA78AE24404869@G5W2743.americas.hpqcorp.net> --===============4563490757436413309== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable --_000_08AA403C8399104A89AF710307FA78AE24404869G5W2743americas_ Content-Type: text/plain; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable Hi Martin, Could you help to review these numa vdsm patches: http://gerrit.ovirt.org/#/c/27399/ http://gerrit.ovirt.org/#/c/27090/ http://gerrit.ovirt.org/#/c/27403/ http://gerrit.ovirt.org/#/c/26876/ You had already reviewed some of the patches, but after that I modified the= =3D transferred data format because of the xmlrpc limitations. So could you pl= =3D ease review them again? Hi Dan, I have modified your comments for some of the above patches. Could you plea= =3D se take a look and merge those if no more comments? Thanks & Best Regards Shi, Xiao-Lei (Bruce) Hewlett-Packard Co., Ltd. HP Servers Core Platform Software China Telephone +86 23 65683093 Mobile +86 18696583447 Email xiao-lei.shi(a)hp.com --_000_08AA403C8399104A89AF710307FA78AE24404869G5W2743americas_ Content-Type: text/html; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable

Hi Martin,

Could you help to review these numa vdsm patches:<= o:=3D p>

h= ttp:=3D //gerrit.ovirt.org/#/c/27399/

h= ttp:=3D //gerrit.ovirt.org/#/c/27090/

h= ttp:=3D //gerrit.ovirt.org/#/c/27403/

h= ttp:=3D //gerrit.ovirt.org/#/c/26876/

You had already reviewed some of the patches, but = af=3D ter that I modified the transferred data format because of the xmlrpc limit= =3D ations. So could you please review them again?

Hi Dan,

I have modified your comments for some of the abov= e =3D patches. Could you please take a look and merge those if no more comments?<= =3D o:p>

Thanks & Best Regards

Shi, Xiao-Lei (Bruce)

Hewlett-Packard Co., Ltd.
HP Servers Core Platform Software China

Telephone += 86=3D 23 65683093

Mobile +86 18696583447<=3D /span>

Email xiao-lei.shi(a)hp.com<= /p>

--_000_08AA403C8399104A89AF710307FA78AE24404869G5W2743americas_-- --===============4563490757436413309== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" LS1fMDAwXzA4QUE0MDNDODM5OTEwNEE4OUFGNzEwMzA3RkE3OEFFMjQ0MDQ4NjlHNVcyNzQzYW1l cmljYXNfCkNvbnRlbnQtVHlwZTogdGV4dC9wbGFpbjsgY2hhcnNldD0idXMtYXNjaWkiCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKCkhpIE1hcnRpbiwKQ291bGQg eW91IGhlbHAgdG8gcmV2aWV3IHRoZXNlIG51bWEgdmRzbSBwYXRjaGVzOgpodHRwOi8vZ2Vycml0 Lm92aXJ0Lm9yZy8jL2MvMjczOTkvCmh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzA5MC8K aHR0cDovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3NDAzLwpodHRwOi8vZ2Vycml0Lm92aXJ0Lm9y Zy8jL2MvMjY4NzYvCllvdSBoYWQgYWxyZWFkeSByZXZpZXdlZCBzb21lIG9mIHRoZSBwYXRjaGVz LCBidXQgYWZ0ZXIgdGhhdCBJIG1vZGlmaWVkIHRoZT0KIHRyYW5zZmVycmVkIGRhdGEgZm9ybWF0 IGJlY2F1c2Ugb2YgdGhlIHhtbHJwYyBsaW1pdGF0aW9ucy4gU28gY291bGQgeW91IHBsPQplYXNl IHJldmlldyB0aGVtIGFnYWluPwoKSGkgRGFuLApJIGhhdmUgbW9kaWZpZWQgeW91ciBjb21tZW50 cyBmb3Igc29tZSBvZiB0aGUgYWJvdmUgcGF0Y2hlcy4gQ291bGQgeW91IHBsZWE9CnNlIHRha2Ug YSBsb29rIGFuZCBtZXJnZSB0aG9zZSBpZiBubyBtb3JlIGNvbW1lbnRzPwoKVGhhbmtzICYgQmVz dCBSZWdhcmRzClNoaSwgWGlhby1MZWkgKEJydWNlKQoKSGV3bGV0dC1QYWNrYXJkIENvLiwgTHRk LgpIUCBTZXJ2ZXJzIENvcmUgUGxhdGZvcm0gU29mdHdhcmUgQ2hpbmEKVGVsZXBob25lICs4NiAy MyA2NTY4MzA5MwpNb2JpbGUgKzg2IDE4Njk2NTgzNDQ3CkVtYWlsIHhpYW8tbGVpLnNoaUBocC5j b208bWFpbHRvOnNoaWd1by5saUBocC5jb20+CgoKLS1fMDAwXzA4QUE0MDNDODM5OTEwNEE4OUFG NzEwMzA3RkE3OEFFMjQ0MDQ4NjlHNVcyNzQzYW1lcmljYXNfCkNvbnRlbnQtVHlwZTogdGV4dC9o dG1sOyBjaGFyc2V0PSJ1cy1hc2NpaSIKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogcXVvdGVk LXByaW50YWJsZQoKPGh0bWwgeG1sbnM6dj0zRCJ1cm46c2NoZW1hcy1taWNyb3NvZnQtY29tOnZt bCIgeG1sbnM6bz0zRCJ1cm46c2NoZW1hcy1taWNyPQpvc29mdC1jb206b2ZmaWNlOm9mZmljZSIg eG1sbnM6dz0zRCJ1cm46c2NoZW1hcy1taWNyb3NvZnQtY29tOm9mZmljZTp3b3JkIiA9CnhtbG5z Om09M0QiaHR0cDovL3NjaGVtYXMubWljcm9zb2Z0LmNvbS9vZmZpY2UvMjAwNC8xMi9vbW1sIiB4 bWxucz0zRCJodHRwOj0KLy93d3cudzMub3JnL1RSL1JFQy1odG1sNDAiPgo8aGVhZD4KPG1ldGEg aHR0cC1lcXVpdj0zRCJDb250ZW50LVR5cGUiIGNvbnRlbnQ9M0QidGV4dC9odG1sOyBjaGFyc2V0 PTNEdXMtYXNjaWkiPQo+CjxtZXRhIG5hbWU9M0QiR2VuZXJhdG9yIiBjb250ZW50PTNEIk1pY3Jv c29mdCBXb3JkIDE0IChmaWx0ZXJlZCBtZWRpdW0pIj4KPHN0eWxlPjwhLS0KLyogRm9udCBEZWZp bml0aW9ucyAqLwpAZm9udC1mYWNlCgl7Zm9udC1mYW1pbHk6U2ltU3VuOwoJcGFub3NlLTE6MiAx IDYgMCAzIDEgMSAxIDEgMTt9CkBmb250LWZhY2UKCXtmb250LWZhbWlseTpTaW1TdW47CglwYW5v c2UtMToyIDEgNiAwIDMgMSAxIDEgMSAxO30KQGZvbnQtZmFjZQoJe2ZvbnQtZmFtaWx5OkNhbGli cmk7CglwYW5vc2UtMToyIDE1IDUgMiAyIDIgNCAzIDIgNDt9CkBmb250LWZhY2UKCXtmb250LWZh bWlseTpTaW1TdW47CglwYW5vc2UtMToyIDEgNiAwIDMgMSAxIDEgMSAxO30KLyogU3R5bGUgRGVm aW5pdGlvbnMgKi8KcC5Nc29Ob3JtYWwsIGxpLk1zb05vcm1hbCwgZGl2Lk1zb05vcm1hbAoJe21h cmdpbjowaW47CgltYXJnaW4tYm90dG9tOi4wMDAxcHQ7Cglmb250LXNpemU6MTEuMHB0OwoJZm9u dC1mYW1pbHk6IkNhbGlicmkiLCJzYW5zLXNlcmlmIjt9CmE6bGluaywgc3Bhbi5Nc29IeXBlcmxp bmsKCXttc28tc3R5bGUtcHJpb3JpdHk6OTk7Cgljb2xvcjpibHVlOwoJdGV4dC1kZWNvcmF0aW9u OnVuZGVybGluZTt9CmE6dmlzaXRlZCwgc3Bhbi5Nc29IeXBlcmxpbmtGb2xsb3dlZAoJe21zby1z dHlsZS1wcmlvcml0eTo5OTsKCWNvbG9yOnB1cnBsZTsKCXRleHQtZGVjb3JhdGlvbjp1bmRlcmxp bmU7fQpzcGFuLkVtYWlsU3R5bGUxNwoJe21zby1zdHlsZS10eXBlOnBlcnNvbmFsLWNvbXBvc2U7 Cglmb250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiOwoJY29sb3I6d2luZG93dGV4dDt9 Ci5Nc29DaHBEZWZhdWx0Cgl7bXNvLXN0eWxlLXR5cGU6ZXhwb3J0LW9ubHk7Cglmb250LWZhbWls eToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiO30KQHBhZ2UgV29yZFNlY3Rpb24xCgl7c2l6ZTo4LjVp biAxMS4waW47CgltYXJnaW46MS4waW4gMS4yNWluIDEuMGluIDEuMjVpbjt9CmRpdi5Xb3JkU2Vj dGlvbjEKCXtwYWdlOldvcmRTZWN0aW9uMTt9Ci0tPjwvc3R5bGU+PCEtLVtpZiBndGUgbXNvIDld Pjx4bWw+CjxvOnNoYXBlZGVmYXVsdHMgdjpleHQ9M0QiZWRpdCIgc3BpZG1heD0zRCIxMDI2IiAv Pgo8L3htbD48IVtlbmRpZl0tLT48IS0tW2lmIGd0ZSBtc28gOV0+PHhtbD4KPG86c2hhcGVsYXlv dXQgdjpleHQ9M0QiZWRpdCI+CjxvOmlkbWFwIHY6ZXh0PTNEImVkaXQiIGRhdGE9M0QiMSIgLz4K PC9vOnNoYXBlbGF5b3V0PjwveG1sPjwhW2VuZGlmXS0tPgo8L2hlYWQ+Cjxib2R5IGxhbmc9M0Qi RU4tVVMiIGxpbms9M0QiYmx1ZSIgdmxpbms9M0QicHVycGxlIj4KPGRpdiBjbGFzcz0zRCJXb3Jk U2VjdGlvbjEiPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPkhpIE1hcnRpbiw8bzpwPjwvbzpwPjwv cD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj5Db3VsZCB5b3UgaGVscCB0byByZXZpZXcgdGhlc2Ug bnVtYSB2ZHNtIHBhdGNoZXM6PG86PQpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFs Ij48YSBocmVmPTNEImh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzM5OS8iPmh0dHA6PQov L2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3Mzk5LzwvYT48bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9 M0QiTXNvTm9ybWFsIj48YSBocmVmPTNEImh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzA5 MC8iPmh0dHA6PQovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3MDkwLzwvYT48bzpwPjwvbzpwPjwv cD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48YSBocmVmPTNEImh0dHA6Ly9nZXJyaXQub3ZpcnQu b3JnLyMvYy8yNzQwMy8iPmh0dHA6PQovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3NDAzLzwvYT48 bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48YSBocmVmPTNEImh0dHA6Ly9n ZXJyaXQub3ZpcnQub3JnLyMvYy8yNjg3Ni8iPmh0dHA6PQovL2dlcnJpdC5vdmlydC5vcmcvIy9j LzI2ODc2LzwvYT48bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj5Zb3UgaGFk IGFscmVhZHkgcmV2aWV3ZWQgc29tZSBvZiB0aGUgcGF0Y2hlcywgYnV0IGFmPQp0ZXIgdGhhdCBJ IG1vZGlmaWVkIHRoZSB0cmFuc2ZlcnJlZCBkYXRhIGZvcm1hdCBiZWNhdXNlIG9mIHRoZSB4bWxy cGMgbGltaXQ9CmF0aW9ucy4gU28gY291bGQgeW91IHBsZWFzZSByZXZpZXcgdGhlbSBhZ2Fpbj88 bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48bzpwPiZuYnNwOzwvbzpwPjwv cD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj5IaSBEYW4sPG86cD48L286cD48L3A+CjxwIGNsYXNz PTNEIk1zb05vcm1hbCI+SSBoYXZlIG1vZGlmaWVkIHlvdXIgY29tbWVudHMgZm9yIHNvbWUgb2Yg dGhlIGFib3ZlID0KcGF0Y2hlcy4gQ291bGQgeW91IHBsZWFzZSB0YWtlIGEgbG9vayBhbmQgbWVy Z2UgdGhvc2UgaWYgbm8gbW9yZSBjb21tZW50cz88PQpvOnA+PC9vOnA+PC9wPgo8cCBjbGFzcz0z RCJNc29Ob3JtYWwiPjxvOnA+Jm5ic3A7PC9vOnA+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwi IHN0eWxlPTNEInRleHQtYXV0b3NwYWNlOm5vbmUiPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQ9Ci1z aXplOjEyLjBwdCI+VGhhbmtzICZhbXA7IEJlc3QgUmVnYXJkczwvc3Bhbj48L2I+PHNwYW4gc3R5 bGU9M0QiZm9udC1zaXplOj0KMTIuMHB0Ij48bzpwPjwvbzpwPjwvc3Bhbj48L3A+CjxwIGNsYXNz PTNEIk1zb05vcm1hbCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5 bGU9M0QiZm9udD0KLXNpemU6MTIuMHB0Ij5TaGksIFhpYW8tTGVpIChCcnVjZSk8bzpwPjwvbzpw Pjwvc3Bhbj48L2I+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiIHN0eWxlPTNEInRleHQtYXV0 b3NwYWNlOm5vbmUiPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQ9Ci1zaXplOjEyLjBwdCI+PG86cD4m bmJzcDs8L286cD48L3NwYW4+PC9iPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0z RCJ0ZXh0LWF1dG9zcGFjZTpub25lIj48Yj48c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4w cHQ7Y29sb3I6YmxhY2siPkhld2xldHQtUGFja2FyZCBDby4sIEx0ZC48YnI+CkhQIFNlcnZlcnMg Q29yZSBQbGF0Zm9ybSBTb2Z0d2FyZSBDaGluYTwvc3Bhbj48L2I+PGI+PHNwYW4gc3R5bGU9M0Qi Zm9udC1zaT0KemU6MTIuMHB0Ij4KPG86cD48L286cD48L3NwYW4+PC9iPjwvcD4KPHAgY2xhc3M9 M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0ZXh0LWF1dG9zcGFjZTpub25lIj48Yj48c3BhbiBzdHls ZT0zRCJmb250PQotc2l6ZToxMi4wcHQiPlRlbGVwaG9uZTwvc3Bhbj48L2I+PHNwYW4gc3R5bGU9 M0QiZm9udC1zaXplOjEyLjBwdCI+ICYjNDM7ODY9CiAyMyA2NTY4MzA5MzxvOnA+PC9vOnA+PC9z cGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0ZXh0LWF1dG9zcGFjZTpu b25lIj48Yj48c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4wcHQiPk1vYmlsZQo8L3NwYW4+ PC9iPjxzcGFuIHN0eWxlPTNEImZvbnQtc2l6ZToxMi4wcHQiPiYjNDM7ODYgMTg2OTY1ODM0NDc8 bzpwPjwvbzpwPjw9Ci9zcGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48Yj48c3BhbiBz dHlsZT0zRCJmb250LXNpemU6MTIuMHB0Ij5FbWFpbCA8L3NwYW4+PC9iPQo+PHNwYW4gc3R5bGU9 M0QiZm9udC1zaXplOjEyLjBwdCI+PGEgaHJlZj0zRCJtYWlsdG86c2hpZ3VvLmxpQGhwLmNvbSI+ PHNwYW49CiBzdHlsZT0zRCJjb2xvcjpibHVlIj54aWFvLWxlaS5zaGlAaHAuY29tPC9zcGFuPjwv YT48L3NwYW4+PG86cD48L286cD48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PG86cD4mbmJz cDs8L286cD48L3A+CjwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4KCi0tXzAwMF8wOEFBNDAzQzgzOTkx MDRBODlBRjcxMDMwN0ZBNzhBRTI0NDA0ODY5RzVXMjc0M2FtZXJpY2FzXy0tCg== --===============4563490757436413309==-- From shangchun.liang at hp.com Wed May 7 22:36:43 2014 Content-Type: multipart/mixed; boundary="===============5345367074424809904==" MIME-Version: 1.0 From: Liang, Shang-Chun (David Liang, HPservers-Core-OE-PSC) To: devel at ovirt.org Subject: Re: [ovirt-devel] Vdsm patches need review and merge Date: Thu, 08 May 2014 02:35:00 +0000 Message-ID: <859B26A348D7AF42A791481A47A8B41A01043C7D@G1W3640.americas.hpqcorp.net> In-Reply-To: 08AA403C8399104A89AF710307FA78AE24404869@G5W2743.americas.hpqcorp.net --===============5345367074424809904== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable --_000_859B26A348D7AF42A791481A47A8B41A01043C7DG1W3640americas_ Content-Type: text/plain; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable Hi Dan, I heard that Martin is in TPO now, would you please take a look on these pa= =3D tches and merge it, thanks. Best regards, David Liang From: Shi, Xiao-Lei (Bruce, HP Servers-PSC-CQ) Sent: Thursday, May 08, 2014 10:31 AM To: Martin Sivak (msivak(a)redhat.com); Dan Kenigsberg (danken(a)redhat.com) Cc: Gilad Chaplik (gchaplik(a)redhat.com); Doron Fe= diuc=3D k (dfediuck(a)redhat.com); Liang, Shang-Chun (David Liang, HPservers-Core-O= E-=3D PSC); Liao, Chuan (Jason Liao, HPservers-Core-OE-PSC); devel(a)ovirt.org Subject: Vdsm patches need review and merge Hi Martin, Could you help to review these numa vdsm patches: http://gerrit.ovirt.org/#/c/27399/ http://gerrit.ovirt.org/#/c/27090/ http://gerrit.ovirt.org/#/c/27403/ http://gerrit.ovirt.org/#/c/26876/ You had already reviewed some of the patches, but after that I modified the= =3D transferred data format because of the xmlrpc limitations. So could you pl= =3D ease review them again? Hi Dan, I have modified your comments for some of the above patches. Could you plea= =3D se take a look and merge those if no more comments? Thanks & Best Regards Shi, Xiao-Lei (Bruce) Hewlett-Packard Co., Ltd. HP Servers Core Platform Software China Telephone +86 23 65683093 Mobile +86 18696583447 Email xiao-lei.shi(a)hp.com --_000_859B26A348D7AF42A791481A47A8B41A01043C7DG1W3640americas_ Content-Type: text/html; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable

Hi Dan,=

I heard that Marti= n is=3D in TPO now, would you please take a look on these patches and merge it, th= =3D anks.

<= /spa=3D n>

Best regards,=

David Liang=3D

<= /spa=3D n>

From: Shi, Xiao-Lei (Bruce, HP Servers-PSC-= CQ=3D )
Sent: Thursday, May 08, 2014 10:31 AM
To: Martin Sivak (msivak(a)redhat.com); Dan Kenigsberg (danken(a)red= hat.=3D com)
Cc: Gilad Chaplik <gchaplik(a)redhat.com> (gchaplik(a)redhat.c= om);=3D Doron Fediuck (dfediuck(a)redhat.com); Liang, Shang-Chun (David Liang, HPs= er=3D vers-Core-OE-PSC); Liao, Chuan (Jason Liao, HPservers-Core-OE-PSC); devel(a= )o=3D virt.org
Subject: Vdsm patches need review and merge

Hi Martin,

Could you help to review these numa vdsm patches:<= o:=3D p>

h= ttp:=3D //gerrit.ovirt.org/#/c/27399/

h= ttp:=3D //gerrit.ovirt.org/#/c/27090/

h= ttp:=3D //gerrit.ovirt.org/#/c/27403/

h= ttp:=3D //gerrit.ovirt.org/#/c/26876/

You had already reviewed some of the patches, but = af=3D ter that I modified the transferred data format because of the xmlrpc limit= =3D ations. So could you please review them again?

Hi Dan,

I have modified your comments for some of the abov= e =3D patches. Could you please take a look and merge those if no more comments?<= =3D o:p>

Thanks & Best Regards

Shi, Xiao-Lei (Bruce)

Hewlett-Packard Co., Ltd.
HP Servers Core Platform Software China

Telephone += 86=3D 23 65683093

Mobile +86 18696583447<=3D /span>

Email xiao-=3D lei.shi(a)hp.com

--_000_859B26A348D7AF42A791481A47A8B41A01043C7DG1W3640americas_-- --===============5345367074424809904== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" LS1fMDAwXzg1OUIyNkEzNDhEN0FGNDJBNzkxNDgxQTQ3QThCNDFBMDEwNDNDN0RHMVczNjQwYW1l cmljYXNfCkNvbnRlbnQtVHlwZTogdGV4dC9wbGFpbjsgY2hhcnNldD0idXMtYXNjaWkiCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKCkhpIERhbiwKSSBoZWFyZCB0 aGF0IE1hcnRpbiBpcyBpbiBUUE8gbm93LCB3b3VsZCB5b3UgcGxlYXNlIHRha2UgYSBsb29rIG9u IHRoZXNlIHBhPQp0Y2hlcyBhbmQgbWVyZ2UgaXQsIHRoYW5rcy4KCkJlc3QgcmVnYXJkcywKRGF2 aWQgTGlhbmcKCkZyb206IFNoaSwgWGlhby1MZWkgKEJydWNlLCBIUCBTZXJ2ZXJzLVBTQy1DUSkK U2VudDogVGh1cnNkYXksIE1heSAwOCwgMjAxNCAxMDozMSBBTQpUbzogTWFydGluIFNpdmFrICht c2l2YWtAcmVkaGF0LmNvbSk7IERhbiBLZW5pZ3NiZXJnIChkYW5rZW5AcmVkaGF0LmNvbSkKQ2M6 IEdpbGFkIENoYXBsaWsgPGdjaGFwbGlrQHJlZGhhdC5jb20+IChnY2hhcGxpa0ByZWRoYXQuY29t KTsgRG9yb24gRmVkaXVjPQprIChkZmVkaXVja0ByZWRoYXQuY29tKTsgTGlhbmcsIFNoYW5nLUNo dW4gKERhdmlkIExpYW5nLCBIUHNlcnZlcnMtQ29yZS1PRS09ClBTQyk7IExpYW8sIENodWFuIChK YXNvbiBMaWFvLCBIUHNlcnZlcnMtQ29yZS1PRS1QU0MpOyBkZXZlbEBvdmlydC5vcmcKU3ViamVj dDogVmRzbSBwYXRjaGVzIG5lZWQgcmV2aWV3IGFuZCBtZXJnZQoKSGkgTWFydGluLApDb3VsZCB5 b3UgaGVscCB0byByZXZpZXcgdGhlc2UgbnVtYSB2ZHNtIHBhdGNoZXM6Cmh0dHA6Ly9nZXJyaXQu b3ZpcnQub3JnLyMvYy8yNzM5OS8KaHR0cDovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3MDkwLwpo dHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjc0MDMvCmh0dHA6Ly9nZXJyaXQub3ZpcnQub3Jn LyMvYy8yNjg3Ni8KWW91IGhhZCBhbHJlYWR5IHJldmlld2VkIHNvbWUgb2YgdGhlIHBhdGNoZXMs IGJ1dCBhZnRlciB0aGF0IEkgbW9kaWZpZWQgdGhlPQogdHJhbnNmZXJyZWQgZGF0YSBmb3JtYXQg YmVjYXVzZSBvZiB0aGUgeG1scnBjIGxpbWl0YXRpb25zLiBTbyBjb3VsZCB5b3UgcGw9CmVhc2Ug cmV2aWV3IHRoZW0gYWdhaW4/CgpIaSBEYW4sCkkgaGF2ZSBtb2RpZmllZCB5b3VyIGNvbW1lbnRz IGZvciBzb21lIG9mIHRoZSBhYm92ZSBwYXRjaGVzLiBDb3VsZCB5b3UgcGxlYT0Kc2UgdGFrZSBh IGxvb2sgYW5kIG1lcmdlIHRob3NlIGlmIG5vIG1vcmUgY29tbWVudHM/CgpUaGFua3MgJiBCZXN0 IFJlZ2FyZHMKU2hpLCBYaWFvLUxlaSAoQnJ1Y2UpCgpIZXdsZXR0LVBhY2thcmQgQ28uLCBMdGQu CkhQIFNlcnZlcnMgQ29yZSBQbGF0Zm9ybSBTb2Z0d2FyZSBDaGluYQpUZWxlcGhvbmUgKzg2IDIz IDY1NjgzMDkzCk1vYmlsZSArODYgMTg2OTY1ODM0NDcKRW1haWwgeGlhby1sZWkuc2hpQGhwLmNv bTxtYWlsdG86c2hpZ3VvLmxpQGhwLmNvbT4KCgotLV8wMDBfODU5QjI2QTM0OEQ3QUY0MkE3OTE0 ODFBNDdBOEI0MUEwMTA0M0M3REcxVzM2NDBhbWVyaWNhc18KQ29udGVudC1UeXBlOiB0ZXh0L2h0 bWw7IGNoYXJzZXQ9InVzLWFzY2lpIgpDb250ZW50LVRyYW5zZmVyLUVuY29kaW5nOiBxdW90ZWQt cHJpbnRhYmxlCgo8aHRtbCB4bWxuczp2PTNEInVybjpzY2hlbWFzLW1pY3Jvc29mdC1jb206dm1s IiB4bWxuczpvPTNEInVybjpzY2hlbWFzLW1pY3I9Cm9zb2Z0LWNvbTpvZmZpY2U6b2ZmaWNlIiB4 bWxuczp3PTNEInVybjpzY2hlbWFzLW1pY3Jvc29mdC1jb206b2ZmaWNlOndvcmQiID0KeG1sbnM6 bT0zRCJodHRwOi8vc2NoZW1hcy5taWNyb3NvZnQuY29tL29mZmljZS8yMDA0LzEyL29tbWwiIHht bG5zPTNEImh0dHA6PQovL3d3dy53My5vcmcvVFIvUkVDLWh0bWw0MCI+CjxoZWFkPgo8bWV0YSBo dHRwLWVxdWl2PTNEIkNvbnRlbnQtVHlwZSIgY29udGVudD0zRCJ0ZXh0L2h0bWw7IGNoYXJzZXQ9 M0R1cy1hc2NpaSI9Cj4KPG1ldGEgbmFtZT0zRCJHZW5lcmF0b3IiIGNvbnRlbnQ9M0QiTWljcm9z b2Z0IFdvcmQgMTUgKGZpbHRlcmVkIG1lZGl1bSkiPgo8c3R5bGU+PCEtLQovKiBGb250IERlZmlu aXRpb25zICovCkBmb250LWZhY2UKCXtmb250LWZhbWlseTpTaW1TdW47CglwYW5vc2UtMToyIDEg NiAwIDMgMSAxIDEgMSAxO30KQGZvbnQtZmFjZQoJe2ZvbnQtZmFtaWx5OiJDYW1icmlhIE1hdGgi OwoJcGFub3NlLTE6MiA0IDUgMyA1IDQgNiAzIDIgNDt9CkBmb250LWZhY2UKCXtmb250LWZhbWls eTpDYWxpYnJpOwoJcGFub3NlLTE6MiAxNSA1IDIgMiAyIDQgMyAyIDQ7fQpAZm9udC1mYWNlCgl7 Zm9udC1mYW1pbHk6IlxAU2ltU3VuIjsKCXBhbm9zZS0xOjIgMSA2IDAgMyAxIDEgMSAxIDE7fQov KiBTdHlsZSBEZWZpbml0aW9ucyAqLwpwLk1zb05vcm1hbCwgbGkuTXNvTm9ybWFsLCBkaXYuTXNv Tm9ybWFsCgl7bWFyZ2luOjBpbjsKCW1hcmdpbi1ib3R0b206LjAwMDFwdDsKCWZvbnQtc2l6ZTox MS4wcHQ7Cglmb250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiO30KYTpsaW5rLCBzcGFu Lk1zb0h5cGVybGluawoJe21zby1zdHlsZS1wcmlvcml0eTo5OTsKCWNvbG9yOmJsdWU7Cgl0ZXh0 LWRlY29yYXRpb246dW5kZXJsaW5lO30KYTp2aXNpdGVkLCBzcGFuLk1zb0h5cGVybGlua0ZvbGxv d2VkCgl7bXNvLXN0eWxlLXByaW9yaXR5Ojk5OwoJY29sb3I6cHVycGxlOwoJdGV4dC1kZWNvcmF0 aW9uOnVuZGVybGluZTt9CnNwYW4uRW1haWxTdHlsZTE3Cgl7bXNvLXN0eWxlLXR5cGU6cGVyc29u YWw7Cglmb250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiOwoJY29sb3I6d2luZG93dGV4 dDt9CnNwYW4uRW1haWxTdHlsZTE4Cgl7bXNvLXN0eWxlLXR5cGU6cGVyc29uYWwtcmVwbHk7Cglm b250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiOwoJY29sb3I6IzFGNDk3RDt9Ci5Nc29D aHBEZWZhdWx0Cgl7bXNvLXN0eWxlLXR5cGU6ZXhwb3J0LW9ubHk7Cglmb250LXNpemU6MTAuMHB0 O30KQHBhZ2UgV29yZFNlY3Rpb24xCgl7c2l6ZTo4LjVpbiAxMS4waW47CgltYXJnaW46MS4waW4g MS4yNWluIDEuMGluIDEuMjVpbjt9CmRpdi5Xb3JkU2VjdGlvbjEKCXtwYWdlOldvcmRTZWN0aW9u MTt9Ci0tPjwvc3R5bGU+PCEtLVtpZiBndGUgbXNvIDldPjx4bWw+CjxvOnNoYXBlZGVmYXVsdHMg djpleHQ9M0QiZWRpdCIgc3BpZG1heD0zRCIxMDI2IiAvPgo8L3htbD48IVtlbmRpZl0tLT48IS0t W2lmIGd0ZSBtc28gOV0+PHhtbD4KPG86c2hhcGVsYXlvdXQgdjpleHQ9M0QiZWRpdCI+CjxvOmlk bWFwIHY6ZXh0PTNEImVkaXQiIGRhdGE9M0QiMSIgLz4KPC9vOnNoYXBlbGF5b3V0PjwveG1sPjwh W2VuZGlmXS0tPgo8L2hlYWQ+Cjxib2R5IGxhbmc9M0QiRU4tVVMiIGxpbms9M0QiYmx1ZSIgdmxp bms9M0QicHVycGxlIj4KPGRpdiBjbGFzcz0zRCJXb3JkU2VjdGlvbjEiPgo8cCBjbGFzcz0zRCJN c29Ob3JtYWwiPjxzcGFuIHN0eWxlPTNEImNvbG9yOiMxRjQ5N0QiPkhpIERhbiw8bzpwPjwvbzpw Pjwvc3A9CmFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBzdHlsZT0zRCJjb2xv cjojMUY0OTdEIj5JIGhlYXJkIHRoYXQgTWFydGluIGlzPQogaW4gVFBPIG5vdywgd291bGQgeW91 IHBsZWFzZSB0YWtlIGEgbG9vayBvbiB0aGVzZSBwYXRjaGVzIGFuZCBtZXJnZSBpdCwgdGg9CmFu a3MuPG86cD48L286cD48L3NwYW4+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIHN0 eWxlPTNEImNvbG9yOiMxRjQ5N0QiPjxvOnA+Jm5ic3A7PC9vOnA+PC9zcGE9Cm4+PC9wPgo8cCBj bGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIHN0eWxlPTNEImNvbG9yOiMxRjQ5N0QiPkJlc3QgcmVn YXJkcyw8bzpwPjwvbzo9CnA+PC9zcGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3Bh biBzdHlsZT0zRCJjb2xvcjojMUY0OTdEIj5EYXZpZCBMaWFuZzxvOnA+PC9vOnA+PQo8L3NwYW4+ PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIHN0eWxlPTNEImNvbG9yOiMxRjQ5N0Qi PjxvOnA+Jm5ic3A7PC9vOnA+PC9zcGE9Cm4+PC9wPgo8ZGl2IHN0eWxlPTNEImJvcmRlcjpub25l O2JvcmRlci1sZWZ0OnNvbGlkIGJsdWUgMS41cHQ7cGFkZGluZzowaW4gMGluIDBpbiA9CjQuMHB0 Ij4KPGRpdj4KPGRpdiBzdHlsZT0zRCJib3JkZXI6bm9uZTtib3JkZXItdG9wOnNvbGlkICNFMUUx RTEgMS4wcHQ7cGFkZGluZzozLjBwdCAwaW4gPQowaW4gMGluIj4KPHAgY2xhc3M9M0QiTXNvTm9y bWFsIj48Yj5Gcm9tOjwvYj4gU2hpLCBYaWFvLUxlaSAoQnJ1Y2UsIEhQIFNlcnZlcnMtUFNDLUNR PQopIDxicj4KPGI+U2VudDo8L2I+IFRodXJzZGF5LCBNYXkgMDgsIDIwMTQgMTA6MzEgQU08YnI+ CjxiPlRvOjwvYj4gTWFydGluIFNpdmFrIChtc2l2YWtAcmVkaGF0LmNvbSk7IERhbiBLZW5pZ3Ni ZXJnIChkYW5rZW5AcmVkaGF0Lj0KY29tKTxicj4KPGI+Q2M6PC9iPiBHaWxhZCBDaGFwbGlrICZs dDtnY2hhcGxpa0ByZWRoYXQuY29tJmd0OyAoZ2NoYXBsaWtAcmVkaGF0LmNvbSk7PQogRG9yb24g RmVkaXVjayAoZGZlZGl1Y2tAcmVkaGF0LmNvbSk7IExpYW5nLCBTaGFuZy1DaHVuIChEYXZpZCBM aWFuZywgSFBzZXI9CnZlcnMtQ29yZS1PRS1QU0MpOyBMaWFvLCBDaHVhbiAoSmFzb24gTGlhbywg SFBzZXJ2ZXJzLUNvcmUtT0UtUFNDKTsgZGV2ZWxAbz0KdmlydC5vcmc8YnI+CjxiPlN1YmplY3Q6 PC9iPiBWZHNtIHBhdGNoZXMgbmVlZCByZXZpZXcgYW5kIG1lcmdlPG86cD48L286cD48L3A+Cjwv ZGl2Pgo8L2Rpdj4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48bzpwPiZuYnNwOzwvbzpwPjwvcD4K PHAgY2xhc3M9M0QiTXNvTm9ybWFsIj5IaSBNYXJ0aW4sPG86cD48L286cD48L3A+CjxwIGNsYXNz PTNEIk1zb05vcm1hbCI+Q291bGQgeW91IGhlbHAgdG8gcmV2aWV3IHRoZXNlIG51bWEgdmRzbSBw YXRjaGVzOjxvOj0KcD48L286cD48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PGEgaHJlZj0z RCJodHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjczOTkvIj5odHRwOj0KLy9nZXJyaXQub3Zp cnQub3JnLyMvYy8yNzM5OS88L2E+PG86cD48L286cD48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1h bCI+PGEgaHJlZj0zRCJodHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjcwOTAvIj5odHRwOj0K Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzA5MC88L2E+PG86cD48L286cD48L3A+CjxwIGNsYXNz PTNEIk1zb05vcm1hbCI+PGEgaHJlZj0zRCJodHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjc0 MDMvIj5odHRwOj0KLy9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzQwMy88L2E+PG86cD48L286cD48 L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PGEgaHJlZj0zRCJodHRwOi8vZ2Vycml0Lm92aXJ0 Lm9yZy8jL2MvMjY4NzYvIj5odHRwOj0KLy9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNjg3Ni88L2E+ PG86cD48L286cD48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+WW91IGhhZCBhbHJlYWR5IHJl dmlld2VkIHNvbWUgb2YgdGhlIHBhdGNoZXMsIGJ1dCBhZj0KdGVyIHRoYXQgSSBtb2RpZmllZCB0 aGUgdHJhbnNmZXJyZWQgZGF0YSBmb3JtYXQgYmVjYXVzZSBvZiB0aGUgeG1scnBjIGxpbWl0PQph dGlvbnMuIFNvIGNvdWxkIHlvdSBwbGVhc2UgcmV2aWV3IHRoZW0gYWdhaW4/PG86cD48L286cD48 L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PG86cD4mbmJzcDs8L286cD48L3A+CjxwIGNsYXNz PTNEIk1zb05vcm1hbCI+SGkgRGFuLDxvOnA+PC9vOnA+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3Jt YWwiPkkgaGF2ZSBtb2RpZmllZCB5b3VyIGNvbW1lbnRzIGZvciBzb21lIG9mIHRoZSBhYm92ZSA9 CnBhdGNoZXMuIENvdWxkIHlvdSBwbGVhc2UgdGFrZSBhIGxvb2sgYW5kIG1lcmdlIHRob3NlIGlm IG5vIG1vcmUgY29tbWVudHM/PD0KbzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFs Ij48bzpwPiZuYnNwOzwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0 ZXh0LWF1dG9zcGFjZTpub25lIj48Yj48c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4wcHQi PlRoYW5rcyAmYW1wOyBCZXN0IFJlZ2FyZHM8L3NwYW4+PC9iPjxzcGFuIHN0eWxlPTNEImZvbnQt c2l6ZTo9CjEyLjBwdCI+PG86cD48L286cD48L3NwYW4+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3Jt YWwiIHN0eWxlPTNEInRleHQtYXV0b3NwYWNlOm5vbmUiPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQ9 Ci1zaXplOjEyLjBwdCI+U2hpLCBYaWFvLUxlaSAoQnJ1Y2UpPG86cD48L286cD48L3NwYW4+PC9i PjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0ZXh0LWF1dG9zcGFjZTpub25l Ij48Yj48c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4wcHQiPjxvOnA+Jm5ic3A7PC9vOnA+ PC9zcGFuPjwvYj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCIgc3R5bGU9M0QidGV4dC1hdXRv c3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9udD0KLXNpemU6MTIuMHB0O2NvbG9yOmJs YWNrIj5IZXdsZXR0LVBhY2thcmQgQ28uLCBMdGQuPGJyPgpIUCBTZXJ2ZXJzIENvcmUgUGxhdGZv cm0gU29mdHdhcmUgQ2hpbmE8L3NwYW4+PC9iPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQtc2k9Cnpl OjEyLjBwdCI+CjxvOnA+PC9vOnA+PC9zcGFuPjwvYj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1h bCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9udD0K LXNpemU6MTIuMHB0Ij5UZWxlcGhvbmU8L3NwYW4+PC9iPjxzcGFuIHN0eWxlPTNEImZvbnQtc2l6 ZToxMi4wcHQiPiAmIzQzOzg2PQogMjMgNjU2ODMwOTM8bzpwPjwvbzpwPjwvc3Bhbj48L3A+Cjxw IGNsYXNzPTNEIk1zb05vcm1hbCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNw YW4gc3R5bGU9M0QiZm9udD0KLXNpemU6MTIuMHB0Ij5Nb2JpbGUKPC9zcGFuPjwvYj48c3BhbiBz dHlsZT0zRCJmb250LXNpemU6MTIuMHB0Ij4mIzQzOzg2IDE4Njk2NTgzNDQ3PG86cD48L286cD48 PQovc3Bhbj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PGI+PHNwYW4gc3R5bGU9M0QiZm9u dC1zaXplOjEyLjBwdCI+RW1haWwgPC9zcGFuPjwvYj0KPjxzcGFuIHN0eWxlPTNEImZvbnQtc2l6 ZToxMi4wcHQiPjxhIGhyZWY9M0QibWFpbHRvOnNoaWd1by5saUBocC5jb20iPnhpYW8tPQpsZWku c2hpQGhwLmNvbTwvYT48L3NwYW4+PG86cD48L286cD48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1h bCI+PG86cD4mbmJzcDs8L286cD48L3A+CjwvZGl2Pgo8L2Rpdj4KPC9ib2R5Pgo8L2h0bWw+Cgot LV8wMDBfODU5QjI2QTM0OEQ3QUY0MkE3OTE0ODFBNDdBOEI0MUEwMTA0M0M3REcxVzM2NDBhbWVy aWNhc18tLQo= --===============5345367074424809904==-- From xiao-lei.shi at hp.com Thu May 8 08:33:45 2014 Content-Type: multipart/mixed; boundary="===============5088316889904542478==" MIME-Version: 1.0 From: Shi, Xiao-Lei (Bruce, HP Servers-PSC-CQ) To: devel at ovirt.org Subject: Re: [ovirt-devel] Vdsm patches need review and merge Date: Thu, 08 May 2014 12:33:14 +0000 Message-ID: <08AA403C8399104A89AF710307FA78AE24404AC1@G5W2743.americas.hpqcorp.net> In-Reply-To: 859B26A348D7AF42A791481A47A8B41A01043C7D@G1W3640.americas.hpqcorp.net --===============5088316889904542478== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable --_000_08AA403C8399104A89AF710307FA78AE24404AC1G5W2743americas_ Content-Type: text/plain; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable Hi Dan/Francesco, Could you help to review and merge these patches: http://gerrit.ovirt.org/#/c/27515/ (This patch is needed by 26876 since so= =3D me comments in that patch need to modify caps module) http://gerrit.ovirt.org/#/c/26876/ http://gerrit.ovirt.org/#/c/27403/ I didn't modify some comments in 26876 since I think they are related with = =3D codes refactor and literal syntax. I will submit a separate patch to modify= =3D these kinds of comments of numa feature altogether later. Could you please= =3D now focus on the functionality since we need to merge these patches ASAP? Thanks & Best Regards Shi, Xiao-Lei (Bruce) Hewlett-Packard Co., Ltd. HP Servers Core Platform Software China Telephone +86 23 65683093 Mobile +86 18696583447 Email xiao-lei.shi(a)hp.com From: Liang, Shang-Chun (David Liang, HPservers-Core-OE-PSC) Sent: Thursday, May 08, 2014 10:35 AM To: Shi, Xiao-Lei (Bruce, HP Servers-PSC-CQ); Martin Sivak (msivak(a)redhat= .c=3D om); Dan Kenigsberg (danken(a)redhat.com) Cc: Gilad Chaplik (gchaplik(a)redhat.com); Doron Fe= diuc=3D k (dfediuck(a)redhat.com); Liao, Chuan (Jason Liao, HPservers-Core-OE-PSC);= d=3D evel(a)ovirt.org Subject: RE: Vdsm patches need review and merge Hi Dan, I heard that Martin is in TPO now, would you please take a look on these pa= =3D tches and merge it, thanks. Best regards, David Liang From: Shi, Xiao-Lei (Bruce, HP Servers-PSC-CQ) Sent: Thursday, May 08, 2014 10:31 AM To: Martin Sivak (msivak(a)redhat.com); Dan Ken= igsb=3D erg (danken(a)redhat.com) Cc: Gilad Chaplik > (gc= hapl=3D ik(a)redhat.com); Doron Fediuck (dfediuck(a)r= edhat.=3D com); Liang, Shang-Chun (David Liang, HPserve= rs=3D -Core-OE-PSC); Liao, Chuan (Jason Liao, HPservers-Core-OE-PSC); devel(a)ovi= rt=3D .org Subject: Vdsm patches need review and merge Hi Martin, Could you help to review these numa vdsm patches: http://gerrit.ovirt.org/#/c/27399/ http://gerrit.ovirt.org/#/c/27090/ http://gerrit.ovirt.org/#/c/27403/ http://gerrit.ovirt.org/#/c/26876/ You had already reviewed some of the patches, but after that I modified the= =3D transferred data format because of the xmlrpc limitations. So could you pl= =3D ease review them again? Hi Dan, I have modified your comments for some of the above patches. Could you plea= =3D se take a look and merge those if no more comments? Thanks & Best Regards Shi, Xiao-Lei (Bruce) Hewlett-Packard Co., Ltd. HP Servers Core Platform Software China Telephone +86 23 65683093 Mobile +86 18696583447 Email xiao-lei.shi(a)hp.com --_000_08AA403C8399104A89AF710307FA78AE24404AC1G5W2743americas_ Content-Type: text/html; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable

Hi Dan/Francesco,<= o:p>=3D

<= /spa=3D n>

Could you help to = revi=3D ew and merge these patches:

http://gerrit.ovirt.org/#/c/27515/ (This= =3D patch is needed by 26876 since some comments in that patch need to modify = =3D caps module)

http://gerrit.ovirt.org/#/c/26876/<= =3D /span>

http://gerrit.ovirt.org/#/c/27403/<= =3D /span>

<= /spa=3D n>

I didn’t mod= ify =3D some comments in 26876 since I think they are related with codes refactor a= =3D nd literal syntax. I will submit a separate patch to modify these kinds of = =3D comments of numa feature altogether later. Could you please now focus on the functionality since we need to merge the= =3D se patches ASAP?

<= /spa=3D n>

Thanks & Best Regards

Shi, Xiao-Lei (Bruce)

Hewlett-Packard Co., Ltd.
HP Servers Core Platform Software China

Telephone +86 23 65683093

Mobile +86 1869658= 34=3D 47

Em=3D ail xiao-lei.shi(a)hp.com

<= /spa=3D n>

From: Liang, S= =3D hang-Chun (David Liang, HPservers-Core-OE-PSC)
Sent: Thursday, May 08, 2014 10:35 AM
To: Shi, Xiao-Lei (Bruce, HP Servers-PSC-CQ); Martin Sivak (msivak(a= )r=3D edhat.com); Dan Kenigsberg (danken(a)redhat.com)
Cc: Gilad Chaplik <gchaplik(a)redhat.com> (gchaplik(a)redhat.c= om);=3D Doron Fediuck (dfediuck(a)redhat.com); Liao, Chuan (Jason Liao, HPservers-= Co=3D re-OE-PSC); devel(a)ovirt.org
Subject: RE: Vdsm patches need review and merge

Hi Dan,=

I heard that Marti= n is=3D in TPO now, would you please take a look on these patches and merge it, th= =3D anks.

<= /spa=3D n>

Best regards,=

David Liang=3D

<= /spa=3D n>

From: Shi, Xiao-Lei (Bruce, HP Servers-PSC-= CQ=3D )
Sent: Thursday, May 08, 2014 10:31 AM
To: Martin Sivak (msivak(a)= redhat=3D .com); Dan Kenigsberg (danken(= a)redh=3D at.com)
Cc: Gilad Chaplik <gch= apli=3D k(a)redhat.com> (gchaplik= (a)redha=3D t.com); Doron Fediuck (dfedi= uck@=3D redhat.com); Liang, Shang-Chun (David Liang, HPservers-Core-OE-PSC); Liao, Chuan (Jason Liao, HPservers-Core-OE-PSC); <= =3D a href=3D3D"mailto:devel(a)ovirt.org"> devel(a)ovirt.org
Subject: Vdsm patches need review and merge

Hi Martin,

Could you help to review these numa vdsm patches:<= o:=3D p>

h= ttp:=3D //gerrit.ovirt.org/#/c/27399/

h= ttp:=3D //gerrit.ovirt.org/#/c/27090/

h= ttp:=3D //gerrit.ovirt.org/#/c/27403/

h= ttp:=3D //gerrit.ovirt.org/#/c/26876/

You had already reviewed some of the patches, but = af=3D ter that I modified the transferred data format because of the xmlrpc limit= =3D ations. So could you please review them again?

Hi Dan,

I have modified your comments for some of the abov= e =3D patches. Could you please take a look and merge those if no more comments?<= =3D o:p>

Thanks & Best Regards

Shi, Xiao-Lei (Bruce)

Hewlett-Packard Co., Ltd.
HP Servers Core Platform Software China

Telephone += 86=3D 23 65683093

Mobile +86 18696583447<=3D /span>

Email xiao-=3D lei.shi(a)hp.com

--_000_08AA403C8399104A89AF710307FA78AE24404AC1G5W2743americas_-- --===============5088316889904542478== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" LS1fMDAwXzA4QUE0MDNDODM5OTEwNEE4OUFGNzEwMzA3RkE3OEFFMjQ0MDRBQzFHNVcyNzQzYW1l cmljYXNfCkNvbnRlbnQtVHlwZTogdGV4dC9wbGFpbjsgY2hhcnNldD0idXMtYXNjaWkiCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKCkhpIERhbi9GcmFuY2VzY28s CgpDb3VsZCB5b3UgaGVscCB0byByZXZpZXcgYW5kIG1lcmdlIHRoZXNlIHBhdGNoZXM6Cmh0dHA6 Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzUxNS8gIChUaGlzIHBhdGNoIGlzIG5lZWRlZCBieSAy Njg3NiBzaW5jZSBzbz0KbWUgY29tbWVudHMgaW4gdGhhdCBwYXRjaCBuZWVkIHRvIG1vZGlmeSBj YXBzIG1vZHVsZSkKaHR0cDovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI2ODc2LwpodHRwOi8vZ2Vy cml0Lm92aXJ0Lm9yZy8jL2MvMjc0MDMvCgpJIGRpZG4ndCBtb2RpZnkgc29tZSBjb21tZW50cyBp biAyNjg3NiBzaW5jZSBJIHRoaW5rIHRoZXkgYXJlIHJlbGF0ZWQgd2l0aCA9CmNvZGVzIHJlZmFj dG9yIGFuZCBsaXRlcmFsIHN5bnRheC4gSSB3aWxsIHN1Ym1pdCBhIHNlcGFyYXRlIHBhdGNoIHRv IG1vZGlmeT0KIHRoZXNlIGtpbmRzIG9mIGNvbW1lbnRzIG9mIG51bWEgZmVhdHVyZSBhbHRvZ2V0 aGVyIGxhdGVyLiBDb3VsZCB5b3UgcGxlYXNlPQogbm93IGZvY3VzIG9uIHRoZSBmdW5jdGlvbmFs aXR5IHNpbmNlIHdlIG5lZWQgdG8gbWVyZ2UgdGhlc2UgcGF0Y2hlcyBBU0FQPwoKVGhhbmtzICYg QmVzdCBSZWdhcmRzClNoaSwgWGlhby1MZWkgKEJydWNlKQoKSGV3bGV0dC1QYWNrYXJkIENvLiwg THRkLgpIUCBTZXJ2ZXJzIENvcmUgUGxhdGZvcm0gU29mdHdhcmUgQ2hpbmEKVGVsZXBob25lICs4 NiAyMyA2NTY4MzA5MwpNb2JpbGUgKzg2IDE4Njk2NTgzNDQ3CkVtYWlsIHhpYW8tbGVpLnNoaUBo cC5jb208bWFpbHRvOnNoaWd1by5saUBocC5jb20+CgpGcm9tOiBMaWFuZywgU2hhbmctQ2h1biAo RGF2aWQgTGlhbmcsIEhQc2VydmVycy1Db3JlLU9FLVBTQykKU2VudDogVGh1cnNkYXksIE1heSAw OCwgMjAxNCAxMDozNSBBTQpUbzogU2hpLCBYaWFvLUxlaSAoQnJ1Y2UsIEhQIFNlcnZlcnMtUFND LUNRKTsgTWFydGluIFNpdmFrIChtc2l2YWtAcmVkaGF0LmM9Cm9tKTsgRGFuIEtlbmlnc2Jlcmcg KGRhbmtlbkByZWRoYXQuY29tKQpDYzogR2lsYWQgQ2hhcGxpayA8Z2NoYXBsaWtAcmVkaGF0LmNv bT4gKGdjaGFwbGlrQHJlZGhhdC5jb20pOyBEb3JvbiBGZWRpdWM9CmsgKGRmZWRpdWNrQHJlZGhh dC5jb20pOyBMaWFvLCBDaHVhbiAoSmFzb24gTGlhbywgSFBzZXJ2ZXJzLUNvcmUtT0UtUFNDKTsg ZD0KZXZlbEBvdmlydC5vcmcKU3ViamVjdDogUkU6IFZkc20gcGF0Y2hlcyBuZWVkIHJldmlldyBh bmQgbWVyZ2UKCkhpIERhbiwKSSBoZWFyZCB0aGF0IE1hcnRpbiBpcyBpbiBUUE8gbm93LCB3b3Vs ZCB5b3UgcGxlYXNlIHRha2UgYSBsb29rIG9uIHRoZXNlIHBhPQp0Y2hlcyBhbmQgbWVyZ2UgaXQs IHRoYW5rcy4KCkJlc3QgcmVnYXJkcywKRGF2aWQgTGlhbmcKCkZyb206IFNoaSwgWGlhby1MZWkg KEJydWNlLCBIUCBTZXJ2ZXJzLVBTQy1DUSkKU2VudDogVGh1cnNkYXksIE1heSAwOCwgMjAxNCAx MDozMSBBTQpUbzogTWFydGluIFNpdmFrIChtc2l2YWtAcmVkaGF0LmNvbTxtYWlsdG86bXNpdmFr QHJlZGhhdC5jb20+KTsgRGFuIEtlbmlnc2I9CmVyZyAoZGFua2VuQHJlZGhhdC5jb208bWFpbHRv OmRhbmtlbkByZWRoYXQuY29tPikKQ2M6IEdpbGFkIENoYXBsaWsgPGdjaGFwbGlrQHJlZGhhdC5j b208bWFpbHRvOmdjaGFwbGlrQHJlZGhhdC5jb20+PiAoZ2NoYXBsPQppa0ByZWRoYXQuY29tPG1h aWx0bzpnY2hhcGxpa0ByZWRoYXQuY29tPik7IERvcm9uIEZlZGl1Y2sgKGRmZWRpdWNrQHJlZGhh dC49CmNvbTxtYWlsdG86ZGZlZGl1Y2tAcmVkaGF0LmNvbT4pOyBMaWFuZywgU2hhbmctQ2h1biAo RGF2aWQgTGlhbmcsIEhQc2VydmVycz0KLUNvcmUtT0UtUFNDKTsgTGlhbywgQ2h1YW4gKEphc29u IExpYW8sIEhQc2VydmVycy1Db3JlLU9FLVBTQyk7IGRldmVsQG92aXJ0PQoub3JnPG1haWx0bzpk ZXZlbEBvdmlydC5vcmc+ClN1YmplY3Q6IFZkc20gcGF0Y2hlcyBuZWVkIHJldmlldyBhbmQgbWVy Z2UKCkhpIE1hcnRpbiwKQ291bGQgeW91IGhlbHAgdG8gcmV2aWV3IHRoZXNlIG51bWEgdmRzbSBw YXRjaGVzOgpodHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjczOTkvCmh0dHA6Ly9nZXJyaXQu b3ZpcnQub3JnLyMvYy8yNzA5MC8KaHR0cDovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3NDAzLwpo dHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjY4NzYvCllvdSBoYWQgYWxyZWFkeSByZXZpZXdl ZCBzb21lIG9mIHRoZSBwYXRjaGVzLCBidXQgYWZ0ZXIgdGhhdCBJIG1vZGlmaWVkIHRoZT0KIHRy YW5zZmVycmVkIGRhdGEgZm9ybWF0IGJlY2F1c2Ugb2YgdGhlIHhtbHJwYyBsaW1pdGF0aW9ucy4g U28gY291bGQgeW91IHBsPQplYXNlIHJldmlldyB0aGVtIGFnYWluPwoKSGkgRGFuLApJIGhhdmUg bW9kaWZpZWQgeW91ciBjb21tZW50cyBmb3Igc29tZSBvZiB0aGUgYWJvdmUgcGF0Y2hlcy4gQ291 bGQgeW91IHBsZWE9CnNlIHRha2UgYSBsb29rIGFuZCBtZXJnZSB0aG9zZSBpZiBubyBtb3JlIGNv bW1lbnRzPwoKVGhhbmtzICYgQmVzdCBSZWdhcmRzClNoaSwgWGlhby1MZWkgKEJydWNlKQoKSGV3 bGV0dC1QYWNrYXJkIENvLiwgTHRkLgpIUCBTZXJ2ZXJzIENvcmUgUGxhdGZvcm0gU29mdHdhcmUg Q2hpbmEKVGVsZXBob25lICs4NiAyMyA2NTY4MzA5MwpNb2JpbGUgKzg2IDE4Njk2NTgzNDQ3CkVt YWlsIHhpYW8tbGVpLnNoaUBocC5jb208bWFpbHRvOnNoaWd1by5saUBocC5jb20+CgoKLS1fMDAw XzA4QUE0MDNDODM5OTEwNEE4OUFGNzEwMzA3RkE3OEFFMjQ0MDRBQzFHNVcyNzQzYW1lcmljYXNf CkNvbnRlbnQtVHlwZTogdGV4dC9odG1sOyBjaGFyc2V0PSJ1cy1hc2NpaSIKQ29udGVudC1UcmFu c2Zlci1FbmNvZGluZzogcXVvdGVkLXByaW50YWJsZQoKPGh0bWwgeG1sbnM6dj0zRCJ1cm46c2No ZW1hcy1taWNyb3NvZnQtY29tOnZtbCIgeG1sbnM6bz0zRCJ1cm46c2NoZW1hcy1taWNyPQpvc29m dC1jb206b2ZmaWNlOm9mZmljZSIgeG1sbnM6dz0zRCJ1cm46c2NoZW1hcy1taWNyb3NvZnQtY29t Om9mZmljZTp3b3JkIiA9CnhtbG5zOm09M0QiaHR0cDovL3NjaGVtYXMubWljcm9zb2Z0LmNvbS9v ZmZpY2UvMjAwNC8xMi9vbW1sIiB4bWxucz0zRCJodHRwOj0KLy93d3cudzMub3JnL1RSL1JFQy1o dG1sNDAiPgo8aGVhZD4KPG1ldGEgaHR0cC1lcXVpdj0zRCJDb250ZW50LVR5cGUiIGNvbnRlbnQ9 M0QidGV4dC9odG1sOyBjaGFyc2V0PTNEdXMtYXNjaWkiPQo+CjxtZXRhIG5hbWU9M0QiR2VuZXJh dG9yIiBjb250ZW50PTNEIk1pY3Jvc29mdCBXb3JkIDE0IChmaWx0ZXJlZCBtZWRpdW0pIj4KPHN0 eWxlPjwhLS0KLyogRm9udCBEZWZpbml0aW9ucyAqLwpAZm9udC1mYWNlCgl7Zm9udC1mYW1pbHk6 U2ltU3VuOwoJcGFub3NlLTE6MiAxIDYgMCAzIDEgMSAxIDEgMTt9CkBmb250LWZhY2UKCXtmb250 LWZhbWlseTpTaW1TdW47CglwYW5vc2UtMToyIDEgNiAwIDMgMSAxIDEgMSAxO30KQGZvbnQtZmFj ZQoJe2ZvbnQtZmFtaWx5OkNhbGlicmk7CglwYW5vc2UtMToyIDE1IDUgMiAyIDIgNCAzIDIgNDt9 CkBmb250LWZhY2UKCXtmb250LWZhbWlseTpUYWhvbWE7CglwYW5vc2UtMToyIDExIDYgNCAzIDUg NCA0IDIgNDt9CkBmb250LWZhY2UKCXtmb250LWZhbWlseTpTaW1TdW47CglwYW5vc2UtMToyIDEg NiAwIDMgMSAxIDEgMSAxO30KLyogU3R5bGUgRGVmaW5pdGlvbnMgKi8KcC5Nc29Ob3JtYWwsIGxp Lk1zb05vcm1hbCwgZGl2Lk1zb05vcm1hbAoJe21hcmdpbjowaW47CgltYXJnaW4tYm90dG9tOi4w MDAxcHQ7Cglmb250LXNpemU6MTEuMHB0OwoJZm9udC1mYW1pbHk6IkNhbGlicmkiLCJzYW5zLXNl cmlmIjt9CmE6bGluaywgc3Bhbi5Nc29IeXBlcmxpbmsKCXttc28tc3R5bGUtcHJpb3JpdHk6OTk7 Cgljb2xvcjpibHVlOwoJdGV4dC1kZWNvcmF0aW9uOnVuZGVybGluZTt9CmE6dmlzaXRlZCwgc3Bh bi5Nc29IeXBlcmxpbmtGb2xsb3dlZAoJe21zby1zdHlsZS1wcmlvcml0eTo5OTsKCWNvbG9yOnB1 cnBsZTsKCXRleHQtZGVjb3JhdGlvbjp1bmRlcmxpbmU7fQpwLk1zb0FjZXRhdGUsIGxpLk1zb0Fj ZXRhdGUsIGRpdi5Nc29BY2V0YXRlCgl7bXNvLXN0eWxlLXByaW9yaXR5Ojk5OwoJbXNvLXN0eWxl LWxpbms6IkJhbGxvb24gVGV4dCBDaGFyIjsKCW1hcmdpbjowaW47CgltYXJnaW4tYm90dG9tOi4w MDAxcHQ7Cglmb250LXNpemU6OC4wcHQ7Cglmb250LWZhbWlseToiVGFob21hIiwic2Fucy1zZXJp ZiI7fQpzcGFuLkVtYWlsU3R5bGUxNwoJe21zby1zdHlsZS10eXBlOnBlcnNvbmFsOwoJZm9udC1m YW1pbHk6IkNhbGlicmkiLCJzYW5zLXNlcmlmIjsKCWNvbG9yOndpbmRvd3RleHQ7fQpzcGFuLkVt YWlsU3R5bGUxOAoJe21zby1zdHlsZS10eXBlOnBlcnNvbmFsOwoJZm9udC1mYW1pbHk6IkNhbGli cmkiLCJzYW5zLXNlcmlmIjsKCWNvbG9yOiMxRjQ5N0Q7fQpzcGFuLkJhbGxvb25UZXh0Q2hhcgoJ e21zby1zdHlsZS1uYW1lOiJCYWxsb29uIFRleHQgQ2hhciI7Cgltc28tc3R5bGUtcHJpb3JpdHk6 OTk7Cgltc28tc3R5bGUtbGluazoiQmFsbG9vbiBUZXh0IjsKCWZvbnQtZmFtaWx5OiJUYWhvbWEi LCJzYW5zLXNlcmlmIjt9CnNwYW4uRW1haWxTdHlsZTIxCgl7bXNvLXN0eWxlLXR5cGU6cGVyc29u YWwtcmVwbHk7Cglmb250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiOwoJY29sb3I6IzFG NDk3RDt9Ci5Nc29DaHBEZWZhdWx0Cgl7bXNvLXN0eWxlLXR5cGU6ZXhwb3J0LW9ubHk7Cglmb250 LXNpemU6MTAuMHB0O30KQHBhZ2UgV29yZFNlY3Rpb24xCgl7c2l6ZTo4LjVpbiAxMS4waW47Cglt YXJnaW46MS4waW4gMS4yNWluIDEuMGluIDEuMjVpbjt9CmRpdi5Xb3JkU2VjdGlvbjEKCXtwYWdl OldvcmRTZWN0aW9uMTt9Ci0tPjwvc3R5bGU+PCEtLVtpZiBndGUgbXNvIDldPjx4bWw+CjxvOnNo YXBlZGVmYXVsdHMgdjpleHQ9M0QiZWRpdCIgc3BpZG1heD0zRCIxMDI2IiAvPgo8L3htbD48IVtl bmRpZl0tLT48IS0tW2lmIGd0ZSBtc28gOV0+PHhtbD4KPG86c2hhcGVsYXlvdXQgdjpleHQ9M0Qi ZWRpdCI+CjxvOmlkbWFwIHY6ZXh0PTNEImVkaXQiIGRhdGE9M0QiMSIgLz4KPC9vOnNoYXBlbGF5 b3V0PjwveG1sPjwhW2VuZGlmXS0tPgo8L2hlYWQ+Cjxib2R5IGxhbmc9M0QiRU4tVVMiIGxpbms9 M0QiYmx1ZSIgdmxpbms9M0QicHVycGxlIj4KPGRpdiBjbGFzcz0zRCJXb3JkU2VjdGlvbjEiPgo8 cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIHN0eWxlPTNEImNvbG9yOiMxRjQ5N0QiPkhpIERh bi9GcmFuY2VzY28sPG86cD49CjwvbzpwPjwvc3Bhbj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1h bCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+PG86cD4mbmJzcDs8L286cD48L3NwYT0K bj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3 RCI+Q291bGQgeW91IGhlbHAgdG8gcmV2aT0KZXcgYW5kIG1lcmdlIHRoZXNlIHBhdGNoZXM6PG86 cD48L286cD48L3NwYW4+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIHN0eWxlPTNE ImNvbG9yOiMxRjQ5N0QiPjxhIGhyZWY9M0QiaHR0cDovL2dlcnI9Cml0Lm92aXJ0Lm9yZy8jL2Mv Mjc1MTUvIj5odHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8jL2MvMjc1MTUvPC9hPiZuYnNwOyAoVGhp cz0KIHBhdGNoIGlzIG5lZWRlZCBieSAyNjg3NiBzaW5jZSBzb21lIGNvbW1lbnRzIGluIHRoYXQg cGF0Y2ggbmVlZCB0byBtb2RpZnkgPQpjYXBzIG1vZHVsZSk8bzpwPjwvbzpwPjwvc3Bhbj48L3A+ CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+PGEg aHJlZj0zRCJodHRwOi8vZ2Vycj0KaXQub3ZpcnQub3JnLyMvYy8yNjg3Ni8iPmh0dHA6Ly9nZXJy aXQub3ZpcnQub3JnLyMvYy8yNjg3Ni88L2E+PG86cD48L286cD48PQovc3Bhbj48L3A+CjxwIGNs YXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+PGEgaHJlZj0z RCJodHRwOi8vZ2Vycj0KaXQub3ZpcnQub3JnLyMvYy8yNzQwMy8iPmh0dHA6Ly9nZXJyaXQub3Zp cnQub3JnLyMvYy8yNzQwMy88L2E+PG86cD48L286cD48PQovc3Bhbj48L3A+CjxwIGNsYXNzPTNE Ik1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+PG86cD4mbmJzcDs8L286 cD48L3NwYT0Kbj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29s b3I6IzFGNDk3RCI+SSBkaWRuJiM4MjE3O3QgbW9kaWZ5ID0Kc29tZSBjb21tZW50cyBpbiAyNjg3 NiBzaW5jZSBJIHRoaW5rIHRoZXkgYXJlIHJlbGF0ZWQgd2l0aCBjb2RlcyByZWZhY3RvciBhPQpu ZCBsaXRlcmFsIHN5bnRheC4gSSB3aWxsIHN1Ym1pdCBhIHNlcGFyYXRlIHBhdGNoIHRvIG1vZGlm eSB0aGVzZSBraW5kcyBvZiA9CmNvbW1lbnRzIG9mIG51bWEgZmVhdHVyZSBhbHRvZ2V0aGVyIGxh dGVyLgogQ291bGQgeW91IHBsZWFzZSBub3cgZm9jdXMgb24gdGhlIGZ1bmN0aW9uYWxpdHkgc2lu Y2Ugd2UgbmVlZCB0byBtZXJnZSB0aGU9CnNlIHBhdGNoZXMgQVNBUD88bzpwPjwvbzpwPjwvc3Bh bj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3 RCI+PG86cD4mbmJzcDs8L286cD48L3NwYT0Kbj48L3A+CjxkaXY+CjxwIGNsYXNzPTNEIk1zb05v cm1hbCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9u dD0KLXNpemU6MTIuMHB0O2NvbG9yOiMxRjQ5N0QiPlRoYW5rcyAmYW1wOyBCZXN0IFJlZ2FyZHM8 L3NwYW4+PC9iPjxzcGFuIHN0eWxlPQo9M0QiZm9udC1zaXplOjEyLjBwdDtjb2xvcjojMUY0OTdE Ij48bzpwPjwvbzpwPjwvc3Bhbj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCIgc3R5bGU9M0Qi dGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9udD0KLXNpemU6MTIuMHB0 O2NvbG9yOiMxRjQ5N0QiPlNoaSwgWGlhby1MZWkgKEJydWNlKTxvOnA+PC9vOnA+PC9zcGFuPjwv Yj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9u ZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9udD0KLXNpemU6MTIuMHB0O2NvbG9yOiMxRjQ5N0QiPjxv OnA+Jm5ic3A7PC9vOnA+PC9zcGFuPjwvYj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCIgc3R5 bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9udD0KLXNpemU6 MTIuMHB0O2NvbG9yOmJsYWNrIj5IZXdsZXR0LVBhY2thcmQgQ28uLCBMdGQuPGJyPgpIUCBTZXJ2 ZXJzIENvcmUgUGxhdGZvcm0gU29mdHdhcmUgQ2hpbmE8L3NwYW4+PC9iPjxiPjxzcGFuIHN0eWxl PTNEImZvbnQtc2k9CnplOjEyLjBwdDtjb2xvcjojMUY0OTdEIj4KPG86cD48L286cD48L3NwYW4+ PC9iPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0ZXh0LWF1dG9zcGFjZTpu b25lIj48Yj48c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4wcHQ7Y29sb3I6IzFGNDk3RCI+ VGVsZXBob25lPC9zcGFuPjwvYj48c3BhbiBzdHlsZT0zRCJmb250LXNpemU6MTI9Ci4wcHQ7Y29s b3I6IzFGNDk3RCI+ICYjNDM7ODYgMjMgNjU2ODMwOTM8bzpwPjwvbzpwPjwvc3Bhbj48L3A+Cjxw IGNsYXNzPTNEIk1zb05vcm1hbCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNw YW4gc3R5bGU9M0QiZm9udD0KLXNpemU6MTIuMHB0O2NvbG9yOiMxRjQ5N0QiPk1vYmlsZQo8L3Nw YW4+PC9iPjxzcGFuIHN0eWxlPTNEImZvbnQtc2l6ZToxMi4wcHQ7Y29sb3I6IzFGNDk3RCI+JiM0 Mzs4NiAxODY5NjU4MzQ9CjQ3PG86cD48L286cD48L3NwYW4+PC9wPgo8cCBjbGFzcz0zRCJNc29O b3JtYWwiPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQtc2l6ZToxMi4wcHQ7Y29sb3I6IzFGNDk3RCI+ RW09CmFpbCA8L3NwYW4+CjwvYj48c3BhbiBzdHlsZT0zRCJmb250LXNpemU6MTIuMHB0O2NvbG9y OiMxRjQ5N0QiPjxhIGhyZWY9M0QibWFpbHRvOnNoaWd1bz0KLmxpQGhwLmNvbSI+eGlhby1sZWku c2hpQGhwLmNvbTwvYT48L3NwYW4+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+PG86PQpw PjwvbzpwPjwvc3Bhbj48L3A+CjwvZGl2Pgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIHN0 eWxlPTNEImNvbG9yOiMxRjQ5N0QiPjxvOnA+Jm5ic3A7PC9vOnA+PC9zcGE9Cm4+PC9wPgo8ZGl2 Pgo8ZGl2IHN0eWxlPTNEImJvcmRlcjpub25lO2JvcmRlci10b3A6c29saWQgI0I1QzRERiAxLjBw dDtwYWRkaW5nOjMuMHB0IDBpbiA9CjBpbiAwaW4iPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxi PjxzcGFuIHN0eWxlPTNEImZvbnQtc2l6ZToxMC4wcHQ7Zm9udC1mYW1pbHk6JnF1b3Q9CjtUYWhv bWEmcXVvdDssJnF1b3Q7c2Fucy1zZXJpZiZxdW90OyI+RnJvbTo8L3NwYW4+PC9iPjxzcGFuIHN0 eWxlPTNEImZvbnQtcz0KaXplOjEwLjBwdDtmb250LWZhbWlseTomcXVvdDtUYWhvbWEmcXVvdDss JnF1b3Q7c2Fucy1zZXJpZiZxdW90OyI+IExpYW5nLCBTPQpoYW5nLUNodW4gKERhdmlkIExpYW5n LCBIUHNlcnZlcnMtQ29yZS1PRS1QU0MpCjxicj4KPGI+U2VudDo8L2I+IFRodXJzZGF5LCBNYXkg MDgsIDIwMTQgMTA6MzUgQU08YnI+CjxiPlRvOjwvYj4gU2hpLCBYaWFvLUxlaSAoQnJ1Y2UsIEhQ IFNlcnZlcnMtUFNDLUNRKTsgTWFydGluIFNpdmFrIChtc2l2YWtAcj0KZWRoYXQuY29tKTsgRGFu IEtlbmlnc2JlcmcgKGRhbmtlbkByZWRoYXQuY29tKTxicj4KPGI+Q2M6PC9iPiBHaWxhZCBDaGFw bGlrICZsdDtnY2hhcGxpa0ByZWRoYXQuY29tJmd0OyAoZ2NoYXBsaWtAcmVkaGF0LmNvbSk7PQog RG9yb24gRmVkaXVjayAoZGZlZGl1Y2tAcmVkaGF0LmNvbSk7IExpYW8sIENodWFuIChKYXNvbiBM aWFvLCBIUHNlcnZlcnMtQ289CnJlLU9FLVBTQyk7IGRldmVsQG92aXJ0Lm9yZzxicj4KPGI+U3Vi amVjdDo8L2I+IFJFOiBWZHNtIHBhdGNoZXMgbmVlZCByZXZpZXcgYW5kIG1lcmdlPG86cD48L286 cD48L3NwYW4+PC9wPQo+CjwvZGl2Pgo8L2Rpdj4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48bzpw PiZuYnNwOzwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBzdHlsZT0zRCJj b2xvcjojMUY0OTdEIj5IaSBEYW4sPG86cD48L286cD48L3NwPQphbj48L3A+CjxwIGNsYXNzPTNE Ik1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+SSBoZWFyZCB0aGF0IE1h cnRpbiBpcz0KIGluIFRQTyBub3csIHdvdWxkIHlvdSBwbGVhc2UgdGFrZSBhIGxvb2sgb24gdGhl c2UgcGF0Y2hlcyBhbmQgbWVyZ2UgaXQsIHRoPQphbmtzLjxvOnA+PC9vOnA+PC9zcGFuPjwvcD4K PHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBzdHlsZT0zRCJjb2xvcjojMUY0OTdEIj48bzpw PiZuYnNwOzwvbzpwPjwvc3BhPQpuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBz dHlsZT0zRCJjb2xvcjojMUY0OTdEIj5CZXN0IHJlZ2FyZHMsPG86cD48L286PQpwPjwvc3Bhbj48 L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gc3R5bGU9M0QiY29sb3I6IzFGNDk3RCI+ RGF2aWQgTGlhbmc8bzpwPjwvbzpwPj0KPC9zcGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFs Ij48c3BhbiBzdHlsZT0zRCJjb2xvcjojMUY0OTdEIj48bzpwPiZuYnNwOzwvbzpwPjwvc3BhPQpu PjwvcD4KPGRpdiBzdHlsZT0zRCJib3JkZXI6bm9uZTtib3JkZXItbGVmdDpzb2xpZCBibHVlIDEu NXB0O3BhZGRpbmc6MGluIDBpbiAwaW4gPQo0LjBwdCI+CjxkaXY+CjxkaXYgc3R5bGU9M0QiYm9y ZGVyOm5vbmU7Ym9yZGVyLXRvcDpzb2xpZCAjRTFFMUUxIDEuMHB0O3BhZGRpbmc6My4wcHQgMGlu ID0KMGluIDBpbiI+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PGI+RnJvbTo8L2I+IFNoaSwgWGlh by1MZWkgKEJydWNlLCBIUCBTZXJ2ZXJzLVBTQy1DUT0KKSA8YnI+CjxiPlNlbnQ6PC9iPiBUaHVy c2RheSwgTWF5IDA4LCAyMDE0IDEwOjMxIEFNPGJyPgo8Yj5Ubzo8L2I+IE1hcnRpbiBTaXZhayAo PGEgaHJlZj0zRCJtYWlsdG86bXNpdmFrQHJlZGhhdC5jb20iPm1zaXZha0ByZWRoYXQ9Ci5jb208 L2E+KTsgRGFuIEtlbmlnc2JlcmcgKDxhIGhyZWY9M0QibWFpbHRvOmRhbmtlbkByZWRoYXQuY29t Ij5kYW5rZW5AcmVkaD0KYXQuY29tPC9hPik8YnI+CjxiPkNjOjwvYj4gR2lsYWQgQ2hhcGxpayAm bHQ7PGEgaHJlZj0zRCJtYWlsdG86Z2NoYXBsaWtAcmVkaGF0LmNvbSI+Z2NoYXBsaT0Ka0ByZWRo YXQuY29tPC9hPiZndDsgKDxhIGhyZWY9M0QibWFpbHRvOmdjaGFwbGlrQHJlZGhhdC5jb20iPmdj aGFwbGlrQHJlZGhhPQp0LmNvbTwvYT4pOyBEb3JvbiBGZWRpdWNrICg8YSBocmVmPTNEIm1haWx0 bzpkZmVkaXVja0ByZWRoYXQuY29tIj5kZmVkaXVja0A9CnJlZGhhdC5jb208L2E+KTsgTGlhbmcs IFNoYW5nLUNodW4gKERhdmlkIExpYW5nLAogSFBzZXJ2ZXJzLUNvcmUtT0UtUFNDKTsgTGlhbywg Q2h1YW4gKEphc29uIExpYW8sIEhQc2VydmVycy1Db3JlLU9FLVBTQyk7IDw9CmEgaHJlZj0zRCJt YWlsdG86ZGV2ZWxAb3ZpcnQub3JnIj4KZGV2ZWxAb3ZpcnQub3JnPC9hPjxicj4KPGI+U3ViamVj dDo8L2I+IFZkc20gcGF0Y2hlcyBuZWVkIHJldmlldyBhbmQgbWVyZ2U8bzpwPjwvbzpwPjwvcD4K PC9kaXY+CjwvZGl2Pgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxvOnA+Jm5ic3A7PC9vOnA+PC9w Pgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPkhpIE1hcnRpbiw8bzpwPjwvbzpwPjwvcD4KPHAgY2xh c3M9M0QiTXNvTm9ybWFsIj5Db3VsZCB5b3UgaGVscCB0byByZXZpZXcgdGhlc2UgbnVtYSB2ZHNt IHBhdGNoZXM6PG86PQpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48YSBocmVm PTNEImh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzM5OS8iPmh0dHA6PQovL2dlcnJpdC5v dmlydC5vcmcvIy9jLzI3Mzk5LzwvYT48bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9y bWFsIj48YSBocmVmPTNEImh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8yNzA5MC8iPmh0dHA6 PQovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3MDkwLzwvYT48bzpwPjwvbzpwPjwvcD4KPHAgY2xh c3M9M0QiTXNvTm9ybWFsIj48YSBocmVmPTNEImh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLyMvYy8y NzQwMy8iPmh0dHA6PQovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI3NDAzLzwvYT48bzpwPjwvbzpw PjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48YSBocmVmPTNEImh0dHA6Ly9nZXJyaXQub3Zp cnQub3JnLyMvYy8yNjg3Ni8iPmh0dHA6PQovL2dlcnJpdC5vdmlydC5vcmcvIy9jLzI2ODc2Lzwv YT48bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj5Zb3UgaGFkIGFscmVhZHkg cmV2aWV3ZWQgc29tZSBvZiB0aGUgcGF0Y2hlcywgYnV0IGFmPQp0ZXIgdGhhdCBJIG1vZGlmaWVk IHRoZSB0cmFuc2ZlcnJlZCBkYXRhIGZvcm1hdCBiZWNhdXNlIG9mIHRoZSB4bWxycGMgbGltaXQ9 CmF0aW9ucy4gU28gY291bGQgeW91IHBsZWFzZSByZXZpZXcgdGhlbSBhZ2Fpbj88bzpwPjwvbzpw PjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48bzpwPiZuYnNwOzwvbzpwPjwvcD4KPHAgY2xh c3M9M0QiTXNvTm9ybWFsIj5IaSBEYW4sPG86cD48L286cD48L3A+CjxwIGNsYXNzPTNEIk1zb05v cm1hbCI+SSBoYXZlIG1vZGlmaWVkIHlvdXIgY29tbWVudHMgZm9yIHNvbWUgb2YgdGhlIGFib3Zl ID0KcGF0Y2hlcy4gQ291bGQgeW91IHBsZWFzZSB0YWtlIGEgbG9vayBhbmQgbWVyZ2UgdGhvc2Ug aWYgbm8gbW9yZSBjb21tZW50cz88PQpvOnA+PC9vOnA+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3Jt YWwiPjxvOnA+Jm5ic3A7PC9vOnA+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiIHN0eWxlPTNE InRleHQtYXV0b3NwYWNlOm5vbmUiPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQ9Ci1zaXplOjEyLjBw dCI+VGhhbmtzICZhbXA7IEJlc3QgUmVnYXJkczwvc3Bhbj48L2I+PHNwYW4gc3R5bGU9M0QiZm9u dC1zaXplOj0KMTIuMHB0Ij48bzpwPjwvbzpwPjwvc3Bhbj48L3A+CjxwIGNsYXNzPTNEIk1zb05v cm1hbCIgc3R5bGU9M0QidGV4dC1hdXRvc3BhY2U6bm9uZSI+PGI+PHNwYW4gc3R5bGU9M0QiZm9u dD0KLXNpemU6MTIuMHB0Ij5TaGksIFhpYW8tTGVpIChCcnVjZSk8bzpwPjwvbzpwPjwvc3Bhbj48 L2I+PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiIHN0eWxlPTNEInRleHQtYXV0b3NwYWNlOm5v bmUiPjxiPjxzcGFuIHN0eWxlPTNEImZvbnQ9Ci1zaXplOjEyLjBwdCI+PG86cD4mbmJzcDs8L286 cD48L3NwYW4+PC9iPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0ZXh0LWF1 dG9zcGFjZTpub25lIj48Yj48c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4wcHQ7Y29sb3I6 YmxhY2siPkhld2xldHQtUGFja2FyZCBDby4sIEx0ZC48YnI+CkhQIFNlcnZlcnMgQ29yZSBQbGF0 Zm9ybSBTb2Z0d2FyZSBDaGluYTwvc3Bhbj48L2I+PGI+PHNwYW4gc3R5bGU9M0QiZm9udC1zaT0K emU6MTIuMHB0Ij4KPG86cD48L286cD48L3NwYW4+PC9iPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9y bWFsIiBzdHlsZT0zRCJ0ZXh0LWF1dG9zcGFjZTpub25lIj48Yj48c3BhbiBzdHlsZT0zRCJmb250 PQotc2l6ZToxMi4wcHQiPlRlbGVwaG9uZTwvc3Bhbj48L2I+PHNwYW4gc3R5bGU9M0QiZm9udC1z aXplOjEyLjBwdCI+ICYjNDM7ODY9CiAyMyA2NTY4MzA5MzxvOnA+PC9vOnA+PC9zcGFuPjwvcD4K PHAgY2xhc3M9M0QiTXNvTm9ybWFsIiBzdHlsZT0zRCJ0ZXh0LWF1dG9zcGFjZTpub25lIj48Yj48 c3BhbiBzdHlsZT0zRCJmb250PQotc2l6ZToxMi4wcHQiPk1vYmlsZQo8L3NwYW4+PC9iPjxzcGFu IHN0eWxlPTNEImZvbnQtc2l6ZToxMi4wcHQiPiYjNDM7ODYgMTg2OTY1ODM0NDc8bzpwPjwvbzpw Pjw9Ci9zcGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48Yj48c3BhbiBzdHlsZT0zRCJm b250LXNpemU6MTIuMHB0Ij5FbWFpbCA8L3NwYW4+PC9iPQo+PHNwYW4gc3R5bGU9M0QiZm9udC1z aXplOjEyLjBwdCI+PGEgaHJlZj0zRCJtYWlsdG86c2hpZ3VvLmxpQGhwLmNvbSI+eGlhby09Cmxl aS5zaGlAaHAuY29tPC9hPjwvc3Bhbj48bzpwPjwvbzpwPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9y bWFsIj48bzpwPiZuYnNwOzwvbzpwPjwvcD4KPC9kaXY+CjwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4K Ci0tXzAwMF8wOEFBNDAzQzgzOTkxMDRBODlBRjcxMDMwN0ZBNzhBRTI0NDA0QUMxRzVXMjc0M2Ft ZXJpY2FzXy0tCg== --===============5088316889904542478==-- From fromani at redhat.com Thu May 8 08:48:53 2014 Content-Type: multipart/mixed; boundary="===============5446166163908915802==" MIME-Version: 1.0 From: Francesco Romani To: devel at ovirt.org Subject: Re: [ovirt-devel] Vdsm patches need review and merge Date: Thu, 08 May 2014 08:48:52 -0400 Message-ID: <1509926854.1928900.1399553332028.JavaMail.zimbra@redhat.com> In-Reply-To: 08AA403C8399104A89AF710307FA78AE24404AC1@G5W2743.americas.hpqcorp.net --===============5446166163908915802== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----- Original Message ----- > From: "Xiao-Lei Shi (Bruce, HP Servers-PSC-CQ)" > To: "Dan Kenigsberg (danken(a)redhat.com)" , "France= sco Romani" > Cc: "Gilad Chaplik (gchaplik(a)redhat.com)" , "Doron Fediuck > (dfediuck(a)redhat.com)" , "Chuan Liao (Jason Liao= , HPservers-Core-OE-PSC)" , > devel(a)ovirt.org, "Shang-Chun Liang (David Liang, HPservers-Core-OE-PSC)= " > Sent: Thursday, May 8, 2014 2:33:14 PM > Subject: RE: Vdsm patches need review and merge > = > Hi Dan/Francesco, > = > Could you help to review and merge these patches: > http://gerrit.ovirt.org/#/c/27515/ (This patch is needed by 26876 since = some > comments in that patch need to modify caps module) > http://gerrit.ovirt.org/#/c/26876/ > http://gerrit.ovirt.org/#/c/27403/ > = > I didn't modify some comments in 26876 since I think they are related with > codes refactor and literal syntax. I will submit a separate patch to modi= fy > these kinds of comments of numa feature altogether later. Could you please > now focus on the functionality since we need to merge these patches ASAP? http://gerrit.ovirt.org/#/c/27403 I'm OK with it (gave +1) http://gerrit.ovirt.org/#/c/26876 I don't have objections and I don't see blockers for the functionality. I'm not 100% happy with the implementation, but the existing one is on par with most of code, and if we're in hurry, I guess refinements can wait. http://gerrit.ovirt.org/#/c/27515 good enough for the same reasons above (gave +1) Bests, -- = Francesco Romani RedHat Engineering Virtualization R & D Phone: 8261328 IRC: fromani --===============5446166163908915802==-- From danken at redhat.com Thu May 8 10:02:53 2014 Content-Type: multipart/mixed; boundary="===============1540526001219539960==" MIME-Version: 1.0 From: Dan Kenigsberg To: devel at ovirt.org Subject: Re: [ovirt-devel] Vdsm patches need review and merge Date: Thu, 08 May 2014 15:02:44 +0100 Message-ID: <20140508140244.GJ7060@redhat.com> In-Reply-To: 1509926854.1928900.1399553332028.JavaMail.zimbra@redhat.com --===============1540526001219539960== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Thu, May 08, 2014 at 08:48:52AM -0400, Francesco Romani wrote: > = > ----- Original Message ----- > > From: "Xiao-Lei Shi (Bruce, HP Servers-PSC-CQ)" > > To: "Dan Kenigsberg (danken(a)redhat.com)" , "Fran= cesco Romani" > > Cc: "Gilad Chaplik (gchaplik(a)redhat.com)" , "Doron Fediuck > > (dfediuck(a)redhat.com)" , "Chuan Liao (Jason Li= ao, HPservers-Core-OE-PSC)" , > > devel(a)ovirt.org, "Shang-Chun Liang (David Liang, HPservers-Core-OE-PS= C)" > > Sent: Thursday, May 8, 2014 2:33:14 PM > > Subject: RE: Vdsm patches need review and merge > > = > > Hi Dan/Francesco, > > = > > Could you help to review and merge these patches: > > http://gerrit.ovirt.org/#/c/27515/ (This patch is needed by 26876 sinc= e some > > comments in that patch need to modify caps module) > > http://gerrit.ovirt.org/#/c/26876/ > > http://gerrit.ovirt.org/#/c/27403/ > > = > > I didn't modify some comments in 26876 since I think they are related w= ith > > codes refactor and literal syntax. I will submit a separate patch to mo= dify > > these kinds of comments of numa feature altogether later. Could you ple= ase > > now focus on the functionality since we need to merge these patches ASA= P? > = > http://gerrit.ovirt.org/#/c/27403 > I'm OK with it (gave +1) > = > http://gerrit.ovirt.org/#/c/26876 > I don't have objections and I don't see blockers for the functionality. > I'm not 100% happy with the implementation, but the existing one is on par > with most of code, and if we're in hurry, I guess refinements can wait. > = > http://gerrit.ovirt.org/#/c/27515 > good enough for the same reasons above (gave +1) Puritans would suggest to squash this into 26876, or fix the commit message - prior to 26876 nothing uses these private methods. But never mind, taken. --===============1540526001219539960==-- From fromani at redhat.com Fri May 16 10:32:38 2014 Content-Type: multipart/mixed; boundary="===============1021572731760777169==" MIME-Version: 1.0 From: Francesco Romani To: devel at ovirt.org Subject: Re: [ovirt-devel] Vdsm patches need review and merge Date: Fri, 16 May 2014 10:32:35 -0400 Message-ID: <192920720.5572628.1400250755266.JavaMail.zimbra@redhat.com> In-Reply-To: 20140508140244.GJ7060@redhat.com --===============1021572731760777169== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Hi, Unfortunately, we just found = https://bugzilla.redhat.com/show_bug.cgi?id=3D1098539 This looks like an engine bug, however. Can someone please check and help with a fix? Thanks, ----- Original Message ----- > From: "Dan Kenigsberg" > To: "Francesco Romani" > Cc: "Xiao-Lei Shi (Bruce, HP Servers-PSC-CQ)" , "G= ilad Chaplik(gchaplik(a)redhat.com)" > , "Doron Fediuck (dfediuck(a)redhat.com)" , "Chuan Liao (Jason Liao, > HPservers-Core-OE-PSC)" , devel(a)ovirt.org, "Shang-= Chun Liang (David Liang, > HPservers-Core-OE-PSC)" > Sent: Thursday, May 8, 2014 4:02:44 PM > Subject: Re: Vdsm patches need review and merge > = > On Thu, May 08, 2014 at 08:48:52AM -0400, Francesco Romani wrote: > > = > > ----- Original Message ----- > > > From: "Xiao-Lei Shi (Bruce, HP Servers-PSC-CQ)" > > > To: "Dan Kenigsberg (danken(a)redhat.com)" , "Fr= ancesco > > > Romani" > > > Cc: "Gilad Chaplik (gchaplik(a)redhat.com)" > > > , "Doron Fediuck > > > (dfediuck(a)redhat.com)" , "Chuan Liao (Jason = Liao, > > > HPservers-Core-OE-PSC)" , > > > devel(a)ovirt.org, "Shang-Chun Liang (David Liang, HPservers-Core-OE-= PSC)" > > > > > > Sent: Thursday, May 8, 2014 2:33:14 PM > > > Subject: RE: Vdsm patches need review and merge > > > = > > > Hi Dan/Francesco, > > > = > > > Could you help to review and merge these patches: > > > http://gerrit.ovirt.org/#/c/27515/ (This patch is needed by 26876 si= nce > > > some > > > comments in that patch need to modify caps module) > > > http://gerrit.ovirt.org/#/c/26876/ > > > http://gerrit.ovirt.org/#/c/27403/ > > > = > > > I didn't modify some comments in 26876 since I think they are related > > > with > > > codes refactor and literal syntax. I will submit a separate patch to > > > modify > > > these kinds of comments of numa feature altogether later. Could you > > > please > > > now focus on the functionality since we need to merge these patches A= SAP? > > = > > http://gerrit.ovirt.org/#/c/27403 > > I'm OK with it (gave +1) > > = > > http://gerrit.ovirt.org/#/c/26876 > > I don't have objections and I don't see blockers for the functionality. > > I'm not 100% happy with the implementation, but the existing one is on = par > > with most of code, and if we're in hurry, I guess refinements can wait. > > = > > http://gerrit.ovirt.org/#/c/27515 > > good enough for the same reasons above (gave +1) > = > Puritans would suggest to squash this into 26876, or fix the commit > message - prior to 26876 nothing uses these private methods. But never > mind, taken. > = -- = Francesco Romani RedHat Engineering Virtualization R & D Phone: 8261328 IRC: fromani --===============1021572731760777169==--