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