Getting empty string as the prouter name from analytics UVE

[VishnuVardhan V]

Getting empty string as the prouter name from analytics UVE.we see this in nodeg25.
[
{
href: "http://nodeg25:8081/analytics/uves/prouter/?flat",
name: ""
},
]

[Sundaresan Rajangam (srajanga)]

contrail-snmp-collector sends PRouter UVE with name="" or None.

Attached snapshot of PRouter UVE with invalid name in UVE cache from the introspect

[Sundaresan Rajangam (srajanga)]

from contrail-snmp-collector.log

05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: @run_scanner(1865): loaded /tmp/tmpnAAErD/out.data
05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: Add uve <, 3558> in the [ObjectPRouter:PRouterUVE] map
05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: @do_work(analyze):State fast_scan(5)->chngd: 5!
05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: @find_fix_name: snmp name differs from configured name bng-contrail-qfx51-8, fixed for this run
05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: @send_uve:Processed 5!
05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: @do_work(1865):Processed 5!
05/20/2015 03:24:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: done work [u'bng-contrail-ex4200-2', '', u'bng-contrail-qfx51-7', u'hooper', u'bng-contrail-ex4500-3']
05/20/2015 03:25:47 PM [nodeg25:contrail-snmp-collector:Analytics:0]: @run: ittr(1866)

[Raj Reddy (rajreddy)]

most likely the empty name is coming from the config..
Ted will fix this by ignoring prouter w/ empty names..

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/10972
Submitter: ted ghose (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/10995
Submitter: ted ghose (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/10972
Committed: http://github.org/Juniper/contrail-controller/commit/db7381a9dc6307ea47642b9496bdd4c3940e91cb
Submitter: Zuul
Branch: master

commit db7381a9dc6307ea47642b9496bdd4c3940e91cb
Author: Ted Ghose <email address hidden>
Date: Thu May 28 00:22:38 2015 -0700

Fix restart and empty prouter name

Change-Id: Idda0a2a5a9ecbae3a2b482178493e59a283b1849
Closes-Bug: #1456981
Closes-Bug: #1451687

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/10995
Committed: http://github.org/Juniper/contrail-controller/commit/5d664fb120fa4bd2d07b0fd083c9569015280e34
Submitter: Zuul
Branch: R2.20

commit 5d664fb120fa4bd2d07b0fd083c9569015280e34
Author: Ted Ghose <email address hidden>
Date: Thu May 28 00:22:38 2015 -0700

Fix restart and empty prouter name

Change-Id: I520d80e99811b1681bd21b7fb503cdf9f018ae0b
Closes-Bug: #1456981
Closes-Bug: #1451687