diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2023-12-06 00:26:34 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2023-12-06 00:26:34 +0000 |
commit | 2ab8544f4ec6df294da88b29d631270641935055 (patch) | |
tree | 415fba94797f79e36b388e9257f570f1e55b276b | |
parent | fbdaf3f9a08d7685d5a9949d5ca74eacc58f2502 (diff) | |
parent | 5c06b39912a5c98a40ce10b79d6c1343ca6021ca (diff) | |
download | update_engine-2ab8544f4ec6df294da88b29d631270641935055.tar.gz |
Snap for 11180250 from 5c06b39912a5c98a40ce10b79d6c1343ca6021ca to 24Q1-release
Change-Id: I1d443167760235e000ae2fb981e1ff81b000d8ff
-rw-r--r-- | aosp/update_attempter_android.cc | 20 | ||||
-rw-r--r-- | common/error_code.h | 2 | ||||
-rw-r--r-- | common/error_code_utils.cc | 4 | ||||
-rw-r--r-- | metrics_utils.cc | 4 |
4 files changed, 20 insertions, 10 deletions
diff --git a/aosp/update_attempter_android.cc b/aosp/update_attempter_android.cc index ea83d9a8..0f4b2e4b 100644 --- a/aosp/update_attempter_android.cc +++ b/aosp/update_attempter_android.cc @@ -242,18 +242,18 @@ bool UpdateAttempterAndroid::ApplyPayload( const vector<string>& key_value_pair_headers, Error* error) { if (status_ == UpdateStatus::UPDATED_NEED_REBOOT) { - return LogAndSetGenericError( - error, - __LINE__, - __FILE__, - "An update already applied, waiting for reboot"); + return LogAndSetError(error, + __LINE__, + __FILE__, + "An update already applied, waiting for reboot", + ErrorCode::kUpdateAlreadyInstalled); } if (processor_->IsRunning()) { - return LogAndSetGenericError( - error, - __LINE__, - __FILE__, - "Already processing an update, cancel it first."); + return LogAndSetError(error, + __LINE__, + __FILE__, + "Already processing an update, cancel it first.", + ErrorCode::kUpdateProcessing); } DCHECK_EQ(status_, UpdateStatus::IDLE); diff --git a/common/error_code.h b/common/error_code.h index 79245797..8954f720 100644 --- a/common/error_code.h +++ b/common/error_code.h @@ -88,6 +88,8 @@ enum class ErrorCode : int { kPackageExcludedFromUpdate = 62, kPostInstallMountError = 63, kOverlayfsenabledError = 64, + kUpdateProcessing = 65, + kUpdateAlreadyInstalled = 66, // VERY IMPORTANT! When adding new error codes: // diff --git a/common/error_code_utils.cc b/common/error_code_utils.cc index 12a98bf8..421e1249 100644 --- a/common/error_code_utils.cc +++ b/common/error_code_utils.cc @@ -177,6 +177,10 @@ string ErrorCodeToString(ErrorCode code) { return "ErrorCode::kPostInstallMountError"; case ErrorCode::kOverlayfsenabledError: return "ErrorCode::kOverlayfsenabledError"; + case ErrorCode::kUpdateProcessing: + return "ErrorCode::kUpdateProcessing"; + case ErrorCode::kUpdateAlreadyInstalled: + return "ErrorCode::kUpdateAlreadyInstalled"; // Don't add a default case to let the compiler warn about newly added // error codes which should be added here. } diff --git a/metrics_utils.cc b/metrics_utils.cc index 94a05201..35bbcf4c 100644 --- a/metrics_utils.cc +++ b/metrics_utils.cc @@ -138,6 +138,8 @@ metrics::AttemptResult GetAttemptResult(ErrorCode code) { case ErrorCode::kTestImageFlag: case ErrorCode::kTestOmahaUrlFlag: case ErrorCode::kSpecialFlags: + case ErrorCode::kUpdateProcessing: + case ErrorCode::kUpdateAlreadyInstalled: break; } @@ -245,6 +247,8 @@ metrics::DownloadErrorCode GetDownloadErrorCode(ErrorCode code) { case ErrorCode::kNotEnoughSpace: case ErrorCode::kDeviceCorrupted: case ErrorCode::kPackageExcludedFromUpdate: + case ErrorCode::kUpdateProcessing: + case ErrorCode::kUpdateAlreadyInstalled: break; // Special flags. These can't happen (we mask them out above) but |