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

Add hawkular admin tenant label to settings #16275

Merged
merged 1 commit into from
Oct 24, 2017
Merged

Add hawkular admin tenant label to settings #16275

merged 1 commit into from
Oct 24, 2017

Conversation

elad661
Copy link
Contributor

@elad661 elad661 commented Oct 24, 2017

Description
Compute -> Containers -> Providers -> Choose provider -> Monitoring -> Ad-hoc metrics

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1503541

ManageIQ/manageiq-ui-classic#2497

Add hawkular admin tenant label to settings, which together with ManageIQ/manageiq-ui-classic#2497 makes it look good in the UI.

Users of OpenShift can create custom tenants in the Hawkular database. The ad-hoc metrics page can handle some pre-defined names, but users can choose to define new ones. This PR adds an example of such setting, and the one in manageiq-ui-classic adds the code to support this kind of setting.

Before

hawkular_admin_before

After

hawkular_admin_after

Settings

hawkular_settings

@elad661
Copy link
Contributor Author

elad661 commented Oct 24, 2017

@miq-bot add-label providers/containers, bug

@simon3z @yaacov please review

@miq-bot
Copy link
Member

miq-bot commented Oct 24, 2017

Checked commit https://github.com/elad661/manageiq/commit/93aa3f440b95b5d9db979f6a76608fd79c4b9f45 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

Copy link
Member

@yaacov yaacov left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@chessbyte chessbyte self-assigned this Oct 24, 2017
@chessbyte chessbyte merged commit a55e9e8 into ManageIQ:master Oct 24, 2017
@chessbyte chessbyte added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants