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

[pull] master from Azure:master #1057

Merged
merged 6 commits into from
Oct 19, 2020
Merged

[pull] master from Azure:master #1057

merged 6 commits into from
Oct 19, 2020

Conversation

pull[bot]
Copy link

@pull pull bot commented Oct 19, 2020

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

jleveque and others added 6 commits October 19, 2020 09:46
Make procdockerstatsd Python 3-compliant and set interpreter to python3 in shebang. Also some other cleanup to improve code reuse.
…5658)

Add sensors labels for human readable output for MSN2010

Signed-off-by: Shlomi Bitton <[email protected]>
…5659)

Add sensors labels for human readable output for MSN2100

Signed-off-by: Shlomi Bitton <[email protected]>
…5660)

Add sensors labels for human readable output for MSN2410
When a large number of changes occur to the ACL table of Config DB, caclmgrd will get flooded with notifications, and previously, it would regenerate and apply the iptables rules for each change, which is unnecessary, as the iptables rules should only get applied once after the last change notification is received. If the ACL table contains a large number of control plane ACL rules, this could cause a large delay in caclmgrd getting the rules applied.

This patch causes caclmgrd to delay updating the iptables rules until it has not received a change notification for at least 0.5 seconds.
Each SFP object inside Chassis will open an SDK client, this is not necessary and SDK client can be shared between SFP objects.
@pull pull bot added the ⤵️ pull label Oct 19, 2020
@pull pull bot merged commit 73f38f6 into pphuchar:master Oct 19, 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