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

[Accton] Remove unused swsssdk import from accton device code #11348

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Jul 6, 2022

Why I did it

Update scripts in sonic-buildimage from py-swsssdk to swsscommon

How I did it

Remove unused swsssdk import from accton device code

How to verify it

Pass all E2E test case

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Remove unused swsssdk import from accton device code

Link to config_db schema for YANG module changes

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

@liuh-80 liuh-80 requested a review from a team as a code owner July 6, 2022 01:59
@liuh-80 liuh-80 requested a review from qiluo-msft July 6, 2022 01:59
@lgtm-com
Copy link

lgtm-com bot commented Jul 6, 2022

This pull request fixes 1 alert when merging cd47b9b0c652815dac7147bce88e3ba49b78b42e into 7b98cf3 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@qiluo-msft
Copy link
Collaborator

@jostar-yang Could you help review?

@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 12, 2022

@jostar-yang, could you help review and give your comments?

@liuh-80
Copy link
Contributor Author

liuh-80 commented Aug 10, 2022

/easycla

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 10, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: liuh-80 / name: Hua Liu (2fad284)

@liuh-80 liuh-80 force-pushed the dev/liuh/replace-swsssdk-accton branch from cd47b9b to 2fad284 Compare August 10, 2022 02:48
@lgtm-com
Copy link

lgtm-com bot commented Aug 10, 2022

This pull request fixes 1 alert when merging 2fad284 into 2af7498 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@liuh-80
Copy link
Contributor Author

liuh-80 commented Aug 19, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@jostar-yang jostar-yang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is fine to me.

@liuh-80
Copy link
Contributor Author

liuh-80 commented Aug 19, 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 Aug 21, 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 Aug 22, 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 Aug 23, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80 liuh-80 merged commit cd33c7f into sonic-net:master Aug 24, 2022
@liuh-80 liuh-80 deleted the dev/liuh/replace-swsssdk-accton branch August 24, 2022 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants