From fabrice.bacchella at orange.fr Thu Apr 6 18:19:30 2017 Content-Type: multipart/mixed; boundary="===============6186509809701437944==" MIME-Version: 1.0 From: Fabrice Bacchella To: users at ovirt.org Subject: Re: [ovirt-users] SKD4 Date: Thu, 06 Apr 2017 20:19:25 +0200 Message-ID: In-Reply-To: CAJgorsbfgsUk1cPUyyMLJ6zKidifumT2JE+4FNJYHTcKNcZ5mQ@mail.gmail.com --===============6186509809701437944== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable --Apple-Mail=3D_9D5F353B-59C7-41DC-8147-56CB3280FDC1 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=3Dutf-8 > Le 6 avr. 2017 =3DC3=3DA0 20:06, Yaniv Kaul a =3DC3= =3DA9crit =3D : >=3D20 >=3D20 >=3D20 > On Thu, Apr 6, 2017 at 5:30 PM, Fabrice Bacchella =3D > =3D wrote: >=3D20 > > Le 6 avr. 2017 =3DC3=3DA0 16:12, Yaniv Kaul > a =3DC3=3DA9crit : >=3D20 > > Seriously though - perhaps you could borrow code from our Ansible =3D module? See[1]. > > >=3D20 > If the code already exists, why it's not already in the sdk instead of = =3D having to dig throw external code ? >=3D20 > It's a good question, which I've asked as well in the past. The reason = =3D is that it's above the SDK, not part of the SDK. > But that doesn't matter - we really ought to have a module/library on =3D top of the SDK, that can be shared. > For example, between ovirt-system-tests, Ansible, oVirtBackup[1] and =3D several others who write on top of our SDK. > We just never got to generalise it enough and split it. You are =3D welcome to begin this work - I believe it has value. > (It's also a good Google Summer of Code project - I'll see if I can =3D update that page on ovirt.org ). I have already started it for sdk3, I will need to restart if almost =3D from scratch to sdk4: https://github.com/fbacchella/ovirtcmd =3D and need to right a lot of very = =3D basic code. --Apple-Mail=3D_9D5F353B-59C7-41DC-8147-56CB3280FDC1 Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=3Dutf-8
Le 6 avr. 2017 =3DC3=3DA0 20:06, Yaniv Kaul <ykaul(a)redhat.com>= ; a =3D =3DC3=3DA9crit :



On Th= u, =3D Apr 6, 2017 at 5:30 PM, Fabrice Bacchella <fabrice.bacchella(a)orange.fr> =3D wrote:

> Le 6 avr. 2017 =3DC3=3DA0 16:12, Yaniv Kaul <ykaul(a)redhat.com>= ; a =3D =3DC3=3DA9crit :

> Seriously though - perhaps you could borrow code from our Ansible =3D module? See[1].
>

If the code already exists, why it's not already in the sdk =3D instead of having to dig throw external code ?

It's a good question,= =3D which I've asked as well in the past. The reason is that it's above the =3D SDK, not part of the SDK.
But that doesn't matter - = =3D we really ought to have a module/library on top of the SDK,  that =3D can be shared.
For example, between =3D ovirt-system-tests, Ansible, oVirtBackup[1] and several others who write = =3D on top of our SDK.
We just never got to generalise = =3D it enough and split it. You are welcome to begin this work - I believe =3D it has value.
(It's also a good Google Summer of =3D Code project - I'll see if I can update that page on ovirt.org).

I have already started it for sdk3, I will need to = =3D restart if almost from scratch to sdk4: https://github.com/fbacchella/ovirtcmd and need to = =3D right a lot of very basic code.

