From j.astrego at netbulae.eu Thu Jul 31 05:01:14 2014 Content-Type: multipart/mixed; boundary="===============4646478776736823469==" MIME-Version: 1.0 From: Jorick Astrego To: users at ovirt.org Subject: Re: [ovirt-users] oVirt 3.5 test day 2 results: foreman advanced integration feature Date: Thu, 31 Jul 2014 11:01:08 +0200 Message-ID: <53DA05D4.8020607@netbulae.eu> In-Reply-To: 1061238518.691346.1406796590443.JavaMail.zimbra@redhat.com --===============4646478776736823469== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable This is a multi-part message in MIME format. --------------050100080105020309000909 Content-Type: text/plain; charset=3DISO-8859-1; format=3Dflowed Content-Transfer-Encoding: 7bit On 07/31/2014 10:49 AM, Moti Asayag wrote: > Hi, > > In the second ovirt test day I've retested the foreman advanced integrati= on. > Using the instructions on the wiki page I was able to test part of the fe= ature: > * Adding foreman provider and test connectivity with certificates import > * Present discovered hosts by foreman > * Present provisioned hosts by foreman > * Add discovered hosts by foreman (reached to "Installing OS" phase only = due to > environmental issues). > > Since the host provisioning wasn't completed, I haven't reach to test the= foreman-->ovirt > connectivity which should have trigger the host installation. > > I've filed a couple of bugs which I encountered during the testing: > Bug 1124870 - Provision of a host without compute resources fails with NPE > Bug 1124878 - "Set Root password" text label is not aligned with its capt= ion > Bug 1124891 - [oVirt][Foreman] Root password length should be validated > Bug 1124887 - [RFE][oVirt][Foreman] Improve error messages when action fa= ils on foreman server > > In addition I've updated the feature page which missed a required package= to be installed > on the foreman server. > > Regards, > Moti > Hi, I tested this feature also. I was able to get things going by manually = installing vdsm because something goes wrong with my puppet classes. Filed one ovirt bug: 1124842 - using = "Use Foreman Hosts Providers" disables "Address" field in "New host" dialog And one foreman bug: Bug #6839: Conflict DHCP records when using the "Use Foreman Hosts = Providers>Provisioned Hosts" of ovirt 3.5 = Also I agree with "Bug 1124887 - [RFE][oVirt][Foreman] Improve error = messages when action fails on foreman server", I had to go look in the = foreman logs a lot to see things configured incorrect. The error in = ovirt is just that the connection doesn't work. But it appears most of it is working correctly. Kind regards, Jorick Astrego Netbulae B.V. --------------050100080105020309000909 Content-Type: text/html; charset=3DISO-8859-1 Content-Transfer-Encoding: 7bit
On 07/31/2014 10:49 AM, Moti Asayag wrote:
Hi,

In the second ovirt test day I've retested the foreman advanced integration.
Using the instructions on the wiki page I was able to test part of the feat=
ure:
* Adding foreman provider and test connectivity with certificates import
* Present discovered hosts by foreman
* Present provisioned hosts by foreman
* Add discovered hosts by foreman (reached to "Installing OS" phase only du=
e to
environmental issues).

Since the host provisioning wasn't completed, I haven't reach to test the f=
oreman-->ovirt
connectivity which should have trigger the host installation. =


I've filed a couple of bugs which I encountered during the testing:
Bug 1124870 - Provision of a host without compute resources fails with NPE
Bug 1124878 - "Set Root password" text label is not aligned with its caption
Bug 1124891 - [oVirt][Foreman] Root password length should be validated
Bug 1124887 - [RFE][oVirt][Foreman] Improve error messages when action fail=
s on foreman server

In addition I've updated the feature page which missed a required package t=
o be installed
on the foreman server.

Regards,
Moti
Hi,

I tested this feature also. I was able to get things going by manually installing vdsm because something goes wrong with my puppet classes.

Filed one ovirt bug:

112484= 2 -  using "Use Foreman Hosts Providers" disables "Address" field in "New host" dialog

And one foreman bug:

Bug #6839: Conflict DHCP records when using the "Use Foreman Hosts Providers>Provisioned Hosts" of ovirt 3.5

