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

Add DSC demo. Bugfix Apache and SystemD demo. #1814

Merged
merged 4 commits into from Aug 17, 2016

Conversation

KrisBash
Copy link
Contributor

Demo updates for Apache, SystemD, and DSC

  • Update Apache & SystemD demo readme
  • Cleanup demo .ps1's
  • Add DSC demo
  • Embed sudo prefixes in .psm1's so PowerShell can be run without sudo
  • bug fixes in Apache & SystemD demos

@hiteshraigandhi
Copy link
Contributor

@TravisEz13 why is it assigned to me?

- `sudo yum install httpd`


Note: Management of Apache requires privileges. THe user must have authorization to elevate with sudo. You will be prompted for a sudo password when running the demo.
Copy link
Contributor

Choose a reason for hiding this comment

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

The smallest nit: "THe" should be "The"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Thanks

@bgelens
Copy link
Contributor

bgelens commented Aug 15, 2016

The DSC example links to an OMI repo not accessible to us currently. It involves OMI 1.1.0 release. On the opengroup website, 1.0.8 is the latest version we can download. If this link will end up merged when the repo goes public, this will be an issue.

@TravisEz13
Copy link
Member

@bgelens, OMI will be handled separately. Thank for calling this out.

@lzybkr lzybkr merged commit 60c3019 into PowerShell:master Aug 17, 2016
JamesWTruher referenced this pull request in JamesWTruher/PowerShell Aug 17, 2016
* Add DSC demo. Bugfix Apache and SystemD demo.

* DSC, Apache, SystemD demos, address comments

* Fix capitalization in apache-demo.ps1
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.

None yet

9 participants