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

[2015.8.0rc1] salt-bootstrap fails to properly install a minion #25270

Closed
iggy opened this issue Jul 8, 2015 · 11 comments
Closed

[2015.8.0rc1] salt-bootstrap fails to properly install a minion #25270

iggy opened this issue Jul 8, 2015 · 11 comments
Labels
Bug broken, incorrect, or confusing behavior Core relates to code central or existential to Salt P1 Priority 1 Packaging Related to packaging of Salt, not Salt's support for package management. Regression The issue is a bug that breaks functionality known to work in previous releases. severity-high 2nd top severity, seen by most users, causes major problems ZRELEASED - Beryllium
Milestone

Comments

@iggy
Copy link
Contributor

iggy commented Jul 8, 2015

curl -s -L https://bootstrap.saltstack.com | sudo sh -s -- -U -P -p python-jinja2 -A master git v2015.8.0rc1

results in the following output:

 *  INFO: Running daemons_running()
 * DEBUG: Waiting 3 seconds for processes to settle before checking for them
 * ERROR: salt-minion was not found running
 * ERROR: Failed to run daemons_running()!!!

Which makes sense because the init script is missing.

bjackson@dev-kmr01:~$ ls /etc/init.d/salt*
ls: cannot access /etc/init.d/salt*: No such file or directory

Not entirely sure if the error is in salt-bootstrap or salt (or some other place).

bjackson@dev-kmr01:~$ salt-call --versions
Salt Version:
           Salt: 2015.8.0rc1

Dependency Versions:
         Jinja2: 2.6
       M2Crypto: 0.21.1
           Mako: Not Installed
         PyYAML: 3.10
          PyZMQ: 13.1.0
         Python: 2.7.3 (default, Mar 13 2014, 11:03:55)
           RAET: Not Installed
        Tornado: 4.2
            ZMQ: 3.2.3
          ioflo: Not Installed
        libnacl: Not Installed
   msgpack-pure: Not Installed
 msgpack-python: 0.1.10
       pycrypto: 2.6

System Versions:
           dist: debian 7.8 
        machine: x86_64
        release: 3.16.0-0.bpo.4-amd64
         system: debian 7.8 

Let me know what other information you need.

@iggy
Copy link
Contributor Author

iggy commented Jul 8, 2015

curl -s -L https://bootstrap.saltstack.com | sudo sh -s -- -U -P -p python-jinja2 -A master git v2015.5.3

works fine

@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior Packaging Related to packaging of Salt, not Salt's support for package management. severity-high 2nd top severity, seen by most users, causes major problems P1 Priority 1 Core relates to code central or existential to Salt labels Jul 9, 2015
@jfindlay jfindlay added this to the Approved milestone Jul 9, 2015
@jfindlay jfindlay added the Regression The issue is a bug that breaks functionality known to work in previous releases. label Jul 9, 2015
@jfindlay
Copy link
Contributor

jfindlay commented Jul 9, 2015

@iggy, thanks for the report.

@s0undt3ch
Copy link
Collaborator

Can you please paste the script's full output while passing -D to it? ... probably in a gist ...

@iggy
Copy link
Contributor Author

iggy commented Jul 15, 2015

I can't. When I try to select the output to copy it, my terminal dies. But it did it again and there was (again) no /etc/init.d/salt-minion

@jfindlay
Copy link
Contributor

@s0undt3ch, @iggy: without -D, with -D. in the with -D case, there was no /etc/init.d/salt-minion.

@iggy
Copy link
Contributor Author

iggy commented Jul 16, 2015

There was no /etc/init.d/salt-minion in either case. But with -D the bootstrap script seems to start the salt-minion in the foreground at the end (which may make it appear to be working correctly).

@s0undt3ch
Copy link
Collaborator

@iggy in debug mode is runs the minion in the foreground to see if there's any complaint with salt starting(missing deps, bad code, etc would show up)

@iggy
Copy link
Contributor Author

iggy commented Jul 16, 2015

Okay, in that sense, the minion starts correctly with -D. But there's still no init script installed.

Are you having trouble reproducing this issue or something? I mean #25456 is open as well that deals with this same bug but in the master. Are you not seeing the same thing when you install the rc with the bootstrap script?

@s0undt3ch
Copy link
Collaborator

@jfindlay is starting up another VM so we can get our hands even dirtier trying to find the source if this problem...

@s0undt3ch
Copy link
Collaborator

I know what the problem is. We removed the debian directory from our repository, so there's no init script to copy from...
Taking care of if...

@s0undt3ch
Copy link
Collaborator

#23219

s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jul 17, 2015
* Make sure setuptools is installed before using it. saltstack/salt-bootstrap#598.
* `systemd` is only fully supported from 15.04 onwards. saltstack/salt-bootstrap#602
* Fix debian mirrors issue. Thanks Wolodja Wentland(babilen). saltstack/salt-bootstrap#606
* Fix python-jinja2 repo move on RHEL6. Thanks lomeroe. saltstack/salt-bootstrap#621
* Allow skipping services. Thanks denmat. saltstack/salt-bootstrap#455
* Fix missing Debian init script. saltstack/salt-bootstrap#607 saltstack#25270 and saltstack#25456
* Fix SmartOS etc path. Thanks Bret Fisher. saltstack/salt-bootstrap#624
* Fix possible unbound variable in Gentoo. saltstack/salt-bootstrap#625
* Properly detect the git binary in SmartOS. saltstack/salt-bootstrap#611
s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jul 17, 2015
* Make sure setuptools is installed before using it. saltstack/salt-bootstrap#598.
* `systemd` is only fully supported from 15.04 onwards. saltstack/salt-bootstrap#602
* Fix debian mirrors issue. Thanks Wolodja Wentland(babilen). saltstack/salt-bootstrap#606
* Fix python-jinja2 repo move on RHEL6. Thanks lomeroe. saltstack/salt-bootstrap#621
* Allow skipping services. Thanks denmat. saltstack/salt-bootstrap#455
* Fix missing Debian init script. saltstack/salt-bootstrap#607 saltstack#25270 and saltstack#25456
* Fix SmartOS etc path. Thanks Bret Fisher. saltstack/salt-bootstrap#624
* Fix possible unbound variable in Gentoo. saltstack/salt-bootstrap#625
* Properly detect the git binary in SmartOS. saltstack/salt-bootstrap#611
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Core relates to code central or existential to Salt P1 Priority 1 Packaging Related to packaging of Salt, not Salt's support for package management. Regression The issue is a bug that breaks functionality known to work in previous releases. severity-high 2nd top severity, seen by most users, causes major problems ZRELEASED - Beryllium
Projects
None yet
Development

No branches or pull requests

3 participants