From xiao-lei.shi at hp.com Wed May 7 22:31:57 2014 Content-Type: multipart/mixed; boundary="===============1532176888576765907==" 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> --===============1532176888576765907== 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_-- --===============1532176888576765907== 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== --===============1532176888576765907==--