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

ssh_auth.absent test=true traceback #8043

Closed
Mrten opened this issue Oct 23, 2013 · 8 comments · Fixed by #8046
Closed

ssh_auth.absent test=true traceback #8043

Mrten opened this issue Oct 23, 2013 · 8 comments · Fixed by #8046

Comments

@Mrten
Copy link
Contributor

Mrten commented Oct 23, 2013

(0.17.1) If I run state.highstate test=true I get this traceback:

database-1:
----------
    State: - ssh_auth
    Name:      AAAAB3NzaC1yc2EAAAADAQABAAABAQDorb3OUWqs5QnHoua6G3AX7QSbaLcl[..]    
Function:  absent
        Result:    False
        Comment:   An exception occurred in this state: Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/salt/state.py", line 1260, in call
    ret = self.states[cdata['full']](*cdata['args'])
  File "/usr/lib/pymodules/python2.7/salt/states/ssh_auth.py", line 258, in absent
    config)
  File "/usr/lib/pymodules/python2.7/salt/modules/ssh.py", line 283, in check_key
    enc = _refine_enc(enc)
  File "/usr/lib/pymodules/python2.7/salt/modules/ssh.py", line 55, in _refine_enc
    raise CommandExecutionError(msg)
CommandExecutionError: Incorrect encryption key type "".

        Changes:

But I don't get it with a normal run.

@holmboe
Copy link
Contributor

holmboe commented Oct 23, 2013

I am sending a PR soonish.

holmboe added a commit to holmboe/salt that referenced this issue Oct 23, 2013
These are present for ssh_auth.present, but not for ssh_auth.absent.
This poses a problem further into the code in ssh.check_key (or rather
_refine_enc() in the ssh module) when it is validating the encryption
type.

Fixes saltstack#8043.
basepi pushed a commit that referenced this issue Oct 30, 2013
These are present for ssh_auth.present, but not for ssh_auth.absent.
This poses a problem further into the code in ssh.check_key (or rather
_refine_enc() in the ssh module) when it is validating the encryption
type.

Fixes #8043.
@Mrten
Copy link
Contributor Author

Mrten commented Nov 19, 2013

0.17.2, still seeing this.

@Mrten
Copy link
Contributor Author

Mrten commented Nov 19, 2013

pokes @basepi should this have been in 0.17.2?

@basepi
Copy link
Contributor

basepi commented Nov 19, 2013

Looks like the answer is yes. The title of the commit message made me think it was a feature, though ("Add ..."). I'll get it in for 0.17.3.

@basepi
Copy link
Contributor

basepi commented Nov 19, 2013

Erm, on second inspection, it looks like it did make it. https://github.com/saltstack/salt/blob/v0.17.2/salt/states/ssh_auth.py#L229-L234

o_O

@Mrten
Copy link
Contributor Author

Mrten commented Nov 19, 2013

I noticed it in the changelog, though: http://docs.saltstack.com/topics/releases/0.17.2.html

I'll double-check the versions, I'm pretty sure that I ran the state that upgrades everything today but salt is kinda flaky lately.

@Mrten
Copy link
Contributor Author

Mrten commented Nov 19, 2013

I did run the upgrade state but it didn't catch on the minion.

root@salt-master:/home/salt/conf/all-syslog# salt '*' state.sls all-salt-minion-config
Executing job with jid 20131119121745708315
-------------------------------------------
Execution is still running on backup-1
...
[and a lot of 'minion did not return's.]

I'm used to salt returning 'minion did not return' lately, but usually it means the state ran OK, but this time it didn't.

root@salt-master:/home/salt/conf/machine/web/users# salt 'database-3' cmd.run 'salt-call --versions'
Executing job with jid 20131119214655274043
-------------------------------------------

database-3:
               Salt: 0.17.1
             Python: 2.7.3 (default, Sep 26 2013, 20:03:06)
             Jinja2: 2.6
           M2Crypto: 0.21.1

I'm sorry for the work, I'll kick it again. Any idea on how to debug something like this (minions not returning)?

@basepi
Copy link
Contributor

basepi commented Nov 19, 2013

Salt has been flaky for awhile as far as its ability to upgrade itself. We're trying to find the problem and fix it, sorry for the inconvenience. =\

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

Successfully merging a pull request may close this issue.

3 participants