On Thu, Apr 7, 2011 at 8:59 AM, Benjamin Reed <ran...@opennms.org> wrote:
> 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
Yeah, I'd played around a little with adding services to the URL in
provisiond-configuration.xml, but haven't had time to get it working
yet. The idea I had was that it would default to ICMP and SNMP if no
services were specified in the URL.
--Bill--
------------------------------------------------------------------------------
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
