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

Ethernet description not shown in show interface ethernetX #89

Closed
mingjunzhang2019 opened this issue Sep 6, 2019 · 9 comments
Closed
Assignees
Labels
Fixed Works fine.

Comments

@mingjunzhang2019
Copy link
Collaborator

mingjunzhang2019 commented Sep 6, 2019

Z9100, Build label:299:

sonic(conf-if-Ethernet4)# description "DelSonic"
Success
sonic(conf-if-Ethernet4)# do show interface status
------------------------------------------------------------------------------------------
Name                Description         Admin          Oper           Speed          MTU
------------------------------------------------------------------------------------------
Ethernet0           -                   down           down                          9100
Ethernet4           DelSonic            down           down                          9100

sonic(conf-if-Ethernet4)# do show interface Ethernet 4
Ethernet4 is down, line protocol is down
Hardware is Eth
Interface index is 2
Mode of IPV4 address assignment: not-set
Mode of IPV6 address assignment: not-set
IP MTU 9100 bytes
LineSpeed , Auto-negotiation off
Input statistics:
        0 packets, 0 octets
        0 Multicasts, 0 Broadcasts, 0 Unicasts
        0 error, 0 discarded
Output statistics:
        0 packets, 0 octets
        0 Multicasts, 0 Broadcasts, 0 Unicasts
        0 error, 0 discarded
sonic(conf-if-Ethernet4)#                      
@mingjunzhang2019
Copy link
Collaborator Author

mingjunzhang2019 commented Sep 6, 2019

no description not work:

sonic(conf-if-Ethernet4)# no description
%Error: Invalid value
sonic(conf-if-Ethernet4)#          

@jeff-yin jeff-yin added the bug Something isn't working label Sep 6, 2019
@justinejose91
Copy link

I will look into it.

@justinejose91
Copy link

Description not present in #show interface ethernet and with id is fixed. #no description command issue is still open.

@justinejose91
Copy link

justinejose91 commented Sep 9, 2019

When we do #no description command for an interface, when I set the description to empty string and set operation, CVL throws failure. Added the CVL logs below.
cvl_logs.txt
When I tried removing the description entry from PORT table, then CVL doesnt throw failure, but app-db is not getting reflected.
Rest Request = {"openconfig-interfaces:description": ""} for any physical interface.

@jeff-yin
Copy link
Collaborator

@sachin-brm @dutta-partha -- can someone help look into the CVL failure when setting description empty string? This had previously worked in our testing.

@dutta-partha
Copy link
Collaborator

@asha-behera Please check the CVL error log once.

@justinejose91 Are you able to reproduce this every time ?

@justinejose91
Copy link

justinejose91 commented Sep 11, 2019

@dutta-partha Yes, this is reproducible every time. I tried deleting description entry as an other option, that won't reflect it in APP-DB, but setting it as empty string used to work earlier, now CVL throws error.

@dutta-partha
Copy link
Collaborator

@justinejose91 CVL fix for resetting description configuration is available in master branch (PR #118). @justinejose91 Can you please check it.

@dutta-partha dutta-partha added the Fix available Fix available in master branch. label Sep 13, 2019
@justinejose91
Copy link

Fixed. Works fine.

@justinejose91 justinejose91 added Fixed Works fine. and removed Fix available Fix available in master branch. bug Something isn't working labels Sep 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed Works fine.
Projects
None yet
Development

No branches or pull requests

6 participants