Project

General

Profile

Bug #12233

Discovery doesn't reboot due to remote execution become extenstion

Added by Ivan Necas almost 7 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

When using foreman remote execution (0.0.10) with discovery (4.1.0), the discovered host doesn't reboot on turning into managed,
getting:

 | NoMethodError: undefined method `drop_execution_interface_cache' for #<Host::Discovered:0x007f6e28913ac8>
 | /opt/rh/ruby193/root/usr/share/gems/gems/activemodel-3.2.8/lib/active_model/attribute_methods.rb:407:in `method_missing'
 | /opt/rh/ruby193/root/usr/share/gems/gems/activerecord-3.2.8/lib/active_record/attribute_methods.rb:149:in `method_missing'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_remote_execution-0.0.10/app/models/concerns/foreman_remote_execution/host_extensions.rb:53:in `becomes_with_remote_execution'
 | /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_discovery-4.1.1/app/models/host/managed_extensions.rb:28:in `setReboot'
 | /usr/share/foreman/app/models/concerns/orchestration.rb:162:in `execute'
 | /usr/share/foreman/app/models/concerns/orchestration.rb:107:in `block in process'
 | /usr/share/foreman/app/models/concerns/orchestration.rb:99:in `each'
 | /usr/share/foreman/app/models/concerns/orchestration.rb:99:in `process'
 | /usr/share/foreman/app/models/concerns/orchestration.rb:39:in `post_commit'

Associated revisions

Revision b9e063e0 (diff)
Added by Ivan Necas almost 7 years ago

Fixes #12233 - clean the interface cache only when becoming Managed::Host

Without the patch, discovery reboot fails on

```
NoMethodError: undefined method `drop_execution_interface_cache' for #<Host::Discovered:0x007f6e28913ac8>
```

Revision 6635732f
Added by Ivan Necas almost 7 years ago

Merge pull request #59 from iNecas/fix-discovery-compatibility

Fixes #12233 - clean the interface cache only when becoming Managed::Host

History

#1 Updated by The Foreman Bot almost 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_remote_execution/pull/59 added
  • Pull request deleted ()

#2 Updated by Ivan Necas almost 7 years ago

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

#3 Updated by Marek Hulán almost 7 years ago

  • Legacy Backlogs Release (now unused) set to 103

Also available in: Atom PDF