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

Update ReadMe OS Support wording #78

Merged
merged 2 commits into from
Oct 21, 2017
Merged

Update ReadMe OS Support wording #78

merged 2 commits into from
Oct 21, 2017

Conversation

rquander
Copy link
Contributor

Update OS support section to clarify that Host tests were tested on HostOS Hosts, NOT regular CentOS hosts.

Update OS support section to clarify that Host tests were tested on HostOS Hosts, NOT regular CentOS hosts.
@mopsfelder
Copy link
Collaborator

What do you think about the following?

  1. OPERATING SYSTEM SUPPORT

A POWER8 bare-metal system running OpenPOWER Host OS is required to execute test cases from this repository. Test cases were validated against the following ppc64le guests:

  • Fedora 23
  • Fedora 24
  • CentOS 7.2
  • CentOS 7.3

Running test cases against other host and guest operating systems may result in unexpected errors.
Proceed at your own risk.

Wording adjustment base don Murilo's suggestions.
@rquander
Copy link
Contributor Author

@mopsfelder I like your suggestion. I edited the proposed ReadMe again with just a few tweaks from me. Thoughts?

Copy link
Collaborator

@mopsfelder mopsfelder left a comment

Choose a reason for hiding this comment

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

LGTM

@rquander rquander merged commit b9ceeff into master Oct 21, 2017
@rquander rquander deleted the rquander-Readme-Update branch October 21, 2017 18:41
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

2 participants