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

grains_dirs is not a valid configuration option #13436

Closed
ryan-lane opened this Issue Jun 12, 2014 · 3 comments

Comments

Projects
None yet
3 participants
@ryan-lane
Contributor

ryan-lane commented Jun 12, 2014

loader.py does not reference grains_dirs. There's no simple way to load extra grains for masterless salt without this.

@ryan-lane ryan-lane changed the title from grains_dir is not a valid configuration option to grains_dirs is not a valid configuration option Jun 13, 2014

@ryan-lane

This comment has been minimized.

Contributor

ryan-lane commented Jun 13, 2014

Seems if you set module_dirs to a location and have _grains in it, you can load extra grains, but this is a really ugly way to do things, especially since there's a documented option for adding additional grains.

@basepi

This comment has been minimized.

Member

basepi commented Jun 13, 2014

Thanks for the report.

@basepi basepi added Bug labels Jun 13, 2014

@basepi basepi added this to the Approved milestone Jun 13, 2014

cachedout added a commit to cachedout/salt that referenced this issue Jun 17, 2014

Include grains_dirs in loader
It is unclear to me why this was missing to begin with. This does
correctly load grains specified by grains_dirs in a minion config in
my testing.

Closes saltstack#13436
@cachedout

This comment has been minimized.

Contributor

cachedout commented Jun 17, 2014

That was a documented configuration option with nothing wired to it so far as I could detect. I have modified the loader to include it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment