Skip to content

RDKEMW-20104:gst-cleanup conditions when cdl_flashed_file_name is not present#556

Merged
nhanasi merged 1 commit into
support/4.5.2from
topicRDKEMW-20104
Jun 16, 2026
Merged

RDKEMW-20104:gst-cleanup conditions when cdl_flashed_file_name is not present#556
nhanasi merged 1 commit into
support/4.5.2from
topicRDKEMW-20104

Conversation

@vrenu2018

Copy link
Copy Markdown
Contributor

Reason for change: gstreamer-cleanup is happening on every reboot when /opt/cdl_flashed_file_name is missing. Fixing the issue as well as re-locating gstreamer-cleanup.service file to recipies-multimedia instead of sysint folder
Test Procedure: Boot the TV and check for gstreamer-cleanup metrics in rdk_milestones.log
Risks: low

… present

Reason for change: gstreamer-cleanup is happening on every reboot when /opt/cdl_flashed_file_name is missing. Fixing the issue as well as re-locating gstreamer-cleanup.service file to recipies-multimedia instead of sysint folder
Test Procedure: Boot the TV and check for gstreamer-cleanup metrics in rdk_milestones.log
Risks: low

Signed-off-by: Renuka Varry <rvarry049_comcast@cable.comcast.com>
@vrenu2018 vrenu2018 requested a review from a team as a code owner June 16, 2026 15:06
Copilot AI review requested due to automatic review settings June 16, 2026 15:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to stop unintended gstreamer-cleanup runs on every reboot when /opt/cdl_flashed_file_name is missing, and to relocate the gstreamer-cleanup.service unit out of this repo’s systemd_units/ directory.

Changes:

  • Removes systemd_units/gstreamer-cleanup.service from this repository.
Comments suppressed due to low confidence (1)

systemd_units/gstreamer-cleanup.service:1

  • This change deletes systemd_units/gstreamer-cleanup.service, but there is no replacement gstreamer-cleanup.service anywhere else in this repo. That means this PR (as-is) removes the gstreamer cleanup service entirely and does not implement the title/description’s “fix + relocation” within this codebase. If the service is being moved to another layer/repo, please either (a) include the new unit file here in its new location, or (b) update the PR description to explicitly state that this PR only removes the unit from sysint and link to the PR/repo where the corrected unit is added.

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

@nhanasi nhanasi merged commit ffc1049 into support/4.5.2 Jun 16, 2026
5 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants