-
Notifications
You must be signed in to change notification settings - Fork 16
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
refactor: Tidy up Operator Dashboard #445
Conversation
@bmtcril can you leave screenshots of the changes? |
@Ian2012 all set |
@bmtcril the k8s jobs are failing. Can you move the translations file to a different folder, create a configmap for it, and attach it to superset? |
@Ian2012 probably not today, I'll have to take a look next week |
- Make names consistent - Remove unusable or useless charts - Add a "slowest ClickHouse queries" chart
Hopefully will fix k8s build issues around the size of the pythonpath configmap.
7c83270
to
e93a095
Compare
- plugins/{{ file }}{% endfor %} | ||
options: | ||
labels: | ||
app.kubernetes.io/name: superset |
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.
nit: update label superset-localization
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.
The others around it are labeled this way, should they be changed as well?
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.
oh sorry, I was confused with this label. The label shows that those config maps are related to the superset aplication
@bmtcril can you upgrade the tutor version to More context here: https://discuss.openedx.org/t/missing-py2neo-package-causing-build-issues-in-all-edx-platform-releases/11371 |
22aa247
to
770ce38
Compare
Closes: #380
Before screenshots:
After screenshots: