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

Display Cluster Resource Pool #785

Merged
merged 1 commit into from
Mar 23, 2017

Conversation

GregP
Copy link
Contributor

@GregP GregP commented Mar 23, 2017

Code fix to display Resource Pools value/icon for selected Cluster.

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

Screen capture prior to code fix below:
cluster resource pools caption prior to code fix

Screen capture post code fix:
cluster resource pools caption post code fix

Code fix to display Resource Pools value/icon for selected Cluster

https://bugzilla.redhat.com/show_bug.cgi?id=1324610
@miq-bot
Copy link
Member

miq-bot commented Mar 23, 2017

Checked commit GregP@01cadcc with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🏆

@h-kataria h-kataria self-assigned this Mar 23, 2017
@h-kataria h-kataria added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 23, 2017
@h-kataria h-kataria merged commit 4592519 into ManageIQ:master Mar 23, 2017
@GregP
Copy link
Contributor Author

GregP commented Mar 23, 2017

@h-kataria Please review.

@simaishi
Copy link
Contributor

Euwe backport (to manageiq repo) details:

$ git log -1
commit 55866ec5b8031c04823c99ad147323b997bdec1d
Author: Harpreet Kataria <[email protected]>
Date:   Thu Mar 23 18:18:59 2017 -0400

    Merge pull request #785 from GregP/cluster_resource_pools
    
    Display Cluster Resource Pool
    (cherry picked from commit 4592519bdc9513bacc2bfe0cee7ab106b35f00fc)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1446773

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