featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

Added component name prefix to process name for Juniper

Open priyaranjannanda opened this issue 2 months ago • 2 comments

This changes are specific to Juniper

  • where gnoi rpc request for sig_kill needs component name to be prefixed.
  • e.g. for rpd it is "re0:routing"

-- Priya Ranjan

priyaranjannanda avatar Nov 05 '25 13:11 priyaranjannanda

Pull Request Functional Test Report for #4780 / 3ace7a22ca7a800e9b7f3a22e1c3865ca37f15e6

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
gNOI-5.3: Copying Debug Files
Cisco 8000E status
gNOI-5.3: Copying Debug Files
Cisco XRd status
gNOI-5.3: Copying Debug Files
Juniper ncPTX status
gNOI-5.3: Copying Debug Files
Nokia SR Linux status
gNOI-5.3: Copying Debug Files
Openconfig Lemming status
gNOI-5.3: Copying Debug Files

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
gNOI-5.3: Copying Debug Files
Cisco 8808 status
gNOI-5.3: Copying Debug Files
Juniper PTX10008 status
gNOI-5.3: Copying Debug Files
Nokia 7250 IXR-10e status
gNOI-5.3: Copying Debug Files

Help

OpenConfigBot avatar Nov 05 '25 13:11 OpenConfigBot

Pull Request Test Coverage Report for Build 19847233303

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.112%

Totals Coverage Status
Change from base Build 19807068140: 0.0%
Covered Lines: 2227
Relevant Lines: 22023

💛 - Coveralls

coveralls avatar Nov 05 '25 13:11 coveralls

/fptest all

singhavnish2516 avatar Nov 10 '25 18:11 singhavnish2516

Note that ncptx is failing this test. I'd expect it to pass. See the Log:

Step #1 - "fp-presubmit": Path Noncompliance Errors:
Step #1 - "fp-presubmit": 	Unmarshal uint_val:57318 into origin:"openconfig"  elem:{name:"system"}  elem:{name:"processes"}  elem:{name:"process"  key:{key:"pid"  value:"973"}}  elem:{name:"state"}  elem:{name:"uptime"}: path "/system/processes/process[pid=973]/state/uptime" is invalid and cannot be matched to a generated GoStruct field: rpc error: code = InvalidArgument desc = no match found in *oc.System_Process, for path elem:{name:"state"}  elem:{name:"uptime"}
Step #1 - "fp-presubmit": Type Noncompliance Errors: None
Step #1 - "fp-presubmit": Value Restriction Noncompliance Errors: None
Step #1 - "fp-presubmit":     copying_debug_files_test.go:91: Failed to restart process rpd with unexpected err: rpc error: code = InvalidArgument desc = error on request {
Step #1 - "fp-presubmit":         pid:  3244
Step #1 - "fp-presubmit":         name:  "RE0:ROUTING"
Step #1 - "fp-presubmit":         signal:  SIGNAL_KILL
Step #1 - "fp-presubmit":         restart:  true
Step #1 - "fp-presubmit":         }: Name does not match PID.
Step #1 - "fp-presubmit": --- FAIL: TestCopyingDebugFiles (3.33s)

dplore avatar Nov 10 '25 20:11 dplore

Hi,

This script changes are no more required. hence closing it.

-- Thanks & Regards, Priya Ranjan

priyaranjannanda avatar Dec 02 '25 04:12 priyaranjannanda