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

infoblox, extracting HTTPPoolConnections,HTTPRequestTimeout #272

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

kuritka
Copy link
Collaborator

@kuritka kuritka commented Feb 1, 2021

related to #255

Infoblox admin complains that tremendous amount of requests is killing his poor Infoblox UI.
Within Infoblox connection I found two other properties, which can be parametrised:

  • HTTPPoolConnections
  • HTTPRequestTimeout

For that purpose I made following steps:

  • update infoblox connection within infoblox provider
  • introduce new variables to depresolver with default values
    • HTTPPoolConnections (default 10, Higher or equal to zero)
    • HTTPRequestTimeout (default 20, Higher than zero)
  • update depresolver tests
  • update controller tests
  • introduce new variables in operator.yaml
  • infoblox-cm.yaml and values.yaml (demonstrative purpose; depresolver can inject the default values)

Infoblox admin complains that tremendous amount of requests is killing his poor Infoblox UI.
Within Infoblox connection I found two other properties, which can be parametrised:
 - HTTPPoolConnections
 - HTTPRequestTimeout

For that purpose I made following steps:

 - update infoblox connection within infoblox provider
 - introduce new variables to `depresolver` with default values
 - update depresolver tests
 - update controller tests
 - introduce new variables in `operator.yaml`
 - `infoblox-cm.yaml` and `values.yaml` (demonstrative purpose, the default values can be injected by `depresolver`)
@kuritka kuritka changed the title infoblox, extracting HTTPPoolConnections,HTTPRequestTimeout infoblox, extracting HTTPPoolConnections,HTTPRequestTimeout Feb 1, 2021
@kuritka kuritka merged commit 2189bc0 into master Feb 2, 2021
@kuritka kuritka deleted the refactor-infoblox-heartbeat branch February 2, 2021 07:20
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