Skip to content

Conversation

@jmcarp
Copy link
Contributor

@jmcarp jmcarp commented Apr 27, 2022

What this PR does

Same as #1761, but for the query-scheduler, which is marked optional and currently not included in the helm chart.

Which issue(s) this PR fixes or relates to

Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@jmcarp jmcarp force-pushed the dashboards-query-scheduler-optional branch from db16021 to ee14a11 Compare April 27, 2022 14:58
@pracucci
Copy link
Collaborator

I'm slightly more dubious about this. I think any production cluster should run with query-scheduler and we should advocate for that. I would rather fix the Helm chart.

@jmcarp
Copy link
Contributor Author

jmcarp commented Apr 27, 2022

Sounds good, I'll close this. If this isn't already the case, maybe the docs should mark this component as recommended.

@jmcarp jmcarp closed this Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants