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

[sonic-utilities] Update submodule #2880

Closed
wants to merge 1 commit into from
Closed

[sonic-utilities] Update submodule #2880

wants to merge 1 commit into from

Conversation

jleveque
Copy link
Contributor

@jleveque jleveque commented May 9, 2019

Includes the following commits:

09806b8 fix name conflict between sonic_platform.py and package sonic_platform (#528)
125e3d9 [reboot] log reboot progress and add a sanity check before reboot (#526)
b7b1070 Fix TODO to get/set active ports only (#494)
0e81375 [teamshow]: Add * to indicate if the state has been synced into database (#395)

NOTE: Merging this PR will break the build due to commit 09806b8, but without it, #2875 will break the build. Therefore, this PR and #2875 should be merged at the same time.

@lguohan
Copy link
Collaborator

lguohan commented May 10, 2019

can you make one pr so that it won't break.

@jleveque
Copy link
Contributor Author

jleveque commented May 10, 2019

@lguohan: Normally I would (and I prefer to), but since we've been maintaining multiple branches in this repo as well as some submodules, adding submodule updates in with other PRs makes cherry-picking more difficult. Thoughts?

Also @yxieca for his input.

@jleveque
Copy link
Contributor Author

Closing because this submodule update will be included in #2875

@jleveque jleveque closed this May 15, 2019
@jleveque jleveque deleted the update_utilities branch May 15, 2019 01:12
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.

2 participants