Skip to content

Commit d966723

Browse files
authored
Merge pull request #839 from kbrock/host_validation
use default verify_credentials implementation
2 parents cd4cc15 + fc52c60 commit d966723

File tree

1 file changed

+0
-15
lines changed
  • app/models/manageiq/providers/openstack/infra_manager

1 file changed

+0
-15
lines changed

app/models/manageiq/providers/openstack/infra_manager/host.rb

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -83,21 +83,6 @@ def authentication_status
8383
end
8484
end
8585

86-
def verify_credentials(auth_type = nil, options = {})
87-
raise MiqException::MiqHostError, "No credentials defined" if missing_credentials?(auth_type)
88-
raise MiqException::MiqHostError, "Logon to platform [#{os_image_name}] not supported" if auth_type.to_s != 'ipmi' && os_image_name !~ /linux_*/
89-
90-
case auth_type.to_s
91-
when 'remote', 'default', 'ssh_keypair' then verify_credentials_with_ssh(auth_type, options)
92-
when 'ws' then verify_credentials_with_ws(auth_type)
93-
when 'ipmi' then verify_credentials_with_ipmi(auth_type)
94-
else
95-
verify_credentials_with_ws(auth_type)
96-
end
97-
98-
true
99-
end
100-
10186
def update_ssh_auth_status!
10287
# Creating just Auth status placeholder, the credentials are stored in parent or this auth, parent is
10388
# EmsOpenstackInfra in this case. We will create Auth per Host where we will store state, if it not exists

0 commit comments

Comments
 (0)