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

[Submodule] Update src/sonic-platform-daemons #10264

Conversation

liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Mar 17, 2022

Updating sonic-utilities sub module with the following commits

f09bd31 Fix UT failed cause by change pycommon to use swsscommon
c092300 Increased pcied unit test coverage to > 80%
7d7c85e Modular chassis: Psud set master led on first run
7195dcc Remove py2 from pipeline
c2e7393 [ycabled] increase UT coverage of ycabled daemon

Why I did it

When change pycommon to use swsscommon UT failed in sonic-platform-daemon, need submodule update with UT issue fix.

How I did it

How to verify it

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

Description for the changelog

    Fix UT failed cause by change pycommon to use swsscommon
    Increased pcied unit test coverage to > 80%
    Modular chassis: Psud set master led on first run
    Remove py2 from pipeline
     [ycabled] increase UT coverage of ycabled daemon

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

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 18, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 21, 2022

This PR blocked by a new UT, will push update after the author fix the new UT.

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 23, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 23, 2022

The UT issue fixed by following PR, will run validation again: #10285

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 24, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 28, 2022

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 28, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 31, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Mar 31, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 6, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 6, 2022

Validation failed because new UT does not support python2:
c092300 Increased pcied unit test coverage to > 80%
There already a fix for this issue, need wait for that PR merge first: #10421

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 11, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 11, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 12, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 12, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 13, 2022

This validation failed 11 days ago and can't re-run: Azure.sonic-buildimage (Build vs)
So will close this PR and create a new one.

@liuh-80 liuh-80 closed this Apr 13, 2022
@liuh-80
Copy link
Contributor Author

liuh-80 commented Apr 14, 2022

Create a new PR for the validation failed issue: #10572

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.

1 participant