-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
state locale.system throws exception #63559
Comments
I assume you get the same exception from What does the Note also your states are missing requisites. For example, |
Ah ya thx for the requisites, I included them now, but it's the same exception. I assumed salt uses the order the states appear in the file. Does this mean requisites are needed everytime there is a dependency in the order of operations? I assume you meant locale.get_locale?
locale returns
|
WTF, now the states work? After I ran salt-call locale.get_locale |
I think your system locale was uninitialised when you reinstalled the system, but something has now set it. Has the I still think it's a bug that Salt doesn't handle that, but if it's hard to reproduce it's not going to be high priority.
Sometimes. You should always explicitly add dependencies where they exist, to avoid cascading failures if nothing else. |
I modified the state from above to include a call to salt-call locale.get_locale. State:
Errors:
|
I could trace it back to the cause: salt's locale state depends on localectl which in turn depends on dbus. These dependencies are not mentioned in the docs. And I am not entierly sure it is good to depend on both. |
A Debian system is expected to have dbus, just as it's expected to have systemd and apt.
|
Probably right. However, dbus is only in systemd's recommended list, so it is not always installed. I suggest updating the doc to include dbus. |
Heads up |
Description of Issue
The following state throws an exception at locale.system. This has worked previously, so I assume it as a recent development.
Also, I reinstalled the system where the state is applied, but I doubt that is the reason for the exception.
Setup
Steps to Reproduce Issue
Versions Report
The text was updated successfully, but these errors were encountered: