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

Update and refactor policy and policy_groups endpoints #177

Merged
merged 5 commits into from
Dec 8, 2015

Conversation

jkeiser
Copy link
Contributor

@jkeiser jkeiser commented Dec 7, 2015

No description provided.

@danielsdeleo
Copy link
Contributor

LGTM 👍

@@ -55,6 +55,11 @@ OptionParser.new do |opts|
options[:log_file] = value
end

opts.on("--enterprise", "Whether to run in enterprise mode") do |value|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realize this PR is just the chef-zero part, but will we need to push this config option up to chef-client -z?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's really a separate question. chef-client -z doesn't support full-on enterprise mode yet (there are separate PRs for that).

@tyler-ball
Copy link
Contributor

Can we uncomment the pedant policies test yet?

Some questions, but overall 👍

@randomcamel
Copy link
Contributor

@tyler-ball Not yet: those are the tests to skip for ChefFS, and that's PR chef/chef#4254, which depends on stuff in this PR, so this has to go first.

@randomcamel randomcamel changed the title Update policy and policy_groups endpoints Update and refactor policy and policy_groups endpoints Dec 8, 2015
randomcamel added a commit that referenced this pull request Dec 8, 2015
Update and refactor policy and policy_groups endpoints
@randomcamel randomcamel merged commit 582f9de into master Dec 8, 2015
@tyler-ball tyler-ball deleted the jk/chef-fs-tests-plus-cd branch December 9, 2015 18:27
@tyler-ball
Copy link
Contributor

@randomcamel 👍

@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants