From 3fd2ea58f71e0efbf515130bcf1b9e8fcb13e49a Mon Sep 17 00:00:00 2001 From: Automerger Merge Worker Date: Fri, 3 Apr 2020 18:14:10 +0000 Subject: [PATCH] Define rcode=12~15 (UNASSIGNED) in rcode enum type RCODE Name 11 NS_R_MAX 12 NS_R_UNASSIGNED12 13 NS_R_UNASSIGNED13 14 NS_R_UNASSIGNED14 15 NS_R_UNASSIGNED15 16 NS_R_BADVERS According the stats data of Dns Resolver module, Some DNS Servers might return undefined code to devices. Without the enum definition, that would be noise for our dashboard Bug: 150194702 Test: Manual test with statsd_testdrive Merged-In: I178164f37b70d0ce86dfb7e37e01b663abbf82f2 Change-Id: I2c2f3307487b85f360fe8733667595617e3716d0 (cherry picked from commit 4ec941aa071c68fe29cdb50d2516678441b82105) --- core/proto/android/stats/dnsresolver/dns_resolver.proto | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/core/proto/android/stats/dnsresolver/dns_resolver.proto b/core/proto/android/stats/dnsresolver/dns_resolver.proto index 448c12dd8d293..61b9b25fe7cf4 100644 --- a/core/proto/android/stats/dnsresolver/dns_resolver.proto +++ b/core/proto/android/stats/dnsresolver/dns_resolver.proto @@ -62,6 +62,13 @@ enum NsRcode { NS_R_NOTAUTH = 9; // Not authoritative for zone NS_R_NOTZONE = 10; // Zone of record different from zone section NS_R_MAX = 11; + // Define rcode=12~15(UNASSIGNED) in rcode enum type. + // Some DNS Servers might return undefined code to devices. + // Without the enum definition, that would be noise for our dashboard. + NS_R_UNASSIGNED12 = 12; // Unassigned + NS_R_UNASSIGNED13 = 13; // Unassigned + NS_R_UNASSIGNED14 = 14; // Unassigned + NS_R_UNASSIGNED15 = 15; // Unassigned // The following are EDNS extended rcodes NS_R_BADVERS = 16; // The following are TSIG errors