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

[sonic-utilities] Update sonic-utilities submodule #11468

Closed
wants to merge 1 commit into from
Closed

[sonic-utilities] Update sonic-utilities submodule #11468

wants to merge 1 commit into from

Conversation

liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Jul 18, 2022 •
edited by qiluo-msft
Loading

Why I did it

Submodule update for sonic-utilities with following change:

1dacb7f Replace pyswsssdk with swsscommon (#2251) 
8e5d478 [ssip]: Add CLI (#2191)
ea11b22 [sonic-bootchart] add sonic-bootchart (#2195)

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

Submodule update for sonic-utilities with following change:

1dacb7f Replace pyswsssdk with swsscommon (#2251) 
8e5d478 [ssip]: Add CLI (#2191)
ea11b22 [sonic-bootchart] add sonic-bootchart (#2195)

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

@liuh-80 liuh-80 requested a review from qiluo-msft July 18, 2022 05:20
@liuh-80 liuh-80 requested a review from lguohan as a code owner July 18, 2022 05:20
@nazariig nazariig mentioned this pull request Jul 18, 2022
Closed
7 tasks
@nazariig nazariig mentioned this pull request Jul 18, 2022
Merged
7 tasks
@nazariig
Copy link
Collaborator

nazariig commented Jul 18, 2022 •
edited
Loading

#10991 must be merged prior to this one to avoid CI failures

@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 19, 2022 •
edited
Loading

Because the CI fix PR #10991 already contains every change in this PR, so after that PR merged, this PR will close without merge.

@nazariig nazariig mentioned this pull request Jul 19, 2022
Merged
6 tasks
@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 25, 2022

Close this PR without merge, because another PR #10991 already include all change in this PR.

@liuh-80 liuh-80 closed this Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nazariig nazariig nazariig approved these changes

@qiluo-msft qiluo-msft qiluo-msft approved these changes

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

@liat-grozovik liat-grozovik Awaiting requested review from liat-grozovik

@volodymyrsamotiy volodymyrsamotiy Awaiting requested review from volodymyrsamotiy

Assignees
No one assigned
Labels
Submodule Update ⏫
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@liuh-80 @nazariig @qiluo-msft