diff options
author | Tianjie Xu <xunchang@google.com> | 2020-01-17 00:17:41 +0100 |
---|---|---|
committer | Tianjie Xu <xunchang@google.com> | 2020-01-17 00:17:41 +0100 |
commit | eb27bfe1e8e64833cb697fe16913e0713bf9af8f (patch) | |
tree | 990ff845639b8a7184fb678c53d69c0a2bf1dbae /recovery-persist.cpp | |
parent | Merge "edify: update for bison 3.5" (diff) | |
download | android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.tar android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.tar.gz android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.tar.bz2 android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.tar.lz android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.tar.xz android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.tar.zst android_bootable_recovery-eb27bfe1e8e64833cb697fe16913e0713bf9af8f.zip |
Diffstat (limited to 'recovery-persist.cpp')
-rw-r--r-- | recovery-persist.cpp | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/recovery-persist.cpp b/recovery-persist.cpp index 6dbf86253..ad101ede2 100644 --- a/recovery-persist.cpp +++ b/recovery-persist.cpp @@ -35,12 +35,10 @@ #include <string.h> #include <unistd.h> -#include <limits> #include <string> #include <android-base/file.h> #include <android-base/logging.h> -#include <metricslogger/metrics_logger.h> #include <private/android_logger.h> /* private pmsg functions */ #include "recovery_utils/logging.h" @@ -112,20 +110,6 @@ ssize_t logsave( return android::base::WriteStringToFile(buffer, destination.c_str()); } -// Parses the LAST_INSTALL file and reports the update metrics saved under recovery mode. -static void report_metrics_from_last_install(const std::string& file_name) { - auto metrics = ParseLastInstall(file_name); - // TODO(xunchang) report the installation result. - for (const auto& [event, value] : metrics) { - if (value > std::numeric_limits<int>::max()) { - LOG(WARNING) << event << " (" << value << ") exceeds integer max."; - } else { - LOG(INFO) << "Uploading " << value << " to " << event; - android::metricslogger::LogHistogram(event, value); - } - } -} - int main(int argc, char **argv) { /* Is /cache a mount?, we have been delivered where we are not wanted */ @@ -157,7 +141,6 @@ int main(int argc, char **argv) { if (has_cache) { // Collects and reports the non-a/b update metrics from last_install; and removes the file // to avoid duplicate report. - report_metrics_from_last_install(LAST_INSTALL_FILE_IN_CACHE); if (access(LAST_INSTALL_FILE_IN_CACHE, F_OK) && unlink(LAST_INSTALL_FILE_IN_CACHE) == -1) { PLOG(ERROR) << "Failed to unlink " << LAST_INSTALL_FILE_IN_CACHE; } @@ -181,7 +164,6 @@ int main(int argc, char **argv) { // For those device without /cache, the last_install file has been copied to // /data/misc/recovery from pmsg. Looks for the sideload history only. if (!has_cache) { - report_metrics_from_last_install(LAST_INSTALL_FILE); if (access(LAST_INSTALL_FILE, F_OK) && unlink(LAST_INSTALL_FILE) == -1) { PLOG(ERROR) << "Failed to unlink " << LAST_INSTALL_FILE; } |