Reject the config if condition config has errors. And add log tag. (c9fa2f6d) · Commits · e / os / android_frameworks_base

cmds/statsd/src/metrics/metrics_manager_util.cpp

+33 −15

Original line number	Diff line number	Diff line
		@@ -14,6 +14,9 @@
		* limitations under the License.
		*/

		#define DEBUG true // STOPSHIP if true
		#include "Log.h"

		#include "../condition/CombinationConditionTracker.h"
		#include "../condition/SimpleConditionTracker.h"
		#include "../external/StatsPullerManager.h"
		@@ -219,9 +222,12 @@ bool initMetrics(const StatsdConfig& config, const unordered_map<string, int>& l

		int conditionIndex = -1;
		if (metric.has_condition()) {
		handleMetricWithConditions(metric.condition(), metricIndex, conditionTrackerMap,
		metric.links(), allConditionTrackers, conditionIndex,
		conditionToMetricMap);
		bool good = handleMetricWithConditions(
		metric.condition(), metricIndex, conditionTrackerMap, metric.links(),
		allConditionTrackers, conditionIndex, conditionToMetricMap);
		if (!good) {
		return false;
		}
		} else {
		if (metric.links_size() > 0) {
		ALOGW("metrics has a EventConditionLink but doesn't have a condition");
		@@ -287,9 +293,12 @@ bool initMetrics(const StatsdConfig& config, const unordered_map<string, int>& l
		int conditionIndex = -1;

		if (metric.has_condition()) {
		handleMetricWithConditions(metric.condition(), metricIndex, conditionTrackerMap,
		metric.links(), allConditionTrackers, conditionIndex,
		conditionToMetricMap);
		bool good = handleMetricWithConditions(
		metric.condition(), metricIndex, conditionTrackerMap, metric.links(),
		allConditionTrackers, conditionIndex, conditionToMetricMap);
		if (!good) {
		return false;
		}
		} else {
		if (metric.links_size() > 0) {
		ALOGW("metrics has a EventConditionLink but doesn't have a condition");
		@@ -321,9 +330,12 @@ bool initMetrics(const StatsdConfig& config, const unordered_map<string, int>& l

		int conditionIndex = -1;
		if (metric.has_condition()) {
		handleMetricWithConditions(metric.condition(), metricIndex, conditionTrackerMap,
		metric.links(), allConditionTrackers, conditionIndex,
		conditionToMetricMap);
		bool good = handleMetricWithConditions(
		metric.condition(), metricIndex, conditionTrackerMap, metric.links(),
		allConditionTrackers, conditionIndex, conditionToMetricMap);
		if (!good) {
		return false;
		}
		} else {
		if (metric.links_size() > 0) {
		ALOGW("metrics has a EventConditionLink but doesn't have a condition");
		@@ -368,9 +380,12 @@ bool initMetrics(const StatsdConfig& config, const unordered_map<string, int>& l

		int conditionIndex = -1;
		if (metric.has_condition()) {
		handleMetricWithConditions(metric.condition(), metricIndex, conditionTrackerMap,
		metric.links(), allConditionTrackers, conditionIndex,
		conditionToMetricMap);
		bool good = handleMetricWithConditions(
		metric.condition(), metricIndex, conditionTrackerMap, metric.links(),
		allConditionTrackers, conditionIndex, conditionToMetricMap);
		if (!good) {
		return false;
		}
		} else {
		if (metric.links_size() > 0) {
		ALOGW("metrics has a EventConditionLink but doesn't have a condition");
		@@ -414,9 +429,12 @@ bool initMetrics(const StatsdConfig& config, const unordered_map<string, int>& l

		int conditionIndex = -1;
		if (metric.has_condition()) {
		handleMetricWithConditions(metric.condition(), metricIndex, conditionTrackerMap,
		metric.links(), allConditionTrackers, conditionIndex,
		conditionToMetricMap);
		bool good = handleMetricWithConditions(
		metric.condition(), metricIndex, conditionTrackerMap, metric.links(),
		allConditionTrackers, conditionIndex, conditionToMetricMap);
		if (!good) {
		return false;
		}
		} else {
		if (metric.links_size() > 0) {
		ALOGW("metrics has a EventConditionLink but doesn't have a condition");

cmds/statsd/tests/MetricsManager_test.cpp

+34 −0

Original line number	Diff line number	Diff line
		@@ -163,6 +163,25 @@ StatsdConfig buildMissingMatchers() {
		return config;
		}

		StatsdConfig buildMissingCondition() {
		StatsdConfig config;
		config.set_name("12345");

		CountMetric* metric = config.add_count_metric();
		metric->set_name("3");
		metric->set_what("SCREEN_EVENT");
		metric->mutable_bucket()->set_bucket_size_millis(30 * 1000L);
		metric->set_condition("SOME_CONDITION");

		LogEntryMatcher* eventMatcher = config.add_log_entry_matcher();
		eventMatcher->set_name("SCREEN_EVENT");

		SimpleLogEntryMatcher* simpleLogEntryMatcher = eventMatcher->mutable_simple_log_entry_matcher();
		simpleLogEntryMatcher->set_tag(2);

		return config;
		}

		StatsdConfig buildDimensionMetricsWithMultiTags() {
		StatsdConfig config;
		config.set_name("12345");
		@@ -308,6 +327,21 @@ TEST(MetricsManagerTest, TestMissingMatchers) {
		trackerToMetricMap, trackerToConditionMap));
		}

		TEST(MetricsManagerTest, TestMissingCondition) {
		StatsdConfig config = buildMissingCondition();
		set<int> allTagIds;
		vector<sp<LogMatchingTracker>> allLogEntryMatchers;
		vector<sp<ConditionTracker>> allConditionTrackers;
		vector<sp<MetricProducer>> allMetricProducers;
		std::vector<sp<AnomalyTracker>> allAnomalyTrackers;
		unordered_map<int, std::vector<int>> conditionToMetricMap;
		unordered_map<int, std::vector<int>> trackerToMetricMap;
		unordered_map<int, std::vector<int>> trackerToConditionMap;
		EXPECT_FALSE(initStatsdConfig(config, allTagIds, allLogEntryMatchers, allConditionTrackers,
		allMetricProducers, allAnomalyTrackers, conditionToMetricMap,
		trackerToMetricMap, trackerToConditionMap));
		}

		TEST(MetricsManagerTest, TestCircleConditionDependency) {
		StatsdConfig config = buildCircleConditions();
		set<int> allTagIds;