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

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

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

CLA assistant check
All CLA requirements met.

@MaratGubaiev MaratGubaiev mentioned this pull request Nov 1, 2021
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

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

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

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?)

fake commit to start pull request tests sonic-net#9130
@MaratGubaiev
Copy link
Contributor Author

MaratGubaiev commented Nov 6, 2021

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.

@pshulik
Copy link

pshulik commented Nov 12, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants