Clarifies the proper way to reference states #34601
Conversation
It seems in some situations, or in some earlier version, using a `/` to reference sls files in subdirectories did work. However, because of the inconsistency it caused occasional confusion. This updates the tutorial to focus on the canonical `.` notation, which always works.
|
A result of a post in the salt user group, https://groups.google.com/d/msg/salt-users/GZJRgOz258s/eQqQzfMqAAAJ. |
|
I'd like to do a little research here and figure out where this is working and where it isn't. If it's a reasonable fix, let's add a test so it doesn't regress and fix it for good. @Ch3LL Can you help figure out which branches this is working in and which it isn't? |
cachedout
added
the
Pending-Discussion
|
@cachedout, makes sense. I intentionally wasn't changing how the code works. I just figured the tutorial should focus on teaching people the canonical salt syntax, and leave such other details for a more advanced usage doc. |
|
@lorengordon That's a good point about removing it from a basic tutorial. Let's go ahead and merge this. Would you mind submitting an issue about the broken behavior so that we can evaluate and hopefully fix it? |
|
FWIW, the user responded on the google list that their problem turned out to be minions with very old versions of salt. Apparently, I had the issue backwards...newer versions support |
|
Ahhh, OK. Makes sense. Yes, I agree with you regarding the tutorial. That is a good change. |
It seems in some situations, or in some earlier version, using a
/to reference sls files in subdirectories did work. However, the inconsistency in when it worked could cause occasional confusion. This updates the tutorial to focus on the canonical.notation, which always works.