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: if routestr does not exist, skip #257

Merged
merged 1 commit into from
May 5, 2022

Conversation

qiluo-msft
Copy link
Contributor

- What I did
Follow up #255.
If route change too fast between keys/hgetall, the original implementation will throw exception and lead to ERR in syslog.

- How I did it

- How to verify it

- Description for the changelog

@qiluo-msft qiluo-msft merged commit 1477c36 into sonic-net:master May 5, 2022
@qiluo-msft qiluo-msft deleted the qiluo/skipnone branch May 5, 2022 00:14
qiluo-msft added a commit to qiluo-msft/sonic-snmpagent that referenced this pull request May 5, 2022
Follow up sonic-net#255.
If route change too fast between keys/hgetall, the original implementation will throw exception and lead to ERR in syslog.
qiluo-msft added a commit that referenced this pull request May 5, 2022
… does not exist, skip (#258)

Manual cherry-pick below commits and resolve conflicts.

1477c36 2022-05-04 | Fix: if routestr does not exist, skip (#257) [Qi Luo]
57f1af6 2022-05-02 | Fix: not to use blocking get_all() after keys() (#255) [Qi Luo]
judyjoseph pushed a commit that referenced this pull request May 8, 2022
Follow up #255.
If route change too fast between keys/hgetall, the original implementation will throw exception and lead to ERR in syslog.
qiluo-msft added a commit that referenced this pull request May 10, 2022
Follow up #255.
If route change too fast between keys/hgetall, the original implementation will throw exception and lead to ERR in syslog.
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Aug 29, 2022
Update sonic-snmpagent submodule pointer to include the following:
* Fix: if routestr does not exist, skip ([sonic-net#257](sonic-net/sonic-snmpagent#257))

Signed-off-by: dprital <drorp@nvidia.com>
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Sep 4, 2022
Update sonic-snmpagent submodule pointer to include the following:
* Fix: if routestr does not exist, skip ([sonic-net#257](sonic-net/sonic-snmpagent#257))

Signed-off-by: dprital <drorp@nvidia.com>
liat-grozovik pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Sep 19, 2022
Update sonic-snmpagent submodule pointer to include the following:
* Fix: if routestr does not exist, skip ([#257](sonic-net/sonic-snmpagent#257))

Signed-off-by: dprital <drorp@nvidia.com>
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.

None yet

3 participants