Project

General

Profile

Actions
Copy link

Bug #21037

closed

"Invalid single-table inheritance type: Host::Discovered is not a subclass of Host::Managed" caused by auditing host

Added by Ivan Necas about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ivan Necas
Category:
Host creation
Target version:
1.17.0
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/4864
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

In the host auditing, we're setting the auditable_class to Host. And in the Host, we're redirecting
all the methods to the Host::Managed. This causes issues, when converting the host from discovered
to managed and creating an audit record of it.

Reproducer (having a discovered host).

host2 = host.become(Host::Managed)
host2.type = "Host::Managed" 
host2.save

Related issues 3 (0 open — 3 closed)

Related to Discovery - Bug #20972: STI issues with Host::Discovered on Rails 5Duplicate09/17/2017Actions
Related to Foreman - Bug #25207: Invalid single-table inheritance type: Host::Discovered is not a subclass of Host::ManagedClosedLukas ZapletalActions
Related to Discovery - Bug #25556: 'Discovery manager' role brokenDuplicateActions
Actions
Copy link
 #1

Updated by The Foreman Bot about 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4863 added
Actions
Copy link
 #2

Updated by The Foreman Bot about 7 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4864 added
Actions
Copy link
 #3

Updated by Anonymous about 7 years ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/4863)
Actions
Copy link
 #4

Updated by Ivan Necas about 7 years ago

  • Related to Bug #20972: STI issues with Host::Discovered on Rails 5 added
Actions
Copy link
 #5

Updated by Ivan Necas about 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions
Copy link
 #6

Updated by Marek Hulán about 7 years ago

  • Translation missing: en.field_release set to 296
Actions
Copy link
 #7

Updated by Lukas Zapletal about 7 years ago

  • Translation missing: en.field_release changed from 296 to 240

I actually need this in 1.16 if Rails5 is part of that branch, this is discovery blocker.

Actions
Copy link
 #8

Updated by Lukas Zapletal about 7 years ago

  • Translation missing: en.field_release changed from 240 to 296

Ok Rails5 is not part of 1.16.

Actions
Copy link
 #9

Updated by Lukas Zapletal about 6 years ago

  • Related to Bug #25207: Invalid single-table inheritance type: Host::Discovered is not a subclass of Host::Managed added
Actions
Copy link
 #10

Updated by Lukas Zapletal about 6 years ago

  • Triaged set to No

Hello, we have an initial version of a patch that solves this. It's one liner, please apply it, restart and get back to me with feedback:

https://github.com/theforeman/foreman/pull/6192/files

Actions
Copy link
 #11

Updated by Lukas Zapletal almost 6 years ago

  • Related to Bug #25556: 'Discovery manager' role broken added
Actions
Copy link

Also available in: Atom PDF