-
Notifications
You must be signed in to change notification settings - Fork 36
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
Request about usage #54
Comments
salt -V Dependency Versions: System Versions: |
Hi @misch42. It is probably good to have the following distinction in mind: data goes into the Pillar, automation logic does into the State files (+ templates, when applicable). In the above you have correctly defined the Pillar data, however the State file is missing: when you execute For example, if you have the following configuration on your Master: file_roots:
base:
- /srv/salt
- /srv/salt/states If you place this State SLS file: https://github.com/napalm-automation/napalm-salt/blob/master/examples/router/ntp.sls under |
Thanks. That was the problem. But I think it will be easiery to work with formulas (napalm-ntp-formula) in real deployments. |
I agree, the napalm-ntp-formula is probably the best (and in similar ways with anything else). The native NTP State is good enough, but can fall short in some particular cases, and it may be deprecated at some point. |
Hi,
I followed you setup in the README. A
salt 'mydevice' test.ping
works. Also all functions work. Nice work. Thanks.
But when I want to test a state, I get an error. According to the doc I add the ntp.peers to the device pillar:
cat /srv/pillar/mydevice.sls
proxy:
proxytype: napalm
driver: junos
host: 192.168.1.2
username: admin
ntp.peers:
The answer of salt is:
salt 'mydevice' state.sls mydevice.ntp
mydevice:
Data failed to compile:
Any idea, what I got wrong?
Michael
The text was updated successfully, but these errors were encountered: