Description of problem: MAIL_SERVER value is not checked so carefully as HTML_MESSAGE_FORMAT for example. See comparison: 2014-01-09 17:41:53,475 INFO [org.ovirt.engine.core.utils.LocalConfig] Value of property "HTML_MESSAGE_FORMAT" is "-11". ...snip... 2014-01-09 17:41:53,723 ERROR [org.ovirt.engine.core.notifier.Notifier] Failed to run the event notification service. java.lang.IllegalArgumentException: The value "-11" for property "HTML_MESSAGE_FORMAT" is not a valid boolean. at org.ovirt.engine.core.utils.LocalConfig.getBoolean(LocalConfig.java:349) vs 2014-01-09 17:42:42,947 INFO [org.ovirt.engine.core.utils.LocalConfig] Value of property "MAIL_SERVER" is "---". ...snip... 2014-01-09 17:42:42,957 ERROR [org.ovirt.engine.core.notifier.Notifier] Failed to run the event notification service. java.net.UnknownHostException: --- at java.net.Inet4AddressImpl.lookupAllHostAddr(Native Method) Version-Release number of selected component (if applicable): is31 - rhevm-tools-3.3.0-0.44.el6ev.noarch How reproducible: 100% Steps to Reproduce: 1. MAIL_SERVER=------ 2. restart 3. Actual results: not so understandable as one would like Expected results: something like... 2014-01-09 17:41:53,723 ERROR [org.ovirt.engine.core.notifier.Notifier] Failed to run the event notification service. java.lang.IllegalArgumentException: The value "----" for property "MAIL_SERVER" is not a ..... (domain|host|...) Additional info:
2014-01-09 17:42:42,957 ERROR [org.ovirt.engine.core.notifier.Notifier] Failed to run the event notification service. java.net.UnknownHostException: --- at java.net.Inet4AddressImpl.lookupAllHostAddr(Native Method) at java.net.InetAddress$1.lookupAllHostAddr(InetAddress.java:901) at java.net.InetAddress.getAddressesFromNameService(InetAddress.java:1293) at java.net.InetAddress.getAllByName0(InetAddress.java:1246) at java.net.InetAddress.getAllByName(InetAddress.java:1162) at java.net.InetAddress.getAllByName(InetAddress.java:1098) at org.ovirt.engine.core.notifier.Notifier.main(Notifier.java:84) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.lang.reflect.Method.invoke(Method.java:606) at org.jboss.modules.Module.run(Module.java:292) at org.jboss.modules.Main.main(Main.java:455)
Moving to assigned due to an abandoned patch
Validation of hostname specified in MAIL_SERVER has been removed completely.
notifier is broken, see BZ1071536.
It's impossible to verify this BZ as due to filter bug, no event would be submitted for sending (see comment 4)
BZ1071536 is fixed in oVirt 3.4.0 RC2, so you can test once new RHEVM build will be prepared
Since BZ1071536 in RHEVM 3.4.0 av3, this can also be verified.
No error message. I got Info message instead of error: Value of property "MAIL_SERVER" is "----------------------------------------"
That's correct, see comment 3, validation of MAIL_SERVER property is now removed completely. There will be an error in notifier.log when an email cannot be sent to specified server.
Verified in av3.1
Closing as part of 3.4.0