Skip to content

Add dependency directive to default.service - #4

Closed
fzahir786 with Copilot wants to merge 2 commits into
topic/NOJIRA-1234from
copilot/sub-pr-2-again
Closed

Add dependency directive to default.service#4
fzahir786 with Copilot wants to merge 2 commits into
topic/NOJIRA-1234from
copilot/sub-pr-2-again

Conversation

Copilot AI commented Feb 17, 2026

Copy link
Copy Markdown

The default.service unit file used After= for ordering but lacked a dependency directive, allowing it to start even if ecfs-init.service failed.

Changes:

  • Added Wants=ecfs-init.service to establish soft dependency relationship
  • Chose Wants= over Requires= since the service has conditional logic and can operate independently
 [Unit]
 Description=Boot version loader for RDK
 After=ecfs-init.service
+Wants=ecfs-init.service

The service now properly tracks its dependency while maintaining tolerance for ecfs-init.service absence.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Add Wants=ecfs-init.service to establish proper dependency relationship.
This ensures the service has a soft dependency on ecfs-init.service,
allowing it to start even if ecfs-init.service is missing, while
maintaining the ordering guarantee with After=.

Co-authored-by: fzahir786 <169052329+fzahir786@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issues in creating default.service implementation Add dependency directive to default.service Feb 17, 2026
Copilot AI requested a review from fzahir786 February 17, 2026 09:00
@fzahir786 fzahir786 closed this Feb 19, 2026
@fzahir786
fzahir786 deleted the copilot/sub-pr-2-again branch February 19, 2026 12:28
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.

2 participants