Re: [Users] hosted-engine deployment doesn't accept legal email address (ovirt-3.4.0-prerelease)
- Original Message - From: Giorgio Bersano giorgio.bers...@gmail.com To: Doron Fediuck dfedi...@redhat.com Cc: Users@ovirt.org, Yedidyah Bar David d...@redhat.com, Sandro Bonazzola sbona...@redhat.com 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 dfedi...@redhat.com: 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. Giorgio, eccellente ;) This is how your patch looks like in Gerrit (Thanks to Sandro)- http://gerrit.ovirt.org/#/c/24476/ Keep sending patches! Doron ___ Users mailing list Users@ovirt.org http://lists.ovirt.org/mailman/listinfo/users
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 dfedi...@redhat.com: 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. ___ Users mailing list Users@ovirt.org http://lists.ovirt.org/mailman/listinfo/users
[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-n...@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. ___ Users mailing list Users@ovirt.org http://lists.ovirt.org/mailman/listinfo/users
Re: [Users] hosted-engine deployment doesn't accept legal email address (ovirt-3.4.0-prerelease)
- Original Message - From: Giorgio Bersano giorgio.bers...@gmail.com To: Users@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-n...@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. 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 ___ Users mailing list Users@ovirt.org http://lists.ovirt.org/mailman/listinfo/users