Skip to content

Commit

Permalink
[local]
Browse files Browse the repository at this point in the history
Remove the workaround in neighbour_mac_noptf, which temporarily adjust the log level to DEBUG for orchagent, neighsync and nbrmgrd ahead of nd test and recover to INFO after that.
This workaround was introduced in order to provide detailed debug information for the issue sonic-net/sonic-buildimage#2414 (DUT takes more than 7 seconds to finish update ip v6 neighbor). However, we found those information can hardly help.

Signed-off-by: Stephen Sun <[email protected]>
  • Loading branch information
Stephen Sun committed May 29, 2019
1 parent a506edf commit fccb890
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions ansible/roles/test/tasks/neighbour-mac-noptf.yml
Original file line number Diff line number Diff line change
Expand Up @@ -107,11 +107,6 @@
v6_nei: "{{ v6_intf_nei }}"
when: v6_nei is not defined

#### Temporary added to capture more log for debug ####
- name: turn on the debug log for orchagent, nbrmgrd and neighsyncd
shell: swssloglevel -l DEBUG -c orchagent; swssloglevel -l DEBUG -c neighsyncd; swssloglevel -l DEBUG -c nbrmgrd;
become: yes

- name: add an ipv6 entry for Ethernet0 if not find v6 neighbor
command: "/sbin/ifconfig Ethernet0 inet6 add {{ v6_intf_ip }}"
when: v6_nei is not defined
Expand Down Expand Up @@ -160,11 +155,6 @@
become: yes

always:
#### turn off the debug log ####
- name: turn off the debug log for orchagent, nbrmgrd and neighsyncd
shell: swssloglevel -l NOTICE -c orchagent; swssloglevel -l NOTICE -c neighsyncd; swssloglevel -l NOTICE -c nbrmgrd;
become: yes

- name: reset all changes
include: "roles/test/tasks/common_tasks/reload_config.yml"
vars:
Expand Down

0 comments on commit fccb890

Please sign in to comment.