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

added config item f5_http_rps_throttle and a proper HTTP rps iRule #131

Merged
merged 1 commit into from
Oct 7, 2016

Conversation

jgruber
Copy link
Contributor

@jgruber jgruber commented Oct 6, 2016

@<reviewer_id>

What issues does this address?

Fixes #127
WIP #127
...

What's this change do?

Creates a agent config entry to toggle if VIP connection limit should be an L4 connection limit or an L7 HTTP rps limit for HTTP VIPs. By default the VIP will stick to current tested L4 connection limits.

If the agent config set the f5_http_rps_throttle setting to True, a new iRule is used to implement the HTTP rps throttle. The new iRule was tested on 11.x and 12.x and is CMP friendly. The old iRule did not work. It never engaged a throttle limit.

Where should the reviewer start?

Create an LBaaSv1 Pool and VIP.with no changes to the agent config. On the VIP set the protocol to HTTP and connection_limit attribute > 1. Confirm the BIG-IP virtual server connection limit attribute tracks the VIP connect_limit attribue.

Create and LBaaSv1 Pool and VIP with the f5_http_rps_throttle = True set in the agent config. On the VIP set the protocol to HTTP connection_limit attribute > 1. Confirm that the agent created the HTTP throttle iRule in the client folder and the iRule was associated with the BIG-IP virtual server created for the VIP.

Any background context?

The only iRule did not work as intended.

@jgruber jgruber assigned jgruber and mattgreene and unassigned jgruber Oct 6, 2016
@mattgreene mattgreene merged commit cfdaab7 into F5Networks:1.0 Oct 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants