Skip to content

Commit

Permalink
Merge pull request #14469 from jhernand/drop_support_for_multiple_ovi…
Browse files Browse the repository at this point in the history
…rt_paths

Drop support for oVirt /api, always use /ovirt-engine/api
(cherry picked from commit ac8b311)
  • Loading branch information
blomquisg authored and simaishi committed Apr 6, 2017
1 parent 6034a20 commit 919c524
Show file tree
Hide file tree
Showing 8 changed files with 2,198 additions and 2,397 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,16 +21,12 @@ def connect(options = {})
raise "version #{version} of the api is not supported by the provider"
end

# If the API path is stored in the endpoints table then use it:
path = options[:path] || default_endpoint.path
_log.info("Using stored API path '#{path}'.") unless path.blank?

# Prepare the options to call the method that creates the actual connection:
connect_options = {
:scheme => options[:scheme] || 'https',
:server => options[:ip] || address,
:port => options[:port] || port,
:path => path,
:path => options[:path] || '/ovirt-engine/api',
:username => options[:user] || authentication_userid(options[:auth_type]),
:password => options[:pass] || authentication_password(options[:auth_type]),
:service => options[:service] || "Service",
Expand All @@ -56,7 +52,7 @@ def connect(options = {})
connection = self.class.public_send(connect_method, connect_options)

# Copy the API path to the endpoints table:
default_endpoint.path = version.to_i == 4 ? '/ovirt-engine/api' : connection.api_path
default_endpoint.path = connect_options[:path]

connection
end
Expand Down
3,067 changes: 1,528 additions & 1,539 deletions spec/vcr_cassettes/manageiq/providers/redhat/infra_manager/refresh/refresher_3_1.yml

Large diffs are not rendered by default.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 919c524

Please sign in to comment.