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

[EUWE] Fill guest_os correctly #100

Merged
merged 1 commit into from
Jan 13, 2017

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented Jan 10, 2017

Fill guest_os correctly, inferring it from Image location

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

Clean cherrypick of 8424752

from the PR #84

the other commit of the PR is not relevant

Fill guest_os correctly, inferring it from Image location

Fixes BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1393547
@Ladas Ladas changed the title [euwe] Fill guest_os correctly [Euwe] Fill guest_os correctly Jan 10, 2017
@Ladas Ladas mentioned this pull request Jan 10, 2017
@chessbyte chessbyte changed the title [Euwe] Fill guest_os correctly [EUWE] Fill guest_os correctly Jan 10, 2017
@simaishi simaishi requested a review from blomquisg January 10, 2017 16:15
@durandom durandom mentioned this pull request Jan 12, 2017
@Ladas
Copy link
Contributor Author

Ladas commented Jan 12, 2017

@simaishi CI is healthy again, so this is ready

@simaishi
Copy link
Contributor

@blomquisg please review

Copy link
Member

@durandom durandom left a comment

Choose a reason for hiding this comment

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

@simaishi this is good to merge

@simaishi simaishi merged commit 722a022 into ManageIQ:euwe Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants