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

Add jitter support to netem qos parameters #2476

Merged
merged 3 commits into from
Mar 15, 2023

Conversation

lime008
Copy link
Contributor

@lime008 lime008 commented Mar 13, 2023

Adds jitter parameter and annotation support for the netem QoS options.

@hongzhen-ma
Copy link
Collaborator

@lime008 Thanks very much for the PR.

Api in windows build also need to be adapted

pkg/daemon/controller_windows.go:217: too many arguments in call to ovs.SetNetemQos
	have (string, string, string, string, string, string, string)
	want (string, string, string, string, string, string)

@lime008
Copy link
Contributor Author

lime008 commented Mar 14, 2023

The windows api should now also be adapted to include the jitter parameter.

@zhangzujian zhangzujian added the feature New network feature label Mar 14, 2023
@hongzhen-ma hongzhen-ma merged commit 47a7055 into kubeovn:master Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New network feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants