chef-boneyard / chef-dk

DEPRECATED: A streamlined development and deployment workflow for Chef Infra platform.
Apache License 2.0
380 stars 170 forks source link

Introduce a shared scm context #3081

Closed Mat-Co closed 3 years ago

Mat-Co commented 3 years ago

In some situations, when a lot of local cookbooks are used, tests can spend a lot of time doing several times the same git commands for each one of them. It can be particularly unefficient when they share the same git repo.

With this change, we make sure commands are launched only once and results are reused if necessary.

Mat-Co commented 3 years ago

Sorry, it was not meant to be pushed :(

tas50 commented 3 years ago

@Mat-Co If you're interested in improving the policyfile experience in the future keep in mind that this repo is deprecated. The code lives in https://github.com/chef/chef-cli now.

Mat-Co commented 3 years ago

Hello ! Thank you for your feedback and again, sorry for the noise ! Yes, we saw that but we are stuck for now with chef-dk. We don't know if others share the same problematic use case we had, but we will consider proposing this patch for chef-cli if it's relevant.