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

[management] [analytics] Configurable http request timeout for analytics requests #1920

Closed
mrkdev opened this Issue Feb 14, 2019 · 0 comments

Comments

@mrkdev
Copy link

mrkdev commented Feb 14, 2019

If you keep a lot of data in your elasticsearch indexes and/or if you query a long period of time, elasticsearch analytics request may take some time. Dependent on the concrete usecase, it should be configurable how long we wait for a response to an analytics query.

Expected Behavior

As a gravitee.io administrator I want to be able to configure the analytics timeout on the elasticsearch-repository as well as in the management UI, so that I am able to search over a lot of data or an extended period of time. There should no timeout be hardcoded neither in the repository nor in the UI.

Current Behavior

The timeout is hardcoded to 30 seconds.

Possible Solution

Add a configuration entry in the contstants.json (WebUI) and in the gravitee.yml (analytics repository)

Steps to Reproduce (for bugs)

  1. Collect a lot of data (1Mio. requests per day over multiple days)
  2. Query for an extended period of time (90d)
  3. The UI displays an error regarding the timeout. It should just await the response from elasticsearch instead.

Context

Currently it is not possible to query for analytics if you have a lot of data and want to retrieve statistics for a long time period. Also, if your elasticsearch cluster is not ultra performant, the same issue may apply.

Your Environment

  • Version used: 1.22.x
  • Browser Name and version: any
  • Operating System and version: any

mrkdev pushed a commit to mrkdev/gravitee-management-webui that referenced this issue Feb 14, 2019

@brasseld brasseld changed the title Configurable http request timeout for analytics requests [management] [analytics] Configurable http request timeout for analytics requests Feb 15, 2019

@NicolasGeraud NicolasGeraud added this to the APIM - 1.24.0 milestone Mar 14, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-management-webui that referenced this issue Mar 18, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-management-webui that referenced this issue Mar 18, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-management-webui that referenced this issue Mar 18, 2019

fix: configurable request timeout for analytics, resolves gravitee-io…
…/issues#1920

move the configuration to the database and group analytics settings in a
single page

NicolasGeraud added a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Mar 18, 2019

@NicolasGeraud NicolasGeraud self-assigned this Mar 18, 2019

aelamrani added a commit to gravitee-io/gravitee-gateway that referenced this issue Mar 20, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Mar 20, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Mar 20, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-management-webui that referenced this issue Mar 20, 2019

fix: configurable request timeout for analytics, resolves gravitee-io…
…/issues#1920

move the configuration to the database and group analytics settings in a
single page

aelamrani added a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Mar 20, 2019

aelamrani added a commit to gravitee-io/gravitee-management-webui that referenced this issue Mar 20, 2019

fix: configurable request timeout for analytics, resolves gravitee-io…
…/issues#1920

move the configuration to the database and group analytics settings in a
single page

aelamrani added a commit to gravitee-io/gravitee-gateway that referenced this issue Mar 20, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue Mar 20, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.