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.

Sign up for GitHub

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

Jump to bottom

Timeout Error when querying variables #71

Closed
Neko-Follower opened this issue Sep 6, 2024 · 7 comments
Closed

Timeout Error when querying variables #71

Neko-Follower opened this issue Sep 6, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@Neko-Follower
Copy link

Neko-Follower commented Sep 6, 2024 •
edited
Loading

Hi! I run into some bugs with new v0.4.0 version. I try to create variable namespace and get 504 Timeout error
Снимок экрана 2024-09-06 в 12 24 56
Снимок экрана 2024-09-06 в 12 30 46
Снимок экрана 2024-09-06 в 12 32 13

For testing purposes I switch back to v0.3.0 version. And I didn't find any errors here.
image
@dmitryk-dk
Copy link
Contributor

dmitryk-dk commented Sep 6, 2024 •
edited
Loading

Hi @Neko-Follower ! Is it reproduced with the previous version? Sorry, I see your next image. So, the previous version was worked on. So, is something wrong with querying values

@dmitryk-dk
Copy link
Contributor

@Neko-Follower ! I have checked the behavior and it works well with my local installation
Screenshot 2024-09-06 at 10 40 55

I used grafana 9.5.21, and the v0.4.0 of the datasource

@dmitryk-dk dmitryk-dk added the question Further information is requested label Sep 6, 2024
@dmitryk-dk
Copy link
Contributor

It takes some time to query those values. Can you check that you do not have any proxy between Datasource and Victorialogs?

@Neko-Follower Neko-Follower mentioned this issue Sep 6, 2024
Closed
@dmitryk-dk
Copy link
Contributor

Hi @Neko-Follower ! We have discussed all described issues. I need to do some testing on both issues and will be back soon. I think I will make all the checks today. But I need to prepare for some of the tests at my local env.

@Neko-Follower
Copy link
Author

Hi @dmitryk-dk ! Ok, I'll be waiting for results. #72 can be closed since it is related to timeout error

hagen1778 added a commit that referenced this issue Sep 10, 2024
@hagen1778
language_provider: fix query time params names
4c46891 
The correct query param names for /field_values are `start` `end`.
See https://docs.victoriametrics.com/victorialogs/querying/#querying-field-values

Sending `from` and `to` params meant time range filter wasn't applied at all.
This made VictoriaLogs make the full data scan.
Which probably caused the following issues:
* #71
* #72
@hagen1778 hagen1778 mentioned this issue Sep 10, 2024
Merged
dmitryk-dk pushed a commit that referenced this issue Sep 10, 2024
@hagen1778
language_provider: fix query time params names (#73)
9916a93 
The correct query param names for /field_values are `start` `end`.
See https://docs.victoriametrics.com/victorialogs/querying/#querying-field-values

Sending `from` and `to` params meant time range filter wasn't applied at all.
This made VictoriaLogs make the full data scan.
Which probably caused the following issues:
* #71
* #72
@dmitryk-dk
Copy link
Contributor

Hi @dmitryk-dk ! Ok, I'll be waiting for results. #72 can be closed since it is related to timeout error

Hi! @hagen1778 fixed the problem. This fix will be included in the next release. You can build the data source from the main branch and try to use it.

@dmitryk-dk
Copy link
Contributor

Hi @Neko-Follower ! The issue was resolved in the release. Please reopen the issue if you find any problems

@valyala valyala added bug Something isn't working and removed waiting for release question Further information is requested labels Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@valyala @hagen1778 @dmitryk-dk @Neko-Follower