Skip to content

RDKEMW-3659: Fix WPE_ERROR_ResourceUnavail message on getModel#92

Open
ponnathdeepak wants to merge 8 commits intodevelopfrom
topic/RDKEMW-3659-1
Open

RDKEMW-3659: Fix WPE_ERROR_ResourceUnavail message on getModel#92
ponnathdeepak wants to merge 8 commits intodevelopfrom
topic/RDKEMW-3659-1

Conversation

@ponnathdeepak
Copy link
Copy Markdown

Reason for change: Fix WPE_ERROR_ResourceUnavail message on getModel
Test Procedure: Refer ticket
Risks: Low

Signed-off-by: Deepak_Ponnath2@comcast.com

Reason for change: Fix WPE_ERROR_ResourceUnavail message on getModel
Test Procedure: Refer ticket
Risks: Low

Signed-off-by: Deepak_Ponnath2@comcast.com
@ponnathdeepak ponnathdeepak requested a review from a team as a code owner September 3, 2025 10:27
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Sep 8, 2025

CLA assistant check
All committers have signed the CLA.

Copilot AI review requested due to automatic review settings January 30, 2026 12:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the WPEFramework System initialiser unit dependencies to ensure required services are started before the System plugin is activated.

Changes:

  • Add wpeframework-deviceinfo.service to Requires= and After= for wpeframework-system.service to enforce startup dependency/order.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +3 to +4
Requires=wpeframework-powermanager.service sysmgr.service mfrmgr.service wpeframework-deviceinfo.service
After=wpeframework-powermanager.service sysmgr.service mfrmgr.service wpeframework-deviceinfo.service
Copy link

Copilot AI Jan 30, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR title/description says it fixes the "WPE_ERROR_ResourceUnavail message on getModel", but the actual change here is only adjusting systemd unit ordering/dependencies (adding wpeframework-deviceinfo.service). Please update the PR description/title (or add the missing code change) to clearly explain how this systemd dependency change resolves the getModel error/message, so reviewers/operators understand the intent and impact.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants