Add provider-based package dependency recovery #461
Merged
Bhargav-qcom (bhargav0610) merged 3 commits intoJul 9, 2026
Conversation
Srikanth Muppandam (smuppand)
force-pushed
the
distro_support
branch
from
May 27, 2026 16:50
8dd23ba to
270af55
Compare
|
@ualcomm/qualcomm-linux-testing.triage This pull request has been marked as stale due to 30 days of inactivity and will automatically close after an additional 5 days. |
Srikanth Muppandam (smuppand)
marked this pull request as draft
July 3, 2026 04:39
Srikanth Muppandam (smuppand)
force-pushed
the
distro_support
branch
2 times, most recently
from
July 8, 2026 11:52
206c217 to
4bc1214
Compare
Srikanth Muppandam (smuppand)
marked this pull request as ready for review
July 8, 2026 11:53
Contributor
Author
|
The framework is ready now, and a sample fastrpc test demonstrates the workflow. |
Add a package-manager based dependency recovery helper for testkit dependencies. The helper supports apt, rpm/dnf/yum, opkg, and check-only providers, with automatic provider detection based on available package manager tools. Add repo-owned provider configuration and command-to-package mapping files. The package map resolves missing dependency commands to provider-specific package names instead of assuming command names match package names. It also supports TESTNAME-scoped package sets so large feature dependencies, such as sensors packages, are installed only when the corresponding test runs. For apt-based systems, install optional *.sources artifacts from /opt/qcom-testkit/metadata when present and create optional apt auth configuration only when secure target-local secret files exist. This supports internal authenticated repositories, future public repositories, local apt sources, and existing target apt sources without hardcoding repository type. The provider reuses existing network recovery helpers when available, prints package manager diagnostics to stdout, retries package operations, and keeps package upgrade disabled by default. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
Update check_dependencies() to optionally recover missing commands through the package provider before marking a test as SKIP. The existing behavior is preserved when package recovery is disabled or when a missing command has no package mapping. In those cases, the test continues to report SKIP for missing dependencies. When recovery is enabled, check_dependencies() sources lib_pkg_provider.sh, resolves the missing command through the command-to-package map, attempts to install the mapped package set using the active package provider, and rechecks PATH before deciding whether to continue or skip. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
Add a generic package-set setup call in the FastRPC runner so Debian/APT targets can install or upgrade the required FastRPC packages before test execution. The setup remains provider-driven and does not affect Yocto/RPM or OPKG flows when no matching package-set mapping is configured. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
Srikanth Muppandam (smuppand)
force-pushed
the
distro_support
branch
from
July 8, 2026 17:11
4bc1214 to
8da7252
Compare
Contributor
Author
|
Appreciate the offline discussions, Phanendra Yarra (@phanendra-qc) . I’ve updated the changes based on the suggestions, and the revised stdout below now shows the versions for packages that were already installed before the upgrade. This snapshot shows the situation where packages are not installed on rootfs, so it configures the install and then runs the tests. |
Srikanth Muppandam (smuppand)
requested review from
Vkenjam and
Bhargav-qcom (bhargav0610)
July 8, 2026 17:18
Bhargav-qcom (bhargav0610)
approved these changes
Jul 9, 2026
Bhargav-qcom (bhargav0610)
merged commit Jul 9, 2026
f09182c
into
qualcomm-linux:main
12 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add an optional package-manager based dependency recovery flow for testkit dependencies.
This change introduces a provider helper that can recover missing commands through the active package manager before a test is marked SKIP. The flow is package-manager based, not distro-profile based, so Yocto images using dnf/rpm are handled through the rpm provider, while apt, opkg, and check-only systems are handled separately.
New files:
The command-to-package map resolves missing dependency commands to provider-specific package names instead of assuming command names match package names. It also supports TESTNAME-scoped package sets, so larger stacks such as sensors packages are installed only when that specific test runs.
For apt systems, optional *.sources files under /opt/qcom-testkit/metadata are installed when present. Optional apt auth is created only when secure target-local secret files exist. This allows the same flow to work with internal authenticated repositories, future public repositories, local apt sources, or already-configured target apt sources.
check_dependencies() now optionally tries package recovery before reporting SKIP. If recovery is disabled, no package mapping exists, no supported package manager exists, or installation fails, the existing SKIP behavior is preserved.
Package upgrade remains disabled by default. Package-manager output is printed to stdout to make CI failures easier to debug.Add an optional package-manager based dependency recovery flow for testkit dependencies.
This change introduces a provider helper that can recover missing commands through the active package manager before a test is marked SKIP. The flow is package-manager based, not distro-profile based, so Yocto images using dnf/rpm are handled through the rpm provider, while apt, opkg, and check-only systems are handled separately.
New files:
The command-to-package map resolves missing dependency commands to provider-specific package names instead of assuming command names match package names. It also supports TESTNAME-scoped package sets, so larger stacks such as sensors packages are installed only when that specific test runs.
For apt systems, optional *.sources files under /opt/qcom-testkit/metadata are installed when present. Optional apt auth is created only when secure target-local secret files exist. This allows the same flow to work with internal authenticated repositories, future public repositories, local apt sources, or already-configured target apt sources.
check_dependencies() now optionally tries package recovery before reporting SKIP. If recovery is disabled, no package mapping exists, no supported package manager exists, or installation fails, the existing SKIP behavior is preserved.
Package upgrade remains disabled by default. Package-manager output is printed to stdout to make CI failures easier to debug.