Skip to content

Improve test_ssh_limit test case not receive EOF issue#14116

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
liuh-80:dev/liuh/improve_ssh_limit
Aug 28, 2024
Merged

Improve test_ssh_limit test case not receive EOF issue#14116
qiluo-msft merged 1 commit intosonic-net:masterfrom
liuh-80:dev/liuh/improve_ssh_limit

Conversation

@liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Aug 14, 2024

Improve test_ssh_limit test case not receive EOF issue

Why I did it

test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

Work item tracking
  • Microsoft ADO: 28874779

How I did it

Read stdout and stderr after exec command via paramiko, and print output for debug

How to verify it

Pass all test case.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Improve test_ssh_limit test case not receive EOF issue

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@liuh-80 liuh-80 marked this pull request as ready for review August 19, 2024 02:11
@liuh-80 liuh-80 requested a review from qiluo-msft August 19, 2024 02:11
@qiluo-msft qiluo-msft merged commit 254f425 into sonic-net:master Aug 28, 2024
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 29, 2024
Improve test_ssh_limit test case not receive EOF issue

#### Why I did it
test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

### How I did it
Read stdout and stderr after exec command via paramiko, and print output for debug

#### How to verify it
Pass all test case.

### Description for the changelog
Improve test_ssh_limit test case not receive EOF issue
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202311: #14306

mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 29, 2024
Improve test_ssh_limit test case not receive EOF issue

#### Why I did it
test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

### How I did it
Read stdout and stderr after exec command via paramiko, and print output for debug

#### How to verify it
Pass all test case.

### Description for the changelog
Improve test_ssh_limit test case not receive EOF issue
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #14307

mssonicbld pushed a commit that referenced this pull request Aug 29, 2024
Improve test_ssh_limit test case not receive EOF issue

#### Why I did it
test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

### How I did it
Read stdout and stderr after exec command via paramiko, and print output for debug

#### How to verify it
Pass all test case.

### Description for the changelog
Improve test_ssh_limit test case not receive EOF issue
mssonicbld pushed a commit that referenced this pull request Aug 29, 2024
Improve test_ssh_limit test case not receive EOF issue

#### Why I did it
test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

### How I did it
Read stdout and stderr after exec command via paramiko, and print output for debug

#### How to verify it
Pass all test case.

### Description for the changelog
Improve test_ssh_limit test case not receive EOF issue
arista-hpandya pushed a commit to arista-hpandya/sonic-mgmt that referenced this pull request Oct 2, 2024
Improve test_ssh_limit test case not receive EOF issue

#### Why I did it
test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

### How I did it
Read stdout and stderr after exec command via paramiko, and print output for debug

#### How to verify it
Pass all test case.

### Description for the changelog
Improve test_ssh_limit test case not receive EOF issue
vikshaw-Nokia pushed a commit to vikshaw-Nokia/sonic-mgmt that referenced this pull request Oct 23, 2024
Improve test_ssh_limit test case not receive EOF issue

#### Why I did it
test_ssh_limit randomly failed, according to syslog, the paramiko connection does not receive EOF after exec command.

### How I did it
Read stdout and stderr after exec command via paramiko, and print output for debug

#### How to verify it
Pass all test case.

### Description for the changelog
Improve test_ssh_limit test case not receive EOF issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants