Skip to content

Adding PF-1.9 Egress handling of TTL Test#4405

Merged
dplore merged 14 commits into
openconfig:mainfrom
open-traffic-generator:fpt_otg_PF-1.9
Sep 11, 2025
Merged

Adding PF-1.9 Egress handling of TTL Test#4405
dplore merged 14 commits into
openconfig:mainfrom
open-traffic-generator:fpt_otg_PF-1.9

Conversation

@keysightgems
Copy link
Copy Markdown
Contributor

@keysightgems keysightgems requested a review from a team as a code owner July 15, 2025 09:03
@OpenConfigBot
Copy link
Copy Markdown

OpenConfigBot commented Jul 15, 2025

Pull Request Functional Test Report for #4405 / df3a2ee

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
PF-1.9: Egress handling of TTL
Cisco 8000E status
PF-1.9: Egress handling of TTL
Cisco XRd status
PF-1.9: Egress handling of TTL
Juniper ncPTX status
PF-1.9: Egress handling of TTL
Nokia SR Linux status
PF-1.9: Egress handling of TTL
Openconfig Lemming status
PF-1.9: Egress handling of TTL

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
PF-1.9: Egress handling of TTL
Cisco 8808 status
PF-1.9: Egress handling of TTL
Juniper PTX10008 status
PF-1.9: Egress handling of TTL
Nokia 7250 IXR-10e status
PF-1.9: Egress handling of TTL

Help

Comment thread proto/metadata.proto
Comment thread proto/metadata.proto Outdated
@nikki-blr nikki-blr assigned dplore and unassigned balaji6 Aug 1, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Aug 1, 2025

Pull Request Test Coverage Report for Build 17601008964

Details

  • 0 of 173 (0.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 12.378%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/deviations/deviations.go 0 3 0.0%
internal/cfgplugins/mpls.go 0 24 0.0%
internal/cfgplugins/policyforwarding.go 0 73 0.0%
proto/metadata_go_proto/metadata.pb.go 0 73 0.0%
Totals Coverage Status
Change from base Build 17582978978: -0.1%
Covered Lines: 2111
Relevant Lines: 17054

💛 - Coveralls

Comment thread internal/cfgplugins/mpls.go
@dplore
Copy link
Copy Markdown
Member

dplore commented Aug 5, 2025

@keysightgems
Copy link
Copy Markdown
Contributor Author

Fixed static checks.
Fixed static checks.
Copy link
Copy Markdown
Contributor

@vvlakshmanamurthy vvlakshmanamurthy left a comment

Choose a reason for hiding this comment

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

Bypass config should be clarified with Arista team and the same needs to be removed from the code once clarified and please do not submit this until the "bypass" knob is clarified.

@ANISH-GOTTAPU
Copy link
Copy Markdown
Contributor

Bypass config should be clarified with Arista team and the same needs to be removed from the code once clarified and please do not submit this until the "bypass" knob is clarified.

Hi @vvlakshmanamurthy : Will you be checking this with Arista or you want us to take it. We don't have permission to raise issue with Arista, we can just ask them in Vendor chat. Please let us know who will take this forward.

@keysightgems
Copy link
Copy Markdown
Contributor Author

Bypass config should be clarified with Arista team and the same needs to be removed from the code once clarified and please do not submit this until the "bypass" knob is clarified.

Clarified with Arista team, they accepted with "access-list bypass". Thanks!

@dplore dplore merged commit 0005981 into openconfig:main Sep 11, 2025
14 of 15 checks passed
nsadhasivam pushed a commit to nsadhasivam/featureprofiles that referenced this pull request Oct 26, 2025
* Adding PF-1.9 Egress handling of TTL Test
priyaranjannanda pushed a commit to priyaranjannanda/featureprofiles that referenced this pull request Jan 22, 2026
* Adding PF-1.9 Egress handling of TTL Test
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.

9 participants