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

FIX the issue "config load_minigraph -y" command fails #822

Merged
merged 9 commits into from
Mar 22, 2020

Conversation

shine4chen
Copy link
Contributor

- What I did
FIX the issue "config load_minigraph -y" command fails

- How I did it
make sure the return value of _get_optional_services() is iterable
This bug is imported from PR #453

- How to verify it
test on nephos platform

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

shine.chen and others added 9 commits February 1, 2019 01:01
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
Signed-off-by: shine.chen <shine.chen@nephosinc.com>
Signed-off-by: shine.chen <shine.chen@mediatek.com>
Signed-off-by: shine.chen <shine.chen@mediatek.com>
Signed-off-by: shine.chen <shine.chen@mediatek.com>
Signed-off-by: shine.chen <shine.chen@mediatek.com>
Signed-off-by: shine.chen <shine.chen@mediatek.com>
@lguohan lguohan added the Bug label Mar 22, 2020
@lguohan lguohan merged commit f1c79d5 into sonic-net:master Mar 22, 2020
lguohan added a commit that referenced this pull request Mar 22, 2020
lguohan added a commit that referenced this pull request Mar 22, 2020
abdosi pushed a commit to abdosi/sonic-utilities that referenced this pull request Aug 4, 2020
the return value of _get_optional_services() must be iterable

This bug is imported from PR sonic-net#453

Co-authored-by: shine.chen <shine.chen@mediatek.com>
abdosi pushed a commit to abdosi/sonic-utilities that referenced this pull request Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants