summaryrefslogtreecommitdiff
path: root/services/incremental/test/IncrementalServiceTest.cpp
AgeCommit message (Collapse)Author
2021-06-08Proper retrying DL installation sessions.Alex Buynytskyy
Plus more robust handling of broken DLs. Bug: 190012477 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest com.google.android.packageinstallerv2proxy.host.gts.IncrementalInstallerHostTest Change-Id: I5cb037d49cd2b140bed1045c99f072112495acfc
2021-05-28[incremental/metrics] report uid of last read errorSongchun Fan
BUG: 187354818 Test: atest service.incremental_test Change-Id: I632b747440ed4faef31266a3a7b18b7a382ff2cf
2021-05-11[incremental] use same sysfs name for setOptionsSongchun Fan
Otherwise we have errors during applyStorageParams: 05-10 15:35:32.363 562 625 E IncrementalService: applyStorageParams failed: Status(-8, EX_SERVICE_SPECIFIC): '-95: ' 05-10 15:35:32.363 562 625 E incfs-dataloaderconnector: setStorageParams failed with error: -95 05-10 15:35:32.363 562 625 E incfs-dataloaderconnector: DataLoader supports UID 05-10 15:35:32.367 0 0 E incfs : Can't change sysfs_name mount option on remount BUG: 187308584 Test: atest CtsContentTestCases:android.content.pm.cts.PackageManagerShellCommandIncrementalTest#testInstallWithIdSigNoMissingPages Change-Id: Ic2146aa7855e13de9f96794639de556f64e93701
2021-05-04[incremental] report new metrics from Incremental ServiceSongchun Fan
BUG: 184844615 Test: atest service.incremental_test Change-Id: Ibbfe68851a4adfd620db9652f1fda550a3c6fb2c
2021-05-03[IncrementalService] add last read error in dumpsysSongchun Fan
BUG: 184844615 Test: dumpsys Change-Id: I3e8178c8a62dc5d09c718b140bb48bda85a3bae5
2021-04-30[IncrementalService] use sysfs_name in mount options and dumpsysSongchun Fan
BUG: 184844615 Test: adb shell dumpsys incremental Change-Id: Ib6bbbede6d4fbed0bcef8d37f377e706b9d1af28
2021-04-27Merge "[incremental] expose more metrics fields" into sc-devSongchun Fan
2021-04-26[incremental] expose more metrics fieldsSongchun Fan
Exposing more data loader states as per go/incremental-crash-reports-1-pager. BUG: 184844615 Test: atest service.incremental_test Test: atest android.cts.statsdatom.incremental.AppErrorAtomTests Change-Id: I532513453411b2ccdb21311d0bc3dee0641837db
2021-04-26Retry on unavailable.Alex Buynytskyy
Bug: 182214420 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: Iaf61b6825ced45ffdc7e9c87dfea830e50633476
2021-04-16[pm] remove incremental startable/unstartable codeSongchun Fan
Also remove streaming health status reporting which could cause startable state change because it is also not needed any more. BUG: 171920377 Test: builds Change-Id: I7284e7a63df79da7dbf3d16ff64302b3d1ce1348
2021-04-08Installation hardening: reducing read timeout during installation.Alex Buynytskyy
And then restoring post-install. Bug: 160635296 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: I0821458bf92db162518a2cbcb7499cd7544e64f2
2021-03-25More logging and better retry logic.Alex Buynytskyy
Bug: 183551663 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: I1b6da82cd4472cc5671b0b60fb12222b4ed08e17
2021-03-24[incfs] Space trimming for IncFS v1Yurii Zubrytskyi
Run a manual timed job that trims all files one by one on the old version of IncFS, where it didn't do it automatically. Bug: 183435580 Fixes: 183436717 Test: atest libincfs-test service.incremental_test Change-Id: I57885b2826e383814822c767802f837135fd8464
2021-03-24[incfs] Fix the mount state callbacks processingYurii Zubrytskyi
- use a never-existing storage ID as a job key - order the jobs in the map to not skip them on changes, or, worse, never hang in a loop - clear the local callbacks vector before moving to the next storage ID - try to resume from the closest place on the next processing iteration Bug: 183435580 Test: atest service.incremental_test Change-Id: I36cd5d30c656bed62c20bd7a7f84fb58046a0933
2021-03-23Add a first chance retry delay of 1s.Alex Buynytskyy
Bug: 182214420 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: Iade9e1355ed0e5e761a3a98f3a7bca8215982315
2021-03-23Unbind from DataLoader when not needed anymore.Alex Buynytskyy
+ simplify adding new callbacks on storage state + streamline lock story for ifs members Bug: 183101753 Fixes: 183101753 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: I86fffa7101eeb42ebccca67ae7f5d133c1ab9dfa
2021-03-19[incfs] Call the new isFullyLoaded api where possibleYurii Zubrytskyi
Replace the remaining calls to getFilledRanges() with isFullyLoaded() where we don't care about the progress Bug: 183067554 Test: atest IncrementalService Change-Id: Ic8dc2e3a0ef078353883feef7969b29e11dfa2d0
2021-03-19[incfs] Use a more efficient getBlockCount() for incfs v2Yurii Zubrytskyi
v2 IncFS driver gives a very lightweight function to check the loading progress on a file, use it instead of counting the filled ranges + remove the unused mockable toString(IncFsFileId) Bug: 183067554 Test: atest IncrementalServiceTest Change-Id: Icd3bd891d671b27654f4194787a15a00cba1eb80
2021-03-18[incfs] Use the new libincfs API for file status checkingYurii Zubrytskyi
libincfs got a new set of functions for checking the file loaded status, which works more efficiently than getting filled ranges. Bug: 183067554 Test: atest IncrementalServiceTest Change-Id: I3b96bf409f1778c5a89e4802e2005197f70ce0cb
2021-03-17[incfs] Preallocate space for IncFS filesYurii Zubrytskyi
Use the new libincfs APIs to preallocate space for all files created via the public makeFile() API. This way we ensure the device won't run out of space much later Bug: 182185202 Test: atest libincfs-test PackageManagerShellCommandTest \ PackageManagerShellCommandIncrementalTest \ IncrementalServiceTest Change-Id: I70af97949b29ff5db63201b0e3487fe026e23160
2021-03-16Limit read log reporting to 2hrs for non-system DLs (non-adb).Alex Buynytskyy
Bug: 182477087 Test: atest IncrementalServiceTest Change-Id: I98c9ed3a2e8a91d26bcb879ab7073903ff7bb2c5
2021-03-11DL lifecycle: handle slow DL binding.Alex Buynytskyy
Bug: 182214420 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: I5959e01177ab702de1f754f4ba433004925ce98b
2021-03-09[incremental] expose duration since oldest pending readSongchun Fan
As requested by go/incremental-disablement-metrics, we will expose the duration since oldest pending read as part of the crash/ANR metrics. This is the first step that exposes the value to Incremental Service. BUG: 180951530 Test: unit test Change-Id: Ic67460072556ef01780a1794b40924ca2092060d
2021-02-22Fix the progress getting for mapped filesYurii Zubrytskyi
Mapped files don't support querying their loading progress, so we should simply skip them - they are already a part of some other file, and will get accounted for loading when that file's progress get queried + a bunch of small improvements Bug: 180535478 Test: atest service.incremental_test, adb install --incremental with mapped native libs Change-Id: Ifc8a402144f2f3669a0419124fb0f35d7002190a (cherry picked from commit 7731ebd1d8187c92a992d1f53c4114a6c40f7563)
2021-02-22[incremental] Use file range mapping for .so if availableYurii Zubrytskyi
Bug: 180535478 Test: IncrementalService unit tests Change-Id: I663dcdce337c289cacc5dc7224dedf5a55605c86
2021-02-11DataLoader lifecycle.Alex Buynytskyy
- restarting DataLoaders for not fully downloaded mounts, - exponential backoff retry on DataLoader restart. Bug: 173223115 Bug: 160634487 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: I5a93fd515ef0fec09452ce7bb4cd79fbd283d45f
2021-01-29Inherited installation support for Incremental.Alex Buynytskyy
Bug: 162345970 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest Change-Id: I360f44bc52e05553eacc448faa26f603d9eaae59 Merged-In: I360f44bc52e05553eacc448faa26f603d9eaae59
2021-01-14Incfs: Add the new .blocks_written control fileYurii Zubrytskyi
Bug: 170231230 Test: incremental and PackageManager unit tests Change-Id: Ia26306a4b1ca70ce70d55ce807fea62540f89334
2021-01-12Require INTERACT_ACROSS_USERS to enable readlogs.Alex Buynytskyy
As requested by PWG. Bug: 177330562 Fixes: 177330562 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest Change-Id: I1b03bc1c5b4c3cffaa0f97d864cc088782290bc2
2021-01-12Per package read timeouts.Alex Buynytskyy
Bug: 162345970 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest Change-Id: I2599db1ed8827fff16387c11254a5d607f27ea46
2020-11-09[incremental] add last pending reads info in dumpsysSongchun Fan
Example section of dumpsys output: lastPendingReads: fileId: 03300000000000000000000000000000 metadataHex: 0330 blockIndex: 857738 bootClockTsUs: 166877488 Test: manual BUG: 162600251 Change-Id: I37b04751c2a11972450ce7e9082116f8047e87df
2020-10-20[incremental/pm] set health listener on commit and on rebootSongchun Fan
This changes allow Incremental Service to directly report health status to package manager service. A health listener is created during package installation session to monitor incremental storage health. After commit, a new listener is created and will overwrite the old one. The new listener will listen to incremental storage health and report the status to package manager service, which will then send the status to IncrementalStates, where the startability state and unstartable reason might change, based on the health status code. During reboot, for each incremental package, if it is not fully loaded, the package manager service will register a health status listener to continue monitor the health status of this package. Test: unit test Test: manual BUG: 170435166 Change-Id: I220f230c523cfaf2c96019f9478554665e6af486
2020-09-15[incremental/pm] register progress listenerSongchun Fan
Incremental Serivce periodically polls loading progress and sends to Package Manager Service. Package Manager provides APIs for other interested parties to listen to the loading progress. BUG: 165841827 Test: unit test Change-Id: I44b9e17c2240b9efe53bc09fc728b6671f1f7dfe
2020-08-27Wait for APK to be fully downloaded for full APK digests.Alex Buynytskyy
Bug: 160605420 Test: atest ChecksumsTest Change-Id: Ib9fd591c67290786268b6dcdc57c7db153612e01
2020-08-25[incremental] storage is regarded as fully loaded if there's no file or only ↵Songchun Fan
empty files Test: atest service.incremental_test BUG: 165799231 Change-Id: I3b6be3918d53ce4ab0a30a4da493d9730a31ed3c
2020-08-22[IncrementalService] getLoadingProgress (v1)Songchun Fan
This is to unblock Launcher's work on progress ring. Currently it uses incfs getFilledBlocks(). Will switch to the new incfs progress reporting API once it is ready. Test: unit test Test: adb shell dumpsys incremental BUG: 165799231 Change-Id: Icd68124806454f888826294da36f109bca9771ac
2020-06-12Don't provide read logs for shell-initiated installations.Alex Buynytskyy
Only if the application is profileable. Bug: 158238023 Fixes: 158238023 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageParserTest Change-Id: I8575830ec3f29850297fdbfbaa157072d6350a28
2020-06-01Healthcheck: proper job allocation and test.Alex Buynytskyy
Bug: 153874006 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: Iede1f2297cc4f8e3c3f0acd43cee597f75dff179
2020-05-29Lifecycle: detecting blocked and unhealthy, part 2.Alex Buynytskyy
Part 2: continuous health checking, blocked and unhealthy states. Bug: 153874006 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: Ifdbff7dfa24b3bd9b96c534a40bb1226082749ca
2020-05-26Lifecycle: detecting blocked and unhealthy.Alex Buynytskyy
Part 1: interfaces and PM implementation. Bug: 153874006 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: I312dd919d2bb552bea3d72fb49fd1579882da14b
2020-05-12Don't try to recreate IncrementalFileStorages on re-commit.Alex Buynytskyy
Bug: 156287164 Fixes 156287164 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: I75e392a1fa84df8b6ac0b855233f9a2662998e96
2020-05-07Lifecycle: detecting pending reads.Alex Buynytskyy
Once pending read detected, try to start the dataloader. Bug: 153874006 Test: test PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: Ia8169ccbb0f710317715e6fddb9bc6a718543766
2020-04-23Keep only service binding in DataLoaderManager.Alex Buynytskyy
This simplifies: - resource management - no extra copies of controls, - state management - all states in one place, no more hidden (bound but not created) state. Bug: b/153874006 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: I3d16a099c7f42fcf14637c5a8e96bd6f99e073d1
2020-04-22[incfs] Use the new IncFs_MakeDirs() functionYurii Zubrytskyi
This gets rid of annoying warnings in logcat about not being able to create a directory and then directory already existing Bug: 153704006 Test: atest IncrementalServiceTest Test: adb install megacity.apk Change-Id: Ib718960287f93cb383c06c9b9e3d0abf1ec42916
2020-04-22[incfs] Use MountRegistry to import existing mounts on startYurii Zubrytskyi
This is a big cleanup in IncrementalService that makes it behave nicely on runtime restart, and more: - fixed a bunch of threading issues in createStorage/bind - made public functions correctly accept any path in any bind mount and translate it to the proper root mount - got rid of "using namespace" in headers, cleaned includes - removed all unused functions - set CLOEXEC bit on all duped FDs Bug: 151241369 Test: atest PackageManagerShellCommandTest \ PackageManagerShellCommandIncrementalTest \ IncrementalServiceTest Change-Id: Ided4415aabfbfca3925b5e71c91896055886ac4a
2020-04-17Switching to FSM-based DL lifecycle.Alex Buynytskyy
This is a pure refactoring. Bug: b/153874006 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: Ieda2be08d7359fa69b2d328c85b3606de6d02b3d
2020-04-14DataLoader lifecycle.Alex Buynytskyy
- mark disconnected DataLoaders as non-existent (destroyed), - remove storage destroy on DataLoader destroy, - more robust destroy on destruct handling, - less race-conditions. Bug: b/153874006 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: Ib77302aac546d66ce40c5345417c7b424a1d601b
2020-04-10[incfs] Fix a crash in worker thread calling JNIYurii Zubrytskyi
Worker thread has to initialize JNI separately to be able to call into managed binders implemented in the same system_server process, e.g. DataLoaderManager Bug: 153513507 Test: adb install megacity.nov4.apk; adb install megacity.v4.apk Change-Id: I668e8664361cd2fb3353ec50efd689c7d613658f
2020-04-09Refactor: move dataLoader details to a separate class.Alex Buynytskyy
Bug: b/151241369 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest Change-Id: If4e7d3bb9f0951dcaee4315f4be525883b7fc333
2020-04-08Better test for setStorageParams.Alex Buynytskyy
Using service connector - as would DataLoaders do - instead of calling service method directly. Bug: b/153530451 Test: atest IncrementalServiceTest Change-Id: I43b9ffa235204b073465c19ab32c0435d2dfe755