Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactors the system checkers #507

Merged
merged 1 commit into from
Oct 16, 2023
Merged

Refactors the system checkers #507

merged 1 commit into from
Oct 16, 2023

Conversation

ribalba
Copy link
Member

@ribalba ribalba commented Oct 13, 2023

No description provided.

@ribalba ribalba requested a review from ArneTR October 13, 2023 14:45
@github-actions
Copy link

Eco-CI Output:

Label 🖥 avg. CPU utilization [%] 🔋 Total Energy [Joules] 🔌 avg. Power [Watts] Duration [Seconds]
Total Run 18.6237 2053.48 2.8963 717
Measurement #1 18.7256 2053.48 2.8963 711

📈 Energy graph:

 
 6.51 ┤                                                                             ╭──╮               ╭─╮     ╭─╮╭─╮
 6.06 ┤         ╭─────╮                             ╭╮                  ╭╮         ╭╯  ╰─────╮╭─╮     ╭╯ ╰──╮ ╭╯ ╰╯ │
 5.61 ┤         │     │                 ╭╮          ││                  ││         │         ╰╯ │     │     │ │     │           ╭╮╭╮                                                                                                                                                                                                       ╭╮                                                                                                                                        ╭╮                                                                                                                                                                                                             ╭╮
 5.16 ┤        ╭╯     ╰╮╭─╮  ╭──╮ ╭╮    │╰╮   ╭╮    │╰╮    ╭╮    ╭╮    ╭╯│         │            │     │     ╰╮│     │        ╭╮ │││╰╮                                                              ╭╮                       ╭╮                                               ╭╮          ╭╮          ╭╮          ╭╮          ╭╮           ╭╯│                         ╭╮                                                                                ╭─╮                          ││                          ╭─╮                                                                                                                                                                                ││                          ╭╮
 4.71 ┤        │       ╰╯ ╰──╯  ╰─╯╰───╮│ ╰───╯╰────╯ ╰───╮│╰────╯╰────╯ │         │            │     │      ││     │        ││ │││ │                         ╭╮           ╭╮          ╭─╮         ││           ╭╮          ││                                               │╰╮         │╰╮         ││          │╰╮         │╰╮          │ │          ╭╮           ╭╮││                        ╭╮                    ╭╮ ╭───╮                         ╭╯ │          ╭╮             ╭╯│                       ╭╮ │ ╰╮                          ╭╮                      ╭─╮                                ╭─╮                            ╭╮                         ╭─╮          ╭╮            ╭──╮ │╰╮                     ╭╮  ││
 4.26 ┤      ╭╮│                       ││                 ││             │ ╭╮      │            │    ╭╯      ││     │        │╰╮│││ │          ╭╮             │╰╮          ││          │ │         │╰╮          │╰╮         │╰╮         ╭─╮         ╭─╮         ╭─╮          │ │         │ │         │╰╮         │ │         │ │          │ ╰╮         ││        ╭╮ │││╰╮                  ╭╮ ╭─╯╰╮                   ││ │   │          ╭╮          ╭╮ │  │          ││          ╭╮ │ ╰╮          ╭╮╭╮        ││ │  │                      ╭╮ ╭╯╰╮         ╭╮          │ │         ╭╮          ╭╮         │ ╰╮          ╭╮               │╰╮          ╭╮            │ │          ││            │  ╰─╯ │         ╭╮          ││ ╭╯│
 3.81 ┤      │││                       ││                 ╰╯             │╭╯│╭╮  ╭─╯            │╭╮  │       ││     ╰╮       │ ╰╯╰╯ │         ╭╯│╭╮        ╭╮ │ ╰╮         │╰╮        ╭╯ │         │ ╰╮        ╭╯ │         │ │         │ │         │ │         │ │         ╭╯ │         │ │         │ │         │ │         │ │         ╭╯  │         ││        ││ │╰╯ │                  ││ │   │                   ││ │   │         ╭╯│╭╮        ││ │  ╰╮         ││          ││ │  │          ││││        ││ │  ╰╮         ╭╮          ││ │  │         ││         ╭╯ │         ││          ││         │  │         ╭╯│╭─╮            │ │          ││            │ │          ││            │      │         │╰╮         ││ │ ╰╮
 3.36 ┤      │││                       ╰╯                                ││ ╰╯│ ╭╯              ╰╯│  │       ╰╯      │      ╭╯      │        ╭╯ │││       ╭╯│╭╯  │        ╭╯ │        │  │         │  │        │  │         │ │         │ │         │ │         │ ╰╮        │  │         │ │         │ │         │ │         │ ╰╮        │   │         ││        ││ │   │                  ││ │   ╰╮        ╭╮        ││ │   │         │ │││       ╭╯│ │   │         ││╭╮        ││ │  ╰╮         ││││        ││ │   │       ╭╮│╰─╮        ││ │  │         │╰╮        │  │         │╰╮        ╭╯│         │  │         │ ││ │╭╮       ╭╮ │ ╰╮         │╰─╮          │ │         ╭╯│            │      │         │ │         ││ │  │
 2.91 ┤    ╭╮│╰╯                                                         ╰╯   │ │                 │  │               │      │       │        │  ╰╯│       │ ││   │        │  │        │  │         │  │        │  │         │ ╰╮        │ ╰╮        │ ╰╮        │  │        │  │         │ │         │ │         │ │         │  │        │   │         ││       ╭╯│╭╯   │        ╭╮        │╰╮│    │        ││        ││ │   │         │ │││       │ │ │   │       ╭╮│╰╯│       ╭╯│ │   │       ╭─╯╰╯│        ││ │   │       │││  │        │╰╮│  │         │ │        │  │         │ │        │ │         │  │         │ ╰╯ ╰╯│       ││ │  │       ╭╮│  │          │ ╰╮       ╭╯ │╭╮       ╭╮╭╯      │        ╭╯ ╰╮   ╭╮   │╰╮│  │
 2.46 ┤  ╭╮│╰╯                                                                ╰─╯                 │ ╭╯               │  ╭╮  │       ╰─╮    ╭╮│    │    ╭╮ │ ││   │  ╭╮    │  │  ╭╮    │  │  ╭╮    ╭╯  │  ╭╮    │  │  ╭╮    ╭╯  │ ╭╮    ╭╯  │  ╭╮    │  │  ╭╮    │  │  ╭╮    │  │   ╭╮    │ │   ╭╮    │ │   ╭╮    │ │    ╭╮   │  │   ╭╮   │   │  ╭╮    ╭╯│    ╭╮ │ ││    │╭╮    ╭╮│╰╮ ╭╮    │ ││    │    ╭╮  ││╭╮    ╭─╯│ │   │  ╭╮   ╭─╯ │││ ╭╮    │ │╭╯   │   ╭╮  │││  │   ╭╮  │ ╰─╯   ╰╮    ╭╮│    │    ╭╮ ╭╯│ │   │ ╭╮    │││  ╰─╮    ╭╮│ ││  │    ╭╮  ╭╯ │    ╭╮  │  │    ╭╮  ╭╯ │     ╭╮ │ │╭╮    ╭╮╭╯  ╰╮    ╭╮╭─╯      │ ╭╮    ││ │  ╰╮    ╭╮│││  │    ╭╮ ╭╮ │  │   ╭╮  │  ╰╯│  ╭╮   │╰╯       │   ╭╮   │   ╰╮ ╭╯│   │ ││  ╰╮    ╭╮
 2.01 ┼──╯╰╯                                                                                      ╰─╯                ╰──╯╰──╯         ╰────╯╰╯    ╰────╯╰─╯ ╰╯   ╰──╯╰────╯  ╰──╯╰────╯  ╰──╯╰────╯   ╰──╯╰────╯  ╰──╯╰────╯   ╰─╯╰────╯   ╰──╯╰────╯  ╰──╯╰────╯  ╰──╯╰────╯  ╰───╯╰────╯ ╰───╯╰────╯ ╰───╯╰────╯ ╰────╯╰───╯  ╰───╯╰───╯   ╰──╯╰────╯ ╰────╯╰─╯ ╰╯    ╰╯╰────╯╰╯ ╰─╯╰────╯ ╰╯    ╰────╯╰──╯╰╯╰────╯  ╰─╯   ╰──╯╰───╯   ╰╯╰─╯╰────╯ ╰╯    ╰───╯╰──╯╰╯  ╰───╯╰──╯        ╰────╯╰╯    ╰────╯╰─╯ ╰─╯   ╰─╯╰────╯╰╯    ╰────╯╰╯ ╰╯  ╰────╯╰──╯  ╰────╯╰──╯  ╰────╯╰──╯  ╰─────╯╰─╯ ╰╯╰────╯╰╯    ╰────╯╰╯        ╰─╯╰────╯╰─╯   ╰────╯╰╯╰╯  ╰────╯╰─╯╰─╯  ╰───╯╰──╯    ╰──╯╰───╯         ╰───╯╰───╯    ╰─╯ ╰───╯ ╰╯   ╰────╯╰─
                                                                                                                                                                                                                                                                                                                                                                  Watts over time

