Skip to content

nacl module documentation/possible bug #27605

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
Closed
jmcook1 opened this issue Oct 1, 2015 · 1 comment
Closed

nacl module documentation/possible bug #27605

jmcook1 opened this issue Oct 1, 2015 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior Documentation Relates to Salt documentation Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix help-wanted Community help is needed to resolve this P2 Priority 2 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone
Approved

Comments

@jmcook1
Copy link

jmcook1 commented Oct 1, 2015

It appears there is an error in the nacl module docs or a bug in 2015.8.0 (https://docs.saltstack.com/en/latest/ref/modules/all/salt.modules.nacl.html). The example config for the master looks like this:

nacl.config:
  key: None
  keyfile: /root/.nacl

But if 'key: none' is set then the key is not loaded from the keyfile and an 'Invalid Key' exception is thrown in the master log.
@jfindlay jfindlay added Documentation Relates to Salt documentation Execution-Module help-wanted Community help is needed to resolve this P2 Priority 2 Platform Relates to OS, containers, platform-based utilities like FS, system based apps Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around labels Oct 6, 2015
@jfindlay jfindlay added this to the Approved milestone Oct 6, 2015
@jfindlay
Copy link
Contributor

jfindlay commented Oct 6, 2015

@jmcook1, thanks for the report.

@rallytime rallytime mentioned this issue Mar 27, 2016
Merged
@rallytime rallytime added the fixed-pls-verify fix is linked, bug author to confirm fix label Mar 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug broken, incorrect, or confusing behavior Documentation Relates to Salt documentation Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix help-wanted Community help is needed to resolve this P2 Priority 2 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
3 participants
@jfindlay @rallytime @jmcook1