SM Lite: R3.1.1 build 42: monitoring is set as false but should show correct error message, not "Not found: '/MonitorInfo'"

[sundarkh]

1) Install SM Lite on a box (R3.1.1 build 42)
2) Use server-manager show monitoring
Not found: '/MonitorInfo'

/opt/contrail/server_manager/sm-config.ini

[SERVER-MANAGER]
listen_ip_addr = 10.204.217.112
listen_port = 9001
database_name = smgr_data.db
server_manager_base_dir = /etc/contrail_smgr/
html_root_dir = /var/www/html/
cobbler = false
monitoring = false
cobbler_ip_address = 10.204.217.112
cobbler_port =
cobbler_username = cobbler
cobbler_password = cobbler
ipmi_username = ADMIN
ipmi_password = ADMIN
ipmi_interface =
ipmi_type = ipmilan
puppet_dir = /etc/puppet/
puppet_agent_retry_count = 10
puppet_agent_retry_poll_interval_seconds = 20
collectors = ['10.204.217.112:8086']
http_introspect_port = 8107
sandesh_log_level = SYS_ERR

[CLUSTER]

[SERVER]

[IMAGE]

[MONITORING]
monitoring_frequency = 300
monitoring_plugin = server_mgr_ipmi_monitoring.ServerMgrIPMIMonitoring

[INVENTORY]
inventory_plugin = server_mgr_inventory.ServerMgrInventory

UI also shows the same issue

[sundarkh (sundar-kh)]

UI MOnitoring

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/26246
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/26247
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/26248
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/26248
Committed: http://github.org/Juniper/contrail-server-manager/commit/ec3b5b4163ca10b4c54577dbb0579c4f77981dd5
Submitter: Zuul
Branch: master

commit ec3b5b4163ca10b4c54577dbb0579c4f77981dd5
Author: nitishkrishna <email address hidden>
Date: Thu Nov 17 12:13:36 2016 -0800

Closes-Bug: #1642351 - SM lite Monitoring set as false doesn't give correct error msg

The base plugin class function calls were no longer being called due to a change made to make monitoring code part of SM
These were not being called because the inventory and monitoring objects were set to None by default instead of base plugin obj

Change-Id: I03e41a6ab4b4451d43b87241e69ea2a13857cf85

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/26246
Committed: http://github.org/Juniper/contrail-server-manager/commit/3ef259ba6223df9841056f56492b403df70e263a
Submitter: Zuul
Branch: R3.1

commit 3ef259ba6223df9841056f56492b403df70e263a
Author: nitishkrishna <email address hidden>
Date: Thu Nov 17 12:13:36 2016 -0800

Closes-Bug: #1642351 - SM lite Monitoring set as false doesn't give correct error msg

The base plugin class function calls were no longer being called due to a change made to make monitoring code part of SM
These were not being called because the inventory and monitoring objects were set to None by default instead of base plugin obj

Change-Id: I03e41a6ab4b4451d43b87241e69ea2a13857cf85

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/26247
Committed: http://github.org/Juniper/contrail-server-manager/commit/1f43c808a8cb17fe003905c3e52da425c046028d
Submitter: Zuul
Branch: R3.2

commit 1f43c808a8cb17fe003905c3e52da425c046028d
Author: nitishkrishna <email address hidden>
Date: Thu Nov 17 12:13:36 2016 -0800

Closes-Bug: #1642351 - SM lite Monitoring set as false doesn't give correct error msg

The base plugin class function calls were no longer being called due to a change made to make monitoring code part of SM
These were not being called because the inventory and monitoring objects were set to None by default instead of base plugin obj

Change-Id: I03e41a6ab4b4451d43b87241e69ea2a13857cf85