Skip to content

RDKEMW-19246 : Enable firmware-independent OTA Updates for MemInsight#910

Open
rosemarybennyy wants to merge 2 commits into
developfrom
topic/RDKEMW-19246
Open

RDKEMW-19246 : Enable firmware-independent OTA Updates for MemInsight#910
rosemarybennyy wants to merge 2 commits into
developfrom
topic/RDKEMW-19246

Conversation

@rosemarybennyy

Copy link
Copy Markdown
Contributor

Reason for change: Enable firmware-independent OTA Updates for MemInsight
Test Procedure: Tested and verified
Risks: Low
Priority: P1
Signed-off-by: RoseMary_Benny@comcast.com

Copilot AI review requested due to automatic review settings June 3, 2026 06:25
@rosemarybennyy rosemarybennyy requested review from a team as code owners June 3, 2026 06:25

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

Enables firmware-independent OTA update support for MemInsight by updating RDM agent and adding MemInsight packaging metadata plus runtime path handling for app installs under /media/apps.

Changes:

  • Bump rdmagent recipe version and source revision to pick up updated OTA/app-management behavior.
  • Add a MemInsight package.json metadata file to the layer and reference it from the MemInsight recipe.
  • Update start_meminsight.sh to re-point /media/apps/meminsight to a model-specific app directory.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
recipes-support/rdmagent/rdmagent.bb Bumps PV and SRCREV to a newer rdm-agent revision.
recipes-devtools/meminsight/meminsight_git.bb Adds package.json to SRC_URI.
recipes-devtools/meminsight/files/start_meminsight.sh Adjusts runtime app path via /media/apps/meminsight symlink and reorders variable setup.
recipes-devtools/meminsight/files/package.json Introduces MemInsight OTA/package metadata including install script reference.

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

