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

ONTAP: Volume - E2E QoS Drilldown show user-defined class volumes #1264

Closed
BrendonA667 opened this issue Sep 8, 2022 · 3 comments · Fixed by #1276 or #1487
Closed

ONTAP: Volume - E2E QoS Drilldown show user-defined class volumes #1264

BrendonA667 opened this issue Sep 8, 2022 · 3 comments · Fixed by #1276 or #1487
Labels
customer feature New feature or request status/done

Comments

@BrendonA667
Copy link

Is your feature request related to a problem? Please describe.
Currently the E2E QoS Drilldown panels only show root volumes because the class is being collected

Describe the solution you'd like
That besides the root volumes also workload volumes are being collected.

Additional context
Was discussed with @rahulguptajss in Discord https://discord.com/channels/855068651522490400/1016280173815009300
@BrendonA667 BrendonA667 added the feature New feature or request label Sep 8, 2022
@rahulguptajss rahulguptajss self-assigned this Sep 12, 2022
@rahulguptajss rahulguptajss linked a pull request Sep 12, 2022 that will close this issue
feat: enable user defined volume workload #1276
Merged
@cgrinds cgrinds mentioned this issue Sep 26, 2022
Closed
@cgrinds
Copy link
Collaborator

cgrinds commented Nov 15, 2022

Verified with 22.11 build commit 6d5f6a4
Confirmed that user defined volume workloads are collected and shown in the Volume End-to-End QoS Drilldown section for all clusters but A250-41-42-43 (9.12.1)

@rahulguptajss please take a look. Once we understand what is going on with that cluster, this issue can be moved to status/done

@rahulguptajss rahulguptajss linked a pull request Nov 16, 2022 that will close this issue
fix: qos rest template fix #1487
Merged
@rahulguptajss
Copy link
Contributor

fixed with #1487 for Rest

@cgrinds cgrinds self-assigned this Nov 16, 2022
@cgrinds
Copy link
Collaborator

cgrinds commented Nov 16, 2022

Verified with 22.11 build commit 062085f
Cluster A250-41-42-43 now collects volume workloads

@cgrinds cgrinds removed their assignment Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
customer feature New feature or request status/done
Projects
None yet
Milestone
No milestone
3 participants
@cgrinds @rahulguptajss @BrendonA667