Fixed code as per the regression test failure#5423
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses regression test failures by enhancing the reliability and scope of the MPLS-over-GUE decap scale test. The changes focus on improving synchronization between the device under test (DUT) and the traffic generator, ensuring that interfaces and protocols are fully operational before traffic transmission begins. Additionally, the test suite has been updated to include IPv6 static routes and traffic flows, providing more comprehensive validation coverage. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request enhances the MPLS GUE IPv4 decap scale test by introducing IPv6 static route configuration, refactoring environment setup, and adding robust validation helpers for LAG and subinterface operational states. The changes include new utility functions for OTG protocol convergence and traffic handling. Review feedback highlights a potential issue with side effects when modifying global flow objects in the createflow helper, a logical mismatch between configured packet counts and traffic duration, and an inconsistent IPv6 prefix length on the core interface.
|
/fptest virtual |
ReadMe Location: https://github.com/open-traffic-generator/featureprofiles/blob/main/feature/policy_forwarding/otg_tests/mpls_gue_ipv4_decap_scale_test/README.md
Attached pass logs here: https://partnerissuetracker.corp.google.com/u/2/issues/415458482