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

Added 'VMware ESXi' to the list of known operating systems #13249

Merged

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Dec 19, 2016

OS VMware ESXi was not listed as known operating system and image name representing VMware ESXi was not defined

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

@miq-bot add-label bug, reporting

\cc @gtanzillo @kbrock

BEFORE:
os image before

AFTER:
os image after

@miq-bot
Copy link
Member

miq-bot commented Dec 19, 2016

Checked commit yrudman@6a8a2d9 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
1 file checked, 0 offenses detected
Everything looks good. 🍪

@gtanzillo
Copy link
Member

👍

@gtanzillo gtanzillo added this to the Sprint 51 Ending Jan 2, 2017 milestone Dec 20, 2016
@gtanzillo gtanzillo merged commit 2c2efdd into ManageIQ:master Dec 20, 2016
@yrudman yrudman deleted the fixed-report-host_summary_for_vms branch December 21, 2016 15:19
simaishi pushed a commit that referenced this pull request Jan 11, 2017
Added 'VMware ESXi' to the list of known operating systems
(cherry picked from commit 2c2efdd)

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

Euwe backport details:

$ git log -1
commit 4fe7b14e0c4347418f85fdfa3a4b05686686ced8
Author: Gregg Tanzillo <gtanzill@redhat.com>
Date:   Tue Dec 20 17:56:07 2016 -0500

    Merge pull request #13249 from yrudman/fixed-report-host_summary_for_vms
    
    Added 'VMware ESXi' to the list of known operating systems
    (cherry picked from commit 2c2efdddfda1417cb5fc3b363ddcb67eb9216491)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1412396

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

5 participants