Skip to content

Commit

Permalink
BZ752975 - multiple executions of aeolus-cleanup/configure not workin…
Browse files Browse the repository at this point in the history
…g as expected

https://bugzilla.redhat.com/show_bug.cgi?id=752975

This occurs because providers that already exist are attempting to be
recreated.  We determine whether or not a provider exists by issuing a
GET on /conductor/providers.  This request issues a 302 response, but
we are not properly following that redirect due to looking for the
follow param in the wrong place.  This corrects that so the follow
param works as expected, and thus the resource is identified and not
wrongfully recreated.
  • Loading branch information
jeckersb authored and calfonso committed Nov 14, 2011
1 parent 2b67daf commit f365b00
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion recipes/aeolus/lib/puppet/provider/web_request/curl.rb
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ def skip_request?(params, cookie = nil)
result = Curl::Easy::web_request(method, condition[method],
:parameters => condition['parameters'],
:file_parameters => condition['file_parameters'],
:cookie => cookie, :follow => condition[:follow])
:cookie => cookie, :follow => params[:follow])
result_succeeded = true
begin
verify_result(result, condition)
Expand Down

0 comments on commit f365b00

Please sign in to comment.