From fabrice.bacchella at orange.fr Sun Apr 9 09:14:40 2017 Content-Type: multipart/mixed; boundary="===============0957308926250045545==" MIME-Version: 1.0 From: Fabrice Bacchella To: users at ovirt.org Subject: Re: [ovirt-users] SKD4 Date: Sun, 09 Apr 2017 11:14:35 +0200 Message-ID: <94B1D56E-ADBF-47A7-800A-BE7426C3DBA9@orange.fr> In-Reply-To: CAJCqMsU0ocEQGHTWMMvdO4Kc2m9gAqLSDWt1yDSRiQk+85o6wQ@mail.gmail.com --===============0957308926250045545== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable --Apple-Mail=3D_E71938C9-E743-4136-A405-57ECD65F94EE Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=3Dutf-8 > Le 6 avr. 2017 =3DC3=3DA0 17:21, Ondra Machacek a= =3D =3DC3=3DA9crit : >=3D20 >=3D20 >=3D20 > On Thu, Apr 6, 2017 at 5:00 PM, Fabrice Bacchella =3D > =3D wrote: > Ho my good, in ovirtsdk.services.py , =3D for every service, there is a different method with a different name =3D that return a associated service, so it's not possible to have a generic = =3D like: >=3D20 > def resolve(service, ...): > id =3D3D ..... > return service.service(id) >=3D20 > because the generic call service is used by something that take a path = =3D argument. But why not a service_by_id(self, id) ? >=3D20 > I am not fully sure I understand what you are missing, but feel free =3D to open > the bug on Python SDK in bugzilla, we will be happy to improve the =3D SDK. I tried : https://bugzilla.redhat.com/show_bug.cgi?id=3D3D1439879 =3D I hit a wall. it seems that some of you are not willing to improve the =3D SDK. --Apple-Mail=3D_E71938C9-E743-4136-A405-57ECD65F94EE Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=3Dutf-8
Le 6 avr. 2017 =3DC3=3DA0 17:21, Ondra Machacek <omachace(a)redhat.com<= /a>>=3D a =3DC3=3DA9crit :



On Th= u, =3D Apr 6, 2017 at 5:00 PM, Fabrice Bacchella <fabrice.bacchella(a)orange.fr> =3D wrote:
Ho my good, in ovirtsdk.services.py, for every service, there is a =3D different method with a different name that return a associated service, = =3D so it's not possible to have a generic like:

def  resolve(service, =3D ...):
id =3D3D .....
return =3D service.service(id)

because the generic call service is used by something that =3D take a path argument. But why not a service_by_id(self, id) ?

I am not fully sure I understand what = =3D you are missing, but feel free to open
the bug on Python = =3D SDK in bugzilla, we will be happy to improve the SDK.

I tried :
https://bugzilla.redhat.com/show_bug.cgi?id=3D3D1439879=3D

I hit a wall. it seems that some of you = =3D are not willing to improve the SDK.

