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

New behavior of NGINX Plus r18 API module breaks prometheus #45

Closed
eepifanova opened this issue Apr 16, 2019 · 1 comment · Fixed by #50
Closed

New behavior of NGINX Plus r18 API module breaks prometheus #45

eepifanova opened this issue Apr 16, 2019 · 1 comment · Fixed by #50
Labels
bug An issue reporting a potential bug

Comments

@eepifanova
Copy link

Since NGINX Plus R18 the API module provides the list of fields according to the configuration instead of using the static list of fields.

In previous versions, the api returned the static list of fields: "nginx","processes","connections","slabs","http","stream","ssl"

Since r18, this list can be different for diffrent configurations. For example, if there are no configured stream{}, then the list of fields will be:
"nginx","processes","connections","slabs","http""ssl"

To reproduce
Steps to reproduce the behavior:

  1. Upgrade NGINX Plus up to the latest r18 release
  2. Comment out the whole stream {} configuration
  3. See error

Expected behavior
The prometheus exporter should request /api/ first, to get the number of latest available API version. After that, request /api/ to get the list of configured fields.

@Dean-Coakley Dean-Coakley added the bug An issue reporting a potential bug label Apr 16, 2019
@pleshakov
Copy link
Contributor

@eepifanova thanks for reporting.

Until we fix the bug, there is a workaround -- putting an empty stream block in the NGINX Plus config:

stream { }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue reporting a potential bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants