summaryrefslogtreecommitdiff
path: root/services/incremental
AgeCommit message (Collapse)Author
2025-02-04[appops] Migrate native IAppOpsCallback to aidl Atneya Nair
Move IAppOpsCallback to frameworks/native for use in both java and native. Fix-up callsites with fully qualified package name and the correct arguments. Test: compiles Test: atest CtsMediaAudioPermissionTestCases Test: atest service.incremental_test Bug: 322692565 Flag: EXEMPT mechanical Change-Id: I8d5fff5ef86e0b2d68df4a95745fe430134fe1f0
2025-01-10Fix -Wreorder-init-list Yi Kong
frameworks/base/services/incremental/IncrementalService.cpp:2004:29: warning: ISO C++ requires field designators to be specified in declaration order; field 'signature' will be initialized after field 'metadata' [-Wreorder-init-list] 2004 | .metadata = {targetLibPath.c_str(), (IncFsSize)targetLibPath.size()}, | ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ frameworks/base/services/incremental/IncrementalService.cpp:2002:30: note: previous initialization for field 'signature' is here 2002 | .signature = {}, | ^~ Test: presubmit Change-Id: I7c71ed46f36f30f3bff947e513b6a021d1c61b8c
2024-09-15Batch migration of frameworks/base TEST_MAPPING to test_module_config modules Ronald Braunstein
As part of go/test-module-config we are moving test options from TEST_MAPPING -> Android.bp files. In previous Cls, we created the new `test_module_config` rules in Android.bp This is updating the TEST_MAPPING file to use those rules. It is also removing "FlakyTest and IgnoreTest" exclude annoations as they are now added in gcl files per run rather than ad-hoc per module. I have a script that looks at the generated tradefed config file for the new options added in Android.bp files, then it looks at TEST_MAPPING files and find the places to update where the options match for the test. I am also doing abtd runs of each TEST_MAPPING file before and after my change and ensuring the number of tests run is the same (or at least as many). There are cases where tradefed would comping include-filters across TEST_MAPPING entries for the same module, but now they will be purposefully split up, causing some tests to be run under two different modules. Flag: TEST_ONLY Test: Ran adbt on each TEST_MAPPING and compared before and after results. Verified we were still running all the tests we were before. i.e. after the adbt run, I would download the test artfiact for the tradefed detailed evenvt and compare test counts. You can see CtsAppTestCases became CtsAppTestCases_cts_requesttileserviceaddtest, etc. I'm not including results for all 100 TEST_MAPPING files, but I did verify with scripts and eyes. Minor differences (like 2011 vs 2009) on a test that didn't change are ignored, but in general there were more tests run, not fewer. % diff <(grep started frameworks_base_services_core_java_com_android_server_statusbar_TEST_MAPPING/BASE.details) <(grep started frameworks_base_services_core_java_com_android_server_statusbar_TEST_MAPPING/NEW.details) | grep run < [run x86_64 CtsAppTestCases (testCount: 6,attempt: 0) started] > [run x86_64 CtsAppTestCases_cts_requesttileserviceaddtest (testCount: 6,attempt: 0) started] < [run x86_64 CtsLocationFineTestCases (testCount: 96,attempt: 0) started] > [run x86_64 CtsLocationFineTestCases_android_server_location (testCount: 96,attempt: 0) started] < [run x86_64 FrameworksNetTests (testCount: 2009,attempt: 0) started] > [run x86_64 FrameworksNetTests (testCount: 2011,attempt: 0) started] Test-Mapping-Slo-Bypass-Bug: b/335015078 Change-Id: If6e3cd0624ac1c16f1cd088566d967769f47199c
2024-06-12libc++fs is part of libc++ now. Elliott Hughes
Change-Id: Icdf645793dbe891d76adf78d70b9cb5287e510e4
2024-01-30Merge changes from topic "udc_qpr2_merge" into aosp-main-future Xin Li
* changes: Fix fs-verity API for secondary users Fix build for ag/25776653 Update @UsesReflection annotation Merge Android 24Q1 Release (ab/11220357)
2024-01-30Update OWNERS files Patrick Baumann
Change-Id: I87e21b50a247f82d65a3cd46cd1612676d9dcbe0
2023-12-06Merge "Fix -Wunused-variable compiler warning" into main am: e6d39e67bb am: ↵ Yi Kong
83933ce167 am: a15a3a80a2 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2859205 Change-Id: I85a3fb0d65ba566c5eddb600133f45395089f6fb Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-12-06Fix -Wunused-variable compiler warning Yi Kong
Test: presubmit Change-Id: Ifd5f633a517d6dbd7344849a739acb32cb142296
2023-09-08Use androidx LargeTest annotation for device tests. Anna Zhuravleva
Platform_testing LargeTest annotation is for host side tests, device tests should use androidx annotation. Bug: 293837621 Test: build Change-Id: I8ea6f256b542a4ed0030b3d366c37f6fdafa6b3b
2023-08-04Merge "Temporarely move CtsPackageManagerTestCases to presubmit-large." into ↵ Treehugger Robot
main
2023-08-04Excluding long incremental stats tests from presubmit Songchun Fan
BUG: 293380233 Test: presubmit Change-Id: I3519bf2260a4c967a033cb4330e41a86f1a124e4
2023-08-04Temporarely move CtsPackageManagerTestCases to presubmit-large. Alex Buynytskyy
To unblock presubmit. Bug: 292174535 Test: atest CtsPackageManagerTestCases Change-Id: I729156f85c072b62c25043b6eabf44616e689239
2023-08-01Fix wait_until(infinity) for condition_variable Yurii Zubrytskyi
libcxx implements wait_until using wait_for, which in its turn has to call a syscall that's basically wait_until, causing a double conversion of the waiting time parameter. Because of this, infinity detection code in those conversions sometimes fail if the delat between the conversions is long enough - causing a syscall error for 'invalid time' This CL explicitly calls non-timed wait() instead. Bug: 293223125 Bug: 292138960 Test: build + unit tests + presubmits Change-Id: Id30fdfbe374b63fcaed627fcdf374f1e501a7807
2023-07-20Point TEST MAPPINGs to the new suites. Alex Buynytskyy
Bug: 288461387 Bug: 288323752 Test: presubmit Change-Id: I453a4151100ca42495e0f2473e5599dec38ab70d
2023-07-19Update Incremental TEST_MAPPING Joanne Chung
Bug: 288460044 Test: atest --test-mapping frameworks/base/services/incremental/TEST_MAPPING Change-Id: Idc5cff6e470ad652835624f9bab66398f8bfb0f5
2023-03-01Merge "Move CtsInstalledLoadingProgressHostTests to "presubmit" group, it's ↵ Julien Desprez
already included that way in another location." am: 12298c9002 am: 9d70d31cd1 am: c2e6d8df4e Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2460715 Change-Id: I1dd7ebb5f654e55ecf7c86f4ceb5713fdc698b61 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-28Merge "Move CtsInstalledLoadingProgressHostTests to "presubmit" group, it's ↵ Julien Desprez
already included that way in another location." am: 12298c9002 am: 9d70d31cd1 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2460715 Change-Id: Ide0d750eb2d00c3a9c8f270af9a4b727e3645034 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-28Move CtsInstalledLoadingProgressHostTests to "presubmit" group, it's already ↵ Julien Desprez
included that way in another location. No need to carry that test exactly the same in both presubmit & presubmit-large Change-Id: I843339f1583e1a163894c52978bdcb4cc9f8ffa6 Test: presubmit Bug: 270214959
2023-02-15Merge "Set app.metadata file permission to 640" William Loh
2023-02-14Set app.metadata file permission to 640 William Loh
This should prevent apps from circumventing the GET_APP_METADATA permission by reading the file directly if they are aware of the file path. Bug: 267823160 Test: atest android.packageinstaller.install.cts.InstallAppMetadataTest Change-Id: I4aab10b48e62234bc252535ab2e2c8b9c77a7ac3
2023-02-09Merge "Remove stale connections to DataLoaders." into tm-qpr-dev am: da38e58db2 TreeHugger Robot
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21342999 Change-Id: I988c7ac970d185bbadd49889d786e990334c111e Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-09Merge "Remove stale connections to DataLoaders." Alex Buynytskyy
2023-02-09Remove stale connections to DataLoaders. Alex Buynytskyy
Bug: 263449831 Fixes: 263449831 Test: atest service.incremental_test Change-Id: Ia3da6ca047c46807cbd72ef797bb4b300c38efb3
2023-02-09Remove stale connections to DataLoaders. Alex Buynytskyy
Bug: 263449831 Fixes: 263449831 Test: atest service.incremental_test Change-Id: Ia3da6ca047c46807cbd72ef797bb4b300c38efb3
2023-02-09Better status message. Alex Buynytskyy
Bug: 263449831 Test: adb shell dumpsys incremental Change-Id: Ie6960d22c1d0e018b50de1c9e431c403fe442458
2023-01-18TEST_MAPPING: Add incremental install tests to kernel-presubmit Edward Liaw
Bug: 258819618 Test: atest :kernel-presubmit Change-Id: Ie887d7f11c2da982e224eb55ef61dfb95a23b438 Signed-off-by: Edward Liaw <edliaw@google.com>
2022-09-30Merge "Explicitly discard released proto strings in test." Krzysztof Kosiński
2022-09-30Explicitly discard released proto strings in test. Krzysztof Kosiński
In newer releases of the protobuf library, the release_* methods are marked [[nodiscard]], which causes a compilation error here. Bug: 203713560 Test: presubmit Change-Id: If8c6392a5d36014f1839deaaf241b636610add71
2022-09-25Explicitly discard released proto strings. Krzysztof Kosiński
In newer releases of the protobuf library, the release_* methods are marked [[nodiscard]], which causes a compilation error here. Bug: 203713560 Test: presubmit Change-Id: I44a9d52730bdc199e34cb25be959a955fff8f128
2022-09-01incfs: tolerate either ptrdiff_t or size_t for std::span Ryan Prichard
The out-of-date std::span in external/libcxx currently uses ptrdiff_t as the type of size() and the second ctor argument. The C++20 std::span actually uses size_t for these values instead. Try to accommodate both versions of std::span until libc++ is upgraded. Bug: http://b/175635923 Test: treehugger Change-Id: Idc43f880b6eb21dad79e79fa456847d5865a1ed2
2022-03-21Mark all installations using PMSC as "ADB". Alex Buynytskyy
This will allow tests to disable verification on case-by-case basis. Bug: 224017187 Bug: 220086205 Test: atest PackageManagerShellCommandTest ResourcesHardeningTest PackageManagerShellCommandIncrementalTest ChecksumsTest Change-Id: Ie0e77f1cbc52b7fe56e00e8b0b36f47b2422391b
2022-01-05Organize incremental test plan for PackageManager Felka Chang
This patch adds libincfs-test and service.incremental_test in the presubmit process. Test: atest service.incremental_test Test: atest libincfs-test Test: atest -p system/incremental_delivery/incfs http://ab/I24700010005521991 Bug: 180650365 Change-Id: Iaea12556d398893875c4ab3bb928477103fa7c18
2021-09-01Merge "[services.incremental] Use shared libutils for remount" am: ↵ Treehugger Robot
44233c693c am: bdf15f2234 am: 65ee23bba4 am: a00b50e072 am: cfa0cf527d Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1810680 Change-Id: I33e5ac11f43881354caf318a68c47cd6f5763ece
2021-09-01Merge "[services.incremental] Use shared libutils for remount" am: 44233c693c Treehugger Robot
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1810680 Change-Id: I2614171ee4fc1822a2a3c1e7e50658487218a34d
2021-08-31[services.incremental] Use shared libutils for remount Pirama Arumuga Nainar
Bug: http://b/197965342 services.incremental depends on libbinder.so, which already depends on libutils.so. When linking services.incremental with libutils.a, the linker tries to open objects from libutils.a to redefine undefined symbols in libbinder.so. This causes a linker failure with upstream LLD. Switching to shared libutils.so instead doesn't increase the dependency closure for services.incremental. Test: build with aosp/1809741 which has lld with the above behavior. Change-Id: I2735461ae64ef2e4c0afc170f7b765c1b5b9432e
2021-08-25Move CtsContentTestCases to group presubmit-large Jackal Guo
The test config takes longer than 900 seconds to run. Move it to a dedicated group for running slow presubmit Test Mapping test. Some more context is in the referenced bug, e.g, b/174495337 The group will work exactly the same as presubmit for now. Bug: 174654670 Bug: 174495337 Test: treehugger Change-Id: Id9769bbc625b42257af603d69098ae69c0fabb80
2021-07-23Mark classes as final that have final destructors Liz Kammer
Having a final destructor prevents the class from being inherited from, adding the designation final to the class makes this behavior more explicit. This is required to re-enable the error for -Wno-final-dtor-non-final-class. Test: m Merged-In: Ia3126d30e19edfd17f7c8da368e9763ca5501e84 Change-Id: Id1d7c607af9cca0109e1f763052894cf179f4af1
2021-06-28Reduce timeout for DL to destroy. Alex Buynytskyy
10secs should be more than enough to stop whatever the DL is doing. Besides, DL has its own set of IncFS FDs and should not crash. Bug: 189222575 Fixes: 189222575 Test: atest IncrementalServiceTest Change-Id: I73cb27d61c7418adeea7536c8263e6ba8c77fd3e
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 error Songchun Fan
BUG: 187354818 Test: atest service.incremental_test Change-Id: I632b747440ed4faef31266a3a7b18b7a382ff2cf
2021-05-21Updates OWNERS files Patrick Baumann
Bug: 186864416 Test: Builds Change-Id: I04dfc5c2c7143825334ebc62742e335b8c889492
2021-05-14Updates OWNERS files Patrick Baumann
Bug: 186864416 Test: Builds Change-Id: I04dfc5c2c7143825334ebc62742e335b8c889492
2021-05-13Duplicate apk names on package update. Alex Buynytskyy
+minor fixes Bug: 187710420 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest Change-Id: Iced479532b38bddcd8655df3ce08611434965cf4
2021-05-11[incremental] use same sysfs name for setOptions Songchun 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-08Avoid sysfs name clashes. Alex Buynytskyy
Bug: 187437944 Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest ApkVerityInstallTest Change-Id: Idc8cb4d170b0bfd98ee3a10bdf57439801739757
2021-05-05Merge "[incremental] report new metrics from Incremental Service" into sc-dev Songchun Fan
2021-05-04[incremental] report new metrics from Incremental Service Songchun Fan
BUG: 184844615 Test: atest service.incremental_test Change-Id: Ibbfe68851a4adfd620db9652f1fda550a3c6fb2c
2021-05-04Merge "[IncrementalService] add last read error in dumpsys" into sc-dev Songchun Fan
2021-05-04Merge "[IncrementalService] use sysfs_name in mount options and dumpsys" ↵ Songchun Fan
into sc-dev
2021-05-03[IncrementalService] add last read error in dumpsys Songchun Fan
BUG: 184844615 Test: dumpsys Change-Id: I3e8178c8a62dc5d09c718b140bb48bda85a3bae5