Also I agree with "Bug 1124887 - [RFE][oVirt][Foreman] Improve error messages when action fails on foreman server", I had to go look in the foreman logs a lot to see things configured incorrect. The error in ovirt is just that the connection doesn't work.

But it appears most of it is working correctly.

Kind regards,

Jorick Astrego
Netbulae B.V.
--------------050100080105020309000909-- --===============4646478776736823469== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4KLS0tLS0tLS0tLS0t LS0wNTAxMDAwODAxMDUwMjAzMDkwMDA5MDkKQ29udGVudC1UeXBlOiB0ZXh0L3BsYWluOyBjaGFy c2V0PUlTTy04ODU5LTE7IGZvcm1hdD1mbG93ZWQKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzog N2JpdAoKCk9uIDA3LzMxLzIwMTQgMTA6NDkgQU0sIE1vdGkgQXNheWFnIHdyb3RlOgo+IEhpLAo+ Cj4gSW4gdGhlIHNlY29uZCBvdmlydCB0ZXN0IGRheSBJJ3ZlIHJldGVzdGVkIHRoZSBmb3JlbWFu IGFkdmFuY2VkIGludGVncmF0aW9uLgo+IFVzaW5nIHRoZSBpbnN0cnVjdGlvbnMgb24gdGhlIHdp a2kgcGFnZSBJIHdhcyBhYmxlIHRvIHRlc3QgcGFydCBvZiB0aGUgZmVhdHVyZToKPiAqIEFkZGlu ZyBmb3JlbWFuIHByb3ZpZGVyIGFuZCB0ZXN0IGNvbm5lY3Rpdml0eSB3aXRoIGNlcnRpZmljYXRl cyBpbXBvcnQKPiAqIFByZXNlbnQgZGlzY292ZXJlZCBob3N0cyBieSBmb3JlbWFuCj4gKiBQcmVz ZW50IHByb3Zpc2lvbmVkIGhvc3RzIGJ5IGZvcmVtYW4KPiAqIEFkZCBkaXNjb3ZlcmVkIGhvc3Rz IGJ5IGZvcmVtYW4gKHJlYWNoZWQgdG8gIkluc3RhbGxpbmcgT1MiIHBoYXNlIG9ubHkgZHVlIHRv Cj4gZW52aXJvbm1lbnRhbCBpc3N1ZXMpLgo+Cj4gU2luY2UgdGhlIGhvc3QgcHJvdmlzaW9uaW5n IHdhc24ndCBjb21wbGV0ZWQsIEkgaGF2ZW4ndCByZWFjaCB0byB0ZXN0IHRoZSBmb3JlbWFuLS0+ b3ZpcnQKPiBjb25uZWN0aXZpdHkgd2hpY2ggc2hvdWxkIGhhdmUgdHJpZ2dlciB0aGUgaG9zdCBp bnN0YWxsYXRpb24uCj4KPiBJJ3ZlIGZpbGVkIGEgY291cGxlIG9mIGJ1Z3Mgd2hpY2ggSSBlbmNv dW50ZXJlZCBkdXJpbmcgdGhlIHRlc3Rpbmc6Cj4gQnVnIDExMjQ4NzAgLSBQcm92aXNpb24gb2Yg YSBob3N0IHdpdGhvdXQgY29tcHV0ZSByZXNvdXJjZXMgZmFpbHMgd2l0aCBOUEUKPiBCdWcgMTEy NDg3OCAtICJTZXQgUm9vdCBwYXNzd29yZCIgdGV4dCBsYWJlbCBpcyBub3QgYWxpZ25lZCB3aXRo IGl0cyBjYXB0aW9uCj4gQnVnIDExMjQ4OTEgLSBbb1ZpcnRdW0ZvcmVtYW5dIFJvb3QgcGFzc3dv cmQgbGVuZ3RoIHNob3VsZCBiZSB2YWxpZGF0ZWQKPiBCdWcgMTEyNDg4NyAtIFtSRkVdW29WaXJ0 XVtGb3JlbWFuXSBJbXByb3ZlIGVycm9yIG1lc3NhZ2VzIHdoZW4gYWN0aW9uIGZhaWxzIG9uIGZv cmVtYW4gc2VydmVyCj4KPiBJbiBhZGRpdGlvbiBJJ3ZlIHVwZGF0ZWQgdGhlIGZlYXR1cmUgcGFn ZSB3aGljaCBtaXNzZWQgYSByZXF1aXJlZCBwYWNrYWdlIHRvIGJlIGluc3RhbGxlZAo+IG9uIHRo ZSBmb3JlbWFuIHNlcnZlci4KPgo+IFJlZ2FyZHMsCj4gTW90aQo+CkhpLAoKSSB0ZXN0ZWQgdGhp cyBmZWF0dXJlIGFsc28uIEkgd2FzIGFibGUgdG8gZ2V0IHRoaW5ncyBnb2luZyBieSBtYW51YWxs eSAKaW5zdGFsbGluZyB2ZHNtIGJlY2F1c2Ugc29tZXRoaW5nIGdvZXMgd3Jvbmcgd2l0aCBteSBw dXBwZXQgY2xhc3Nlcy4KCkZpbGVkIG9uZSBvdmlydCBidWc6CgoxMTI0ODQyIDxodHRwczovL2J1 Z3ppbGxhLnJlZGhhdC5jb20vc2hvd19idWcuY2dpP2lkPTExMjQ4NDI+IC0gIHVzaW5nIAoiVXNl IEZvcmVtYW4gSG9zdHMgUHJvdmlkZXJzIiBkaXNhYmxlcyAiQWRkcmVzcyIgZmllbGQgaW4gIk5l dyBob3N0IiBkaWFsb2cKCkFuZCBvbmUgZm9yZW1hbiBidWc6CgpCdWcgIzY4Mzk6IENvbmZsaWN0 IERIQ1AgcmVjb3JkcyB3aGVuIHVzaW5nIHRoZSAiVXNlIEZvcmVtYW4gSG9zdHMgClByb3ZpZGVy cz5Qcm92aXNpb25lZCBIb3N0cyIgb2Ygb3ZpcnQgMy41IAo8aHR0cDovL3Byb2plY3RzLnRoZWZv cmVtYW4ub3JnL2lzc3Vlcy82ODM5I2NoYW5nZS0yMzg0Nz4KCkFsc28gSSBhZ3JlZSB3aXRoICJC dWcgMTEyNDg4NyAtIFtSRkVdW29WaXJ0XVtGb3JlbWFuXSBJbXByb3ZlIGVycm9yIAptZXNzYWdl cyB3aGVuIGFjdGlvbiBmYWlscyBvbiBmb3JlbWFuIHNlcnZlciIsIEkgaGFkIHRvIGdvIGxvb2sg aW4gdGhlIApmb3JlbWFuIGxvZ3MgYSBsb3QgdG8gc2VlIHRoaW5ncyBjb25maWd1cmVkIGluY29y cmVjdC4gVGhlIGVycm9yIGluIApvdmlydCBpcyBqdXN0IHRoYXQgdGhlIGNvbm5lY3Rpb24gZG9l c24ndCB3b3JrLgoKQnV0IGl0IGFwcGVhcnMgbW9zdCBvZiBpdCBpcyB3b3JraW5nIGNvcnJlY3Rs eS4KCktpbmQgcmVnYXJkcywKCkpvcmljayBBc3RyZWdvCk5ldGJ1bGFlIEIuVi4KCi0tLS0tLS0t LS0tLS0tMDUwMTAwMDgwMTA1MDIwMzA5MDAwOTA5CkNvbnRlbnQtVHlwZTogdGV4dC9odG1sOyBj aGFyc2V0PUlTTy04ODU5LTEKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzogN2JpdAoKPGh0bWw+ CiAgPGhlYWQ+CiAgICA8bWV0YSBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9SVNPLTg4NTkt MSIKICAgICAgaHR0cC1lcXVpdj0iQ29udGVudC1UeXBlIj4KICA8L2hlYWQ+CiAgPGJvZHkgdGV4 dD0iIzAwMDAwMCIgYmdjb2xvcj0iI0ZGRkZGRiI+CiAgICA8YnI+CiAgICA8ZGl2IGNsYXNzPSJt b3otY2l0ZS1wcmVmaXgiPk9uIDA3LzMxLzIwMTQgMTA6NDkgQU0sIE1vdGkgQXNheWFnCiAgICAg IHdyb3RlOjxicj4KICAgIDwvZGl2PgogICAgPGJsb2NrcXVvdGUKICAgICAgY2l0ZT0ibWlkOjEw NjEyMzg1MTguNjkxMzQ2LjE0MDY3OTY1OTA0NDMuSmF2YU1haWwuemltYnJhQHJlZGhhdC5jb20i CiAgICAgIHR5cGU9ImNpdGUiPgogICAgICA8cHJlIHdyYXA9IiI+SGksCgpJbiB0aGUgc2Vjb25k IG92aXJ0IHRlc3QgZGF5IEkndmUgcmV0ZXN0ZWQgdGhlIGZvcmVtYW4gYWR2YW5jZWQgaW50ZWdy YXRpb24uClVzaW5nIHRoZSBpbnN0cnVjdGlvbnMgb24gdGhlIHdpa2kgcGFnZSBJIHdhcyBhYmxl IHRvIHRlc3QgcGFydCBvZiB0aGUgZmVhdHVyZToKKiBBZGRpbmcgZm9yZW1hbiBwcm92aWRlciBh bmQgdGVzdCBjb25uZWN0aXZpdHkgd2l0aCBjZXJ0aWZpY2F0ZXMgaW1wb3J0CiogUHJlc2VudCBk aXNjb3ZlcmVkIGhvc3RzIGJ5IGZvcmVtYW4KKiBQcmVzZW50IHByb3Zpc2lvbmVkIGhvc3RzIGJ5 IGZvcmVtYW4KKiBBZGQgZGlzY292ZXJlZCBob3N0cyBieSBmb3JlbWFuIChyZWFjaGVkIHRvICJJ bnN0YWxsaW5nIE9TIiBwaGFzZSBvbmx5IGR1ZSB0bwplbnZpcm9ubWVudGFsIGlzc3VlcykuCgpT aW5jZSB0aGUgaG9zdCBwcm92aXNpb25pbmcgd2Fzbid0IGNvbXBsZXRlZCwgSSBoYXZlbid0IHJl YWNoIHRvIHRlc3QgdGhlIGZvcmVtYW4tLSZndDtvdmlydApjb25uZWN0aXZpdHkgd2hpY2ggc2hv dWxkIGhhdmUgdHJpZ2dlciB0aGUgaG9zdCBpbnN0YWxsYXRpb24uIAoKSSd2ZSBmaWxlZCBhIGNv dXBsZSBvZiBidWdzIHdoaWNoIEkgZW5jb3VudGVyZWQgZHVyaW5nIHRoZSB0ZXN0aW5nOgpCdWcg MTEyNDg3MCAtIFByb3Zpc2lvbiBvZiBhIGhvc3Qgd2l0aG91dCBjb21wdXRlIHJlc291cmNlcyBm YWlscyB3aXRoIE5QRQpCdWcgMTEyNDg3OCAtICJTZXQgUm9vdCBwYXNzd29yZCIgdGV4dCBsYWJl bCBpcyBub3QgYWxpZ25lZCB3aXRoIGl0cyBjYXB0aW9uCkJ1ZyAxMTI0ODkxIC0gW29WaXJ0XVtG b3JlbWFuXSBSb290IHBhc3N3b3JkIGxlbmd0aCBzaG91bGQgYmUgdmFsaWRhdGVkCkJ1ZyAxMTI0 ODg3IC0gW1JGRV1bb1ZpcnRdW0ZvcmVtYW5dIEltcHJvdmUgZXJyb3IgbWVzc2FnZXMgd2hlbiBh Y3Rpb24gZmFpbHMgb24gZm9yZW1hbiBzZXJ2ZXIKCkluIGFkZGl0aW9uIEkndmUgdXBkYXRlZCB0 aGUgZmVhdHVyZSBwYWdlIHdoaWNoIG1pc3NlZCBhIHJlcXVpcmVkIHBhY2thZ2UgdG8gYmUgaW5z dGFsbGVkCm9uIHRoZSBmb3JlbWFuIHNlcnZlci4KClJlZ2FyZHMsCk1vdGkKCjwvcHJlPgogICAg PC9ibG9ja3F1b3RlPgogICAgSGksPGJyPgogICAgPGJyPgogICAgSSB0ZXN0ZWQgdGhpcyBmZWF0 dXJlIGFsc28uIEkgd2FzIGFibGUgdG8gZ2V0IHRoaW5ncyBnb2luZyBieQogICAgbWFudWFsbHkg aW5zdGFsbGluZyB2ZHNtIGJlY2F1c2Ugc29tZXRoaW5nIGdvZXMgd3Jvbmcgd2l0aCBteSBwdXBw ZXQKICAgIGNsYXNzZXMuPGJyPgogICAgPGJyPgogICAgRmlsZWQgb25lIG92aXJ0IGJ1Zzo8YnI+ CiAgICA8YnI+CiAgICA8YSBuYW1lPSJiMTEyNDg0MiIKICAgICAgaHJlZj0iaHR0cHM6Ly9idWd6 aWxsYS5yZWRoYXQuY29tL3Nob3dfYnVnLmNnaT9pZD0xMTI0ODQyIj4xMTI0ODQyPC9hPgogICAg LSZuYnNwOyB1c2luZyAiVXNlIEZvcmVtYW4gSG9zdHMgUHJvdmlkZXJzIiBkaXNhYmxlcyAiQWRk cmVzcyIgZmllbGQgaW4KICAgICJOZXcgaG9zdCIgZGlhbG9nIDxicj4KICAgIDxicj4KICAgIEFu ZCBvbmUgZm9yZW1hbiBidWc6PGJyPgogICAgPGJyPgogICAgPGEgaHJlZj0iaHR0cDovL3Byb2pl Y3RzLnRoZWZvcmVtYW4ub3JnL2lzc3Vlcy82ODM5I2NoYW5nZS0yMzg0NyI+QnVnCiAgICAgICM2 ODM5OiBDb25mbGljdCBESENQIHJlY29yZHMgd2hlbiB1c2luZyB0aGUgIlVzZSBGb3JlbWFuIEhv c3RzCiAgICAgIFByb3ZpZGVycyZndDtQcm92aXNpb25lZCBIb3N0cyIgb2Ygb3ZpcnQgMy41PC9h Pjxicj4KICAgIDxicj4KICAgIEFsc28gSSBhZ3JlZSB3aXRoICJCdWcgMTEyNDg4NyAtIFtSRkVd W29WaXJ0XVtGb3JlbWFuXSBJbXByb3ZlIGVycm9yCiAgICBtZXNzYWdlcyB3aGVuIGFjdGlvbiBm YWlscyBvbiBmb3JlbWFuIHNlcnZlciIsIEkgaGFkIHRvIGdvIGxvb2sgaW4KICAgIHRoZSBmb3Jl bWFuIGxvZ3MgYSBsb3QgdG8gc2VlIHRoaW5ncyBjb25maWd1cmVkIGluY29ycmVjdC4gVGhlIGVy cm9yCiAgICBpbiBvdmlydCBpcyBqdXN0IHRoYXQgdGhlIGNvbm5lY3Rpb24gZG9lc24ndCB3b3Jr Ljxicj4KICAgIDxicj4KICAgIEJ1dCBpdCBhcHBlYXJzIG1vc3Qgb2YgaXQgaXMgd29ya2luZyBj b3JyZWN0bHkuPGJyPgogICAgPGJyPgogICAgS2luZCByZWdhcmRzLDxicj4KICAgIDxicj4KICAg IEpvcmljayBBc3RyZWdvPGJyPgogICAgTmV0YnVsYWUgQi5WLjxicj4KICA8L2JvZHk+CjwvaHRt bD4KCi0tLS0tLS0tLS0tLS0tMDUwMTAwMDgwMTA1MDIwMzA5MDAwOTA5LS0KCg== --===============4646478776736823469==--