Project

General

Profile

Actions

Bug #2002

closed

Problem editing Libvirt Host in Foreman

Added by Greg Sutcliffe almost 12 years ago. Updated over 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
VM management
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

This is on my bleeding edge Foreman (ruby 1.9, puppet 3, psql 9.2)

I can create and destroy Libvirt Hosts in Foreman fine, but if I try to edit one, I get this error/trace:

"undefined method `id' for #<Fog::Compute::Libvirt::Nic:0xda7a000>" from app/views/compute_resources_vms/form/_libvirt.html.erb:11


Related issues 3 (0 open3 closed)

Related to Foreman - Bug #2586: Upgrade fog when releasedClosedMarek Hulán05/30/2013Actions
Has duplicate Foreman - Bug #2024: (Fog / Libvirt / Rub1.9) cannot edit a Libvirt host Duplicate12/09/2012Actions
Has duplicate Smart Proxy - Bug #2456: Error while testing compute resourcesClosed05/05/2013Actions
Actions #1

Updated by Ohad Levy almost 12 years ago

can you confirm this happens on ruby 1.9 only?

Actions #2

Updated by Marek Hulán over 11 years ago

  • Status changed from New to Assigned
  • Assignee set to Marek Hulán

I can confirm. In ruby 1.8 every object has a method id that was replaced by object_id in ruby 1.9. The proper fix is in unreleased fog master branch (see https://github.com/fog/fog/blob/master/lib/fog/libvirt/models/compute/nic.rb#L10). We can either wait for a new release or make some monkey patch. However they claim to release soon.

Actions #3

Updated by Greg Sutcliffe over 11 years ago

I don't want to put bundler git gems back in the packaging, it makes a mess :). Let's hope they release a new version.

Actions #4

Updated by Dominic Cleal over 11 years ago

  • Target version set to 1.2.0

I think this is important to fix for 1.2, as we're moving to Ruby 1.9 on the RPMs. I don't think the fix is in the 1.11.1 release, so can we monkey patch?

Actions #5

Updated by Marek Hulán over 11 years ago

  • Status changed from Assigned to Ready For Testing

PR https://github.com/theforeman/foreman/pull/663
Before we close this one, we should create a new issue to remove this quick fix in future.

Actions #6

Updated by Ohad Levy over 11 years ago

  • Status changed from Ready For Testing to Resolved
Actions

Also available in: Atom PDF