Skip to content
This repository was archived by the owner on Jul 14, 2021. It is now read-only.

Add a description of policyfile design and status #122

Merged
merged 4 commits into from
Aug 4, 2014

Conversation

danielsdeleo
Copy link
Contributor

I've added a readme for the Policyfile feature so we can point people with questions there for additional info. It describes the overall plan and has a list of not-yet-implemented stuff. The idea is we'll update this as we go.

Thoughts? @opscode/client-engineers

@sersut
Copy link

sersut commented Aug 1, 2014

👍 Neat write up. Might be a good idea to share this in the ML and maybe use of of next weeks Chef Office Hours to get questions / talk about this stuff.

@damm
Copy link

damm commented Aug 2, 2014

I have to say this is the first RFC i'm actually excited to see.

policyfile, and run the recipes specified by the policyfile's run list.
A given revision of a policyfile can be promoted through deployment
stages to safely and reliably deploy new configuration to your
infrastructure.
Copy link

Choose a reason for hiding this comment

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

Maybe would be nice to specify where to save the Policifyle: what is its place from the FS/repo point of view. Or at least list some examples. AFAIK, Policyfile can be saved within a non-chef application, within a cookbook (similar to Berksfile) or even in your chef-repo/policies (or similar).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, you can put it wherever you want, depending on your workflow, but those are the 3 examples we have in mind.

@zuazo
Copy link

zuazo commented Aug 4, 2014

@danielsdeleo, nice doc. This new feature sounds very interesting 😃 Excuse me for the flood of questions.

upstream ID of the cookbook (such as git revision); for cookbooks loaded
from local disk, the Policyfile implementation detects if they are in a
git repo and collects the upstream URL, current revision ID, whether the
repo is dirty, and whether the commits are pushed to the remote.
Copy link

Choose a reason for hiding this comment

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

Why does it need to read the details of the git repo when working with local disk cookbooks? For what purpose? What happens if you are using a non-git SCM? and if you have multiple remotes? I see no need for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As the first paragraph of the section says, the primary purpose is to provide information that might be useful when looking at multiple cookbook revisions with the same name and version number and trying to make sense of them. Additionally, it gives you a head start when debugging issues, especially if you use a workflow that involves uploading cookbooks from path sources on a regular basis.

@danielsdeleo
Copy link
Contributor Author

@zuazo I added some sections based on your questions.

@danielsdeleo danielsdeleo merged commit a09fbc0 into master Aug 4, 2014
@danielsdeleo danielsdeleo deleted the policyfile-readme branch August 4, 2014 20:42
@zuazo
Copy link

zuazo commented Aug 5, 2014

OK, thanks @danielsdeleo. No more questions. I'll take a look at the code when I have a free moment.

@chef-boneyard chef-boneyard locked and limited conversation to collaborators Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants