example/complete fails with salt '*' saltutil.sync_all and "stdin: is not a tty"

[vlcinsky]

Trying to run "complete" example, learning about preseeding minion keys to a master, the sample fails.

vagrant version: 1.2.7
OS: Ubuntu 13.04 64 bit

1. installed salty-vagrant from source code as described in readme - Installing from source. Git shows last commit being 0476bcb
2. cd to examples/complete
3. in Vagrantfile, commented out the first line:

#require '../../../salty-vagrant/lib/vagrant-salt/plugin'

1. run $ vagrant up
2. got and error as follows:

Salt successfully configured and installed!
ATTENTION: 'salt.accept_keys' is deprecated. Please use salt.seed_master to upload your minion keys
Waiting for minion key...
Adding 2 key(s) for minion(s)
Calling state.highstate... (this may take a while)
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

salt '*' saltutil.sync_all

Stdout from the command:

No minions matched the target. No command was sent, no jid was assigned.



Stderr from the command:

stdin: is not a tty

1. checking status on VM:

$ vagrant ssh
$ sudo su
$ salt-key
Accepted Keys:
Unaccepted Keys:
precise64
Rejected Keys:
$ cd /etc/salt/pki
$ find .
.
./minions
./minions/seed-minion.pub
./minion
./minion/minion.pub
./minion/minion.pem
./master
./master/minions
./master/minions_rejected
./master/master.pem
./master/minions_pre
./master/minions_pre/precise64
./master/master.pub

[ifnull]

+1

[bashu]

+1

[claytondaley]

+1

Incidentally, the TTY problem is an unrelated Ubuntu issue that can be fixed by adding config.ssh.shell = "bash -c 'BASH_ENV=/etc/profile exec bash'" to the Vagrantfile. For a more complete discussion, see hashicorp/vagrant#1673

The real issue is No minions matched the target. No command was sent, no jid was assigned. Similar issues have been discussed in other places, but I've yet to sort out the actual fix:

[claytondaley]

... and I've found a solution. Basically,

1. The salt/minion config file doesn't correctly set the id for the minion. Uncomment line 24 and set it to id:minion otherwise the minion identifies itself as precise64 and doesn't match the keys.
2. (EDIT: REMOVED, SEE NEXT COMMENT FOR ACCURATE ISSUE AND FIX)

[claytondaley]

Good News! I hadn't made it to the actual root cause so the fix for 2 is FAR easier (and safer). Turns out that there's an issue in the custom bootstrap file... and (at the moment) the current version in the salt-bootstrap repo. To fix, edit your (custom) bootstrap file, find lines that look like:

        src_keyfile="${TEMP_KEYS_DIR}/${keyfile}"
        dst_keyfile="${PKI_DIR}/minions/${keyfile}"

change them to:

        src_keyfile="${_TEMP_KEYS_DIR}/$(basename ${keyfile})"
        dst_keyfile="${SEED_DEST}/$(basename ${keyfile})"