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.

Sign up for GitHub

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

Jump to bottom

[OC-6762] Prereq checks for OPC #104

Closed
wants to merge 2 commits into from
Closed

[OC-6762] Prereq checks for OPC #104

wants to merge 2 commits into from

Conversation

sean-horn
Copy link
Contributor

  • Added ruby port checks
  • Port checking is only active on a pre-bootstrap machine to avoid self-DOS
  • Checking for common packages and hostname configurations that cause problems
  • Convert strings to numbers for OPC service port attributes

@sean-horn
Convert strings to numbers for ports
44ddd29
@sean-horn
[OC-6762] Prereq checks for OPC
673b13b 
* Added ruby port checks
* Port checking is only active on a pre-bootstrap machine to avoid self-DOS
* Checking for common packages and hostname configurations that cause problems
@sdelano
Copy link
Contributor

sdelano commented Jul 23, 2014

@sean-horn This looks pretty handy. Is it still needed?

@sean-horn
Copy link
Contributor Author

Yeah, it would be useful, but it potentially needs some work. It was written against OPC something.

@galenemery galenemery mentioned this pull request Oct 21, 2014
Open
@stevendanna
Copy link
Contributor

@sean-horn I'm going to close this PR. I think all of the maintainers are +1 on a feature like this, but with an implementation that allows it to run as a separate command. Apologies this sat for so long without a response; it is definitely going to be helpful as the first checks that are implemented will probably be the one's you've provided here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Signed CLA
Milestone
No milestone
4 participants
@sean-horn @sdelano @stevendanna @chef-supermarket