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

Fixed incorrect No data in Grafana dashboard panels #33612

Closed
wants to merge 3 commits into from

Conversation

msvticket
Copy link

@msvticket msvticket commented Jun 24, 2021

Many panels show No data if Prometheus has a scrape interval configured to be larger than 15s. Using
the $__rate_interval introduced in Grafana 7.2 fixes just that.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[X] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure

Pull Request Attributes

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@istio-policy-bot
Copy link

😊 Welcome @msvticket! This is either your first contribution to the Istio istio repo, or it's been
awhile since you've been here.

You can learn more about the Istio working groups, code of conduct, and contributing guidelines
by referring to Contributing to Istio.

Thanks for contributing!

Courtesy of your friendly welcome wagon.

@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 24, 2021
@istio-testing
Copy link
Collaborator

Hi @msvticket. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@google-cla
Copy link

google-cla bot commented Jun 24, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. label Jun 24, 2021
@msvticket
Copy link
Author

@googlebot I signed it!

@google-cla google-cla bot added cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. and removed cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. labels Jun 24, 2021
@devincd
Copy link
Member

devincd commented Jun 25, 2021

/ok-to-test

@istio-testing istio-testing added ok-to-test Set this label allow normal testing to take place for a PR not submitted by an Istio org member. and removed needs-ok-to-test labels Jun 25, 2021
@howardjohn
Copy link
Member

One thing we should look into is how many users are on 7.1 or older. I have no idea currently - could be 1% or 99%

@istio-testing istio-testing added the needs-rebase Indicates a PR needs to be rebased before being merged label Jun 28, 2021
@istio-policy-bot istio-policy-bot added the lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while label Jul 26, 2021
msvticket and others added 3 commits August 2, 2021 17:58
Many panels show No data if Prometheus has a scrape interval configured to be larger than 15s. Using
the $__rate_interval introduced in Grafana 7.2 fixes just that.
Co-authored-by: Shamsher Ansari <shaansar@redhat.com>
Co-authored-by: Shamsher Ansari <shaansar@redhat.com>
@istio-testing istio-testing removed the needs-rebase Indicates a PR needs to be rebased before being merged label Aug 2, 2021
@google-cla
Copy link

google-cla bot commented Aug 2, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. and removed cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. labels Aug 2, 2021
@shamsher31
Copy link
Member

@googlebot I consent

@istio-policy-bot istio-policy-bot removed the lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while label Aug 3, 2021
@google-cla
Copy link

google-cla bot commented Aug 3, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@douglas-reid
Copy link
Contributor

@howardjohn anything more to review here (i'm just digging through my backlog).

@howardjohn
Copy link
Member

I think we need to understand impact of Grafana version requirement

One thing we should look into is how many users are on 7.1 or older. I have no idea currently - could be 1% or 99%

@douglas-reid
Copy link
Contributor

I think we need to understand impact of Grafana version requirement

One thing we should look into is how many users are on 7.1 or older. I have no idea currently - could be 1% or 99%

Do we have any updated insights here?

@msvticket are there ways to achieve the same goal with older primitives?

@msvticket
Copy link
Author

I think we need to understand impact of Grafana version requirement

One thing we should look into is how many users are on 7.1 or older. I have no idea currently - could be 1% or 99%

Do we have any updated insights here?

@msvticket are there ways to achieve the same goal with older primitives?

No.

My rationale for it not being that much of a problem to require 7.2 is that if you install Grafana using istio you will as far as I understand currently (master) get version 7.5.5. But it would be good to add some more information also in the other main distribution route for these Dashboards: https://grafana.com/grafana/dashboards

I'm thinking of something along the lines of:

Check revisions to get the dashboard corresponding to your version of istio. Not that from version 1.11.? of istio the Dashboards require Grafana 7.2 or newer.
See https://istio.io/latest/docs/ops/integrations/grafana/ for more details on the integration of istio with Grafana.

@istio-policy-bot istio-policy-bot added the lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while label Sep 23, 2021
@istio-policy-bot
Copy link

🚧 This issue or pull request has been closed due to not having had activity from an Istio team member since 2021-08-23. If you feel this issue or pull request deserves attention, please reopen the issue. Please see this wiki page for more information. Thank you for your contributions.

Created by the issue and PR lifecycle manager.

@istio-policy-bot istio-policy-bot added the lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. label Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user experience cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. lifecycle/stale Indicates a PR or issue hasn't been manipulated by an Istio team member for a while ok-to-test Set this label allow normal testing to take place for a PR not submitted by an Istio org member. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants