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

fixing #7283: Move system-health daemon into sonic-host-services/soni… #9130

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

fixing #7283: Move system-health daemon into sonic-host-services/soni… #9130

wants to merge 8 commits into from

Conversation

MaratGubaiev
Copy link
Contributor

…c-host-services-data packages

Why I did it

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

healthd is moved to sonic-host-services & sonic-host-services-data

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

@ghost
Copy link

ghost commented Nov 1, 2021 •
edited by ghost
Loading

CLA assistant check
All CLA requirements met.

@MaratGubaiev MaratGubaiev mentioned this pull request Nov 1, 2021
Draft
4 tasks
@MaratGubaiev
Copy link
Contributor Author

The test error is dget: curl net-snmp_5.7.3+dfsg-5.debian.tar.xz https://sonicstorage.blob.core.windows.net/debian/pool/main/n/net-snmp/net-snmp_5.7.3+dfsg-5.debian.tar.xz failed https://dev.azure.com/mssonic/build/_build/results?buildId=48413&view=logs&j=cef3d8a9-152e-5193-620b-567dc18af272&t=359769c4-8b5e-5976-a793-85da132e0a6f&l=467

So it looks like the problem with tests themselves.

I think we should try to rerun the tests.

Could anybody please do /azp run?
@lguohan @qiluo-msft @xumia

@MaratGubaiev
Copy link
Contributor Author

MaratGubaiev commented Nov 4, 2021 •
edited
Loading

dget: curl net-snmp_5.7.3+dfsg-5.debian.tar.xz https://sonicstorage.blob.core.windows.net/debian/pool/main/n/net-snmp/net-snmp_5.7.3+dfsg-5.debian.tar.xz failed https://dev.azure.com/mssonic/build/_build/results?buildId=48639&view=logs&j=d37bc48d-29a0-534f-a1dc-3d699deb17a6&t=933dfdc6-d074-504d-ee66-71743fd9ae4e&l=432

@Junchao-Mellanox @jleveque

@MaratGubaiev
Copy link
Contributor Author

MaratGubaiev commented Nov 5, 2021 •
edited
Loading

Build broadcom failed: curl: (56) OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 104 https://dev.azure.com/mssonic/build/_build/results?buildId=48899&view=logs&j=88ce9a53-729c-5fa9-7b6e-3d98f2488e3f&t=8d99be27-49d0-54d0-99b1-cfc0d47f0318&l=174

@MaratGubaiev
/azp run
e32a1bb 
fake commit to start pull request tests sonic-net#9130
rerun tests after build broadcom failed because of of pipeline error (curl: (56) OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 104)
@MaratGubaiev
Copy link
Contributor Author

MaratGubaiev commented Nov 6, 2021 •
edited
Loading

Broadcom build failed with [ FAILED ] LinkManagerStateMachineTest.MuxStandbyAsymetricLinkDrop (link)
but you can see that previous time everything was OK with broadcom build
and practically no changes were made: ec86cb9

@lguohan @qiluo-msft @xumia @Junchao-Mellanox @jleveque @zhangyanzhao @prsunny @pshulik

BTW, does anybody know what LinkManagerStateMachineTest.MuxStandbyAsymetricLinkDrop means? (As well as other test names?)

@MaratGubaiev
/azp run
7b1c955 
fake commit to start pull request tests sonic-net#9130
@MaratGubaiev
Copy link
Contributor Author

MaratGubaiev commented Nov 6, 2021 •
edited
Loading

All checks have passed

The issue was marked by @jleveque as "Help Wanted sos". FYI SOS(...---...) means that somebody's dying, and anybody who hear it shall start rescue operation immediately.

Unfortunately I didn't see any help from @jleveque and others when I asked it.

@MaratGubaiev MaratGubaiev mentioned this pull request Nov 10, 2021
Open
@pshulik
Copy link

pshulik commented Nov 12, 2021

@lguohan
@qiluo-msft
@xumia Could you please review these changes?

@MaratGubaiev MaratGubaiev mentioned this pull request Nov 12, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft is a code owner

@xumia xumia Awaiting requested review from xumia xumia is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MaratGubaiev @pshulik