Skip to content

RDKBACCL-1463: DB and other customized values are overwritten to defa…#402

Open
manigandanj wants to merge 1 commit intordkcentral:developfrom
manigandanj:RDKBACCL-1463_nvram_mount
Open

RDKBACCL-1463: DB and other customized values are overwritten to defa…#402
manigandanj wants to merge 1 commit intordkcentral:developfrom
manigandanj:RDKBACCL-1463_nvram_mount

Conversation

@manigandanj
Copy link
Contributor

…ult without FR after firmware upgrade

Reason for change: Mounting nvram partition in /nvram folder
Test procedure: Sanity
Risks: Low

@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch 2 times, most recently from 7c97503 to b36c57b Compare February 4, 2026 15:03
@manigandanj manigandanj marked this pull request as draft February 4, 2026 15:05
@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch 2 times, most recently from 8dbcc04 to 40b40e2 Compare February 18, 2026 11:21
@@ -0,0 +1,12 @@
[Unit]
Description=Mount nvram partition
ConditionPathExists=/nvram
Copy link
Contributor

Choose a reason for hiding this comment

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

This check shouldn't be present for this case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay will remove it.

@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch 4 times, most recently from ab0b390 to 643f761 Compare February 25, 2026 12:06
@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch from 643f761 to a572f1d Compare March 3, 2026 11:28
@@ -0,0 +1,11 @@
[Unit]
Copy link
Contributor

Choose a reason for hiding this comment

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

pls create folder inside banana pi as recipes-nonvol and create a bb file and keep .sh and service file and install in the rootfs and add it as dependency in core rdkb.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i did not find any reference for recipes-nonvol. can i host recipe in recipes-support folder?

Copy link
Contributor

Choose a reason for hiding this comment

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

Please try creating recipe-nonvol folder

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is done.

FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:${THISDIR}/files:"
SRC_URI_append = " \
file://gwprovapp.conf \
file://mount-nvram.service \
Copy link
Contributor

Choose a reason for hiding this comment

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

pls create folder inside banana pi as recipes-nonvol and create a bb file and keep .sh and service file and install in the rootfs and add it as dependency in core rdkb.

Copy link
Contributor

@cpokuru cpokuru left a comment

Choose a reason for hiding this comment

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

Also please take care of bind mount of nvram , opt/secure

fi
mkdir -p /nvram/secure
# OneWifi DB stored in /opt/secure/wifi
mount /dev/mmcblk0p9 /opt/secure
Copy link
Contributor

Choose a reason for hiding this comment

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

we cannot mount same device again.we should go for bind

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is taken care.

@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch from a572f1d to d78d039 Compare March 4, 2026 13:16
@manigandanj manigandanj marked this pull request as ready for review March 4, 2026 14:41
@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/d06cfd7a23f0f573e934a982de996547

@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch 2 times, most recently from e5a7be8 to 84f9396 Compare March 5, 2026 14:21
@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/meta-cmf-bananapi/402/rdkcentral/meta-cmf-bananapi

  • Commit: 84f9396

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/meta-cmf-bananapi/402/rdkcentral/meta-cmf-bananapi

  • Commit: 84f9396

Report detail: gist'

@@ -0,0 +1,40 @@
DESCRIPTION = "Mounting nvram partition for storing customized data"

LICENSE = "CLOSED"

Choose a reason for hiding this comment

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

LICENSE cannot be closed as this repo is open and SRC_URI points to two files here. Incidentally, mount-nvram.sh should have an Apache-2 license header.

@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/e2d08e06656591936c96551bfeb8d7ac

@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/5782c60bc4ef555ad50290bd19660450

@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch from 84f9396 to 7c52e78 Compare March 6, 2026 09:46
@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/meta-cmf-bananapi/402/rdkcentral/meta-cmf-bananapi

  • Commit: 7c52e78

Report detail: gist'

…ult without FR after firmware upgrade

Reason for change: Mounting nvram partition in /nvram folder
Test procedure: Sanity and FWUpgrade
Risks: Low

Signed-off-by: Manigandan Gopalakrishnan <[email protected]>
@manigandanj manigandanj force-pushed the RDKBACCL-1463_nvram_mount branch from 7c52e78 to c618ca0 Compare March 6, 2026 10:10
@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/meta-cmf-bananapi/402/rdkcentral/meta-cmf-bananapi

  • Commit: c618ca0

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/bd8d04dd52a0eac9d8fc784ffbd05be8

@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/c01aeed24b824264722c483564d61303

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.

5 participants