Project

General

Profile

Actions

Bug #7297

closed

Virsh error handlers throw "uninitialized constant Proxy::DNS" error

Added by Dominic Cleal about 10 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
DNS
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

When a virsh update fails, the error handlers in lib/proxy/virsh.rb throw an uninitialized constant error themselves:

D, [2014-08-29T11:12:17.142706 #24336] DEBUG -- : /usr/bin/sudo /usr/bin/virsh net-update default add-last dns-host --xml '<host ip="192.168.122.112"><hostname>pdb2.example.com</hostname></host>' --live --config 2>&1
E, [2014-08-29T11:12:17.195826 #24336] ERROR -- : uninitialized constant Proxy::DNS
D, [2014-08-29T11:12:17.195996 #24336] DEBUG -- : /home/dcleal/code/foreman/smart-proxy/lib/proxy/virsh.rb:54:in `rescue in virsh_update_dns'
/home/dcleal/code/foreman/smart-proxy/lib/proxy/virsh.rb:47:in `virsh_update_dns'
/home/dcleal/code/foreman/smart-proxy/modules/dns/providers/virsh.rb:17:in `create'
/home/dcleal/code/foreman/smart-proxy/modules/dns/dns_api.rb:47:in `block in <class:Api>'

Related issues 1 (0 open1 closed)

Related to Smart Proxy - Refactor #7298: Remove Proxy::Virsh dependencies on dns and dhcp modulesClosed08/29/2014Actions
Actions #1

Updated by The Foreman Bot about 10 years ago

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

Updated by Dominic Cleal about 10 years ago

  • Translation missing: en.field_release changed from 10 to 22
Actions #3

Updated by Anonymous about 10 years ago

  • Target version changed from 1.7.4 to 1.7.3
Actions #4

Updated by Dominic Cleal about 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #5

Updated by Dominic Cleal over 8 years ago

  • Related to Refactor #7298: Remove Proxy::Virsh dependencies on dns and dhcp modules added
Actions

Also available in: Atom PDF