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.

Sign up for GitHub

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

Jump to bottom

tlm_teamd still crashing when running test_po_update test #5755

Closed
lguohan opened this issue Oct 30, 2020 · 1 comment · Fixed by sonic-net/sonic-swss#1490
Closed

tlm_teamd still crashing when running test_po_update test #5755

lguohan opened this issue Oct 30, 2020 · 1 comment · Fixed by sonic-net/sonic-swss#1490
Assignees
@pavel-shirshov
Labels
Bug 🐛 Master Branch Quality P0 Priority of the issue

Comments

@lguohan
Copy link
Collaborator

lguohan commented Oct 30, 2020

check out this test.

https://sonic-jenkins.westus2.cloudapp.azure.com/job/common/job/sonic-mgmt-testing/job/sonic-mgmt-pr/1792/testReport/junit/route/test_default_route/test_default_route_set_src/

Oct 29 21:53:00.911408 vlab-01 NOTICE teamd#teammgrd: :- setLagMtu: Set port channel PortChannel999 MTU to 9100
Oct 29 21:53:01.570923 vlab-01 INFO ansible-command: Invoked with creates=None executable=None _uses_shell=True strip_empty_ends=True _raw_params=config portchannel member add PortChannel999 Ethernet112
removes=None argv=None warn=True chdir=None stdin_add_newline=True stdin=None
Oct 29 21:53:01.867700 vlab-01 INFO kernel: [ 2539.062561] tlm_teamd[18333]: segfault at 0 ip 000055bbfc143bc8 sp 00007ffcb598b190 error 4 in tlm_teamd[55bbfc140000+9000]

you can find the dump file in here.

https://sonic-jenkins.westus2.cloudapp.azure.com/job/common/job/sonic-mgmt-testing/job/sonic-mgmt-pr/1792/
@lguohan lguohan transferred this issue from sonic-net/sonic-swss Oct 30, 2020
@lguohan lguohan mentioned this issue Oct 30, 2020
Merged
3 tasks
@lguohan
Copy link
Collaborator Author

lguohan commented Oct 30, 2020 •
edited
Loading

the test is using latest kvm image 531, which has latest swss submodule. @pavel-shirshov , it looks like your previous fix sonic-net/sonic-swss@495816d still not solving the issue.

@pavel-shirshov pavel-shirshov mentioned this issue Oct 30, 2020
Merged
lguohan pushed a commit to sonic-net/sonic-swss that referenced this issue Nov 1, 2020
@pavel-shirshov
[tlm_teamd]: Make the destionation for std::transform() to use std::b…
d7643f2 
…ack_inserter() for allocating new space for the copied objects (#1490)

Fixes: sonic-net/sonic-buildimage#5755
Fixes: sonic-net/sonic-buildimage#5433

Make the destination for std::transform() to use std::back_inserter() for allocating new space for the copied objects
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pavel-shirshov pavel-shirshov

Labels
Bug 🐛 Master Branch Quality P0 Priority of the issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[tlm_teamd]: Fix segfault, when retrying to do teamd_connect() again pavel-shirshov/sonic-swss
2 participants
@lguohan @pavel-shirshov