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.

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 trees cannot have 'dots' in the name #26264

Closed
grep4linux opened this issue Aug 12, 2015 · 3 comments
Closed

state trees cannot have 'dots' in the name #26264

grep4linux opened this issue Aug 12, 2015 · 3 comments
Labels
Bug broken, incorrect, or confusing behavior Documentation Relates to Salt documentation help-wanted Community help is needed to resolve this P3 Priority 3 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@grep4linux
Copy link
Contributor

It looks like wee need to either document that states cannot have dots in the name or add that functionality. I had a customer ask about this.

Example:
Given the following state tree:

bogus.dot.state/
└── init.sls

Salt sees it as:

salt minion01 cp.list_states
minion01:
    - bogus.dot.state

Running the state produces the following error:

# salt minion01 state.sls bogus.dot.state  test=True
minion01:
    Data failed to compile:
----------
    No matching sls found for 'bogus.dot.state' in env 'base'

If I rename the state to remove 'dots' then it works.

@jfindlay jfindlay added Documentation Relates to Salt documentation Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around help-wanted Community help is needed to resolve this P3 Priority 3 labels Aug 12, 2015
@jfindlay jfindlay added this to the Approved milestone Aug 12, 2015
@jfindlay
Copy link
Contributor

@grep4linux, thanks for the report.

@whiteinge
Copy link
Contributor

We had one at one point but it looks like it got unlinked at some point.

Orig is here: https://github.com/saltstack/salt/blob/develop/doc/_incl/_incl/sls_filename_cant_contain_period.rst

cc @jacobhammons

@jacobhammons
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Documentation Relates to Salt documentation help-wanted Community help is needed to resolve this P3 Priority 3 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

No branches or pull requests

4 participants