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

Remove API from required installs for debian. #436

Closed
wants to merge 1 commit into from
Closed

Remove API from required installs for debian. #436

wants to merge 1 commit into from

Conversation

brendandburns
Copy link

Debian doesn't appear to have salt-api installable, and when I try to install I see:

 * INFO: Running install_debian_check_services()
 * DEBUG: Checking if service salt-minion is enabled
 * DEBUG: Service salt-minion is enabled
 * DEBUG: Checking if service salt-master is enabled
 * DEBUG: Service salt-master is enabled
 * DEBUG: Checking if service salt-api is enabled
 * DEBUG: Service salt-api is NOT enabled
 * ERROR: Failed to run install_debian_check_services()!!!
 * DEBUG: Removing the logging pipe /tmp/bootstrap-salt.logpipe

@ghost
Copy link

ghost commented Jul 28, 2014

Test PASSed.
Refer to this link for build results: http://cookingwithsalt.org/job/bootstrap/job/pr/job/build/53/

brendandburns added a commit to brendandburns/kubernetes that referenced this pull request Jul 28, 2014
@brendandburns
Disable early error exit, since the salt script is broken for now.
1530da0 
See: saltstack/salt-bootstrap#436
@brendandburns brendandburns mentioned this pull request Jul 28, 2014
Merged
@gravyboat
Copy link
Contributor

#437 has been opened to resolve all occurrences of this problem. Please merge 437 and close this one.

@s0undt3ch
Copy link
Contributor

Fixed in current stable.

This was not the right fix. Thanks anyway.

@s0undt3ch s0undt3ch closed this Jul 29, 2014
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@brendandburns @gravyboat @s0undt3ch