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

Remove error logging on "failed in fdb_vlanmac" #272

Merged
merged 3 commits into from
Nov 14, 2022

Conversation

qiluo-msft
Copy link
Contributor

- What I did
It is considered normal that the dependences of FDB_ENTRY are temporarily not available.

- How I did it

- How to verify it

- Description for the changelog

that the dependences of FDB_ENTRY are temporarily not available.
This reverts commit ee781de.
prsunny
prsunny previously approved these changes Nov 14, 2022
Copy link
Contributor

@prsunny prsunny left a comment

Choose a reason for hiding this comment

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

agree with @zhenggen-xu to have it at debug level.

@qiluo-msft qiluo-msft merged commit bf6cd4c into sonic-net:master Nov 14, 2022
@qiluo-msft qiluo-msft deleted the qiluo/noinvalidfdb branch November 14, 2022 21:50
qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Nov 16, 2022
#### Why I did it

Update sonic-snmpagent ubmodule pointer to include the following:

* bf6cd4c Remove error logging on "failed in fdb_vlanmac" ([#272](sonic-net/sonic-snmpagent#272))
* 05f2a28 Remove LOGLEVEL DB since is no longer used ([#271](sonic-net/sonic-snmpagent#271))
* 06299e1 Don't cache the vlan-id if it is not valid from DB ([#273](sonic-net/sonic-snmpagent#273))
* 8a515f2 Install libyang to azure pipeline ([#268](sonic-net/sonic-snmpagent#268))

#### How I did it

Advance sonic-snmpagent pointer
qiluo-msft added a commit that referenced this pull request Nov 17, 2022
It is considered normal that the dependences of FDB_ENTRY are temporarily not available.
richardyu-ms pushed a commit to richardyu-ms/sonic-buildimage that referenced this pull request Nov 18, 2022
#### Why I did it

Update sonic-snmpagent ubmodule pointer to include the following:

* bf6cd4c Remove error logging on "failed in fdb_vlanmac" ([sonic-net#272](sonic-net/sonic-snmpagent#272))
* 05f2a28 Remove LOGLEVEL DB since is no longer used ([sonic-net#271](sonic-net/sonic-snmpagent#271))
* 06299e1 Don't cache the vlan-id if it is not valid from DB ([sonic-net#273](sonic-net/sonic-snmpagent#273))
* 8a515f2 Install libyang to azure pipeline ([sonic-net#268](sonic-net/sonic-snmpagent#268))

#### How I did it

Advance sonic-snmpagent pointer
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