summaryrefslogtreecommitdiff
path: root/lib/chef/knife/cookbook_upload.rb
Commit message (Collapse)AuthorAgeFilesLines
* Use double quotes by defaultThom May2016-01-141-13/+13
| | | | | | | This is an entirely mechanically generated (chefstyle -a) change, to go along with chef/chefstyle#5 . We should pick something and use it consistently, and my opinion is that double quotes are the appropriate thing.
* Updated "cbs" variable to "cookbooks_for_upload"Josh Murphy2015-01-251-4/+4
|
* Added cookbook_path configuration value to warning message for uploading ↵Josh Murphy2015-01-251-1/+2
| | | | cookbooks and updated tests
* Updated cookbook upload all logic to check that cookbooks exist in the path ↵Josh Murphy2015-01-251-6/+10
| | | | before attempting to upload
* Remove unused argument to CookbookUploader#initializedanielsdeleo2014-07-301-1/+1
|
* Make FileVendor configuration specific to the two implementationsdanielsdeleo2014-07-301-1/+1
| | | | | | | | | | | | | | | | FileVendor previously was configured by storing a closure/anonymous function as a class instance variable. This had the following downsides: * The API was too general, which caused a lot of code repetition * The block was lazily evaluated, which hid errors and made testing more difficult * The closures captured references to classes with references to large data structures, which complicates GC. Since we've only ever had the same two implementations of FileVendor, we can encapsulate configuration of the FileVendor factory by wrapping each configuration option in a method. As a side benefit, arguments to these methods will be eagerly evaluated, which makes it easier to detect errors.
* Restore warning for cookbook shadowingdanielsdeleo2014-03-271-0/+6
|
* CHEF-4851 - Replaced #select with negation for finding missing dependencies ↵Josh Murphy2014-03-191-2/+2
| | | | with #reject
* CHEF-4851 - Display all missing dependencies in cookbook uploadJosh Murphy2014-03-191-7/+11
|
* support configurable concurrencyRanjib Dey2013-11-261-1/+7
|
* [CHEF-3452] uploading frozen cookbook version explodesZachary Stevens2013-04-121-9/+3
| | | | | | | This change make Chef::CookbookUploader for testing whether a cookbook upload has failed due to a frozen version existing on the server. This fixes a regression introduced by 7c7b4ea1.
* Added require 'chef/cookbook_uploader' to file 'chef/knife/cookbook_upload.rb'Stathy Touloumis2013-04-121-0/+1
|
* [CHEF-3919] Removed Chef::Version::Cookbook code and replaced by a ↵Xabier de Zuazo2013-04-111-2/+2
| | | | Chef::Version::Platform class, reverting Chef::Version class changes
* [CHEF-3919] cookbook version specific code factored out from Chef::Version, ↵Xabier de Zuazo2013-04-111-2/+2
| | | | new classes created: Chef::Version::Cookbook and Chef::VersionConstraint::Cookbook
* Quote cookbook name/version in error messagedanielsdeleo2012-12-051-1/+1
| | | | | | | | | User was getting an error message: > ERROR: Cookbook git depends on cookbook runit, version >= 0.0.0, Issue was the metadata for git had a dependency on 'runit,'. Should probably also add validation to metadata depends for cookbook names, but generally quoting user-supplied values in error messages makes this kind of issue more obvious, so this should be a worthwhile quick fix.
* Fix 'knife cookbook upload --all'Christopher Maier2012-11-301-1/+1
| | | | | | | | | | | | | Cookbooks could be uploaded individually, but attempting to load all at once was resulting in the following error against Chef 11 servers: ERROR: The data in your request was invalid Response: Field 'checksums' invalid The immediate cause of this message is that knife was trying to create a sandbox with no files, which we explicitly forbid on Chef 11. The ultimate cause of this was introduced on August 14, 2012 with commit 70c019f3. It appears this was an incomplete refactoring of the CookbookLoader class. Before the refactoring, a cookbook loader would load its cookbooks lazily in each method that needed them. The refactoring made this explicit, but an explicit call was not added to the knife cookbook upload command. This addresses CHEF-3638.
* [OC-3564] move core Chef to the repo root \o/ \m/Seth Chisamore2012-10-301-0/+295
The opscode/chef repository now only contains the core Chef library code used by chef-client, knife and chef-solo!