Refactor #80
Reports data should be accessed in a simple way
| Status: | Closed | Start: | 11/17/2009 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned to: |  Ohad Levy |
% Done: | 100% |
|
| Category: | Puppet Reports | |||
| Target version: | 0.1-3 | |||
| Backlog: | Difficulity: | |||
| Votes: | 0 |
Description
Currently report status is stored in one integer, it should be broken down into multiple database tables allowing to reduce the size of the database (by not storing the entire Puppet::Transaction::Report).
this should allow to search in a more rails way for all hosts that has a report with change count > 20
Related issues
| blocks Foreman - Bug #78: Puppet reports fails with 0.24.4-5 master | Closed | 11/12/2009 |
Associated revisions
Revision ff1cc6b1ff54003ac9916ecf389710fdfed2ed7b
fixes #106 - redid the status calcuation, this fixes #80 but not in an optimal way - db migration would be required for that.
History
Updated by Ohad Levy over 2 years ago
- Target version deleted (
0.1-3)
as the new version of puppet is going to be a massive rewrite of how reports are handled, I'm going to push this db schema changes until its clear what info puppet will provide.
additionally, I was never really happy with my ideas for the schema structure, so any ideas are welcomed.
Updated by Ohad Levy over 2 years ago
I've created http://github.com/ohadlevy/foreman/tree/report_scopes
I think it solves all of the topics in this issue without changing the database schema - I'll appreciate a code review :)
Updated by Ohad Levy over 2 years ago
Ohad Levy wrote:
I've created http://github.com/ohadlevy/foreman/tree/report_scopes
I think it solves all of the topics in this issue without changing the database schema - I'll appreciate a code review :)
well.. I've just learned that indexes do not work on bitwise columns.. even though I really like that code...
Updated by Ohad Levy over 2 years ago
- Target version set to 0.1-3
Updated by Ohad Levy over 2 years ago
- Status changed from New to Ready For Testing
- % Done changed from 0 to 100
Applied in changeset ff1cc6b1ff54003ac9916ecf389710fdfed2ed7b.
Updated by Ohad Levy over 2 years ago
Applied in changeset 16cb774245863cfba6ceca91deec13449b7a1ce2.
Updated by Ohad Levy over 2 years ago
- Status changed from Ready For Testing to Closed