=3D --Apple-Mail=3D_E71938C9-E743-4136-A405-57ECD65F94EE-- --===============0957308926250045545== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" Ci0tQXBwbGUtTWFpbD1fRTcxOTM4QzktRTc0My00MTM2LUE0MDUtNTdFQ0Q2NUY5NEVFCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKQ29udGVudC1UeXBlOiB0ZXh0 L3BsYWluOwoJY2hhcnNldD11dGYtOAoKCj4gTGUgNiBhdnIuIDIwMTcgPUMzPUEwIDE3OjIxLCBP bmRyYSBNYWNoYWNlayA8b21hY2hhY2VAcmVkaGF0LmNvbT4gYSA9Cj1DMz1BOWNyaXQgOgo+PTIw Cj49MjAKPj0yMAo+IE9uIFRodSwgQXByIDYsIDIwMTcgYXQgNTowMCBQTSwgRmFicmljZSBCYWNj aGVsbGEgPQo8ZmFicmljZS5iYWNjaGVsbGFAb3JhbmdlLmZyIDxtYWlsdG86ZmFicmljZS5iYWNj aGVsbGFAb3JhbmdlLmZyPj4gPQp3cm90ZToKPiBIbyBteSBnb29kLCBpbiBvdmlydHNkay5zZXJ2 aWNlcy5weSA8aHR0cDovL292aXJ0c2RrLnNlcnZpY2VzLnB5Lz4sID0KZm9yIGV2ZXJ5IHNlcnZp Y2UsIHRoZXJlIGlzIGEgZGlmZmVyZW50IG1ldGhvZCB3aXRoIGEgZGlmZmVyZW50IG5hbWUgPQp0 aGF0IHJldHVybiBhIGFzc29jaWF0ZWQgc2VydmljZSwgc28gaXQncyBub3QgcG9zc2libGUgdG8g aGF2ZSBhIGdlbmVyaWMgPQpsaWtlOgo+PTIwCj4gZGVmICByZXNvbHZlKHNlcnZpY2UsIC4uLik6 Cj4gCWlkID0zRCAuLi4uLgo+IAlyZXR1cm4gc2VydmljZS5zZXJ2aWNlKGlkKQo+PTIwCj4gYmVj YXVzZSB0aGUgZ2VuZXJpYyBjYWxsIHNlcnZpY2UgaXMgdXNlZCBieSBzb21ldGhpbmcgdGhhdCB0 YWtlIGEgcGF0aCA9CmFyZ3VtZW50LiBCdXQgd2h5IG5vdCBhIHNlcnZpY2VfYnlfaWQoc2VsZiwg aWQpID8KPj0yMAo+IEkgYW0gbm90IGZ1bGx5IHN1cmUgSSB1bmRlcnN0YW5kIHdoYXQgeW91IGFy ZSBtaXNzaW5nLCBidXQgZmVlbCBmcmVlID0KdG8gb3Blbgo+IHRoZSBidWcgb24gUHl0aG9uIFNE SyBpbiBidWd6aWxsYSwgd2Ugd2lsbCBiZSBoYXBweSB0byBpbXByb3ZlIHRoZSA9ClNESy4KCkkg dHJpZWQgOgpodHRwczovL2J1Z3ppbGxhLnJlZGhhdC5jb20vc2hvd19idWcuY2dpP2lkPTNEMTQz OTg3OSA9CjxodHRwczovL2J1Z3ppbGxhLnJlZGhhdC5jb20vc2hvd19idWcuY2dpP2lkPTNEMTQz OTg3OT4KCkkgaGl0IGEgd2FsbC4gaXQgc2VlbXMgdGhhdCBzb21lIG9mIHlvdSBhcmUgbm90IHdp bGxpbmcgdG8gaW1wcm92ZSB0aGUgPQpTREsuCgoKLS1BcHBsZS1NYWlsPV9FNzE5MzhDOS1FNzQz LTQxMzYtQTQwNS01N0VDRDY1Rjk0RUUKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogcXVvdGVk LXByaW50YWJsZQpDb250ZW50LVR5cGU6IHRleHQvaHRtbDsKCWNoYXJzZXQ9dXRmLTgKCjxodG1s PjxoZWFkPjxtZXRhIGh0dHAtZXF1aXY9M0QiQ29udGVudC1UeXBlIiBjb250ZW50PTNEInRleHQv aHRtbCA9CmNoYXJzZXQ9M0R1dGYtOCI+PC9oZWFkPjxib2R5IHN0eWxlPTNEIndvcmQtd3JhcDog YnJlYWstd29yZDsgPQotd2Via2l0LW5ic3AtbW9kZTogc3BhY2U7IC13ZWJraXQtbGluZS1icmVh azogYWZ0ZXItd2hpdGUtc3BhY2U7IiA9CmNsYXNzPTNEIiI+PGJyIGNsYXNzPTNEIiI+PGRpdj48 YmxvY2txdW90ZSB0eXBlPTNEImNpdGUiIGNsYXNzPTNEIiI+PGRpdiA9CmNsYXNzPTNEIiI+TGUg NiBhdnIuIDIwMTcgPUMzPUEwIDE3OjIxLCBPbmRyYSBNYWNoYWNlayAmbHQ7PGEgPQpocmVmPTNE Im1haWx0bzpvbWFjaGFjZUByZWRoYXQuY29tIiBjbGFzcz0zRCIiPm9tYWNoYWNlQHJlZGhhdC5j b208L2E+Jmd0Oz0KIGEgPUMzPUE5Y3JpdCA6PC9kaXY+PGJyIGNsYXNzPTNEIkFwcGxlLWludGVy Y2hhbmdlLW5ld2xpbmUiPjxkaXYgPQpjbGFzcz0zRCIiPjxkaXYgZGlyPTNEImx0ciIgY2xhc3M9 M0QiIj48YnIgY2xhc3M9M0QiIj48ZGl2ID0KY2xhc3M9M0QiZ21haWxfZXh0cmEiPjxiciBjbGFz cz0zRCIiPjxkaXYgY2xhc3M9M0QiZ21haWxfcXVvdGUiPk9uIFRodSwgPQpBcHIgNiwgMjAxNyBh dCA1OjAwIFBNLCBGYWJyaWNlIEJhY2NoZWxsYSA8c3BhbiBkaXI9M0QibHRyIiA9CmNsYXNzPTNE IiI+Jmx0OzxhIGhyZWY9M0QibWFpbHRvOmZhYnJpY2UuYmFjY2hlbGxhQG9yYW5nZS5mciIgPQp0 YXJnZXQ9M0QiX2JsYW5rIiBjbGFzcz0zRCIiPmZhYnJpY2UuYmFjY2hlbGxhQG9yYW5nZS5mcjwv YT4mZ3Q7PC9zcGFuPiA9Cndyb3RlOjxiciBjbGFzcz0zRCIiPjxibG9ja3F1b3RlIGNsYXNzPTNE ImdtYWlsX3F1b3RlIiBzdHlsZT0zRCJtYXJnaW46MCA9CjAgMCAuOGV4O2JvcmRlci1sZWZ0OjFw eCAjY2NjIHNvbGlkO3BhZGRpbmctbGVmdDoxZXgiPjxkaXYgPQpzdHlsZT0zRCJ3b3JkLXdyYXA6 YnJlYWstd29yZCIgY2xhc3M9M0QiIj5IbyBteSBnb29kLCBpbiA8YSA9CmhyZWY9M0QiaHR0cDov L292aXJ0c2RrLnNlcnZpY2VzLnB5LyIgdGFyZ2V0PTNEIl9ibGFuayIgPQpjbGFzcz0zRCIiPm92 aXJ0c2RrLnNlcnZpY2VzLnB5PC9hPiwgZm9yIGV2ZXJ5IHNlcnZpY2UsIHRoZXJlIGlzIGEgPQpk aWZmZXJlbnQgbWV0aG9kIHdpdGggYSBkaWZmZXJlbnQgbmFtZSB0aGF0IHJldHVybiBhIGFzc29j aWF0ZWQgc2VydmljZSwgPQpzbyBpdCdzIG5vdCBwb3NzaWJsZSB0byBoYXZlIGEgZ2VuZXJpYyBs aWtlOjxkaXYgY2xhc3M9M0QiIj48YnIgPQpjbGFzcz0zRCIiPjwvZGl2PjxkaXYgY2xhc3M9M0Qi Ij5kZWYgJm5ic3A7cmVzb2x2ZShzZXJ2aWNlLCA9Ci4uLik6PC9kaXY+PGRpdiBjbGFzcz0zRCIi PjxzcGFuID0KY2xhc3M9M0QibV8tNDgxNjg1OTYwNTI2ODk4MTkxOUFwcGxlLXRhYi1zcGFuIiA9 CnN0eWxlPTNEIndoaXRlLXNwYWNlOnByZS13cmFwIj4JPC9zcGFuPmlkID0zRCAuLi4uLjwvZGl2 PjxkaXYgPQpjbGFzcz0zRCIiPjxzcGFuIGNsYXNzPTNEIm1fLTQ4MTY4NTk2MDUyNjg5ODE5MTlB cHBsZS10YWItc3BhbiIgPQpzdHlsZT0zRCJ3aGl0ZS1zcGFjZTpwcmUtd3JhcCI+CTwvc3Bhbj5y ZXR1cm4gPQpzZXJ2aWNlLnNlcnZpY2UoaWQpPC9kaXY+PGRpdiBjbGFzcz0zRCIiPjxiciBjbGFz cz0zRCIiPjwvZGl2PjxkaXYgPQpjbGFzcz0zRCIiPmJlY2F1c2UgdGhlIGdlbmVyaWMgY2FsbCBz ZXJ2aWNlIGlzIHVzZWQgYnkgc29tZXRoaW5nIHRoYXQgPQp0YWtlIGEgcGF0aCBhcmd1bWVudC4g QnV0IHdoeSBub3QgYSBzZXJ2aWNlX2J5X2lkKHNlbGYsIGlkKSA/PGJyID0KY2xhc3M9M0QiIj48 L2Rpdj48L2Rpdj48L2Jsb2NrcXVvdGU+PGRpdiBjbGFzcz0zRCIiPjxiciA9CmNsYXNzPTNEIiI+ PC9kaXY+PGRpdiBjbGFzcz0zRCIiPkkgYW0gbm90IGZ1bGx5IHN1cmUgSSB1bmRlcnN0YW5kIHdo YXQgPQp5b3UgYXJlIG1pc3NpbmcsIGJ1dCBmZWVsIGZyZWUgdG8gb3BlbjxiciBjbGFzcz0zRCIi PnRoZSBidWcgb24gUHl0aG9uID0KU0RLIGluIGJ1Z3ppbGxhLCB3ZSB3aWxsIGJlIGhhcHB5IHRv IGltcHJvdmUgdGhlIFNESy48YnIgPQpjbGFzcz0zRCIiPjwvZGl2PjwvZGl2PjwvZGl2PjwvZGl2 PjwvZGl2PjwvYmxvY2txdW90ZT48YnIgPQpjbGFzcz0zRCIiPjwvZGl2PjxkaXY+SSB0cmllZCA6 PC9kaXY+PGRpdj48YSA9CmhyZWY9M0QiaHR0cHM6Ly9idWd6aWxsYS5yZWRoYXQuY29tL3Nob3df YnVnLmNnaT9pZD0zRDE0Mzk4NzkiID0KY2xhc3M9M0QiIj5odHRwczovL2J1Z3ppbGxhLnJlZGhh dC5jb20vc2hvd19idWcuY2dpP2lkPTNEMTQzOTg3OTwvYT48L2Rpdj49CjxkaXY+PGJyIGNsYXNz PTNEIiI+PC9kaXY+PGRpdj5JIGhpdCBhIHdhbGwuIGl0IHNlZW1zIHRoYXQgc29tZSBvZiB5b3Ug PQphcmUgbm90IHdpbGxpbmcgdG8gaW1wcm92ZSB0aGUgU0RLLjwvZGl2PjxiciBjbGFzcz0zRCIi PjwvYm9keT48L2h0bWw+PQoKLS1BcHBsZS1NYWlsPV9FNzE5MzhDOS1FNzQzLTQxMzYtQTQwNS01 N0VDRDY1Rjk0RUUtLQo= --===============0957308926250045545==--