Project

General

Profile

Actions

Bug #9441

closed

Trends background processing is unnecessarily complicated

Added by Bryan Kearney about 9 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
gem foreman_statistics
Difficulty:
easy
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1192176
Description of problem:
I'll admit I don't fully understand what Trends will end up showing me, since I haven't gotten it working, but it seems odd that we force users to use both the Web UI and the CLI to get them set up.

When you first visit the Trends page, and there's nothing configured, you see:
"No trend counter defined.
To define trend counters, use the Add Trend Counter button.To start collecting trend data, set a cron job to execute 'rake trends:counter' every Puppet Interval (30 minutes)."

If you click on New Trend Counter, and make something, the message changes:
"No trend counter found.
To start collecting trend data, set a cron job to execute 'RAILS_ENV=production bundle exec rake trends:counter' every Puppet Interval (30 minutes)"

Why can't I fully configure Trends through the Web UI? Couldn't we configure that through Satellite and have Sat push any CLI commands needed in the background?

Also, there's no ending punctuation for that second message.

Version-Release number of selected component (if applicable):
6.0.6

Actions #1

Updated by Dominic Cleal about 9 years ago

  • Category set to Trends
  • Assignee deleted (Ohad Levy)
  • Priority changed from Normal to Low
  • Difficulty set to easy

Incorrect, it is configured by the installer and packages. Perhaps just remove the message and ensure it's in the documentation just in case.

Actions #2

Updated by Dominic Cleal about 9 years ago

  • Subject changed from Fully configuring Trends cannot be done through the Web UI, have to finish task using CLI to Trends web UI mentions unnecessary command in default install
Actions #3

Updated by Ondřej Pražák over 7 years ago

  • Target version set to 115
Actions #4

Updated by Tomer Brisker almost 4 years ago

  • Project changed from Foreman to Statistics
  • Category deleted (Trends)
Actions #5

Updated by Ondřej Ezr over 3 years ago

  • Subject changed from Trends web UI mentions unnecessary command in default install to Trends background processing is unecessary complicated
  • Category set to gem foreman_statistics
  • Assignee set to Ondřej Ezr
  • Priority changed from Low to Normal
  • Target version set to foreman_statistics-1.0.0
  • Triaged set to Yes
Actions #6

Updated by Ondřej Ezr over 3 years ago

  • Subject changed from Trends background processing is unecessary complicated to Trends background processing is unnecessarily complicated
Actions #7

Updated by Ondřej Ezr over 3 years ago

  • Pull request https://github.com/theforeman/foreman_statistics/pull/19 added
Actions #8

Updated by Ondřej Ezr over 3 years ago

  • Status changed from New to Ready For Testing
Actions #9

Updated by Ondřej Ezr over 3 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF