On 4/7/11 10:26 AM, ayresw wrote:
> Relative to testing the release candidates,
> opennms-provisiond/src/main/java/org/opennms/netmgt/provision/service/dns/DnsRequisitionUrlConnection.java
>  hard-codes ICMP and SNMP services for all nodes provisioned via DNS.
>
> This causes problems for nodes that don't support SNMP. Scans of these
> nodes are aborted in
> opennms-provisiond/src/main/java/org/opennms/netmgt/provision/service/NodeInfoScan.java,
>  and the nodes are provisioned with SNMP which is always down.
>
> I'd like to remove the line
> i.insertMonitoredService(newRequisitionMonitoredService("SNMP"));
> from DnsRequisitionUrlConnection.java.
>
> SNMP will still be found on the nodes that support it, and the problems
> will be fixed for those that don't.
>
> Are there any reasons for NOT removing this line? Can it be done before
> the upcoming release?

It's a requirement of the folks who original commissioned the code, I 
think.  That said, we need to come up with a way to make it 
configurable, since obviously not everyone will want those things.  
Regardless, that's been the behavior since the creation of the monitor, 
if we're going to change it, I'd say 1.8.11/1.9.7 are too late.  =)

-- 
Benjamin Reed
The OpenNMS Group
http://www.opennms.org/



------------------------------------------------------------------------------
Xperia(TM) PLAY
It's a major breakthrough. An authentic gaming
smartphone on the nation's most reliable network.
And it wants your games.
http://p.sf.net/sfu/verizon-sfdev
_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/index.php/Mailing_List_FAQ

opennms-devel mailing list

To *unsubscribe* or change your subscription options, see the bottom of this 
page:
https://lists.sourceforge.net/lists/listinfo/opennms-devel

Reply via email to