Skip to content

Conversation

@shashwats1
Copy link

@shashwats1 shashwats1 commented Oct 3, 2025

RDKBDEV-3295: Adding configuration for new labels "webpa-interface-label" and "wan-ipv4-address" in webpa convey header.

DESCRIPTION: Added configurations for the labels mentioned, "webpa-interface-label" can be configured with "Fixed" or "Mobile" values for representing primary or backup WAN respectively. "wan-ipv4-address" can represent the ipv4 address of the current active interface like ipv4 address of erouter0 interface.

TESTING DONE: Test results captured in RDKBDEV-3295

@CLAassistant
Copy link

CLAassistant commented Oct 3, 2025

CLA assistant check
All committers have signed the CLA.

@shashwats1 shashwats1 force-pushed the master branch 3 times, most recently from d6d1279 to 2aa3e76 Compare October 6, 2025 13:08
@pradeeptakdas
Copy link

Internal ticket for tracking https://ccp.sys.comcast.net/browse/RDKCOM-5455

Copy link
Contributor

@sadhyama sadhyama left a comment

Choose a reason for hiding this comment

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

Change looks good to me.
Please add the 2 new fields in parodus README file and its usage similar to existing command line arguments.

Github work flow builds are not running due to ubuntu 20 deprecation, we are planning to fix this as a separate bug.
So once changes are done , will merge PR as it is already unit tested.

@shashwats1 shashwats1 force-pushed the master branch 2 times, most recently from dc45285 to b0fa359 Compare October 24, 2025 10:02
…bel" and "wan-ipv4-address" in webpa convey header.

DESCRIPTION: Added configurations for the labels mentioned, "webpa-interface-label" can be configured with "Fixed" or "Mobile" values for representing primary or backup WAN respectively. "wan-ipv4-address" can represent the ipv4 address of the current active interface like ipv4 address of erouter0 interface.

TESTING DONE: Test results captured in RDKBDEV-3295

Signed-off-by: shashwats1 <[email protected]>
@shashwats1
Copy link
Author

Addressed the review comment and updated the README file

@shashwats1 shashwats1 requested a review from sadhyama October 27, 2025 09:00
@sadhyama sadhyama merged commit e11acb9 into xmidt-org:master Oct 28, 2025
1 of 2 checks passed
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.

4 participants