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

make ganglia optional #67

Closed
wants to merge 6 commits into from
Closed

make ganglia optional #67

wants to merge 6 commits into from

Conversation

qianyuanzhu
Copy link
Contributor

@qianyuanzhu qianyuanzhu commented Nov 19, 2018

Changes:

  • Both clusters with and without Ganglia are shown on the main page.
  • Clusters without Ganglia is not clickable on the main page.
  • Clusters without Ganglia is hidden in the navigation bar.

fixes #65

<div class="panel panel-default bs-callout bs-callout-info text-center panel-unclickable">
<h4><strong><%= cluster.cluster_title %> Cluster Status</strong></h4>
<%= erb :_node_status, :locals => { :showqer => cluster } %>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

create a partial with a local "clickable" which is true or false - that's the only thing that changes between the two. Make sure whitespace is good also.

@ericfranz
Copy link
Contributor

We are taking a different approach with this now that we have grafana.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the ganglia graphs optional
2 participants