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

Connect clusterless hosts directly to the provider in infra topology #1156

Merged
merged 1 commit into from Apr 26, 2017

Conversation

skateman
Copy link
Member

@skateman skateman commented Apr 25, 2017

If an infra provider has clusterless hosts, they aren't displayed in the topology view. This PR connects these hosts directly to the provider using the clusterless_hosts method. Depends on: ManageIQ/manageiq#14884

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

@miq-bot
Copy link
Member

miq-bot commented Apr 25, 2017

Checked commit skateman@4e23c99 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. ⭐

@skateman skateman closed this Apr 25, 2017
@skateman skateman reopened this Apr 25, 2017
@martinpovolny martinpovolny added this to the Sprint 60 Ending May 8, 2017 milestone Apr 26, 2017
@martinpovolny martinpovolny merged commit 093b264 into ManageIQ:master Apr 26, 2017
@skateman skateman deleted the clusterless-hosts branch April 26, 2017 08:05
@jteehan
Copy link

jteehan commented Apr 26, 2017

I verified the fix using 5.8.0.11 plus copy/paste. Now seeing a proper topology for non-clustered environments.

simaishi pushed a commit that referenced this pull request Jun 6, 2017
Connect clusterless hosts directly to the provider in infra topology
(cherry picked from commit 093b264)

https://bugzilla.redhat.com/show_bug.cgi?id=1458930
@simaishi
Copy link
Contributor

simaishi commented Jun 6, 2017

Fine backport details:

$ git log -1
commit a316463d6b0136526af184cd6b6f78f4ef609a10
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Wed Apr 26 08:54:33 2017 +0200

    Merge pull request #1156 from skateman/clusterless-hosts
    
    Connect clusterless hosts directly to the provider in infra topology
    (cherry picked from commit 093b26439fe817f7da6fadedfaa861a1f61d61e3)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1458930

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.

None yet

6 participants