scope-aggregator

log sometimes does not state scope id of agg

Bug #1549844 reported by Kyle Nitzsche
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
scope-aggregator
Fix Released
Medium
Kyle Nitzsche

Bug Description

When the agg scope starts reporting which scopes are NOT REGISTERED or NOT ENABLED, it does not prefix the current runtime scope ID but it should:

"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.dupday_day, local_id: day_localId"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.weatherchannel, local_id: weather_channel_localId"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.holidays_holidays, local_id: holidays_localID"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.events_events, local_id: events_localID"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.tasks_sctasks, local_id: tasks_localID"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.contacts_contacts, local_id: contacts"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.fitbit_fitbit, local_id: fitbit_localId"
"com.canonical.scopes.dashboard_dashboard: ADDING CATEGORY KEYWORD child scope: com.canonical.scopes.calls_calls, by keyword recent, to department: "
"com.canonical.scopes.dashboard_dashboard: ADDING CATEGORY KEYWORD child scope: com.canonical.scopes.texts_texts, by keyword recent, to department: "
"com.canonical.scopes.aggregator_aggregator: NOT using departments"
"com.canonical.scopes.aggregator_aggregator: scope is NOT REGISTERED, skipping: com.canonical.scopes.dupday_day"
"com.canonical.scopes.aggregator_aggregator: scope is NOT ENABLED, skipping: com.canonical.scopes.weatherchannel"
"com.canonical.scopes.aggregator_aggregator: scope is NOT ENABLED, skipping: com.canonical.scopes.events_events"
"com.canonical.scopes.aggregator_aggregator: scope is NOT ENABLED, skipping: com.canonical.scopes.tasks_sctasks"
"com.canonical.scopes.aggregator_aggregator: scope is NOT ENABLED, skipping: com.canonical.scopes.fitbit_fitbit"

Related branches

lp:~knitzsche/scope-aggregator/fixes-bug-1549844-runtime-scopeid-in-log-always
Jin: Pending requested
Zhang Enwei: Pending requested
Penk Chen: Pending requested
Gary.Wang: Pending requested
Diff: 39 lines (+4/-4)
2 files modified
CMakeLists.txt (+1/-1)
src/query.cpp (+3/-3)
lp:scope-aggregator
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

With the MR, here's the log results:

"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.dupday_day, local_id: day_localId"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.weatherchannel, local_id: weather_channel_localId"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.holidays_holidays, local_id: holidays_localID"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.events_events, local_id: events_localID"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.tasks_sctasks, local_id: tasks_localID"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.contacts_contacts, local_id: contacts"
"com.canonical.scopes.dashboard_dashboard: ADDING DECLARED child scope: com.canonical.scopes.fitbit_fitbit, local_id: fitbit_localId"
"com.canonical.scopes.dashboard_dashboard: ADDING CATEGORY KEYWORD child scope: com.canonical.scopes.calls_calls, by keyword recent, to department: "
"com.canonical.scopes.dashboard_dashboard: ADDING CATEGORY KEYWORD child scope: com.canonical.scopes.texts_texts, by keyword recent, to department: "
"com.canonical.scopes.dashboard_dashboard: NOT using departments"
"com.canonical.scopes.dashboard_dashboard: scope is NOT REGISTERED, skipping: com.canonical.scopes.dupday_day"
"com.canonical.scopes.dashboard_dashboard: scope is NOT ENABLED, skipping: com.canonical.scopes.weatherchannel"
"com.canonical.scopes.dashboard_dashboard: scope is NOT ENABLED, skipping: com.canonical.scopes.events_events"
"com.canonical.scopes.dashboard_dashboard: scope is NOT ENABLED, skipping: com.canonical.scopes.tasks_sctasks"
"com.canonical.scopes.dashboard_dashboard: scope is NOT ENABLED, skipping: com.canonical.scopes.fitbit_fitbit"

Kyle Nitzsche (knitzsche)
Changed in scope-aggregator:
assignee: nobody → Kyle Nitzsche (knitzsche)
status: New → In Progress
importance: Undecided → Medium
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

fixed in scope-agg 4.3.1 (bzr rev 159) (which is not released & bzr tagged because other changes will be merged for a 4.4 release)

Changed in scope-aggregator:
status: In Progress → Fix Committed
Kyle Nitzsche (knitzsche)
Changed in scope-aggregator:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.