Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gNMI UT failures #120

Open
PrabhuSreenivasan opened this issue Sep 13, 2019 · 4 comments
Open

gNMI UT failures #120

PrabhuSreenivasan opened this issue Sep 13, 2019 · 4 comments
Assignees

Comments

@PrabhuSreenivasan
Copy link
Collaborator

PrabhuSreenivasan commented Sep 13, 2019

Two tests are failing on gNMI UT. Since its an UT issue it needs to be fixed on priority.

+--------+---------+-------------+-------+------+------+------+
| STATUS | ELAPSED |   PACKAGE   | COVER | PASS | FAIL | SKIP |
+--------+---------+-------------+-------+------+------+------+
| FAIL   | 0.00s   | gnmi_server | 30.9% |   15 |    2 |    0 |
+--------+---------+-------------+-------+------+------+------+
FAIL: gnmi_server
-----------------
--- FAIL: TestGnmiSet (12.99s)
   --- FAIL: TestGnmiSet/Check_OC_Interface_values_set (0.01s)
       server_test.go:155: got: map[openconfig-interfaces:interface:[map[config:map[description: mtu:9000 name:Ethernet0] name:Ethernet0 openconfig-if-ethernet:ethernet:map[state:map[port-speed:openconfig-if-ethernet:SPEED_10GB] config:map[port-speed:openconfig-if-ethernet:SPEED_10GB]] state:map[mtu:9000 name:Ethernet0 oper-status:DOWN admin-status:DOWN counters:map[in-discards:0 in-octets:0 in-pkts:0 out-broadcast-pkts:0 out-pkts:0 in-errors:0 in-unicast-pkts:0 out-errors:0 out-multicast-pkts:0 out-octets:0 out-unicast-pkts:0 out-discards:0 in-broadcast-pkts:0 in-multicast-pkts:0] description: ifindex:0] subinterfaces:map[subinterface:[map[index:0]]]]]] (map[string]interface {}),
           want map[openconfig-interfaces:interface:[map[openconfig-if-ethernet:ethernet:map[state:map[port-speed:openconfig-if-ethernet:SPEED_40GB] config:map[port-speed:openconfig-if-ethernet:SPEED_40GB]] state:map[admin-status:UP counters:map[out-unicast-pkts:0 in-octets:0 out-errors:0 in-errors:0 out-octets:0 in-broadcast-pkts:0 in-discards:0 in-unicast-pkts:0 out-broadcast-pkts:0 out-discards:0 out-multicast-pkts:0 out-pkts:0 in-multicast-pkts:0 in-pkts:0] description: ifindex:0 mtu:9000 name:Ethernet0 oper-status:DOWN] subinterfaces:map[subinterface:[map[index:0 openconfig-if-ip:ipv4:map[addresses:map[address:[map[ip:9.9.9.9 state:map[ip:9.9.9.9 prefix-length:24] config:map[ip:9.9.9.9 prefix-length:24]]]]]]]] config:map[description: mtu:9000 name:Ethernet0] name:Ethernet0]]] (map[string]interface {})
@jeff-yin
Copy link
Collaborator

@Tejaswi-Goel -- please just disable the UT cases for now until a platform-independent solution is devised.

@jeff-yin
Copy link
Collaborator

@PrabhuSreenivasan -- UT cases are disabled in this PR project-arlo/sonic-telemetry#28

Let's leave this issue open for the follow-up fix

@PrabhuSreenivasan
Copy link
Collaborator Author

@jeff-yin I started seeing new issues after using the new code.

------------------------------------------------------------------
Summary: PASS: 471 FAIL: 2 ERRORS: 0 TOTAL: 473   PASS(%): 99% 
------------------------------------------------------------------
+--------+---------+--------------------+-------+------+------+------+
| STATUS | ELAPSED |      PACKAGE       | COVER | PASS | FAIL | SKIP |
+--------+---------+--------------------+-------+------+------+------+
| PASS   | 0.00s   | rest/server        | 90.0% |  193 |    0 |    0 |
| PASS   | 0.01s   | cvl                | 88.4% |   85 |    0 |    0 |
| PASS   | 0.01s   | translib/db        | 63.6% |    6 |    0 |    0 |
| PASS   | 0.01s   | translib           | 40.2% |  173 |    0 |    0 |
| FAIL   | 0.01s   | gnmi_server/server | 31.4% |   14 |    2 |    0 |
+--------+---------+--------------------+-------+------+------+------+
FAIL: gnmi_server/server
------------------------
--- FAIL: TestGnmiGet (6.64s)
    --- FAIL: TestGnmiGet/Get_OC_System_CPU (0.00s)
        server_test.go:117: err:  rpc error: code = NotFound desc = GET failed for this message
        server_test.go:118: got return code NotFound, want OK