@ArneTR
Copy link
Member

ArneTR commented Oct 16, 2023

LGTM. Tested on Linux box and works fine

Copy link
Member

@ArneTR ArneTR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tested on Linux box and works fine

@ArneTR ArneTR merged commit a3b9a9e into main Oct 16, 2023
4 checks passed
@ArneTR ArneTR deleted the refactors-system-checks branch October 16, 2023 09:28
ArneTR added a commit that referenced this pull request Oct 20, 2023
* main: (103 commits)
  Removing some noise debug
  BCC typos
  Bump actions/checkout from 3 to 4 (#450)
  Bump orjson from 3.9.7 to 3.9.9 (#506)
  Bump fastapi from 0.103.2 to 0.104.0 (#511)
  Added BCC functionality for email sending (#510)
  Refactors the system checkers (#507)
  Bump psutil from 5.9.5 to 5.9.6 (#508)
  FIXME disabled in Pylint
  Added simple HTML5 validation to request.html
  Network key is optional for HOG
  Clarified error message and clarified actual exception caught
  extends the install script to have a random password option (#504)
  Jobs.py now sends email when old jobs are present; Bugfixes in SQL query
  Flow process runtime bumped to 60+ minutes
  Sorting must be additionaly done by table to show indicator
  Repositories generated table also shall show sorting indicators
  Removing legacy datatable includes for energy-timeline
  Removed 'sortable' class where we are using datatables and not jQuery-Sort
  Bugifx: If Jobs object is not set do not access
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants