Skip to content

[sonic-snmpagent] Update sonic-snmpagent submodule#11467

Merged
lguohan merged 2 commits intosonic-net:masterfrom
liuh-80:dev/liuh/update-sonic-snmpagent
Jul 27, 2022
Merged

[sonic-snmpagent] Update sonic-snmpagent submodule#11467
lguohan merged 2 commits intosonic-net:masterfrom
liuh-80:dev/liuh/update-sonic-snmpagent

Conversation

@liuh-80
Copy link
Contributor

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

Why I did it

Submodule update for sonic-snmpagent with following change:

151638b Change diff coverage threshold to 80% (#261)
a45f623 Replace swsssdk with swsscommon (#263) 
3bd4a23 Remove python .whl package dependency to swsssdk (#265) 

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-snmpagent with following change:
151638b Change diff coverage threshold to 80% (#261)
a45f623 Replace swsssdk with swsscommon (#263)
3bd4a23 Remove python .whl package dependency to swsssdk (#265)

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

@liuh-80 liuh-80 requested a review from qiluo-msft July 18, 2022 05:13
@liuh-80 liuh-80 requested a review from lguohan as a code owner July 18, 2022 05:13
@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 18, 2022

This submodule changes need update again, because .whl package still depends on swsssdk, created another PR to fix that issue: sonic-net/sonic-snmpagent#265

@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 20, 2022

There is another PR will fix the .whl dependency issue, so need wait that PR merge first and update this PR again:
sonic-net/sonic-snmpagent#265

@liuh-80
Copy link
Contributor Author

liuh-80 commented Jul 26, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit 78e2194 into sonic-net:master Jul 27, 2022
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.

4 participants