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

[platform/nephos] Optimize the code to reduce changes due to the kernel upgrade #4332

Merged
merged 5 commits into from
May 21, 2020

Conversation

simonJi2018
Copy link
Contributor

@simonJi2018 simonJi2018 commented Mar 27, 2020

…el upgrade

- What I did

  1. bug fix : Fixed an issue which the nps ko file was not loaded due to the wrong service file name
  2. Optimize the code to reduce changes due to the kernel upgrade
  3. Remove nephos ko file loaded in swss.service.j2 because it has loaded at syncd.service.j2
    - How I did it

- How to verify it
make sonic-nephos.bin and test it at accton as7116 platform

- Description for the changelog

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

jleveque
jleveque previously approved these changes Mar 30, 2020
@lguohan
Copy link
Collaborator

lguohan commented Mar 31, 2020

can you resolve the conflict?

@simonJi2018
Copy link
Contributor Author

I have resolved the conflict ,please have time to review it

@lguohan
Copy link
Collaborator

lguohan commented May 1, 2020

retest vsimage please

1 similar comment
@lguohan
Copy link
Collaborator

lguohan commented May 21, 2020

retest vsimage please

@lguohan lguohan merged commit 0b6253b into sonic-net:master May 21, 2020
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.

3 participants