-
Notifications
You must be signed in to change notification settings - Fork 680
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
Add supported OS list to readme #859
Conversation
|
||
Remote Targets | ||
|
||
Platform | Versions | Architectures | Package Format |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we do not need the package format column for remote targets.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup. that makes sense! just pushed up the fix
84e389a
to
b27d9a5
Compare
Forgot that, but we should add a warning for windows 2008 and windows 2008 R2 e.g. via a |
* RHEL 6 & 7 | ||
* Ubuntu 12.04+ | ||
* Windows 7+ | ||
* Windows 2012+ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we use the same table style as above?
Awesome job @vjeffrey ! would love to have the table style on the list below |
b27d9a5
to
3dcf866
Compare
3dcf866
to
485e765
Compare
485e765
to
ffbe540
Compare
@arlimus @chris-rock updated :) |
Gentoo Linux | | x86_64 | ||
Arch Linux | | x86_64 | ||
|
||
* Powershell with Windows Management Framework 5.0 required |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe: For Windows 2008 and 2008 R2 an updated Powershell (Windows Management Framework 5.0) is required.
Looks great. Tables are much better. Lets just make the Windows 2008 requirements a little more clear and we are good to go. |
ffbe540
to
5b80875
Compare
5b80875
to
987a0d5
Compare
should probably make sure this info eventually gets to: https://docs.chef.io/platforms.html |
@vjeffrey this is only a restriction for inspec |
ya, I meant more like, is inspec gonna have a section on that page, since automate and a bunch of things are listed on there, as a one place to look at chef-things requirements? just curious. also, why is the build failing? that error msg is pretty vague..i assume it's just a transient error thing? |
#842
Let me know if that's ok/if more/different things are needed! :)
@chris-rock