Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d23b1fe6 authored by Chiachang Wang's avatar Chiachang Wang Committed by android-build-merger
Browse files

Merge "Migrate NetworkStack metrics off StatsLog.write" into qt-dev 

am: fa124d7bf4

Change-Id: I1bfe856d771bf56fe33790aea22eceae2001921a
parents efc6a309 53817ca8

Android.bp

+9 −0
Original line number Diff line number Diff line
@@ -37,6 +37,7 @@ android_library { 
        "src/**/*.java",

        ":framework-networkstack-shared-srcs",

        ":services-networkstack-shared-srcs",

        ":statslog-networkstack-java-gen",

    ],

    static_libs: [

        "androidx.annotation_annotation",
@@ -104,3 +105,11 @@ android_app { 
    certificate: "networkstack",

    manifest: "AndroidManifest.xml",

}



genrule {

    name: "statslog-networkstack-java-gen",

    tools: ["stats-log-api-gen"],

    cmd: "$(location stats-log-api-gen) --java $(out) --module network_stack" +

         " --javaPackage com.android.networkstack.metrics --javaClass NetworkStackStatsLog",

    out: ["com/android/networkstack/metrics/NetworkStackStatsLog.java"],

}

src/android/net/metrics/DataStallDetectionStats.java→src/com/android/networkstack/metrics/DataStallDetectionStats.java

+1 −1
Original line number Diff line number Diff line
@@ -14,7 +14,7 @@ 
 * limitations under the License.

 */



package android.net.metrics;

package com.android.networkstack.metrics;



import android.annotation.NonNull;

import android.annotation.Nullable;

src/android/net/metrics/DataStallStatsUtils.java→src/com/android/networkstack/metrics/DataStallStatsUtils.java

+2 −6
Original line number Diff line number Diff line
@@ -14,13 +14,12 @@ 
 * limitations under the License.

 */



package android.net.metrics;

package com.android.networkstack.metrics;



import android.annotation.NonNull;

import android.annotation.Nullable;

import android.net.captiveportal.CaptivePortalProbeResult;

import android.util.Log;

import android.util.StatsLog;



import com.android.internal.util.HexDump;

import com.android.server.connectivity.nano.DataStallEventProto;
@@ -37,13 +36,11 @@ import com.android.server.connectivity.nano.DataStallEventProto; 
 */

public class DataStallStatsUtils {

    private static final String TAG = DataStallStatsUtils.class.getSimpleName();

    private static final int DATA_STALL_EVENT_ID = 121;

    private static final boolean DBG = false;



    private static int probeResultToEnum(@Nullable final CaptivePortalProbeResult result) {

        if (result == null) return DataStallEventProto.INVALID;



        // TODO: Add partial connectivity support.

        if (result.isSuccessful()) {

            return DataStallEventProto.VALID;

        } else if (result.isPortal()) {
@@ -65,8 +62,7 @@ public class DataStallStatsUtils { 
            Log.d(TAG, "write: " + stats + " with result: " + validationResult

                    + ", dns: " + HexDump.toHexString(stats.mDns));

        }

        // TODO(b/124613085): Update API once the public StatsLog API is ready.

        StatsLog.write(DATA_STALL_EVENT_ID,

        NetworkStackStatsLog.write(NetworkStackStatsLog.DATA_STALL_EVENT,

                stats.mEvaluationType,

                validationResult,

                stats.mNetworkType,

src/com/android/server/connectivity/NetworkMonitor.java

+2 −2
Original line number Diff line number Diff line
@@ -65,8 +65,6 @@ import android.net.TrafficStats; 
import android.net.Uri;

import android.net.captiveportal.CaptivePortalProbeResult;

import android.net.captiveportal.CaptivePortalProbeSpec;

import android.net.metrics.DataStallDetectionStats;

import android.net.metrics.DataStallStatsUtils;

import android.net.metrics.IpConnectivityLog;

import android.net.metrics.NetworkEvent;

import android.net.metrics.ValidationProbeEvent;
@@ -101,6 +99,8 @@ import com.android.internal.util.RingBufferIndices; 
import com.android.internal.util.State;

import com.android.internal.util.StateMachine;

import com.android.networkstack.R;

import com.android.networkstack.metrics.DataStallDetectionStats;

import com.android.networkstack.metrics.DataStallStatsUtils;



import java.io.IOException;

import java.net.HttpURLConnection;

tests/src/com/android/server/connectivity/NetworkMonitorTest.java

+3 −2
Original line number Diff line number Diff line
@@ -59,8 +59,6 @@ import android.net.Network; 
import android.net.NetworkCapabilities;

import android.net.NetworkInfo;

import android.net.captiveportal.CaptivePortalProbeResult;

import android.net.metrics.DataStallDetectionStats;

import android.net.metrics.DataStallStatsUtils;

import android.net.metrics.IpConnectivityLog;

import android.net.util.SharedLog;

import android.net.wifi.WifiInfo;
@@ -78,6 +76,9 @@ import android.util.ArrayMap; 
import androidx.test.filters.SmallTest;

import androidx.test.runner.AndroidJUnit4;



import com.android.networkstack.metrics.DataStallDetectionStats;

import com.android.networkstack.metrics.DataStallStatsUtils;



import org.junit.After;

import org.junit.Before;

import org.junit.Test;