summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefano Cianciulli <scianciulli@google.com>2024-05-10 10:49:46 +0000
committerStefano Cianciulli <scianciulli@google.com>2024-05-10 10:49:46 +0000
commit2638c85b928af6659f6b7d3d0ef6fa735f791145 (patch)
tree6a3b0084393510e12cf8642e081a6f8d2834695e
parentd76327758c7263601dc8177d0b3ab2afdd8985fa (diff)
downloadproto_logging-2638c85b928af6659f6b7d3d0ef6fa735f791145.tar.gz
Create new PreRebootDexoptJobEnded pushed atom in stats/atoms/art and reference it from atoms.proto
Bug: 336239721 Metrics Design Review: eldar/824633423 Test: presubmits Merged-In: I3143615f0741c786354a1b1e6ba69ab576a05d7 Change-Id: I0e3eb50bca2f805f38e7b4655139219b1c770796
-rw-r--r--stats/atoms.proto1
-rw-r--r--stats/atoms/art/background_extension_dexopt_atoms.proto46
2 files changed, 47 insertions, 0 deletions
diff --git a/stats/atoms.proto b/stats/atoms.proto
index 170e6b87..944a46c9 100644
--- a/stats/atoms.proto
+++ b/stats/atoms.proto
@@ -1141,6 +1141,7 @@ message Atom {
extensions 873; // LeScanAbused le_scan_abused
extensions 874; // LeAdvStateChanged le_adv_state_changed
extensions 875; // LeAdvErrorReported le_adv_error_reported
+ extensions 883; // PreRebootDexoptJobEnded pre_reboot_dexopt_job_ended
extensions 9999; // Atom9999 atom_9999
// StatsdStats tracks platform atoms with ids up to 900.
// Update StatsdStats::kMaxPushedAtomId when atom ids here approach that value.
diff --git a/stats/atoms/art/background_extension_dexopt_atoms.proto b/stats/atoms/art/background_extension_dexopt_atoms.proto
index 9a4e5030..383e2c1f 100644
--- a/stats/atoms/art/background_extension_dexopt_atoms.proto
+++ b/stats/atoms/art/background_extension_dexopt_atoms.proto
@@ -11,6 +11,7 @@ option java_multiple_files = true;
extend Atom {
optional BackgroundDexoptJobEnded background_dexopt_job_ended = 467 [(module) = "art"];
+ optional PreRebootDexoptJobEnded prereboot_dexopt_job_ended = 883 [(module) = "art"];
}
/**
@@ -114,3 +115,48 @@ extend Atom {
optional Pass pass = 8;
}
+
+/**
+ * Logs when a pre-reboot dexopt job is ended.
+ *
+ * TODO(b/336239721): Mention the file containing the code logging this atom.
+ */
+message PreRebootDexoptJobEnded {
+ // Overall status of the job right before the reboot.
+ enum Status {
+ STATUS_UNKNOWN = 0;
+ STATUS_NOT_RUN = 1;
+ STATUS_FINISHED = 2;
+ STATUS_FAILED = 3;
+ STATUS_CANCELLED = 4;
+ }
+ optional Status status = 1;
+
+ // Number of packages successfully optimized.
+ optional int32 optimized_package_count = 2;
+ // Number of packages failed to optimize.
+ optional int32 failed_package_count = 3;
+ // Number of packages skipped.
+ optional int32 skipped_package_count = 4;
+ // Total number of packages scanned.
+ optional int32 total_package_count = 5;
+
+ // Duration of the job run (delta between when then job is run and when the
+ // job is finished or failed), in milliseconds.
+ optional int64 job_duration_millis = 6;
+ // Latency of the job run (delta between when the job is scheduled and when
+ // the job is finished or failed, including the time period between
+ // cancellations and reruns), in milliseconds.
+ optional int64 job_latency_millis = 7;
+
+ // Number of packages that have Pre-reboot Dexopt artifacts after the
+ // reboot. Normally, this should be equal to `optimized_package_count`, but
+ // artifacts may be lost due to bugs.
+ optional int32 packages_with_artifacts_after_reboot_count = 8;
+ // Number of packages that have Pre-reboot Dexopt artifacts that are usable
+ // after the reboot. Normally, this should be equal to
+ // `packages_with_artifacts_after_reboot_count`, but artifacts may be
+ // invalid due to bugs.
+ optional int32 packages_with_artifacts_usable_after_reboot_count = 9;
+}
+