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

publish.host_keys function #659

Closed
thatch45 opened this issue Feb 8, 2012 · 5 comments
Closed

publish.host_keys function #659

thatch45 opened this issue Feb 8, 2012 · 5 comments
Milestone
0.9.8

Comments

@thatch45
Copy link
Contributor

thatch45 commented Feb 8, 2012

the publish module is the place to put convenience functions for the peer publisher
@SEJeff
Copy link
Contributor

SEJeff commented Mar 16, 2012

What exactly is this? Are you wanting a way for the minion to return it's private keys? If so, should they be encrypted with the master public key so only the master can read / decrypt them? These keys + the right id could be used to impersonate a minion, right?

@thatch45
Copy link
Contributor Author

you are overthinking this, we just want to be able to easily gather all of the ssh public host keys

@SEJeff
Copy link
Contributor

SEJeff commented Mar 16, 2012

Hence why I asked what this actually was. I can do a basic version of this hopefully this weekend.

@thatch45
Copy link
Contributor Author

it should be an easy thing to add

@thatch45
Copy link
Contributor Author

This can be done with publish.publish('*', 'ssh.host_keys')

arthurzenika pushed a commit to arthurzenika/salt that referenced this issue Sep 17, 2013
@techhat
Merge pull request saltstack#659 from KB1JWQ/update-debian-changelog-…
2e24ed2 
…0.8.9

Bump changelog
meaksh added a commit to meaksh/salt that referenced this issue Jul 8, 2024
@meaksh @m-czernek
Fix "status.diskusage" and exclude some tests to run when testing Sal…
4555f21 
…t Bundle (saltstack#659)

* Show warning instead of crashing when stats cannot be fetched

* Skip tests that are not compatible with Salt Bundle

* test_syndic_eauth: do not produce error if docker service is not running

* test_cmdmod: assert properly in case of DeprecationsWarnings

* Include path as part of output in case of errors

Co-authored-by: Marek Czernek <[email protected]>

---------

Co-authored-by: Marek Czernek <[email protected]>
meaksh added a commit to meaksh/salt that referenced this issue Nov 12, 2024
@meaksh @m-czernek
Fix "status.diskusage" and exclude some tests to run when testing Sal…
785b882 
…t Bundle (saltstack#659)

* Show warning instead of crashing when stats cannot be fetched

* Skip tests that are not compatible with Salt Bundle

* test_syndic_eauth: do not produce error if docker service is not running

* test_cmdmod: assert properly in case of DeprecationsWarnings

* Include path as part of output in case of errors

Co-authored-by: Marek Czernek <[email protected]>

---------

Co-authored-by: Marek Czernek <[email protected]>
agraul pushed a commit to agraul/salt that referenced this issue Jan 27, 2025
@meaksh @agraul
Fix "status.diskusage" and exclude some tests to run when testing Sal…
33b1446 
…t Bundle (saltstack#659)

* Show warning instead of crashing when stats cannot be fetched

* Skip tests that are not compatible with Salt Bundle

* test_syndic_eauth: do not produce error if docker service is not running

* test_cmdmod: assert properly in case of DeprecationsWarnings

* Include path as part of output in case of errors

Co-authored-by: Marek Czernek <[email protected]>

---------

Co-authored-by: Marek Czernek <[email protected]>

BACKPORT-UPSTREAM=saltstack#66647
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9.8
Development

No branches or pull requests
2 participants
@SEJeff @thatch45