commit d18f0d36c31a685a6153b11e1966b8c6533d19d6
author Girish Gowdra <girish@opennetworking.org> Fri Oct 11 20:46:15 2019 +0530
committer Girish Gowdra <girish@opennetworking.org> Mon Oct 14 11:52:43 2019 +0530
tree 18d1459124815c40bd1f59e4cf2e216fc5e78f07
parent 3de9d34e325ebb9e02cbcdee8234338b30978727

[VOL-2043] : OLT Enable Test Case

- Added more test cases for OLT Enable. Now covers all scenarios.
- Fixed typos in unit test README.md

Change-Id: Ib931d89cedee78fe40ca67761a028365a047ebc3

agent/src/core.cc

diff --git a/agent/src/core.cc b/agent/src/core.cc
index 9f30bde..9c27296 100644
--- a/agent/src/core.cc
+++ b/agent/src/core.cc
@@ -741,6 +741,7 @@
     bcmos_errno err;
     bcmolt_host_init_parms init_parms = {};
     init_parms.transport.type = BCM_HOST_API_CONN_LOCAL;
+    unsigned int failed_enable_device_cnt = 0;
 
     if (!state.is_activated()) {
 
@@ -808,7 +809,12 @@
                     }
                     err = bcmolt_oper_submit(dev_id, &oper.hdr);
                     if (err) {
+                        failed_enable_device_cnt ++;
                         OPENOLT_LOG(ERROR, openolt_log_id, "Enable PON device %d failed, err %d\n", dev, err);
+                        if (failed_enable_device_cnt == BCM_MAX_DEVS_PER_LINE_CARD) {
+                            OPENOLT_LOG(ERROR, openolt_log_id, "failed to enable all the pon ports\n");
+                            return Status(grpc::StatusCode::INTERNAL, "Failed to activate all PON ports");
+                        }
                     }
                     bcmos_usleep(200000);
                 }