Skip to content

Conversation

@uainap
Copy link

@uainap uainap commented Oct 14, 2025

Reason for change: IPV6 ping failure is high on 8.2 release. Deprecated IPv6 ping and implemented rdisc mechanism to check connectivity
Test Procedure: Load build on GW and wait for 1 - 2 hours of uptime. Check Selfheal.txt.0 logs. Should not see IPv6 ping failures.
Risks: Low
Priority: P0
Signed-off-by: Umesh_Ainapuram@comcast.com

Reason for change: IPV6 ping failure is high on 8.2 release. Deprecated IPv6 ping and implemented rdisc mechanism to check connectivity
Test Procedure: Load build on GW and wait for 1 - 2 hours of uptime. Check Selfheal.txt.0 logs. Should not see IPv6 ping failures.
Risks: Low
Priority: P0
Signed-off-by: Umesh_Ainapuram@comcast.com
@uainap uainap requested review from a team as code owners October 14, 2025 02:37
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

fi
fi

if [ "$IPv6_Gateway_addr" != "" ]
Copy link
Contributor

Choose a reason for hiding this comment

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

Whether in current sprint, whether this variable is properly set, I believe we have two addresses set on this variable. can you confirm

if [ "$ping4_success" -ne 1 ] && [ "$connectivity_ipv6_success" -ne 1 ]
then
echo_t "RDKB_SELFHEAL : Ping to both IPv4 and IPv6 Gateway Address failed."
t2CountNotify "RF_ERROR_IPV4IPV6PingFailed"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please modify this telemetry as well

if [ "$CHECK_PACKET_RECEIVED" -ne 100 ]
then
ping6_success=1
connectivity_ipv6_success=1
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we really need our connectivity test variable here. This is based on ipv4 server address and ipv6 address configured. It need not to be CMTS address. Can you please check the expected value here.

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