Merge pull request #167 from teto/fixes

Fixes
This commit is contained in:
Dmitry Vasilets 2014-03-16 08:17:36 +01:00
commit 2d088d43df
2 changed files with 15 additions and 3 deletions

View File

@ -19,8 +19,14 @@ module VagrantPlugins
def read_state(libvirt, machine) def read_state(libvirt, machine)
return :not_created if machine.id.nil? return :not_created if machine.id.nil?
# Find the machine begin
server = libvirt.servers.get(machine.id) server = libvirt.servers.get(machine.id)
rescue Libvirt::RetrieveError => e
server = nil
@logger.debug('Machine not found #{e}.')
end
# Find the machine
if server.nil? || [:'shutting-down', :terminated].include?(server.state.to_sym) if server.nil? || [:'shutting-down', :terminated].include?(server.state.to_sym)
# The machine can't be found # The machine can't be found
@logger.info('Machine not found or terminated, assuming it got destroyed.') @logger.info('Machine not found or terminated, assuming it got destroyed.')

View File

@ -21,10 +21,16 @@ module VagrantPlugins
env[:domain_name] << '_' env[:domain_name] << '_'
env[:domain_name] << env[:machine].name.to_s env[:domain_name] << env[:machine].name.to_s
begin
@logger.info("Looking for domain #{env[:domain_name]} through list #{env[:libvirt_compute].servers.all}") @logger.info("Looking for domain #{env[:domain_name]} through list #{env[:libvirt_compute].servers.all}")
# Check if the domain name is not already taken # Check if the domain name is not already taken
domain = ProviderLibvirt::Util::Collection.find_matching( domain = ProviderLibvirt::Util::Collection.find_matching(
env[:libvirt_compute].servers.all, env[:domain_name]) env[:libvirt_compute].servers.all, env[:domain_name])
rescue Fog::Errors::Error => e
@logger.info("#{e}")
domain = nil
end
@logger.info("Looking for domain #{env[:domain_name]}") @logger.info("Looking for domain #{env[:domain_name]}")