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

salt-ssh writes to /etc #7531

Closed
joehealy opened this issue Oct 1, 2013 · 7 comments
Closed

salt-ssh writes to /etc #7531

joehealy opened this issue Oct 1, 2013 · 7 comments
Assignees
@thatch45
Labels
Bug broken, incorrect, or confusing behavior
Milestone
Approved

Comments

@joehealy
Copy link
Contributor

joehealy commented Oct 1, 2013

salt-ssh wants to create the file /etc/salt/pki/minion

This seems to be unnecessary for salt-ssh.
@joehealy
Copy link
Contributor Author

joehealy commented Oct 1, 2013

We should also think about whether we need to leave /var/cache/salt/minion/ lying about as well

@basepi
Copy link
Contributor

basepi commented Oct 1, 2013

Ah, thanks for pointing this out. Not sure what purpose the cache is serving, but we'll look into that as well.

@ghost ghost assigned thatch45 Oct 1, 2013
@basepi
Copy link
Contributor

basepi commented Oct 1, 2013

@thatch45 ping

@ajithhub ajithhub mentioned this issue Oct 1, 2013
Closed
@malinoff malinoff mentioned this issue Oct 4, 2013
Closed
@pfalcon
Copy link

pfalcon commented Oct 17, 2013

Any news?

@pfalcon
Copy link

pfalcon commented Oct 17, 2013

See also #7913 for related issue.

@basepi
Copy link
Contributor

basepi commented Oct 18, 2013

Talking to @thatch45 this is going to be a non-trivial fix. It has not been resolved yet.

@pniederw pniederw mentioned this issue Nov 21, 2013
Closed
@basepi basepi modified the milestones: Approved, Outstanding Bugs Apr 21, 2014
@thatch45
Copy link
Member

This was fixed a long time ago

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thatch45 thatch45

Labels
Bug broken, incorrect, or confusing behavior
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
4 participants
@pfalcon @thatch45 @basepi @joehealy