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

fix: gpu dashboard temp graph #3206

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: gpu dashboard temp graph #3206

wants to merge 1 commit into from

Conversation

tuxtof
Copy link
Contributor

@tuxtof tuxtof commented Feb 24, 2025 •
edited
Loading

What problem does this PR solve?:

in GPU dashboard the GPU Temperatur is not displaying correctly, we need to remove the instant flag

image

also add some tags on the dashboard for cosmetic

image

@tuxtof tuxtof requested a review from faiq February 24, 2025 20:53
@github-actions github-actions bot added services/nvidia-gpu-operator size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 24, 2025
@mesosphere-ci mesosphere-ci added ok-to-test Signals mergebot that CI checks are ready to be kicked off do-not-merge/testing Do not merge because there is still on-going testing open-kommander-pr Automatically triggers the creation of a PR in Kommander repo update-licenses signals mergebot to update licenses.d2iq.yaml and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 24, 2025
@github-actions GitHub Actions
Copy link
Contributor

✅ Created Kommander branch to test kommander-applications changes: https://github.com/mesosphere/kommander/tree/kapps/main/fix/gpu-dashboard

@coveralls
Copy link

Pull Request Test Coverage Report for Build 13507613157

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 51.703%
Totals Coverage Status
Change from base Build 13504291153: 0.0%
Covered Lines: 167
Relevant Lines: 323
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@faiq faiq faiq approved these changes

Assignees
No one assigned
Labels
do-not-merge/testing Do not merge because there is still on-going testing ok-to-test Signals mergebot that CI checks are ready to be kicked off open-kommander-pr Automatically triggers the creation of a PR in Kommander repo services/nvidia-gpu-operator update-licenses signals mergebot to update licenses.d2iq.yaml
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tuxtof @coveralls @faiq @mesosphere-ci