Project

General

Profile

Feature #4151

Add STI to reports and allow for registering new report types

Added by Marek Hulán almost 6 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Reporting
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

We need to support different report types for OpenScap, ABRT and other future reporting tools.

This means there must be a way to register new report types and importers from plugin (exactly the same we have for facts now). Reports will probably only differ in metrics and their own statuses, therefore we should make metrics optional and mapping to global status should be different per specific status.

There should be a generic API for uploading reports. Common things for all types are:
- message (text, status, level)
- reported_at
- status (custom hash of format "status": n, each type different statuses)

User stories:
- As a plugin developer I want to register new report type.
- As a plugin developer I want to register new report importer for a given type.
- As a user I want to search hosts based on report type and its specific status.
- As a user I want to have be able to set different report permissions per report types. (implies report STI)


Related issues

Related to Foreman - Feature #8488: Make ReportImporters a registerable service like FactImportersNew2014-11-24
Related to Foreman - Feature #10782: Add global status for hostsClosed2015-06-11
Related to Foreman - Refactor #12344: Use Rails 4 update_all syntax migrationsClosed2015-10-29
Related to Foreman - Bug #13920: Unable to view config reports in forman 1.11 rc1 as non admin userClosed2016-02-26
Related to OpenSCAP - Bug #14478: compliance report searches failsClosed2016-04-05

Associated revisions

Revision e8d6d2d6 (diff)
Added by Shlomi Zadok about 4 years ago

fixes #4151 - enable reports STI

Permits subclassing of ReportImporter and Report to import and store
new types of reports associated to hosts.

Revision 90334715 (diff)
Added by Marek Hulán over 3 years ago

Refs #4151 - Use new report API introduced in Foreman 1.11

History

#1 Updated by Marek Hulán almost 6 years ago

This is becoming a blocker for foreman_abrt plugin soon.

#2 Updated by Dmitri Dolguikh almost 6 years ago

  • Target version set to 1.9.0

#3 Updated by Dmitri Dolguikh over 5 years ago

  • Target version deleted (1.9.0)

#4 Updated by Tomáš Strachota over 4 years ago

  • Subject changed from Add STI to reports to Add STI to reports and allow for registering new report types
  • Description updated (diff)
  • Assignee set to Tomáš Strachota

#5 Updated by Tomáš Strachota over 4 years ago

  • Related to Feature #8488: Make ReportImporters a registerable service like FactImporters added

#6 Updated by Tomáš Strachota over 4 years ago

#7 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2530 added
  • Pull request deleted ()

#8 Updated by Marek Hulán about 4 years ago

  • Pull request https://github.com/theforeman/foreman/pull/2750 added
  • Pull request deleted (https://github.com/theforeman/foreman/pull/2530)

#9 Updated by Tomáš Strachota about 4 years ago

  • Assignee changed from Tomáš Strachota to Shlomi Zadok

Switching the owner to Shlomi since he has actually been implementing it.

#10 Updated by Ondřej Pražák about 4 years ago

  • Blocks Refactor #12261: host_id references host(id) constraint on reports STI table is too restrictive for ForemanOpenscap added

#11 Updated by Dominic Cleal about 4 years ago

  • Blocks deleted (Refactor #12261: host_id references host(id) constraint on reports STI table is too restrictive for ForemanOpenscap)

#12 Updated by Dominic Cleal about 4 years ago

  • Legacy Backlogs Release (now unused) set to 71

#13 Updated by Shlomi Zadok about 4 years ago

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

#14 Updated by Dominic Cleal about 4 years ago

  • Related to Refactor #12344: Use Rails 4 update_all syntax migrations added

#15 Updated by Dominic Cleal almost 4 years ago

  • Related to Bug #13920: Unable to view config reports in forman 1.11 rc1 as non admin user added

#16 Updated by Marek Hulán over 3 years ago

  • Related to Bug #14478: compliance report searches fails added

Also available in: Atom PDF