CBR2-2262 : set primary_lan_l3net sysevent#302
Conversation
Reason for change: Fix the typo in setting primary_lan_l3net sysevent Test Procedure: Flash the image and test the brlan0 in CBR2 Risks: Low Priority: P1
There was a problem hiding this comment.
Pull request overview
Fixes a typo in the LAN initialization handler intended to ensure the primary_lan_l3net sysevent is set correctly for CBR2/TCCBR scenarios.
Changes:
- Replace an invalid
syseveninvocation withsysevent set primary_lan_l3net 4. - Replace a no-op
echo(redirected to/dev/null) with anecho_tlog message in the same branch.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| elif [ "$BOX_TYPE" = "TCCBR" ]; then | ||
| if [ -z "$INST" ]; then | ||
| echo "*****SET THE PRIMARY LAN ******" > /dev/null | ||
| syseven set primary_lan_l3net 4 | ||
| echo_t "SET THE PRIMARY LAN $INST" | ||
| sysevent set primary_lan_l3net 4 | ||
| fi |
There was a problem hiding this comment.
@copilot apply changes based on this feedback
There was a problem hiding this comment.
Updated in commit e0301c9. I moved the TCCBR fallback out of the if [ x != x$INST ] block so it now executes correctly when INST is empty, and still defaults primary_lan_l3net to 4 for that case.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Agent-Logs-Url: https://github.com/rdkcentral/utopia/sessions/fcfd6e0e-8b1c-4360-b167-6809a7b5b990 Co-authored-by: rajkamal-cv <72071311+rajkamal-cv@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Reason for change: Fix the typo in setting primary_lan_l3net sysevent
Test Procedure: Flash the image and test the brlan0 in CBR2
Risks: Low
Priority: P1