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

I was having the same issue. I changed lines 319-322 in vmwaretools\m… #51

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

esarakaitis
Copy link

@esarakaitis esarakaitis commented Jul 8, 2016

Services we're not starting on RHEL 6.8

Update …anifests\init.pp from:

        start      => "/sbin/start ${service_name_real}",
        stop       => "/sbin/stop ${service_name_real}",
        status     => "/sbin/status ${service_name_real} | grep -q 'start/'",
        restart    => "/sbin/restart ${service_name_real}",

to:

        start      => "service ${service_name_real} start",
        stop       => "service ${service_name_real} stop",
        status     => "service ${service_name_real} status",
        restart    => "service ${service_name_real} restart",

It now works properly.

Eric Sarakaitis and others added 6 commits July 8, 2016 10:20
…anifests\init.pp from:

            start      => "/sbin/start ${service_name_real}",
            stop       => "/sbin/stop ${service_name_real}",
            status     => "/sbin/status ${service_name_real} | grep -q 'start/'",
            restart    => "/sbin/restart ${service_name_real}",
to:

            start      => "service ${service_name_real} start",
            stop       => "service ${service_name_real} stop",
            status     => "service ${service_name_real} status",
            restart    => "service ${service_name_real} restart",
It now works properly.
Updated spacing
@divad
Copy link

divad commented Jul 19, 2016

This is not the correct fix, it looks like the fix because the module is currently installing old and out of date VMware Tools. :(

@razorsedge
Copy link
Contributor

@esarakaitis Thank you for your submission. As per the CONTRIBUTING.md file, could you please resubmit from the develop branch?

@razorsedge razorsedge self-assigned this Jun 22, 2017
@razorsedge razorsedge added bug Something isn't working waiting on submiter labels Jun 22, 2017
@razorsedge razorsedge removed their assignment Apr 10, 2019
@vox-pupuli-tasks
Copy link

Dear @esarakaitis, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs?
If you need any help, you can reach out to us on our IRC channel voxpupuli on Freenode or our Slack channel voxpupuli at slack.puppet.com.
You can find my sourcecode at voxpupuli/vox-pupuli-tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants