Project

General

Profile

Bug #6908

Chef and Puppet request test failures

Added by Dominic Cleal over 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Proxy tests are failing with a difference between "/" and "//" in the Chef request URLs. Looks like it's from the release of Chef 11.14.2.

http://ci.theforeman.org/job/test_proxy_develop_pull_request/425/

Real HTTP connections are disabled. Unregistered request: GET https://chef.example.com/clients/testnode1 with headers {'Accept'=>'application/json', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'Host'=>'chef.example.com:443', 'User-Agent'=>'Chef Client/11.14.2 (ruby-1.8.7-p371; ohai-7.2.0; x86_64-linux; +http://opscode.com)', 'X-Chef-Version'=>'11.14.2', 'X-Ops-Authorization-1'=>'vCAKNPYkc4+kWJtzPTtVx5IhpmwQucRzHnivkcxigNoxihu5+hT6UI7jqDK9', 'X-Ops-Authorization-2'=>'IeBbUf0NU8eUmM7Kx0E+H/dooCSZ08A+hg2tNXXv8xesX01v7qyEXCDDpm23', 'X-Ops-Authorization-3'=>'ZesXNacclo6vwctvougiHXRL6eNFLgk7dqHAQLOBbaxqmqxjA8Re7RvoWHLB', 'X-Ops-Authorization-4'=>'b57yFmW2NL5YDZnQ8KgnGz21hsoWJ461GTpMDIqSLlfHsMm8RWL8+g700jL2', 'X-Ops-Authorization-5'=>'Ih0nFxD86kXgnsA23aHyXmc6Qryv6TB6FQVwZMsw5+q40KV0nm7CeA7eyteJ', 'X-Ops-Authorization-6'=>'k3hVHpjnRcktwRh6TAizrKZqZfV6KcEeg6gCfAfTJw==', 'X-Ops-Content-Hash'=>'2jmj7l5rSw0yVb/vlWAYkK/YBwk=', 'X-Ops-Sign'=>'algorithm=sha1;version=1.0;', 'X-Ops-Timestamp'=>'2014-08-04T10:58:05Z', 'X-Ops-Userid'=>'testnode1', 'X-Remote-Request-Id'=>'7dff1b07-740a-4c87-bbcb-e07c33e66f92'}

You can stub this request with the following snippet:

stub_request(:get, "https://chef.example.com/clients/testnode1").
  with(:headers => {'Accept'=>'application/json', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'Host'=>'chef.example.com:443', 'User-Agent'=>'Chef Client/11.14.2 (ruby-1.8.7-p371; ohai-7.2.0; x86_64-linux; +http://opscode.com)', 'X-Chef-Version'=>'11.14.2', 'X-Ops-Authorization-1'=>'vCAKNPYkc4+kWJtzPTtVx5IhpmwQucRzHnivkcxigNoxihu5+hT6UI7jqDK9', 'X-Ops-Authorization-2'=>'IeBbUf0NU8eUmM7Kx0E+H/dooCSZ08A+hg2tNXXv8xesX01v7qyEXCDDpm23', 'X-Ops-Authorization-3'=>'ZesXNacclo6vwctvougiHXRL6eNFLgk7dqHAQLOBbaxqmqxjA8Re7RvoWHLB', 'X-Ops-Authorization-4'=>'b57yFmW2NL5YDZnQ8KgnGz21hsoWJ461GTpMDIqSLlfHsMm8RWL8+g700jL2', 'X-Ops-Authorization-5'=>'Ih0nFxD86kXgnsA23aHyXmc6Qryv6TB6FQVwZMsw5+q40KV0nm7CeA7eyteJ', 'X-Ops-Authorization-6'=>'k3hVHpjnRcktwRh6TAizrKZqZfV6KcEeg6gCfAfTJw==', 'X-Ops-Content-Hash'=>'2jmj7l5rSw0yVb/vlWAYkK/YBwk=', 'X-Ops-Sign'=>'algorithm=sha1;version=1.0;', 'X-Ops-Timestamp'=>'2014-08-04T10:58:05Z', 'X-Ops-Userid'=>'testnode1', 'X-Remote-Request-Id'=>'7dff1b07-740a-4c87-bbcb-e07c33e66f92'}).
  to_return(:status => 200, :body => "", :headers => {})

registered request stubs:

stub_request(:get, "https://chef.example.com//clients/testnode1")

also this, possibly due to ffi_yajl being added by chef-zero:

undefined method `bytesize' for nil:NilClass (NoMethodError)
/usr/local/rvm/gems/ruby-1.8.7-p371@test_proxy_develop_pull_request-3/gems/ffi-yajl-1.0.1/lib/ffi_yajl/ffi/parser.rb:131:in `do_yajl_parse'
/usr/local/rvm/gems/ruby-1.8.7-p371@test_proxy_develop_pull_request-3/gems/ffi-yajl-1.0.1/lib/ffi_yajl/parser.rb:58:in `parse'
/usr/local/rvm/gems/ruby-1.8.7-p371@test_proxy_develop_pull_request-3/gems/ffi-yajl-1.0.1/lib/ffi_yajl/parser.rb:24:in `parse'
/usr/local/rvm/gems/ruby-1.8.7-p371@test_proxy_develop_pull_request-3/gems/ffi-yajl-1.0.1/lib/ffi_yajl/json_gem.rb:50:in `load'
./test/../modules/puppet_proxy/api_request.rb:65:in `find_environments'

Associated revisions

Revision 37c1113f (diff)
Added by Dominic Cleal over 6 years ago

fixes #6908 - update request tests for latest Chef release

Revision 2781dc7d
Added by Dmitri Dolguikh over 6 years ago

Merge pull request #194 from domcleal/6908-request-tests

fixes #6908 - update request tests for latest Chef release

History

#1 Updated by The Foreman Bot over 6 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/194 added
  • Pull request deleted ()

#2 Updated by Dominic Cleal over 6 years ago

  • Legacy Backlogs Release (now unused) set to 10

#3 Updated by Dmitri Dolguikh over 6 years ago

  • Status changed from Ready For Testing to Resolved

commit 37c1113f5c1b7c896238f8e0f764241c7bd1cdb7

#4 Updated by Dominic Cleal over 6 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF