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

'chef export' respects chefignore #606

Merged
merged 2 commits into from
Nov 17, 2015
Merged

Conversation

danielsdeleo
Copy link
Contributor

Makes chef export respect the chefignore file. Frequently very large files like a VM image are stored in a cookbook's ignored directories, which can be bad when trying to copy cookbooks into a VM for testing.

Fixes #519

@tbunnyman I pulled in your commit and cleaned it up a little. It turns out there is already a chefignored file in the fixture data, so the existing tests only needed to be modified a little bit to handle the new behavior.

cc @chef/workflow

onlyhavecans and others added 2 commits November 16, 2015 12:32
This could maybe be a bit more in depth. The way I did it automatically
excludes all dotfiles and only really checks the base level directories.
I'm not sure if we need to get any more in depth in the chefignore here.

Fixes #519
This is consistent with how we're implementing chefignore elsewhere.
@onlyhavecans
Copy link
Contributor

👍

@tyler-ball tyler-ball changed the title Export use chefignore chef export respects chefignore Nov 17, 2015
@tyler-ball tyler-ball changed the title chef export respects chefignore 'chef export' respects chefignore Nov 17, 2015
@tyler-ball
Copy link
Contributor

👍

I changed the title of this PR so the automatically generated CHANGELOG will be more clear

@danielsdeleo danielsdeleo merged commit bee7d99 into master Nov 17, 2015
@danielsdeleo
Copy link
Contributor Author

Thanks for the initial patch @tbunnyman !

@danielsdeleo danielsdeleo deleted the export-use-chefignore branch November 17, 2015 17:49
@onlyhavecans
Copy link
Contributor

Sorry for my horrible follow through. It's "the time" right now in my industry.

@danielsdeleo
Copy link
Contributor Author

No worries, stressful time of year.

@thommay thommay added Type: Bug Doesn't work as expected. and removed Bug labels Feb 1, 2017
@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
Type: Bug Doesn't work as expected.
Development

Successfully merging this pull request may close these issues.

policyfile export doesn't use chefignore
5 participants