update_engine: Update complete event with exclusions When updates are complete, currently all the AppIDs within the request parameter are considered to be updated. This however is not true with exclusions as non-critical AppIDs (e.g. DLCs) can be excluded. This change sends the correct event for |kTypeUpdateComplete| event type. BUG=chromium:928805 TEST=FEATURES=test emerge-$B update_engine update_engine-client Change-Id: I8c21721688fb8a6501316cb87bd0a6f8e005b7ae Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2247489 Tested-by: Jae Hoon Kim <[email protected]> Reviewed-by: Amin Hassani <[email protected]> Reviewed-by: Manoj Gupta <[email protected]> Auto-Submit: Jae Hoon Kim <[email protected]> Commit-Queue: Andrew Lassalle <[email protected]>
diff --git a/metrics_utils.cc b/metrics_utils.cc index efbd067..0d333ca 100644 --- a/metrics_utils.cc +++ b/metrics_utils.cc
@@ -122,6 +122,7 @@ case ErrorCode::kOmahaUpdateIgnoredOverCellular: case ErrorCode::kNoUpdate: case ErrorCode::kFirstActiveOmahaPingSentPersistenceError: + case ErrorCode::kPackageExcludedFromUpdate: return metrics::AttemptResult::kInternalError; // Special flags. These can't happen (we mask them out above) but @@ -236,6 +237,7 @@ case ErrorCode::kRollbackNotPossible: case ErrorCode::kFirstActiveOmahaPingSentPersistenceError: case ErrorCode::kVerityCalculationError: + case ErrorCode::kPackageExcludedFromUpdate: break; // Special flags. These can't happen (we mask them out above) but