-
Notifications
You must be signed in to change notification settings - Fork 43
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
feat: add ioxQuery limits #6875
Conversation
Add support for viewing and modifying the new IOx Query limits.§
Is it possible to add a before and after screenshot in the description? Something like this #6657 |
@@ -339,6 +339,28 @@ export const OrgOverlay: FC = () => { | |||
</Grid.Column> | |||
</Grid.Row> | |||
)} | |||
{limits?.ioxQuery && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like ioxQuery is defined under the type OrgLimits, but limits
is a type of OperatorOrgLimits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is now resolved after influxdata/openapi#649
@mhilton and I discussed this: Given the current implementation for mocking these limits, the addition of this limit on the operator page will not reflect meaningful data in local dev. I will add more details in the 🧵 , but staging is the better place to revalidate this. |
Add support for viewing and modifying the new IOx Query limits.
Closes #
Checklist
Authors and Reviewer(s), please verify the following: