From: "Giorgio Bersano" <giorgio.bersano(a)gmail.com&gt; To: Users(a)ovirt.org Sent: Wednesday, February 12, 2014 12:28:33 PM Subject: [Users] hosted-engine deployment doesn't accept legal email address (ovirt-3.4.0-prerelease) Hi, I am happily testing the hosted engine feature, sorry I was late on the test day but I dare to provide some feedback. To be more clear I prefer to report one item per email. This is related to a really minor issue I think it doesn't deserve a Bugzilla but I'm ready to do it if requested. So, during # hosted-engine --deploy in the step named "--== HOSTED ENGINE CONFIGURATION ==--" we are asked to provide a couple of email addresses but addresses with an hypen before the @ sign (ex. my-name(a)mydomain.com ) aren't allowed and throwns the following: [ ERROR ] Invalid input, please try again As a workaround I used another address but, looking at /usr/share/ovirt-hosted-engine-setup/plugins/ovirt-hosted-engine-setup/ha/ha_notifications.py (ovirt-hosted-engine-setup-1.1.0-0.5.beta2.el6.noarch) I've found that _RE_EMAIL_ADDRESS is using a simplified pattern (hope that gmail doesn't mangles text too much): [a-zA-Z0-9_.+]+ @ [a-z0-9.-]+ even avoiding the intricacy of the whole possibilities (see "Local part" in http://en.wikipedia.org/wiki/Email_address ) I think the following (not tested) could be more appropriate: [a-zA-Z0-9] [a-zA-Z0-9_.+-]+ [a-zA-Z0-9] @ [a-z0-9.-]+ don't know if it really works as I've never programmed in python but I hope you get the idea. Best regards, Giorgio.

From: "Giorgio Bersano" <giorgio.bersano(a)gmail.com&gt; To: "Doron Fediuck" <dfediuck(a)redhat.com&gt; Cc: Users(a)ovirt.org, "Yedidyah Bar David" <didi(a)redhat.com&gt;, "Sandro Bonazzola" <sbonazzo(a)redhat.com&gt; Sent: Friday, February 14, 2014 10:54:25 AM Subject: Re: [Users] hosted-engine deployment doesn't accept legal email address (ovirt-3.4.0-prerelease) 2014-02-12 17:28 GMT+01:00 Doron Fediuck <dfediuck(a)redhat.com&gt;: > > Hi Giorgio. > Patches are always welcomed! > > I'd start with opening a bug, as mail verification may need > some attention. > > Next, to get the code you can use- > git clone git://gerrit.ovirt.org/ovirt-hosted-engine-setup > > and then make a patch and push it to gerrit. > > Thanks again! > Doron Hi Doron, here it is: https://bugzilla.redhat.com/show_bug.cgi?id=1065269 . To speedup things I've put the patch in there, as I'm not familiar with gerrit ATM . Please push it by yourself if you agree; in the meantime I'll try to learn how to use it. TIA, Giorgio.