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

core pillars missing in 2014.7.0-830-g4085cfa #16693

Closed
smithjm opened this issue Oct 16, 2014 · 4 comments
Closed

core pillars missing in 2014.7.0-830-g4085cfa #16693

smithjm opened this issue Oct 16, 2014 · 4 comments
Labels
Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged

Comments

@smithjm
Copy link

smithjm commented Oct 16, 2014

Today I checked out 2014.7.0-830-g4085cfa from git, and noticed that all the "core" pillars (e.g. pillar['master']) have vanished.

They were present as of 2014.7.0-790-g40e8840, so sometime between those two revisions the behavior changed.

We are using the core pillars, plus pillars defined by pepa. We do not have any other custom pillars, or pillar.top file.

@thatch45
Copy link
Contributor

We had quite a few requests to turn off the master object in the pillar because of security concerns. It is still on in the 2014.7 release but is off by default in the develop branch. This is just a toggled config value pillar_opts, it used to be True by default and now is False. So setting it to True in the master config will restore it.

@thatch45 thatch45 added the Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged label Oct 17, 2014
@smithjm
Copy link
Author

smithjm commented Oct 17, 2014

I'm closing this bug. The default setting for pillar_opts in /etc/salt/master is now false. Setting it to true restores the old behavior.

@smithjm smithjm closed this as completed Oct 17, 2014
@thatch45
Copy link
Contributor

Thank you for understanding, I am sorry I have not better publicized the change.

@UtahDave
Copy link
Contributor

I've updated the default master config to reflect this change: #16718

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged
Projects
None yet
Development

No branches or pull requests

3 participants