DNM: log first IPv6 CIDR association entry for debugging - #261
Conversation
Add a temporary klog.Infof in getSubnet to print the first entry in Ipv6CidrBlockAssociationSet (CIDR value and state) so we can observe whether the old code was picking a stale 'disassociated' entry. This is an experimental/diagnostic-only commit and should not be merged. Signed-off-by: Jean Chen <jechen@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
Important Review skippedIgnore keyword(s) in the title. ⛔ Ignored keywords (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jechen0648 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Summary
This is a Do Not Merge experimental PR for diagnostic purposes only.
Adds a temporary
klog.InfofingetSubnetto log the first entry inIpv6CidrBlockAssociationSet(CIDR value and its state) before the existing code uses it. This allows us to observe whether the old code was picking a staledisassociatedentry — answering the reviewer question on #228 about what the incorrect subnet value actually was.How to use
Deploy this build on a dualstack cluster whose subnet has a stale IPv6 CIDR association, then check the CNCC pod logs:
oc logs -n openshift-cloud-network-config-controller \ deployment/cloud-network-config-controller -f | grep getSubnetExpected output (on affected cluster):
Related
Made with Cursor