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.

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

be able to tell what dashboards to create #78

Merged
merged 1 commit into from
Jul 15, 2020

Conversation

jmazzitelli
Copy link
Contributor

@jmazzitelli jmazzitelli commented Jul 14, 2020

fixes: kiali/kiali#2989
Users probably won't want all of them, and they just cause the Kiali CR processing to be slower than need be.
The default is backward compat to previous behavior (all monitoring dashboard resources are created) but you now have the ability to not install those you do not want. e.g., in the Kiali CR:

deployment:
  custom_dashboards:
    includes: ['go*', 'kiali*']

This will install the Go and Kiali monitoring dashboards but no others.

deployment:
  custom_dashboards:
    includes: ['vertx*']
    excludes: ['vertx-server*']

This will install all the vertx dashboards except the server.

@jmazzitelli jmazzitelli self-assigned this Jul 14, 2020
@jmazzitelli jmazzitelli added this to In Review in Sprint 42 via automation Jul 14, 2020
@jmazzitelli
Copy link
Contributor Author

jmazzitelli commented Jul 14, 2020

You would think the default excludes can be an empty list but you will notice the default is a list with one empty string [''] - this is because if it is empty, a bug in ansible find task will exclude everything.

@jmazzitelli
Copy link
Contributor Author

This PR includes a molecule test that shows this working.

@jmazzitelli jmazzitelli force-pushed the monitoringdashboard-options branch 2 times, most recently from 0e652a8 to f8009b2 Compare July 14, 2020 19:27
@jmazzitelli
Copy link
Contributor Author

@jotak what are we calling the "monitoring dashboards" today? Is it still "monitoring dashboards"?

This PR introduces a new section in the Kiali CR under deployments called dashboards - I want to make sure I name that new section correctly. I was thinking monitoring_dashboards but I do not know if that is the official name for these things.

For details of this new section, see the kiali_cr.yaml comments.

@jmazzitelli jmazzitelli requested a review from jotak July 14, 2020 19:40
@jmazzitelli jmazzitelli force-pushed the monitoringdashboard-options branch 2 times, most recently from 5aea7d8 to d22346e Compare July 14, 2020 21:09
@jotak
Copy link
Contributor

jotak commented Jul 15, 2020

@jmazzitelli they are still 'monitoringdashboards' resources, although if I had to name them today perhaps I would probably choose 'customdashboards'. In our documentation they are referred to as Custom Dashboards, but the CRD itself is still 'monitoringdashboards.

I'm not sure if this setting should be named just 'dashboard', to me it's a very broad term, for many users Kiali itself is a dashboard. I fear it could cause confusion.

Copy link
Contributor

@jotak jotak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beside the comment about the name of that setting, lgtm. I've tested locally.

@jmazzitelli
Copy link
Contributor Author

I'll rename to custom_dashboards

… all of them, and they just cause the Kiali CR processing to be slower than need be
Copy link
Contributor

@jotak jotak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jmazzitelli jmazzitelli merged commit d566633 into kiali:master Jul 15, 2020
Sprint 42 automation moved this from In Review to Done Jul 15, 2020
@jmazzitelli jmazzitelli deleted the monitoringdashboard-options branch July 15, 2020 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Sprint 42
  
Done
2 participants