Project

General

Profile

Actions
Copy link

Refactor #23665

closed

Use Patternfly charts in Runtime

Added by boaz shust almost 6 years ago. Updated over 2 years ago.

Status:
Duplicate
Priority:
Normal
Assignee:
boaz shust
Category:
JavaScript stack
Target version:
3.0.0
Difficulty:
medium
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/5600, https://github.com/theforeman/foreman/pull/8522
Fixed in Releases:
3.0.0
Found in Releases:
Red Hat JIRA:

Description

This is related to http://projects.theforeman.org/issues/22812 and projects.theforeman.org/issues/23010.

The flot chart package is deprecated although Foreman is using it to plot charts.
One of the charts that needs to be migrated is the Runtime chart which is in "app/views/hosts/show.html.erb"

Related issues 3 (0 open — 3 closed)

Related to Foreman - Refactor #22812: Migrate jquery-flot to patternfly-react chartsClosedAvi SharvitActions
Related to Foreman - Bug #33310: Host Detail Graphs VERY slow to load after upgrade to 3.0.0-rc1ClosedActions
Follows Foreman - Refactor #31595: deprecate flot_chart (timeseries area chart)ClosedOndřej EzrActions
Actions
Copy link
 #1

Updated by boaz shust almost 6 years ago

  • Subject changed from Replace flot chart with Patternfly c3 chart in the Runtime chart to Use Patternfly charts in Runtime
Actions
Copy link
 #2

Updated by The Foreman Bot almost 6 years ago

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

Updated by Ohad Levy over 4 years ago

  • Related to Refactor #22812: Migrate jquery-flot to patternfly-react charts added
Actions
Copy link
 #4

Updated by Ohad Levy over 4 years ago

  • Related to Refactor #23010: add deprecation warnings to all flotcharts functions added
Actions
Copy link
 #5

Updated by Ohad Levy over 4 years ago

  • Related to deleted (Refactor #23010: add deprecation warnings to all flotcharts functions)
Actions
Copy link
 #6

Updated by Ondřej Ezr almost 3 years ago

  • Tracker changed from Bug to Refactor
Actions
Copy link
 #7

Updated by Ondřej Ezr almost 3 years ago

  • Is duplicate of Refactor #31595: deprecate flot_chart (timeseries area chart) added
Actions
Copy link
 #8

Updated by Ondřej Ezr almost 3 years ago

  • Status changed from Ready For Testing to Duplicate

This has been done within #31595

Actions
Copy link
 #9

Updated by Ondřej Ezr almost 3 years ago

  • Is duplicate of deleted (Refactor #31595: deprecate flot_chart (timeseries area chart))
Actions
Copy link
 #10

Updated by Ondřej Ezr almost 3 years ago

  • Follows Refactor #31595: deprecate flot_chart (timeseries area chart) added
Actions
Copy link
 #11

Updated by The Foreman Bot almost 3 years ago

  • Fixed in Releases 3.0.0 added
Actions
Copy link
 #12

Updated by Amit Upadhye over 2 years ago

  • Target version set to 3.0.0
Actions
Copy link
 #13

Updated by Amit Upadhye over 2 years ago

boaz shust wrote:

This is related to http://projects.theforeman.org/issues/22812 and projects.theforeman.org/issues/23010.

The flot chart package is deprecated although Foreman is using it to plot charts.
One of the charts that needs to be migrated is the Runtime chart which is in "app/views/hosts/show.html.erb"

boaz shust wrote:

This is related to http://projects.theforeman.org/issues/22812 and projects.theforeman.org/issues/23010.

The flot chart package is deprecated although Foreman is using it to plot charts.
One of the charts that needs to be migrated is the Runtime chart which is in "app/views/hosts/show.html.erb"

I can see this is marked duplicate however fixed by combined pr in separate commit. Accordingly I am marking this fixed in 3.0

Actions
Copy link
 #14

Updated by Amit Upadhye over 2 years ago

  • Category set to JavaScript stack
Actions
Copy link
 #15

Updated by Tomer Brisker over 2 years ago

  • Related to Bug #33310: Host Detail Graphs VERY slow to load after upgrade to 3.0.0-rc1 added
Actions
Copy link
 #16

Updated by Ondřej Ezr over 2 years ago

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

Also available in: Atom PDF