Comment on lines +44 to +46
APP_HOME_DIR=/media/apps/${MODEL_NUM}-meminsight
rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight
Comment on lines +48 to +50
SRC_BIN="/media/apps/meminsight/usr/bin/meminsight"
DST_BIN="/run/meminsight/usr/bin/meminsight"
RFC_PARAM="Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.meminsight.Trigger"
Comment on lines +1 to +8
{
  "name": "meminsight",
  "description": "Performance Monitoring tool - meminsight",
  "version": "1.1",
  "contents": ["meminsight-pkg.tar"],
  "size": "72K",
"installScript": "/etc/rdm/post-services/start_meminsight.sh"
}
Comment on lines 18 to 22
file://meminsight-upload.service \
file://meminsight-upload.path \
file://upload_MemReports.sh \
file://package.json \
"
Copilot AI review requested due to automatic review settings June 4, 2026 05:29

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Comment on lines +1 to +5
# Generate RDM package only when 'meminsight' is passed to GENERATE_RDM_CERTS
# GENERATE_RDM_CERTS can be passed through jenkins or can be set locally in auto.conf
inherit ${@bb.utils.contains('GENERATE_RDM_CERTS', '${BPN}', 'comcast-package-deploy', '', d)}
DOWNLOAD_APPS= "${@bb.utils.contains('DISTRO_FEATURES', 'rdm', bb.utils.contains('DISTRO_FEATURES', 'enable_xmeminsight', '', '${BPN}', d), '', d)}"
CUSTOM_PKG_EXTNS = "dl"
Comment on lines +14 to +20
DOWNLOADABLE_FILES = "${@bb.utils.contains('DOWNLOAD_APPS', '${PKG_BUNDLE_NAME}', '\
${bindir}/meminsight \
/etc/rdm/post-services/start_meminsight.sh \
', '', d)}"
DOWNLOADABLE_FILES += "${@bb.utils.contains('ENABLE_RDM_VERSIONING', 'true', '\
${sysconfdir}/apps/${PKG_BUNDLE_NAME}_package.json \
', '', d)}"
Comment on lines +21 to +33
PACKAGE_BEFORE_PN += "${PN}-dl "
RDEPENDS_${PN} += " ${PN}-dl"
do_install_append() {
if [ "${ENABLE_RDM_VERSIONING}" = "true" ]; then
install -d ${D}${sysconfdir}/apps
install -m 644 ${WORKDIR}/package.json ${D}${sysconfdir}/apps/${PKG_BUNDLE_NAME}_package.json
fi
}
FILES_${PN}-dl += "${sysconfdir}/apps/${PKG_BUNDLE_NAME}_package.json \
${bindir}/meminsight \
/etc/rdm/post-services/start_meminsight.sh \
"
pkg_postinst_${PN}-dl () {
Comment on lines +44 to +50
APP_HOME_DIR=/media/apps/${MODEL_NUM}-meminsight
rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight

SRC_BIN="/media/apps/meminsight/usr/bin/meminsight"
DST_BIN="/run/meminsight/usr/bin/meminsight"
RFC_PARAM="Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.meminsight.Trigger"
Comment on lines +1 to +8
{
  "name": "meminsight",
  "description": "Performance Monitoring tool - meminsight",
  "version": "1.1",
  "contents": ["meminsight-pkg.tar"],
  "size": "72K",
"installScript": "/etc/rdm/post-services/start_meminsight.sh"
}
Copilot AI review requested due to automatic review settings June 4, 2026 06:47

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

@@ -0,0 +1,29 @@
# Generate RDM package only when 'meminsight' is passed to GENERATE_RDM_CERTS
# GENERATE_RDM_CERTS can be passed through jenkins or can be set locally in auto.conf
inherit ${@bb.utils.contains('GENERATE_RDM_CERTS', '${BPN}', 'comcast-package-deploy', '', d)}
Comment on lines +14 to +17
DOWNLOADABLE_FILES = "${@bb.utils.contains('DOWNLOAD_APPS', '${PKG_BUNDLE_NAME}', '\
${bindir}/meminsight \
/etc/rdm/post-services/start_meminsight.sh \
', '', d)}"
Comment on lines +44 to +46
APP_HOME_DIR=/media/apps/${MODEL_NUM}-meminsight
rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight
Comment on lines +1 to +5
{
  "name": "meminsight",
  "description": "Performance Monitoring tool - meminsight",
  "version": "1.1",
  "contents": ["meminsight-pkg.tar"],
Copilot AI review requested due to automatic review settings June 4, 2026 11:18

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

Comment on lines +44 to +47
APP_HOME_DIR=/media/apps/${MODEL_NUM}-meminsight
rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight

rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight

SRC_BIN="/media/apps/meminsight/usr/bin/meminsight"
Comment on lines 18 to 22
file://meminsight-upload.service \
file://meminsight-upload.path \
file://upload_MemReports.sh \
file://package.json \
"
Comment on lines +1 to +9
{
  "name": "meminsight",
  "description": "Performance Monitoring tool - meminsight",
  "version": "1.0.0",
  "contents": ["meminsight-pkg.tar"],
  "size": "72K",
"installScript": "/etc/rdm/post-services/start_meminsight.sh"
}

Copilot AI review requested due to automatic review settings June 5, 2026 09:30

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

Comment thread recipes-devtools/meminsight/files/package.json
Comment on lines 20 to 22
file://upload_MemReports.sh \
file://package.json \
"
Comment on lines +44 to +46
APP_HOME_DIR=/media/apps/${MODEL_NUM}-meminsight
rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight
Comment on lines +48 to +50
SRC_BIN="/media/apps/meminsight/usr/bin/meminsight"
DST_BIN="/run/meminsight/usr/bin/meminsight"
RFC_PARAM="Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.meminsight.Trigger"
Copilot AI review requested due to automatic review settings June 9, 2026 06:30

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

Comment on lines 19 to 22
file://meminsight-upload.path \
file://upload_MemReports.sh \
file://package.json \
"
Comment on lines +44 to +46
APP_HOME_DIR=/media/apps/${MODEL_NUM}-meminsight
rm -rf /media/apps/meminsight
ln -snf "$APP_HOME_DIR" /media/apps/meminsight
Comment on lines +48 to +50
SRC_BIN="/media/apps/meminsight/usr/bin/meminsight"
DST_BIN="/run/meminsight/usr/bin/meminsight"
RFC_PARAM="Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.meminsight.Trigger"
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