Skip to content

RDKEMW-19100:gst-cleanup conditions when cdl_flashed_file_name is not present - #551

Merged
nhanasi merged 1 commit into
support/4.2.1from
topic/RDKEMW-19100
May 26, 2026
Merged

RDKEMW-19100:gst-cleanup conditions when cdl_flashed_file_name is not present#551
nhanasi merged 1 commit into
support/4.2.1from
topic/RDKEMW-19100

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 issues 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 issues 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>
Copilot AI review requested due to automatic review settings May 25, 2026 19:50
@vrenu2018
vrenu2018 requested a review from a team as a code owner May 25, 2026 19:50

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 intends to stop gstreamer-cleanup from running on every reboot when /opt/cdl_flashed_file_name is missing, and to relocate the gstreamer-cleanup.service unit out of systemd_units/.

Changes:

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

systemd_units/gstreamer-cleanup.service:1

  • The PR description says this change fixes the cleanup conditions (when /opt/cdl_flashed_file_name is missing) and relocates gstreamer-cleanup.service under recipes-multimedia, but the PR diff shown here only deletes this unit. In this repo checkout there is no replacement unit/recipe (no other occurrences of "gstreamer-cleanup"), so this appears to remove the cleanup behavior entirely rather than adjusting its conditions. Please either add the new unit/recipe to this PR (and ensure it’s installed/enabled), or update the PR description/scope to reflect that the relocation happens in a different repo.

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

@nhanasi
nhanasi merged commit 6606a5c into support/4.2.1 May 26, 2026
8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 26, 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