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

Add Grafana dashboard for Fleet performance #4106

Merged

Conversation

p-se
Copy link
Contributor

@p-se p-se commented Jun 19, 2024

Issue

Partly implements rancher/fleet#2460
Shares parent issue with #4091

Problem

This dashboard utilizes generic metrics available by just using controller-runtime and enables insights into the performance of Fleet controllers.

Solution

image

Testing

Imported the dashboard into a Grafana instance installed by rancher-monitoring (from dev-v2.9 branch to contain the necessary ServiceMonitor custom resource) chart using Rancher 2.9. Metrics in Fleet are available since v0.10.0-rc.13.

Engineering Testing

Manual Testing

See Testing.

Automated Testing

No automated tests for this dashboard.

QA Testing Considerations

Regressions Considerations

New dashboard, no regressions possible.

Backporting considerations

Does not need to be backported.

@p-se p-se requested a review from a team as a code owner June 19, 2024 08:07
Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

1 similar comment
Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

Copy link

@thehejik thehejik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test report

Verified on rancher v2.9-head v2.9-d68a9b4627050b3f2bbe5a8e4cbca1ec7df82200-head with fleet:104.0.0+up0.10.0-rc.16 and rancher-monitoring:104.0.0-rc1+up45.31.1

There is a new dashboard named Fleet / Controller-Runtime:
image

The other fleet dashboards are outdated as #4091 is not merged yet.

p-se added 2 commits June 21, 2024 10:30
This dashboard utilizes generic metrics available by just using
controller-runtime and enables insights into the performance of Fleet
controllers.

Related to rancher/fleet#2460
@p-se p-se force-pushed the add-fleet-controller-runtime-dashboard branch from c97210d to 0749777 Compare June 21, 2024 08:34
Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

@thehejik
Copy link

After rebase the new dashboard is still working and the other fleet dashboards were updated.

@thehejik thehejik merged commit f68ec41 into rancher:dev-v2.9 Jun 21, 2024
5 checks passed
krunalhinguu pushed a commit to krunalhinguu/charts that referenced this pull request Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants