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

Format trend max cpu usage rate with percent #15272

Merged
merged 1 commit into from
Jun 2, 2017

Conversation

hayesr
Copy link
Contributor

@hayesr hayesr commented Jun 1, 2017

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

A percent sign is missing from values under Trend Max % Used column on the C&U CPU report.

Before
screen shot 2017-06-01 at 1 13 08 pm

After
screen shot 2017-06-01 at 1 09 42 pm

@dclarizio @h-kataria

@hayesr
Copy link
Contributor Author

hayesr commented Jun 1, 2017

@miq-bot add_label ui, reporting, bug

@miq-bot
Copy link
Member

miq-bot commented Jun 1, 2017

Checked commit hayesr@1724f17 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@h-kataria
Copy link
Contributor

@Fryguy @chessbyte Looks good to be merged

@chessbyte chessbyte self-assigned this Jun 2, 2017
@chessbyte chessbyte merged commit 8a00aac into ManageIQ:master Jun 2, 2017
@chessbyte chessbyte added this to the Sprint 62 Ending Jun 5, 2017 milestone Jun 2, 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