isAvailable returning true due to pingDevice method at NetconfControllerImpl even if netconf data tree can't be fetched from device.
Change-Id: If4ee29d4db266561b63421b4aae538874daf7a53
diff --git a/protocols/netconf/ctl/src/main/java/org/onosproject/netconf/ctl/impl/NetconfControllerImpl.java b/protocols/netconf/ctl/src/main/java/org/onosproject/netconf/ctl/impl/NetconfControllerImpl.java
index a3f02b0..aa44e6d 100644
--- a/protocols/netconf/ctl/src/main/java/org/onosproject/netconf/ctl/impl/NetconfControllerImpl.java
+++ b/protocols/netconf/ctl/src/main/java/org/onosproject/netconf/ctl/impl/NetconfControllerImpl.java
@@ -577,6 +577,10 @@
CompletableFuture<CharSequence> future = getNetconfDevice(deviceId).getSession().asyncGet();
CharSequence reply = Tools.futureGetOrElse(future, NETCONF_REPLY_TIMEOUT_DEFAULT, TimeUnit.SECONDS,
"Unable to read netconf data tree from device");
+ if (reply.equals("Unable to read netconf data tree from device")) {
+ log.error("Failed to get netconf data tree for device : {}", deviceId);
+ return false;
+ }
log.debug("Netconf data tree for device : {} -> {}", deviceId, reply);
} catch (NetconfException e) {
log.error("Error while getting netconf data tree for device : {}", deviceId);