summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Remove a require statement now that we dont need it anymoreirving/streaming_uploader_http_fixIrving Popovetsky2015-05-051-1/+0
|
* Use Chef::HTTP::BasicClient instead of Net::HTTP to get both proxy and ssl ↵Irving Popovetsky2015-05-051-6/+1
| | | | goodness
* Remove now-redundant verify_mode because DefaultSSLPolicy takes care of it ↵Irving Popovetsky2015-05-051-12/+0
| | | | for us
* Apply an SSL Policy to CookbookSiteStreamingUploader, fixing SSL errors ↵Irving Popovetsky2015-05-041-1/+3
| | | | uploading to private Supermarkets
* Nice error handling for unsupported API version in formatters/*.rb.tylercloke2015-05-046-0/+29
|
* Nice error formatting for unsupported API version in Knife.tylercloke2015-05-041-0/+8
|
* Implemented X-Ops-Server-API-Version in Chef requests.tylercloke2015-05-041-0/+3
| | | | | | | | | | This takes advantage of Chef::HTTP::Authenticator.handle_request. After the authentication_headers have been merged, it adds: {'X-Ops-Server-API-Version' => Chef::HTTP::Authenticator::SERVER_API_VERSION} Chef::HTTP::Authenticator::SERVER_API_VERSION should corrospond to the max version of the server the client is compatible with. We are starting at "0". I also removed outer level lets :base_header and :req_with_body_headers from rest_spec.rb as they were never used anywhere. Another :base_header was defined in a tighter part of the spec and was only used in that scope, and :req_with_body_headers was never used anywhere.
* Merge pull request #3320 from chef/lcg/node-utf8-sanitizeLamont Granquist2015-05-042-2/+15
|\ | | | | Lcg/node utf8 sanitize
| * add a spec test + linebreaksLamont Granquist2015-05-041-1/+9
| |
| * use validate_utf8=false option to json encoder for nodeLamont Granquist2015-05-042-2/+7
| | | | | | | | | | | | | | with validation off any bad utf8 data will not cause the ffi-yajl encoder to raise. with ffi-yajl >= 2.2.0 the bad data will be scrubbed to produce valid JSON to POST/PUT to the server.
* | Merge pull request #3298 from juliandunn/fix-aix-source-packageJulian C. Dunn2015-05-041-0/+1
|\ \ | |/ |/| fix AIX package installs using a 'source' attribute
| * Raise exception if a package provided by 'source' doesn't actually provideJulian C. Dunn2015-04-281-0/+1
| | | | | | | | that package
* | Cache service_resource_providers for the duration of the run.Noah Kantrowitz2015-05-011-21/+21
| |
* | Fix installation of yum packages with version constraintsojab2015-05-011-4/+1
| | | | | | | | Fixes #2778
* | Fix debug messageojab2015-05-011-1/+1
| | | | | | | | | | | | | | << has higher precedence than == & ?: => "mdbtools >= 0.7.1: Unable to match package 'mdbtools >= 0.7.1' but matched 1 \u0001" Obvious fix.
* | fix smartos_package for new "pkgin" output, fixes #3112Dave Eddy2015-05-011-5/+5
| |
* | test state_for_resource_reporter on file resourceLamont Granquist2015-05-011-1/+1
| | | | | | | | and fix it to hit the right key.
* | create Chef::Resource#state_for_resource_reporterLamont Granquist2015-05-014-12/+36
| | | | | | | | | | | | Use this to override the state reported by the resource reporter while avoiding the collision over Chef::Resource#state being used by some LWRPs.
* | don't mutate the new resourceLamont Granquist2015-05-011-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this line violates the rule that we never mutate the new resource. in Chef 12 this causes real problems because if a file resource is notified (run twice) then the rendered content is loaded into the new resource and if the file resource content changes (i.e. its a template with logic which changes when its notified) then it will cause a failure because the rendered content will not match the 'requested' checksum. where the 'requested' checksum is actually the checksum loaded by this line. fundamentally we have three different states that we're trying to track: - current state - initial state - requested state the removed line tries to use the @new_resource for reporting initial state, which then bleeds over into what chef thinks is requested state in the next invokation. without constructing a third @initial_resource and using that for resource reporting comparison we can't solve this problem "right". the tradeoff is we either break reporting here or break chef-client runs. this patch sacrifices reporting in order to make chef-client work.
* | Merge pull request #3227 from higanworks/show_chef_version_on_shellThom May2015-05-011-1/+1
|\ \ | | | | | | Show Chef::VERSION at prompt_c and prompt_i on shell session
| * | reflect https://github.com/chef/chef/pull/3227#issuecomment-94560685sawanoboly2015-04-211-1/+1
| | |
| * | Merge branch 'master' into show_chef_version_on_shellsawanoboly2015-04-2117-107/+796
| |\ \
| * | | Show Chef::VERSION at prompt_c and prompt_i on shell sessionsawanoboly2015-04-151-2/+2
| | | |
* | | | Merge pull request #3311 from chef/tc/key-editTyler Cloke2015-04-3010-9/+559
|\ \ \ \ | | | | | | | | | | Implemented `knife user key edit` and `knife client key edit`
| * | | | Implemented `knife user key show` and `knife client key show`.tylercloke2015-04-308-58/+209
| | | | |
| * | | | Fixed small bug in Chef::Key.load_by_(user|client) where actor wasn't populated.tylercloke2015-04-301-3/+5
| | | | |
| * | | | Implemented `knife user key edit` and `knife client key edit`.tylercloke2015-04-306-1/+384
| | | | |
| * | | | Fixed a bug where create_key would conflict with public_key in hash returned ↵tylercloke2015-04-301-0/+8
| | | | | | | | | | | | | | | | | | | | from server on update.
| * | | | Changes Chef::Key update to allow updating the name of a key.tylercloke2015-04-301-4/+10
| | | | |
* | | | | windows_package is idempotent againjdm/winstaller-idempotentJay Mundrawala2015-04-301-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was broken in #3034. The issue is that the package does not get detected as installed because the version that is installed has \x00 at the end, while the version from the msi does not. This fails comparison, and we fall into code that does not use source and thus requires candidate version and we die. (Or something like that) Solves #3316
* | | | Merge pull request #3207 from Igorshp/exception_handlersThom May2015-04-302-6/+4
|\ \ \ \ | | | | | | | | | | patch to always run exception handlers
| * | | | updated based on @lamont-granquist commentsIgor Shpakov2015-04-132-2/+1
| | | | |
| * | | | do not overwrite whole @run_status in #build_nodeIgor Shpakov2015-04-112-6/+5
| | | | |
| * | | | patch to always run exception handlersIgor Shpakov2015-04-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | exception handlers require @run_status to be set. This patch moves @run_status declaration higher up so exception handlers report on all problems inside do_run begin block
* | | | | Implemented `knife user key delete` and `knife client key delete`.tylercloke2015-04-293-0/+207
| | | | |
* | | | | Merge pull request #3235 from jaymzh/yum_archPhil Dibowitz2015-04-292-31/+73
|\ \ \ \ \ | | | | | | | | | | | | Make multipackage and arch play nicely together
| * | | | | Make multipackage and arch play nicely togetherPhil Dibowitz2015-04-292-31/+73
| | |_|_|/ | |/| | | | | | | | | | | | | Fixes #3233
* | | | | Implemented `knife user key list` and `knife client key list`.tc/key-listtylercloke2015-04-294-0/+271
|/ / / / | | | | | | | | | | | | Abstracted out a bunch of common tests with the key create code to reduce redundancy.
* | | | Implemented `knife user key create` and `knife client key create`.tylercloke2015-04-286-1/+298
| | | | | | | | | | | | | | | | Implemented using a common service class that contains most of the actual functionality outside of parsing and inheriting Chef::Knife.
* | | | Fixed bug in Chef::Key where create failed to return the full key.tylercloke2015-04-281-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Code assumed POST to keys endpoint returned the key body like other parts of the API. Now, it simply generated the new key returned by create from the original key plus the private_key returned by the API if any. Specs also updated.
* | | | Set chef_server_root config value to sane defaults.tylercloke2015-04-281-0/+10
| | | | | | | | | | | | | | | | If chef_server_url matches /\/organizations\/\S*$/ aka .../organizations/someorg, then strip /organizations/someorg and use that as the chef_server_root default. Otherwise, just use whatever chef_server_url is as the default.
* | | | Merge pull request #3279 from chef/lcg/fix-provider-resolver-api-breakThom May2015-04-272-1/+4
|\ \ \ \ | | | | | | | | | | Lcg/fix provider resolver api break
| * | | | typo fix in commentlcg/fix-provider-resolver-api-breakLamont Granquist2015-04-271-1/+1
| | | | |
| * | | | Fix API break in 12.3.0.rc.0Lamont Granquist2015-04-242-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | provides? on a provider was passed the full resource. provides? on a resource class cannot be passed the resource and must be passed the declared_type instead. i tried to make them the same, but that would change the API on the provider.
* | | | | Merge pull request #3280 from chef/sersut/3234Thom May2015-04-271-0/+1
|\ \ \ \ \ | | | | | | | | | | | | Configure serverspec correctly on windows.
| * | | | | Configure serverspec correctly on windows.sersut/3234Serdar Sutay2015-04-241-0/+1
| |/ / / /
* | | | | Merge pull request #3166 from chef/ssd/find-pythonThom May2015-04-251-1/+29
|\ \ \ \ \ | |/ / / / |/| | | | Use the same python interpreter as yum when possible
| * | | | Use the same python interpreter as yum when possiblessd/find-pythonSteven Danna2015-03-311-1/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We rely on a system python installation with the correct python libraries. To improve the chances that the python interpreter we call has the yum libary available, we use the same interpreter as yum uses. Fixes #3143
* | | | | Revert "Fix API break in 12.3.0.rc.0"Lamont Granquist2015-04-242-4/+1
| | | | | | | | | | | | | | | | | | | | This reverts commit 57cdbe124f63e47cab960e714572539d3979b90f.
* | | | | Fix API break in 12.3.0.rc.0Lamont Granquist2015-04-242-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | provides? on a provider was passed the full resource. provides? on a resource class cannot be passed the resource and must be passed the declared_type instead. i tried to make them the same, but that would change the API on the provider.