@jeff-yin
Copy link
Collaborator

@arunsundark -- seems there are still gnmi failures?

@project-arlo project-arlo deleted a comment from Tejaswi-Goel Sep 18, 2019
dell-engops pushed a commit that referenced this issue Jul 5, 2022
* [JIRA SONIC-59502] User name field in not applicable cases should be displayed as Not Applicable instead of empty.

Change-Id: Ie96be7eacf13a9ebe4c9e95abbd6a88f1e47533f

* [JIRA SONIC-60245] Field to display the operational value of the session timeout in the output of the show authentication clients.

Change-Id: Ieddba8ac30568d55197fe1ce172b8cc208ffdc30

* [JIRA SONIC-56405] Added support for pac configuration over range of interfaces.

Change-Id: I0ce68fc2497fcc45fcd0c15d6113542967d974c2

* [JIRA SONIC-61478] KLISH: add 50G AN support

Signed-off-by: Dante Su <[email protected]>
Change-Id: If6c05a49f18469858b66c58cd456e7b4f6a3c0e0

* [JIRA SONIC-61803] GetTablePattern ExistKeysPattern API in DB Layer

- Added Statistics

Change-Id: I3732e78e13881842e9394a571788abe3ab1ac337

* [JIRA SONIC-61314] go.sum update

Change-Id: I7576ffc6933f9727747d3463b022c23ad7def609

* [JIRA SONIC-61807] - Model Identity as enumeration

Used Enum to represent identities
This will allow swagger ui to show drop-down in case of parameters.

Change-Id: I8e438545d0b28c9c2ee545a2704366da512b5c74

* [JIRA SONIC-60508]: ospf neighbor details template fix

Change-Id: Iee1dbdc72e02a6d23e0dfac71220843be3730ddd

* [JIRA SONIC-61879] PAC - Added required changes for config knobs dymanic_vlan_enable, vlan_assign_enable.

Change-Id: Ib38db8f2e9917b4261fc709e1548ea78f0196453

* [JIRA SONIC-61168] Klish support for ND config commands

Change-Id: I7227cb99ecf82bddac169c3b544bd28758f40c3b

* [JIRA SONIC-61223] With user added explicit snooping configuration on Vlan, actioner has called PATCH with default value for the no form of the user configuration, this has resulted in running configuration showing default value when there is no explicit configuration for the default

- Fixed actioner script to trigger DELETE for the snooping configuration instead of PATCH with the default values.
- Fix added in l2mcmgrd to fill snooping configuration IPC message by reading redisDB update instead of keeping separate local cache for snooping configurations which comes before snooping enable.

Change-Id: I3ace4e6eca5cb7608d48bb9e3a11bb1c4976e4d9

* [JIRA SONIC-62134] - Corrected the sag xml order to come after vrf config

Change-Id: I0d463d9d16e014d39b343d14d36ff801019017d4

* [JIRA SONIC-61997] using escap_chars option to skip '\' for special chars

Change-Id: I3b654ea105931c0e3605b84ed5db560cb5e13e95

Co-authored-by: Sravani Maddukuri <[email protected]>
Co-authored-by: Shiva Kumar Boddula <[email protected]>
Co-authored-by: Dante Su <[email protected]>
Co-authored-by: Sonic Build <[email protected]>
Co-authored-by: Arun Barboza <[email protected]>
Co-authored-by: Sachin Holla <[email protected]>
Co-authored-by: Faraaz <[email protected]>
Co-authored-by: Ramachandran Sathianandan <[email protected]>
Co-authored-by: sonicmergeuser <[email protected]>
Co-authored-by: sudhanshukumar22 <[email protected]>
Co-authored-by: Ashok Krishnegowda <[email protected]>
Co-authored-by: Akhilesh Samineni <[email protected]>
Co-authored-by: suresh-rupanagudi <[email protected]>
Co-authored-by: SONiC Syncbot <[email protected]>
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

No branches or pull requests

5 participants