featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

OC-26.1: Removing ntp_non_default_vrf_unsupported deviation

Open pshivash opened this issue 3 months ago • 3 comments

  • Removed ntp_non_default_vrf_unsupported deviation from OC-26.1 and from deviations

"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

pshivash avatar Oct 06 '25 17:10 pshivash

Pull Request Functional Test Report for #4679 / 6109592c1dec84fec656d11fc4f680d8de032486

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
OC-26.1: Network Time Protocol (NTP)
Cisco 8000E status
OC-26.1: Network Time Protocol (NTP)
Cisco XRd status
OC-26.1: Network Time Protocol (NTP)
Juniper ncPTX status
OC-26.1: Network Time Protocol (NTP)
Nokia SR Linux status
OC-26.1: Network Time Protocol (NTP)
Openconfig Lemming status
OC-26.1: Network Time Protocol (NTP)

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
OC-26.1: Network Time Protocol (NTP)
Cisco 8808 status
OC-26.1: Network Time Protocol (NTP)
Juniper PTX10008 status
OC-26.1: Network Time Protocol (NTP)
Nokia 7250 IXR-10e status
OC-26.1: Network Time Protocol (NTP)

Help

OpenConfigBot avatar Oct 06 '25 17:10 OpenConfigBot

Pull Request Test Coverage Report for Build 20071570703

Details

  • 0 of 18 (0.0%) changed or added relevant lines in 1 file are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.03%) to 10.072%

Changes Missing Coverage Covered Lines Changed/Added Lines %
proto/metadata_go_proto/metadata.pb.go 0 18 0.0%
<!-- Total: 0 18
Files with Coverage Reduction New Missed Lines %
proto/metadata_go_proto/metadata.pb.go 4 0.0%
<!-- Total: 4
Totals Coverage Status
Change from base Build 20058892338: 0.03%
Covered Lines: 2227
Relevant Lines: 22111

💛 - Coveralls

coveralls avatar Oct 30 '25 05:10 coveralls

The test fails without the deviation. The system ntp network-instance (mgmtVrf) must be the same as the provided for every server

@AmrNJ, As mentioned by @trathod1, could you please update the baseline config and resubmit the tests?

pshivash avatar Nov 06 '25 12:11 pshivash

@AmrNJ Could you please update the baseline config and resubmit the tests, as requested? cc: @trathod1

pshivash avatar Dec 09 '25 07:12 pshivash