=3D --Apple-Mail=3D_9D5F353B-59C7-41DC-8147-56CB3280FDC1-- --===============6186509809701437944== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" Ci0tQXBwbGUtTWFpbD1fOUQ1RjM1M0ItNTlDNy00MURDLTgxNDctNTZDQjMyODBGREMxCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKQ29udGVudC1UeXBlOiB0ZXh0 L3BsYWluOwoJY2hhcnNldD11dGYtOAoKCj4gTGUgNiBhdnIuIDIwMTcgPUMzPUEwIDIwOjA2LCBZ YW5pdiBLYXVsIDx5a2F1bEByZWRoYXQuY29tPiBhID1DMz1BOWNyaXQgPQo6Cj49MjAKPj0yMAo+ PTIwCj4gT24gVGh1LCBBcHIgNiwgMjAxNyBhdCA1OjMwIFBNLCBGYWJyaWNlIEJhY2NoZWxsYSA9 CjxmYWJyaWNlLmJhY2NoZWxsYUBvcmFuZ2UuZnIgPG1haWx0bzpmYWJyaWNlLmJhY2NoZWxsYUBv cmFuZ2UuZnI+PiA9Cndyb3RlOgo+PTIwCj4gPiBMZSA2IGF2ci4gMjAxNyA9QzM9QTAgMTY6MTIs IFlhbml2IEthdWwgPHlrYXVsQHJlZGhhdC5jb20gPQo8bWFpbHRvOnlrYXVsQHJlZGhhdC5jb20+ PiBhID1DMz1BOWNyaXQgOgo+PTIwCj4gPiBTZXJpb3VzbHkgdGhvdWdoIC0gcGVyaGFwcyB5b3Ug Y291bGQgYm9ycm93IGNvZGUgZnJvbSBvdXIgQW5zaWJsZSA9Cm1vZHVsZT8gU2VlWzFdLgo+ID4K Pj0yMAo+IElmIHRoZSBjb2RlIGFscmVhZHkgZXhpc3RzLCB3aHkgaXQncyBub3QgYWxyZWFkeSBp biB0aGUgc2RrIGluc3RlYWQgb2YgPQpoYXZpbmcgdG8gZGlnIHRocm93IGV4dGVybmFsIGNvZGUg Pwo+PTIwCj4gSXQncyBhIGdvb2QgcXVlc3Rpb24sIHdoaWNoIEkndmUgYXNrZWQgYXMgd2VsbCBp biB0aGUgcGFzdC4gVGhlIHJlYXNvbiA9CmlzIHRoYXQgaXQncyBhYm92ZSB0aGUgU0RLLCBub3Qg cGFydCBvZiB0aGUgU0RLLgo+IEJ1dCB0aGF0IGRvZXNuJ3QgbWF0dGVyIC0gd2UgcmVhbGx5IG91 Z2h0IHRvIGhhdmUgYSBtb2R1bGUvbGlicmFyeSBvbiA9CnRvcCBvZiB0aGUgU0RLLCAgdGhhdCBj YW4gYmUgc2hhcmVkLgo+IEZvciBleGFtcGxlLCBiZXR3ZWVuIG92aXJ0LXN5c3RlbS10ZXN0cywg QW5zaWJsZSwgb1ZpcnRCYWNrdXBbMV0gYW5kID0Kc2V2ZXJhbCBvdGhlcnMgd2hvIHdyaXRlIG9u IHRvcCBvZiBvdXIgU0RLLgo+IFdlIGp1c3QgbmV2ZXIgZ290IHRvIGdlbmVyYWxpc2UgaXQgZW5v dWdoIGFuZCBzcGxpdCBpdC4gWW91IGFyZSA9CndlbGNvbWUgdG8gYmVnaW4gdGhpcyB3b3JrIC0g SSBiZWxpZXZlIGl0IGhhcyB2YWx1ZS4KPiAoSXQncyBhbHNvIGEgZ29vZCBHb29nbGUgU3VtbWVy IG9mIENvZGUgcHJvamVjdCAtIEknbGwgc2VlIGlmIEkgY2FuID0KdXBkYXRlIHRoYXQgcGFnZSBv biBvdmlydC5vcmcgPGh0dHA6Ly9vdmlydC5vcmcvPikuCgpJIGhhdmUgYWxyZWFkeSBzdGFydGVk IGl0IGZvciBzZGszLCBJIHdpbGwgbmVlZCB0byByZXN0YXJ0IGlmIGFsbW9zdCA9CmZyb20gc2Ny YXRjaCB0byBzZGs0OiBodHRwczovL2dpdGh1Yi5jb20vZmJhY2NoZWxsYS9vdmlydGNtZCA9Cjxo dHRwczovL2dpdGh1Yi5jb20vZmJhY2NoZWxsYS9vdmlydGNtZD4gYW5kIG5lZWQgdG8gcmlnaHQg YSBsb3Qgb2YgdmVyeSA9CmJhc2ljIGNvZGUuCgoKLS1BcHBsZS1NYWlsPV85RDVGMzUzQi01OUM3 LTQxREMtODE0Ny01NkNCMzI4MEZEQzEKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogcXVvdGVk LXByaW50YWJsZQpDb250ZW50LVR5cGU6IHRleHQvaHRtbDsKCWNoYXJzZXQ9dXRmLTgKCjxodG1s PjxoZWFkPjxtZXRhIGh0dHAtZXF1aXY9M0QiQ29udGVudC1UeXBlIiBjb250ZW50PTNEInRleHQv aHRtbCA9CmNoYXJzZXQ9M0R1dGYtOCI+PC9oZWFkPjxib2R5IHN0eWxlPTNEIndvcmQtd3JhcDog YnJlYWstd29yZDsgPQotd2Via2l0LW5ic3AtbW9kZTogc3BhY2U7IC13ZWJraXQtbGluZS1icmVh azogYWZ0ZXItd2hpdGUtc3BhY2U7IiA9CmNsYXNzPTNEIiI+PGJyIGNsYXNzPTNEIiI+PGRpdj48 YmxvY2txdW90ZSB0eXBlPTNEImNpdGUiIGNsYXNzPTNEIiI+PGRpdiA9CmNsYXNzPTNEIiI+TGUg NiBhdnIuIDIwMTcgPUMzPUEwIDIwOjA2LCBZYW5pdiBLYXVsICZsdDs8YSA9CmhyZWY9M0QibWFp bHRvOnlrYXVsQHJlZGhhdC5jb20iIGNsYXNzPTNEIiI+eWthdWxAcmVkaGF0LmNvbTwvYT4mZ3Q7 IGEgPQo9QzM9QTljcml0IDo8L2Rpdj48YnIgY2xhc3M9M0QiQXBwbGUtaW50ZXJjaGFuZ2UtbmV3 bGluZSI+PGRpdiA9CmNsYXNzPTNEIiI+PGRpdiBkaXI9M0QibHRyIiBjbGFzcz0zRCIiPjxiciBj bGFzcz0zRCIiPjxkaXYgPQpjbGFzcz0zRCJnbWFpbF9leHRyYSI+PGJyIGNsYXNzPTNEIiI+PGRp diBjbGFzcz0zRCJnbWFpbF9xdW90ZSI+T24gVGh1LCA9CkFwciA2LCAyMDE3IGF0IDU6MzAgUE0s IEZhYnJpY2UgQmFjY2hlbGxhIDxzcGFuIGRpcj0zRCJsdHIiID0KY2xhc3M9M0QiIj4mbHQ7PGEg aHJlZj0zRCJtYWlsdG86ZmFicmljZS5iYWNjaGVsbGFAb3JhbmdlLmZyIiA9CnRhcmdldD0zRCJf YmxhbmsiIGNsYXNzPTNEIiI+ZmFicmljZS5iYWNjaGVsbGFAb3JhbmdlLmZyPC9hPiZndDs8L3Nw YW4+ID0Kd3JvdGU6PGJyIGNsYXNzPTNEIiI+PGJsb2NrcXVvdGUgY2xhc3M9M0QiZ21haWxfcXVv dGUiIHN0eWxlPTNEIm1hcmdpbjowcHg9CiAwcHggMHB4IDAuOGV4O2JvcmRlci1sZWZ0OjFweCBz b2xpZCA9CnJnYigyMDQsMjA0LDIwNCk7cGFkZGluZy1sZWZ0OjFleCI+PHNwYW4gY2xhc3M9M0Qi Z21haWwtIj48YnIgY2xhc3M9M0QiIj4KJmd0OyBMZSA2IGF2ci4gMjAxNyA9QzM9QTAgMTY6MTIs IFlhbml2IEthdWwgJmx0OzxhID0KaHJlZj0zRCJtYWlsdG86eWthdWxAcmVkaGF0LmNvbSIgY2xh c3M9M0QiIj55a2F1bEByZWRoYXQuY29tPC9hPiZndDsgYSA9Cj1DMz1BOWNyaXQgOjxiciBjbGFz cz0zRCIiPgo8YnIgY2xhc3M9M0QiIj4KJmd0OyBTZXJpb3VzbHkgdGhvdWdoIC0gcGVyaGFwcyB5 b3UgY291bGQgYm9ycm93IGNvZGUgZnJvbSBvdXIgQW5zaWJsZSA9Cm1vZHVsZT8gU2VlWzFdLjxi ciBjbGFzcz0zRCIiPgomZ3Q7PGJyIGNsYXNzPTNEIiI+CjxiciBjbGFzcz0zRCIiPgo8L3NwYW4+ SWYgdGhlIGNvZGUgYWxyZWFkeSBleGlzdHMsIHdoeSBpdCdzIG5vdCBhbHJlYWR5IGluIHRoZSBz ZGsgPQppbnN0ZWFkIG9mIGhhdmluZyB0byBkaWcgdGhyb3cgZXh0ZXJuYWwgY29kZSA/PC9ibG9j a3F1b3RlPjxkaXYgPQpjbGFzcz0zRCIiPjxiciBjbGFzcz0zRCIiPjwvZGl2PjxkaXYgY2xhc3M9 M0QiIj5JdCdzIGEgZ29vZCBxdWVzdGlvbiwgPQp3aGljaCBJJ3ZlIGFza2VkIGFzIHdlbGwgaW4g dGhlIHBhc3QuIFRoZSByZWFzb24gaXMgdGhhdCBpdCdzIGFib3ZlIHRoZSA9ClNESywgbm90IHBh cnQgb2YgdGhlIFNESy48L2Rpdj48ZGl2IGNsYXNzPTNEIiI+QnV0IHRoYXQgZG9lc24ndCBtYXR0 ZXIgLSA9CndlIHJlYWxseSBvdWdodCB0byBoYXZlIGEgbW9kdWxlL2xpYnJhcnkgb24gdG9wIG9m IHRoZSBTREssICZuYnNwO3RoYXQgPQpjYW4gYmUgc2hhcmVkLjwvZGl2PjxkaXYgY2xhc3M9M0Qi Ij5Gb3IgZXhhbXBsZSwgYmV0d2VlbiA9Cm92aXJ0LXN5c3RlbS10ZXN0cywgQW5zaWJsZSwgb1Zp cnRCYWNrdXBbMV0gYW5kIHNldmVyYWwgb3RoZXJzIHdobyB3cml0ZSA9Cm9uIHRvcCBvZiBvdXIg U0RLLjwvZGl2PjxkaXYgY2xhc3M9M0QiIj5XZSBqdXN0IG5ldmVyIGdvdCB0byBnZW5lcmFsaXNl ID0KaXQgZW5vdWdoIGFuZCBzcGxpdCBpdC4gWW91IGFyZSB3ZWxjb21lIHRvIGJlZ2luIHRoaXMg d29yayAtIEkgYmVsaWV2ZSA9Cml0IGhhcyB2YWx1ZS48L2Rpdj48ZGl2IGNsYXNzPTNEIiI+KEl0 J3MgYWxzbyBhIGdvb2QgR29vZ2xlIFN1bW1lciBvZiA9CkNvZGUgcHJvamVjdCAtIEknbGwgc2Vl IGlmIEkgY2FuIHVwZGF0ZSB0aGF0IHBhZ2Ugb24gPGEgPQpocmVmPTNEImh0dHA6Ly9vdmlydC5v cmcvIiA9CmNsYXNzPTNEIiI+b3ZpcnQub3JnPC9hPikuPC9kaXY+PC9kaXY+PC9kaXY+PC9kaXY+ PC9kaXY+PC9ibG9ja3F1b3RlPjxiciA9CmNsYXNzPTNEIiI+PC9kaXY+PGRpdj5JIGhhdmUgYWxy ZWFkeSBzdGFydGVkIGl0IGZvciBzZGszLCBJIHdpbGwgbmVlZCB0byA9CnJlc3RhcnQgaWYgYWxt b3N0IGZyb20gc2NyYXRjaCB0byBzZGs0OiZuYnNwOzxhID0KaHJlZj0zRCJodHRwczovL2dpdGh1 Yi5jb20vZmJhY2NoZWxsYS9vdmlydGNtZCIgPQpjbGFzcz0zRCIiPmh0dHBzOi8vZ2l0aHViLmNv bS9mYmFjY2hlbGxhL292aXJ0Y21kPC9hPiZuYnNwO2FuZCBuZWVkIHRvID0KcmlnaHQgYSBsb3Qg b2YgdmVyeSBiYXNpYyBjb2RlLjwvZGl2PjxiciBjbGFzcz0zRCIiPjwvYm9keT48L2h0bWw+PQoK LS1BcHBsZS1NYWlsPV85RDVGMzUzQi01OUM3LTQxREMtODE0Ny01NkNCMzI4MEZEQzEtLQo= --===============6186509809701437944==--