some introspect links missing for contrail-collector, contrail-qe
| Affects | Status | Importance | Assigned to | Milestone | ||
|---|---|---|---|---|---|---|
| Juniper Openstack | Status tracked in Trunk | |||||
| | R3.2 |
In Progress
|
High
|
Megh Bhatt | ||
| | R4.0 |
Fix Committed
|
High
|
Megh Bhatt | ||
| | Trunk |
Fix Committed
|
High
|
Megh Bhatt | ||
Bug Description
R4.0 Build 7 Ubuntu 16.04.2
Similar to bug 1691016 and bug 1691349, some links are missing for contrail-collector and contail-
QE:
====
Seen :
cpuinfo.xml
qe.xml
task.xml
version.xml
Expected:
cpuinfo.xml
qe.xml
sandesh_trace.xml
sandesh_uve.xml
task.xml
version.xml
Collector:
============
Seen:
analytics.xml
collector_uve.xml
cpuinfo.xml
kafka.xml
nodeinfo.xml
process_info.xml
redis.xml
sflow.xml
task.xml
uflow.xml
uve_aggregator.xml
version.xml
Expected:
analytics.xml
collector_uve.xml
cpuinfo.xml
kafka.xml
nodeinfo.xml
process_info.xml
redis.xml
sandesh_trace.xml
sandesh_uve.xml
sflow.xml
task.xml
uflow.xml
uve_aggregator.xml
version.xml
| Changed in juniperopenstack: | |
| assignee: | Raj Reddy (rajreddy) → Nikhil Bansal (nikhilb-u) |
| tags: | added: blocker |
| Changed in juniperopenstack: | |
| assignee: | Nikhil Bansal (nikhilb-u) → Megh Bhatt (meghb) |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: R4.0
commit 4074d8af7592a56
Author: Megh Bhatt <email address hidden>
Date: Sun May 21 00:52:14 2017 -0700
Introspect of daemons missing links to some request files
Global static initialization is used to populate a SandeshHttp
static map with the names of the sandesh IDL files that contain
requests for introspect for a daemon. The static declaration and
definition is done in _html.cpp. In order for the linker to include
the symbols in that translation unit a standalone int is also defined
in the file and used in _types.cpp. However the current usage was just
assigning another static variable this int and as a result the symbols
in the _html.cpp were not included and the map was not populated.
Changed the generated code to assign a value to the standalone int
as part of RequestFromHttp function so that symbols from the
_html.cpp translation units are pulled in.
Change-Id: Id5515bb75fab09
Closes-Bug: #1691605
Closes-Bug: #1691949
Review in progress for https:/
Submitter: Megh Bhatt (<email address hidden>)
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit f04e697dac10146
Author: Megh Bhatt <email address hidden>
Date: Sun May 21 00:52:14 2017 -0700
Introspect of daemons missing links to some request files
Global static initialization is used to populate a SandeshHttp
static map with the names of the sandesh IDL files that contain
requests for introspect for a daemon. The static declaration and
definition is done in _html.cpp. In order for the linker to include
the symbols in that translation unit a standalone int is also defined
in the file and used in _types.cpp. However the current usage was just
assigning another static variable this int and as a result the symbols
in the _html.cpp were not included and the map was not populated.
Changed the generated code to assign a value to the standalone int
as part of RequestFromHttp function so that symbols from the
_html.cpp translation units are pulled in.
Change-Id: Id5515bb75fab09
Closes-Bug: #1691605
Closes-Bug: #1691949
(cherry picked from commit 4074d8af7592a56
Review in progress for https:/
Submitter: Antoine Eiche (<email address hidden>)
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: R3.2
commit 60c4f43433d73bf
Author: Megh Bhatt <email address hidden>
Date: Sun May 21 00:52:14 2017 -0700
Introspect of daemons missing links to some request files
Global static initialization is used to populate a SandeshHttp
static map with the names of the sandesh IDL files that contain
requests for introspect for a daemon. The static declaration and
definition is done in _html.cpp. In order for the linker to include
the symbols in that translation unit a standalone int is also defined
in the file and used in _types.cpp. However the current usage was just
assigning another static variable this int and as a result the symbols
in the _html.cpp were not included and the map was not populated.
Changed the generated code to assign a value to the standalone int
as part of RequestFromHttp function so that symbols from the
_html.cpp translation units are pulled in.
Change-Id: Id5515bb75fab09
Closes-Bug: #1691605
Closes-Bug: #1691949
(cherry picked from commit 4074d8af7592a56
Review in progress for https:/
Submitter: Megh Bhatt (<email address hidden>)